Thread (26 messages) flat view 26 messages, 5 authors, 2018-04-09

Re: [PATCH net-next 6/6] netdevsim: Add simple FIB resource controller via devlink

From: David Miller <davem@davemloft.net>
Date: 2018-03-29 18:11:20

From: Jakub Kicinski <redacted>
Date: Tue, 27 Mar 2018 18:34:50 -0700
On Tue, 27 Mar 2018 18:22:00 -0700, David Ahern wrote:
quoted
+void nsim_devlink_setup(struct netdevsim *ns)
+{
 ...
nit: DaveM expressed preference to not have silent failures in a
     discussion about DebugFS, not sure it applies here, but why not
     handle errors?
Yes it is a concern.

David please address this as a follow-up.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help