Commit c5398ac1 authored by Vitaly Kuznetsov's avatar Vitaly Kuznetsov Committed by Ben Hutchings

hv: allocate synic pages for all present CPUs

commit 421b8f20 upstream.

It may happen that not all CPUs are online when we do hv_synic_alloc() and
in case more CPUs come online later we may try accessing these allocated
structures.
Signed-off-by: default avatarVitaly Kuznetsov <vkuznets@redhat.com>
Signed-off-by: default avatarK. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
[bwh: Backported to 3.16: adjust context]
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
parent d1f5156e
...@@ -264,7 +264,7 @@ int hv_synic_alloc(void) ...@@ -264,7 +264,7 @@ int hv_synic_alloc(void)
size_t size = sizeof(struct tasklet_struct); size_t size = sizeof(struct tasklet_struct);
int cpu; int cpu;
for_each_online_cpu(cpu) { for_each_present_cpu(cpu) {
hv_context.event_dpc[cpu] = kmalloc(size, GFP_ATOMIC); hv_context.event_dpc[cpu] = kmalloc(size, GFP_ATOMIC);
if (hv_context.event_dpc[cpu] == NULL) { if (hv_context.event_dpc[cpu] == NULL) {
pr_err("Unable to allocate event dpc\n"); pr_err("Unable to allocate event dpc\n");
...@@ -317,7 +317,7 @@ void hv_synic_free(void) ...@@ -317,7 +317,7 @@ void hv_synic_free(void)
{ {
int cpu; int cpu;
for_each_online_cpu(cpu) for_each_present_cpu(cpu)
hv_synic_free_cpu(cpu); hv_synic_free_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