Commit 3230bb47 authored by Zachary Amsden's avatar Zachary Amsden Committed by Avi Kivity

KVM: Fix hotplug of CPUs

Both VMX and SVM require per-cpu memory allocation, which is done at module
init time, for only online cpus.

Backend was not allocating enough structure for all possible CPUs, so
new CPUs coming online could not be hardware enabled.
Signed-off-by: default avatarZachary Amsden <zamsden@redhat.com>
Signed-off-by: default avatarMarcelo Tosatti <mtosatti@redhat.com>
parent e6732a5a
...@@ -482,7 +482,7 @@ static __init int svm_hardware_setup(void) ...@@ -482,7 +482,7 @@ static __init int svm_hardware_setup(void)
kvm_enable_efer_bits(EFER_SVME); kvm_enable_efer_bits(EFER_SVME);
} }
for_each_online_cpu(cpu) { for_each_possible_cpu(cpu) {
r = svm_cpu_init(cpu); r = svm_cpu_init(cpu);
if (r) if (r)
goto err; goto err;
...@@ -516,7 +516,7 @@ static __exit void svm_hardware_unsetup(void) ...@@ -516,7 +516,7 @@ static __exit void svm_hardware_unsetup(void)
{ {
int cpu; int cpu;
for_each_online_cpu(cpu) for_each_possible_cpu(cpu)
svm_cpu_uninit(cpu); svm_cpu_uninit(cpu);
__free_pages(pfn_to_page(iopm_base >> PAGE_SHIFT), IOPM_ALLOC_ORDER); __free_pages(pfn_to_page(iopm_base >> PAGE_SHIFT), IOPM_ALLOC_ORDER);
......
...@@ -1350,15 +1350,17 @@ static void free_kvm_area(void) ...@@ -1350,15 +1350,17 @@ static void free_kvm_area(void)
{ {
int cpu; int cpu;
for_each_online_cpu(cpu) for_each_possible_cpu(cpu) {
free_vmcs(per_cpu(vmxarea, cpu)); free_vmcs(per_cpu(vmxarea, cpu));
per_cpu(vmxarea, cpu) = NULL;
}
} }
static __init int alloc_kvm_area(void) static __init int alloc_kvm_area(void)
{ {
int cpu; int cpu;
for_each_online_cpu(cpu) { for_each_possible_cpu(cpu) {
struct vmcs *vmcs; struct vmcs *vmcs;
vmcs = alloc_vmcs_cpu(cpu); vmcs = alloc_vmcs_cpu(cpu);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment