diff options
author | Mahesh Bandewar <maheshb@google.com> | 2017-01-18 18:02:49 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-01-20 12:22:25 -0500 |
commit | 1b7cd0044e4a9f69aaf00511870b07fcdeda591d (patch) | |
tree | fce5cccf4992d44ea8c74986c8c21123e5d6f687 | |
parent | 264b87fa617e758966108db48db220571ff3d60e (diff) |
net: remove duplicate code.
netdev_rx_handler_register() checks to see if the handler is already
busy which was recently separated into netdev_is_rx_handler_busy(). So
use the same function inside register() to avoid code duplication.
Essentially this change should be a no-op
Signed-off-by: Mahesh Bandewar <maheshb@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/core/dev.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/net/core/dev.c b/net/core/dev.c index ad5959e56116..c8f1f67ff16c 100644 --- a/net/core/dev.c +++ b/net/core/dev.c | |||
@@ -3961,9 +3961,7 @@ int netdev_rx_handler_register(struct net_device *dev, | |||
3961 | rx_handler_func_t *rx_handler, | 3961 | rx_handler_func_t *rx_handler, |
3962 | void *rx_handler_data) | 3962 | void *rx_handler_data) |
3963 | { | 3963 | { |
3964 | ASSERT_RTNL(); | 3964 | if (netdev_is_rx_handler_busy(dev)) |
3965 | |||
3966 | if (dev->rx_handler) | ||
3967 | return -EBUSY; | 3965 | return -EBUSY; |
3968 | 3966 | ||
3969 | /* Note: rx_handler_data must be set before rx_handler */ | 3967 | /* Note: rx_handler_data must be set before rx_handler */ |