> -----Original Message----- > From: Dexuan Cui > Sent: Friday, July 17, 2015 3:07 AM > To: KY Srinivasan; da...@davemloft.net; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; jasow...@redhat.com; vkuzn...@redhat.com > Cc: KY Srinivasan > Subject: RE: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be > processed during probe > > > From: K. Y. Srinivasan > > Sent: Friday, July 17, 2015 3:17 > > Subject: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be > processed > > during probe > > > > The current code returns from probe without waiting for the proper > handling > > of subchannels that may be requested. If the netvsc driver were to be > rapidly > > loaded/unloaded, we can trigger a panic as the unload will be tearing > > down state that may not have been fully setup yet. We fix this issue by > making > > sure that we return from the probe call only after ensuring that the > > sub-channel offers in flight are properly handled. > > > > --- > > drivers/net/hyperv/hyperv_net.h | 2 ++ > > drivers/net/hyperv/rndis_filter.c | 25 +++++++++++++++++++++++++ > > 2 files changed, 27 insertions(+), 0 deletions(-) > > BTW, not sure if we should make the same fix to storvsc. > > IMO storvsc should have the same issue, at least in theory, though usually > it's > unlikely to unload storvsc. :-)
You are right; I am planning to submit a similar patch for storvsc. As you note, this scenario is unlikely for sorvsc. K. Y > > Thanks, > -- Dexuan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
