diff options
| author | Vitaly Kuznetsov <vkuznets@redhat.com> | 2016-12-07 04:16:26 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-12 00:41:49 -0500 |
| commit | 6e936b06795ae5dcb5248e0c4344d8303de4c2db (patch) | |
| tree | 6c10946b40e360306b9aa98cd2213a6f59d9260e /drivers/hv | |
| parent | 879dc37cc3a81c99527410a83ae9cc35c637e8b1 (diff) | |
hv: init percpu_list in hv_synic_alloc()
commit 3c7630d35009e6635e5b58d62de554fd5b6db5df upstream.
Initializing hv_context.percpu_list in hv_synic_alloc() helps to prevent a
crash in percpu_channel_enq() when not all CPUs were online during
initialization and it naturally belongs there.
Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/hv')
| -rw-r--r-- | drivers/hv/hv.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/hv/hv.c b/drivers/hv/hv.c index e4bb498e41ae..a2567a4be1a8 100644 --- a/drivers/hv/hv.c +++ b/drivers/hv/hv.c | |||
| @@ -457,6 +457,8 @@ int hv_synic_alloc(void) | |||
| 457 | pr_err("Unable to allocate post msg page\n"); | 457 | pr_err("Unable to allocate post msg page\n"); |
| 458 | goto err; | 458 | goto err; |
| 459 | } | 459 | } |
| 460 | |||
| 461 | INIT_LIST_HEAD(&hv_context.percpu_list[cpu]); | ||
| 460 | } | 462 | } |
| 461 | 463 | ||
| 462 | return 0; | 464 | return 0; |
| @@ -552,8 +554,6 @@ void hv_synic_init(void *arg) | |||
| 552 | rdmsrl(HV_X64_MSR_VP_INDEX, vp_index); | 554 | rdmsrl(HV_X64_MSR_VP_INDEX, vp_index); |
| 553 | hv_context.vp_index[cpu] = (u32)vp_index; | 555 | hv_context.vp_index[cpu] = (u32)vp_index; |
| 554 | 556 | ||
| 555 | INIT_LIST_HEAD(&hv_context.percpu_list[cpu]); | ||
| 556 | |||
| 557 | /* | 557 | /* |
| 558 | * Register the per-cpu clockevent source. | 558 | * Register the per-cpu clockevent source. |
| 559 | */ | 559 | */ |
