Thread (8 messages) flat view 8 messages, 3 authors, 2018-07-02

Re: [PATCH net] ipvlan: call dev_change_flags when reset ipvlan mode

From: David Miller <davem@davemloft.net>
Date: 2018-06-20 05:31:50

From: Hangbin Liu <redacted>
Date: Wed, 20 Jun 2018 11:22:54 +0800
The only case dev_change_flags() return an err is when we change IFF_UP flag.
Since we only set/reset IFF_NOARP, do you think we still need to check the
return value?
It is bad to try and take shortcuts on error handling using assumptions
like that.

If dev_change_flags() is adjusted to return error codes in more
situations, nobody is going to remember to undo your "optimziation"
here.

Please check for errors, thank you.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help