RE: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-06-01 Thread Dexuan Cui
> >> [5.472143] BUG: unable to handle kernel paging request at > >> 00079fff5288 > >> [5.477107] IP: [] vmbus_onoffer+0x311/0x570 > >> [hv_vmbus] > >> ... > >> Vitaly > > > > I can't reproduce the panic somehow, but I did find a bug in > vmbus_process_offer(): > > > >

RE: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-06-01 Thread Dexuan Cui
> >> [5.472143] BUG: unable to handle kernel paging request at > >> 00079fff5288 > >> [5.477107] IP: [] vmbus_onoffer+0x311/0x570 > >> [hv_vmbus] > >> ... > >> Vitaly > > > > I can't reproduce the panic somehow, but I did find a bug in > vmbus_process_offer(): > > > >

Re: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-06-01 Thread Vitaly Kuznetsov
...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; >> jasow...@redhat.com; KY Srinivasan <k...@microsoft.com>; Haiyang >> Zhang <haiya...@microsoft.com>; rolf.neugeba...@docker.com; >> dave.sc...@docker.com; ian.campb...@docker.com >> Subjec

Re: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-06-01 Thread Vitaly Kuznetsov
; a...@canonical.com; >> jasow...@redhat.com; KY Srinivasan ; Haiyang >> Zhang ; rolf.neugeba...@docker.com; >> dave.sc...@docker.com; ian.campb...@docker.com >> Subject: Re: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & >> updating the percpu list >>

RE: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-06-01 Thread Dexuan Cui
jasow...@redhat.com; KY Srinivasan <k...@microsoft.com>; Haiyang > Zhang <haiya...@microsoft.com>; rolf.neugeba...@docker.com; > dave.sc...@docker.com; ian.campb...@docker.com > Subject: Re: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & > updating the perc

RE: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-06-01 Thread Dexuan Cui
inivasan ; Haiyang > Zhang ; rolf.neugeba...@docker.com; > dave.sc...@docker.com; ian.campb...@docker.com > Subject: Re: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & > updating the percpu list > > Dexuan Cui writes: > > > There is a rare

Re: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-05-31 Thread Vitaly Kuznetsov
Dexuan Cui writes: > There is a rare race when we remove an entry from the global list > hv_context.percpu_list[cpu] in hv_process_channel_removal() -> > percpu_channel_deq() -> list_del(): at this time, if vmbus_on_event() -> > process_chn_event() -> pcpu_relid2channel() is

Re: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-05-31 Thread Vitaly Kuznetsov
Dexuan Cui writes: > There is a rare race when we remove an entry from the global list > hv_context.percpu_list[cpu] in hv_process_channel_removal() -> > percpu_channel_deq() -> list_del(): at this time, if vmbus_on_event() -> > process_chn_event() -> pcpu_relid2channel() is trying to query the