> -----Original Message----- > From: Dexuan Cui > Sent: Friday, July 17, 2015 3:07 AM > To: KY Srinivasan; davem@xxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux- > kernel@xxxxxxxxxxxxxxx; devel@xxxxxxxxxxxxxxxxxxxxxx; olaf@xxxxxxxxx; > apw@xxxxxxxxxxxxx; jasowang@xxxxxxxxxx; vkuznets@xxxxxxxxxx > 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 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
