Re: [PATCH net-next v12 2/5] netvsc: refactor notifier/event handling code to use the failover framework

From
Stephen Hemminger <stephen@networkplumber.org>
Date
2018-05-31 12:58:20
Reply to
Samudrala, Sridhar, 2018-05-31
On Wed, 30 May 2018 20:03:11 -0700
"Samudrala, Sridhar" [off-list ref] wrote:
On 5/30/2018 7:06 PM, Stephen Hemminger wrote:
quoted
On Thu, 24 May 2018 09:55:14 -0700
Sridhar Samudrala [off-list ref] wrote:
 
quoted
Use the registration/notification framework supported by the generic
failover infrastructure.

Signed-off-by: Sridhar Samudrala <sridhar.samudrala@intel.com>  
Why was this merged? It was never signed off by any of the netvsc maintainers,
and there were still issues unresolved.

There are also namespaces issues I am fixing and this breaks them.
Will start my patch set with a revert for this. Sorry  
I would appreciate if you can make the fixes on top of this patch series. I tried hard
to make sure that netvsc functionality and behavior doesn't change.

It is possible that there could be some bugs introduced, but they can be fixed.
Looks like Wei already found a bug and submitted a fix for that.
Ok, but several of these may clash with what you want for virtio.
Like:
	- VF should be moved to namespace of virt device
	- VF should be associated based on message from host with serial # not
	  registration notifier and MAC address.
	- control operations should use master device reference rather than
	  searching based on MAC.

As you can see these are structural changes.
Thread (28 messages) 28 messages, 6 authors, last reply 2018-05-31
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help