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

From: Dexuan Cui
Date: Tue May 17 2016 - 04:37:59 EST


> From: devel [mailto:driverdev-devel-bounces@xxxxxxxxxxxxxxxxxxxxxx] On Behalf
> Of Dexuan Cui
> >
> > Do we need to do tasklet_schedule() to make sure there are no events
> > pending? This is probably not a big issue as some other event will
> > trigger scheduling but in some corner cases it may bite. Same question
> > applies to the code below and to vmbus_close_internal().
>
> Hi Vitaly,
> Thanks for spotting this!
> I think you're correct.
> I'll add tasklet_schedul() before the tasklet_enable() in the 2 places.

Sorry for the typo -- I meant "after" rather than "before".

Thanks
-- Dexuan