diff options
author | dingtianhong <dingtianhong@huawei.com> | 2013-05-27 19:53:31 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-05-27 23:18:50 -0700 |
commit | da6e378ba918cd0feeb90eeb84d8b42148bb0c82 (patch) | |
tree | 7ffc6e756d49f67205b5e05d3284d81cda06cc7a /net/core/dev.c | |
parent | 87227b8b2d4d556a6924ad9af87450fdc3fcd7e3 (diff) | |
download | linux-stable-da6e378ba918cd0feeb90eeb84d8b42148bb0c82.tar.gz linux-stable-da6e378ba918cd0feeb90eeb84d8b42148bb0c82.tar.bz2 linux-stable-da6e378ba918cd0feeb90eeb84d8b42148bb0c82.zip |
netpoll: remove return value from netpoll_rx_disable()
The netpoll_rx_disable() will always return 0, it is no use and looks wordy,
so remove the unnecessary code and get rid of it in _dev_open and _dev_close.
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/dev.c')
-rw-r--r-- | net/core/dev.c | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/net/core/dev.c b/net/core/dev.c index 2f09cb29cc95..5f747974ac58 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -1198,9 +1198,7 @@ static int __dev_open(struct net_device *dev) * If we don't do this there is a chance ndo_poll_controller * or ndo_poll may be running while we open the device */ - ret = netpoll_rx_disable(dev); - if (ret) - return ret; + netpoll_rx_disable(dev); ret = call_netdevice_notifiers(NETDEV_PRE_UP, dev); ret = notifier_to_errno(ret); @@ -1309,9 +1307,7 @@ static int __dev_close(struct net_device *dev) LIST_HEAD(single); /* Temporarily disable netpoll until the interface is down */ - retval = netpoll_rx_disable(dev); - if (retval) - return retval; + netpoll_rx_disable(dev); list_add(&dev->unreg_list, &single); retval = __dev_close_many(&single); @@ -1353,14 +1349,11 @@ static int dev_close_many(struct list_head *head) */ int dev_close(struct net_device *dev) { - int ret = 0; if (dev->flags & IFF_UP) { LIST_HEAD(single); /* Block netpoll rx while the interface is going down */ - ret = netpoll_rx_disable(dev); - if (ret) - return ret; + netpoll_rx_disable(dev); list_add(&dev->unreg_list, &single); dev_close_many(&single); @@ -1368,7 +1361,7 @@ int dev_close(struct net_device *dev) netpoll_rx_enable(dev); } - return ret; + return 0; } EXPORT_SYMBOL(dev_close); |