net: remove useless comments in net/core/dev.c
The code itself can explain what it is doing, no need these comments. Signed-off-by: WANG Cong <amwang@redhat.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
e0bccd315d
commit
3b261ade42
|
@ -1140,9 +1140,6 @@ static int __dev_open(struct net_device *dev)
|
||||||
|
|
||||||
ASSERT_RTNL();
|
ASSERT_RTNL();
|
||||||
|
|
||||||
/*
|
|
||||||
* Is it even present?
|
|
||||||
*/
|
|
||||||
if (!netif_device_present(dev))
|
if (!netif_device_present(dev))
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
|
@ -1151,9 +1148,6 @@ static int __dev_open(struct net_device *dev)
|
||||||
if (ret)
|
if (ret)
|
||||||
return ret;
|
return ret;
|
||||||
|
|
||||||
/*
|
|
||||||
* Call device private open method
|
|
||||||
*/
|
|
||||||
set_bit(__LINK_STATE_START, &dev->state);
|
set_bit(__LINK_STATE_START, &dev->state);
|
||||||
|
|
||||||
if (ops->ndo_validate_addr)
|
if (ops->ndo_validate_addr)
|
||||||
|
@ -1162,31 +1156,12 @@ static int __dev_open(struct net_device *dev)
|
||||||
if (!ret && ops->ndo_open)
|
if (!ret && ops->ndo_open)
|
||||||
ret = ops->ndo_open(dev);
|
ret = ops->ndo_open(dev);
|
||||||
|
|
||||||
/*
|
|
||||||
* If it went open OK then:
|
|
||||||
*/
|
|
||||||
|
|
||||||
if (ret)
|
if (ret)
|
||||||
clear_bit(__LINK_STATE_START, &dev->state);
|
clear_bit(__LINK_STATE_START, &dev->state);
|
||||||
else {
|
else {
|
||||||
/*
|
|
||||||
* Set the flags.
|
|
||||||
*/
|
|
||||||
dev->flags |= IFF_UP;
|
dev->flags |= IFF_UP;
|
||||||
|
|
||||||
/*
|
|
||||||
* Enable NET_DMA
|
|
||||||
*/
|
|
||||||
net_dmaengine_get();
|
net_dmaengine_get();
|
||||||
|
|
||||||
/*
|
|
||||||
* Initialize multicasting status
|
|
||||||
*/
|
|
||||||
dev_set_rx_mode(dev);
|
dev_set_rx_mode(dev);
|
||||||
|
|
||||||
/*
|
|
||||||
* Wakeup transmit queue engine
|
|
||||||
*/
|
|
||||||
dev_activate(dev);
|
dev_activate(dev);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1209,22 +1184,13 @@ int dev_open(struct net_device *dev)
|
||||||
{
|
{
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
/*
|
|
||||||
* Is it already up?
|
|
||||||
*/
|
|
||||||
if (dev->flags & IFF_UP)
|
if (dev->flags & IFF_UP)
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
/*
|
|
||||||
* Open device
|
|
||||||
*/
|
|
||||||
ret = __dev_open(dev);
|
ret = __dev_open(dev);
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
return ret;
|
return ret;
|
||||||
|
|
||||||
/*
|
|
||||||
* ... and announce new interface.
|
|
||||||
*/
|
|
||||||
rtmsg_ifinfo(RTM_NEWLINK, dev, IFF_UP|IFF_RUNNING);
|
rtmsg_ifinfo(RTM_NEWLINK, dev, IFF_UP|IFF_RUNNING);
|
||||||
call_netdevice_notifiers(NETDEV_UP, dev);
|
call_netdevice_notifiers(NETDEV_UP, dev);
|
||||||
|
|
||||||
|
@ -1240,10 +1206,6 @@ static int __dev_close_many(struct list_head *head)
|
||||||
might_sleep();
|
might_sleep();
|
||||||
|
|
||||||
list_for_each_entry(dev, head, unreg_list) {
|
list_for_each_entry(dev, head, unreg_list) {
|
||||||
/*
|
|
||||||
* Tell people we are going down, so that they can
|
|
||||||
* prepare to death, when device is still operating.
|
|
||||||
*/
|
|
||||||
call_netdevice_notifiers(NETDEV_GOING_DOWN, dev);
|
call_netdevice_notifiers(NETDEV_GOING_DOWN, dev);
|
||||||
|
|
||||||
clear_bit(__LINK_STATE_START, &dev->state);
|
clear_bit(__LINK_STATE_START, &dev->state);
|
||||||
|
@ -1272,15 +1234,7 @@ static int __dev_close_many(struct list_head *head)
|
||||||
if (ops->ndo_stop)
|
if (ops->ndo_stop)
|
||||||
ops->ndo_stop(dev);
|
ops->ndo_stop(dev);
|
||||||
|
|
||||||
/*
|
|
||||||
* Device is now down.
|
|
||||||
*/
|
|
||||||
|
|
||||||
dev->flags &= ~IFF_UP;
|
dev->flags &= ~IFF_UP;
|
||||||
|
|
||||||
/*
|
|
||||||
* Shutdown NET_DMA
|
|
||||||
*/
|
|
||||||
net_dmaengine_put();
|
net_dmaengine_put();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1309,9 +1263,6 @@ static int dev_close_many(struct list_head *head)
|
||||||
|
|
||||||
__dev_close_many(head);
|
__dev_close_many(head);
|
||||||
|
|
||||||
/*
|
|
||||||
* Tell people we are down
|
|
||||||
*/
|
|
||||||
list_for_each_entry(dev, head, unreg_list) {
|
list_for_each_entry(dev, head, unreg_list) {
|
||||||
rtmsg_ifinfo(RTM_NEWLINK, dev, IFF_UP|IFF_RUNNING);
|
rtmsg_ifinfo(RTM_NEWLINK, dev, IFF_UP|IFF_RUNNING);
|
||||||
call_netdevice_notifiers(NETDEV_DOWN, dev);
|
call_netdevice_notifiers(NETDEV_DOWN, dev);
|
||||||
|
@ -1371,11 +1322,6 @@ EXPORT_SYMBOL(dev_disable_lro);
|
||||||
|
|
||||||
static int dev_boot_phase = 1;
|
static int dev_boot_phase = 1;
|
||||||
|
|
||||||
/*
|
|
||||||
* Device change register/unregister. These are not inline or static
|
|
||||||
* as we export them to the world.
|
|
||||||
*/
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* register_netdevice_notifier - register a network notifier block
|
* register_netdevice_notifier - register a network notifier block
|
||||||
* @nb: notifier
|
* @nb: notifier
|
||||||
|
|
Loading…
Reference in New Issue