> -----Original Message-----
> From: Stephen Hemminger [mailto:step...@networkplumber.org]
> Sent: Thursday, June 29, 2017 4:31 PM
> To: KY Srinivasan <k...@microsoft.com>; Haiyan Yang
> <haiy...@microsoft.com>
> Cc: de...@linuxdriverproject.org; sta...@vger.kernel.org; Greg KH
> <g...@kroah.com>
> Subject: Re: [PATCH] vmbus: re-enable tasklet
> 
> On Thu, 22 Jun 2017 16:16:54 -0700
> Stephen Hemminger <step...@networkplumber.org> wrote:
> 
> > This problem shows up in 4.11 when netvsc driver is removed and
> reloaded.
> > The problem is that the channel is closed during module removal and the
> > tasklet for processing responses is disabled. When module is reloaded
> > the channel is reopened but the tasklet is marked as disabled.
> >
> > Patch for upstream is pending, but less urgent since problem doesn't
> > happen with network driver in 4.12.
> >
> > Signed-off-by: Stephen Hemminger <sthem...@microsoft.com>
> > ---
> >  drivers/hv/channel.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c
> > index 321b8833fa6f..36c9ce191b74 100644
> > --- a/drivers/hv/channel.c
> > +++ b/drivers/hv/channel.c
> > @@ -606,6 +606,8 @@ static int vmbus_close_internal(struct
> vmbus_channel *channel)
> >             get_order(channel->ringbuffer_pagecount * PAGE_SIZE));
> >
> >  out:
> > +   /* re-enable tasklet for use on re-open */
> > +   tasklet_enable(&channel->callback_event);
> >     return ret;
> >  }
> >
> 
> Ping.
> 
> What is the state of this patch? Why is it not queued for 4.11 stable?

I will submit this shortly.

K. Y
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to