2005-04-17 06:20:36 +08:00
|
|
|
/*
|
|
|
|
* net/sched/sch_generic.c Generic packet scheduler routines.
|
|
|
|
*
|
|
|
|
* This program is free software; you can redistribute it and/or
|
|
|
|
* modify it under the terms of the GNU General Public License
|
|
|
|
* as published by the Free Software Foundation; either version
|
|
|
|
* 2 of the License, or (at your option) any later version.
|
|
|
|
*
|
|
|
|
* Authors: Alexey Kuznetsov, <kuznet@ms2.inr.ac.ru>
|
|
|
|
* Jamal Hadi Salim, <hadi@cyberus.ca> 990601
|
|
|
|
* - Ingress support
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include <linux/bitops.h>
|
|
|
|
#include <linux/module.h>
|
|
|
|
#include <linux/types.h>
|
|
|
|
#include <linux/kernel.h>
|
|
|
|
#include <linux/sched.h>
|
|
|
|
#include <linux/string.h>
|
|
|
|
#include <linux/errno.h>
|
|
|
|
#include <linux/netdevice.h>
|
|
|
|
#include <linux/skbuff.h>
|
|
|
|
#include <linux/rtnetlink.h>
|
|
|
|
#include <linux/init.h>
|
|
|
|
#include <linux/rcupdate.h>
|
|
|
|
#include <linux/list.h>
|
|
|
|
#include <net/pkt_sched.h>
|
|
|
|
|
|
|
|
/* Main transmission queue. */
|
|
|
|
|
2007-04-17 08:02:10 +08:00
|
|
|
/* Modifications to data participating in scheduling must be protected with
|
2008-07-09 08:18:23 +08:00
|
|
|
* queue->lock spinlock.
|
2007-04-17 08:02:10 +08:00
|
|
|
*
|
|
|
|
* The idea is the following:
|
|
|
|
* - enqueue, dequeue are serialized via top level device
|
2008-07-09 08:18:23 +08:00
|
|
|
* spinlock queue->lock.
|
2007-04-17 08:07:08 +08:00
|
|
|
* - ingress filtering is serialized via top level device
|
2008-07-09 08:33:13 +08:00
|
|
|
* spinlock dev->rx_queue.lock.
|
2007-04-17 08:02:10 +08:00
|
|
|
* - updates to tree and tree walking are only done under the rtnl mutex.
|
2005-04-17 06:20:36 +08:00
|
|
|
*/
|
|
|
|
|
|
|
|
void qdisc_lock_tree(struct net_device *dev)
|
2008-07-09 08:33:13 +08:00
|
|
|
__acquires(dev->rx_queue.lock)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2008-07-17 15:34:19 +08:00
|
|
|
unsigned int i;
|
|
|
|
|
|
|
|
local_bh_disable();
|
|
|
|
for (i = 0; i < dev->num_tx_queues; i++) {
|
|
|
|
struct netdev_queue *txq = netdev_get_tx_queue(dev, i);
|
|
|
|
spin_lock(&txq->lock);
|
|
|
|
}
|
2008-07-09 08:33:13 +08:00
|
|
|
spin_lock(&dev->rx_queue.lock);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
2008-01-23 14:10:23 +08:00
|
|
|
EXPORT_SYMBOL(qdisc_lock_tree);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
void qdisc_unlock_tree(struct net_device *dev)
|
2008-07-09 08:33:13 +08:00
|
|
|
__releases(dev->rx_queue.lock)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2008-07-17 15:34:19 +08:00
|
|
|
unsigned int i;
|
|
|
|
|
2008-07-09 08:33:13 +08:00
|
|
|
spin_unlock(&dev->rx_queue.lock);
|
2008-07-17 15:34:19 +08:00
|
|
|
for (i = 0; i < dev->num_tx_queues; i++) {
|
|
|
|
struct netdev_queue *txq = netdev_get_tx_queue(dev, i);
|
|
|
|
spin_unlock(&txq->lock);
|
|
|
|
}
|
|
|
|
local_bh_enable();
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
2008-01-23 14:10:23 +08:00
|
|
|
EXPORT_SYMBOL(qdisc_unlock_tree);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2007-06-11 08:31:24 +08:00
|
|
|
static inline int qdisc_qlen(struct Qdisc *q)
|
|
|
|
{
|
|
|
|
return q->q.qlen;
|
|
|
|
}
|
|
|
|
|
2008-07-09 14:11:25 +08:00
|
|
|
static inline int dev_requeue_skb(struct sk_buff *skb,
|
2008-07-09 14:10:33 +08:00
|
|
|
struct netdev_queue *dev_queue,
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
struct Qdisc *q)
|
2007-06-11 08:31:24 +08:00
|
|
|
{
|
|
|
|
if (unlikely(skb->next))
|
2008-07-09 14:10:33 +08:00
|
|
|
dev_queue->gso_skb = skb;
|
2007-06-11 08:31:24 +08:00
|
|
|
else
|
|
|
|
q->ops->requeue(skb, q);
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
|
2008-07-09 14:11:25 +08:00
|
|
|
netif_schedule_queue(dev_queue);
|
2007-06-11 08:31:24 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2008-07-09 14:12:38 +08:00
|
|
|
static inline struct sk_buff *dequeue_skb(struct netdev_queue *dev_queue,
|
|
|
|
struct Qdisc *q)
|
2007-06-11 08:31:24 +08:00
|
|
|
{
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
struct sk_buff *skb;
|
2007-06-11 08:31:24 +08:00
|
|
|
|
2008-07-09 14:10:33 +08:00
|
|
|
if ((skb = dev_queue->gso_skb))
|
|
|
|
dev_queue->gso_skb = NULL;
|
2007-06-11 08:31:24 +08:00
|
|
|
else
|
|
|
|
skb = q->dequeue(q);
|
|
|
|
|
|
|
|
return skb;
|
|
|
|
}
|
|
|
|
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
static inline int handle_dev_cpu_collision(struct sk_buff *skb,
|
2008-07-09 14:10:33 +08:00
|
|
|
struct netdev_queue *dev_queue,
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
struct Qdisc *q)
|
2007-06-11 08:31:24 +08:00
|
|
|
{
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
int ret;
|
2007-06-11 08:31:24 +08:00
|
|
|
|
2008-07-09 14:13:53 +08:00
|
|
|
if (unlikely(dev_queue->xmit_lock_owner == smp_processor_id())) {
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
/*
|
|
|
|
* Same CPU holding the lock. It may be a transient
|
|
|
|
* configuration error, when hard_start_xmit() recurses. We
|
|
|
|
* detect it by checking xmit owner and drop the packet when
|
|
|
|
* deadloop is detected. Return OK to try the next skb.
|
|
|
|
*/
|
2007-06-11 08:31:24 +08:00
|
|
|
kfree_skb(skb);
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
if (net_ratelimit())
|
|
|
|
printk(KERN_WARNING "Dead loop on netdevice %s, "
|
2008-07-09 14:13:53 +08:00
|
|
|
"fix it urgently!\n", dev_queue->dev->name);
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
ret = qdisc_qlen(q);
|
|
|
|
} else {
|
|
|
|
/*
|
|
|
|
* Another cpu is holding lock, requeue & delay xmits for
|
|
|
|
* some time.
|
|
|
|
*/
|
|
|
|
__get_cpu_var(netdev_rx_stat).cpu_collision++;
|
2008-07-09 14:11:25 +08:00
|
|
|
ret = dev_requeue_skb(skb, dev_queue, q);
|
2007-06-11 08:31:24 +08:00
|
|
|
}
|
|
|
|
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
return ret;
|
2007-06-11 08:31:24 +08:00
|
|
|
}
|
|
|
|
|
2007-02-09 22:25:16 +08:00
|
|
|
/*
|
2008-07-09 08:18:23 +08:00
|
|
|
* NOTE: Called under queue->lock with locally disabled BH.
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
*
|
2008-07-09 14:14:46 +08:00
|
|
|
* __QUEUE_STATE_QDISC_RUNNING guarantees only one CPU can process
|
|
|
|
* this queue at a time. queue->lock serializes queue accesses for
|
|
|
|
* this queue AND txq->qdisc pointer itself.
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
*
|
|
|
|
* netif_tx_lock serializes accesses to device driver.
|
|
|
|
*
|
2008-07-09 08:18:23 +08:00
|
|
|
* queue->lock and netif_tx_lock are mutually exclusive,
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
* if one is grabbed, another must be free.
|
|
|
|
*
|
|
|
|
* Note, that this procedure can be called by a watchdog timer
|
|
|
|
*
|
|
|
|
* Returns to the caller:
|
|
|
|
* 0 - queue is empty or throttled.
|
|
|
|
* >0 - queue is not empty.
|
|
|
|
*
|
|
|
|
*/
|
2008-07-09 14:12:38 +08:00
|
|
|
static inline int qdisc_restart(struct netdev_queue *txq)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2008-07-09 08:42:10 +08:00
|
|
|
struct Qdisc *q = txq->qdisc;
|
2007-11-14 12:40:55 +08:00
|
|
|
int ret = NETDEV_TX_BUSY;
|
2008-07-09 14:12:38 +08:00
|
|
|
struct net_device *dev;
|
|
|
|
struct sk_buff *skb;
|
2005-04-17 06:20:36 +08:00
|
|
|
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
/* Dequeue packet */
|
2008-07-09 14:12:38 +08:00
|
|
|
if (unlikely((skb = dequeue_skb(txq, q)) == NULL))
|
2007-06-11 08:31:24 +08:00
|
|
|
return 0;
|
2006-06-22 17:57:17 +08:00
|
|
|
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
|
|
|
|
/* And release queue */
|
2008-07-09 08:42:10 +08:00
|
|
|
spin_unlock(&txq->lock);
|
2007-06-11 08:31:24 +08:00
|
|
|
|
2008-07-09 14:12:38 +08:00
|
|
|
dev = txq->dev;
|
|
|
|
|
2008-07-09 14:13:53 +08:00
|
|
|
HARD_TX_LOCK(dev, txq, smp_processor_id());
|
2007-11-14 12:40:55 +08:00
|
|
|
if (!netif_subqueue_stopped(dev, skb))
|
2008-07-17 16:56:23 +08:00
|
|
|
ret = dev_hard_start_xmit(skb, dev, txq);
|
2008-07-09 14:13:53 +08:00
|
|
|
HARD_TX_UNLOCK(dev, txq);
|
2007-06-11 08:31:24 +08:00
|
|
|
|
2008-07-09 08:42:10 +08:00
|
|
|
spin_lock(&txq->lock);
|
|
|
|
q = txq->qdisc;
|
2007-06-11 08:31:24 +08:00
|
|
|
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
switch (ret) {
|
|
|
|
case NETDEV_TX_OK:
|
|
|
|
/* Driver sent out skb successfully */
|
|
|
|
ret = qdisc_qlen(q);
|
|
|
|
break;
|
|
|
|
|
|
|
|
case NETDEV_TX_LOCKED:
|
|
|
|
/* Driver try lock failed */
|
2008-07-09 14:12:38 +08:00
|
|
|
ret = handle_dev_cpu_collision(skb, txq, q);
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
break;
|
|
|
|
|
|
|
|
default:
|
|
|
|
/* Driver returned NETDEV_TX_BUSY - requeue skb */
|
|
|
|
if (unlikely (ret != NETDEV_TX_BUSY && net_ratelimit()))
|
|
|
|
printk(KERN_WARNING "BUG %s code %d qlen %d\n",
|
|
|
|
dev->name, ret, q->q.qlen);
|
|
|
|
|
2008-07-09 14:11:25 +08:00
|
|
|
ret = dev_requeue_skb(skb, txq, q);
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
break;
|
|
|
|
}
|
2007-06-11 08:31:24 +08:00
|
|
|
|
[NET]: qdisc_restart - readability changes plus one bug fix.
New changes :
- Incorporated Peter Waskiewicz's comments.
- Re-added back one warning message (on driver returning wrong value).
Previous changes :
- Converted to use switch/case code which looks neater.
- "if (ret == NETDEV_TX_LOCKED && lockless)" is buggy, and the lockless
check should be removed, since driver will return NETDEV_TX_LOCKED only
if lockless is true and driver has to do the locking. In the original
code as well as the latest code, this code can result in a bug where
if LLTX is not set for a driver (lockless == 0) but the driver is written
wrongly to do a trylock (despite LLTX being set), the driver returns
LOCKED. But since lockless is zero, the packet is requeue'd instead of
calling collision code which will issue warning and free up the skb.
Instead this skb will be retried with this driver next time, and the same
result will ensue. Removing this check will catch these driver bugs instead
of hiding the problem. I am keeping this change to readability section
since :
a. it is confusing to check two things as it is; and
b. it is difficult to keep this check in the changed 'switch' code.
- Changed some names, like try_get_tx_pkt to dev_dequeue_skb (as that is
the work being done and easier to understand) and do_dev_requeue to
dev_requeue_skb, merged handle_dev_cpu_collision and tx_islocked to
dev_handle_collision (handle_dev_cpu_collision is a small routine with only
one caller, so there is no need to have two separate routines which also
results in getting rid of two macros, etc.
- Removed an XXX comment as it should never fail (I suspect this was related
to batch skb WIP, Jamal ?). Converted some functions to original coding
style of having the return values and the function name on same line, eg
prio2list.
Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-06-25 10:56:09 +08:00
|
|
|
return ret;
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
2008-07-09 14:12:38 +08:00
|
|
|
void __qdisc_run(struct netdev_queue *txq)
|
2006-06-20 14:57:59 +08:00
|
|
|
{
|
2008-03-29 07:25:26 +08:00
|
|
|
unsigned long start_time = jiffies;
|
|
|
|
|
2008-07-09 14:12:38 +08:00
|
|
|
while (qdisc_restart(txq)) {
|
2008-07-17 16:56:23 +08:00
|
|
|
if (netif_tx_queue_stopped(txq))
|
2008-03-29 07:25:26 +08:00
|
|
|
break;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Postpone processing if
|
|
|
|
* 1. another process needs the CPU;
|
|
|
|
* 2. we've been doing it for too long.
|
|
|
|
*/
|
|
|
|
if (need_resched() || jiffies != start_time) {
|
2008-07-09 14:12:38 +08:00
|
|
|
netif_schedule_queue(txq);
|
2007-05-10 19:55:14 +08:00
|
|
|
break;
|
2008-03-29 07:25:26 +08:00
|
|
|
}
|
|
|
|
}
|
2006-06-20 14:57:59 +08:00
|
|
|
|
2008-07-09 14:14:46 +08:00
|
|
|
clear_bit(__QUEUE_STATE_QDISC_RUNNING, &txq->state);
|
2006-06-20 14:57:59 +08:00
|
|
|
}
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
static void dev_watchdog(unsigned long arg)
|
|
|
|
{
|
|
|
|
struct net_device *dev = (struct net_device *)arg;
|
|
|
|
|
2006-06-10 03:20:56 +08:00
|
|
|
netif_tx_lock(dev);
|
2008-07-17 15:34:19 +08:00
|
|
|
if (!qdisc_tx_is_noop(dev)) {
|
2005-04-17 06:20:36 +08:00
|
|
|
if (netif_device_present(dev) &&
|
|
|
|
netif_running(dev) &&
|
|
|
|
netif_carrier_ok(dev)) {
|
2008-07-17 15:34:19 +08:00
|
|
|
int some_queue_stopped = 0;
|
|
|
|
unsigned int i;
|
|
|
|
|
|
|
|
for (i = 0; i < dev->num_tx_queues; i++) {
|
|
|
|
struct netdev_queue *txq;
|
|
|
|
|
|
|
|
txq = netdev_get_tx_queue(dev, i);
|
|
|
|
if (netif_tx_queue_stopped(txq)) {
|
|
|
|
some_queue_stopped = 1;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
2006-05-17 06:02:12 +08:00
|
|
|
|
2008-07-17 15:34:19 +08:00
|
|
|
if (some_queue_stopped &&
|
|
|
|
time_after(jiffies, (dev->trans_start +
|
|
|
|
dev->watchdog_timeo))) {
|
|
|
|
printk(KERN_INFO "NETDEV WATCHDOG: %s: "
|
|
|
|
"transmit timed out\n",
|
2006-05-17 06:02:12 +08:00
|
|
|
dev->name);
|
2005-04-17 06:20:36 +08:00
|
|
|
dev->tx_timeout(dev);
|
2008-05-03 07:21:07 +08:00
|
|
|
WARN_ON_ONCE(1);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
2008-07-17 15:34:19 +08:00
|
|
|
if (!mod_timer(&dev->watchdog_timer,
|
|
|
|
round_jiffies(jiffies +
|
|
|
|
dev->watchdog_timeo)))
|
2005-04-17 06:20:36 +08:00
|
|
|
dev_hold(dev);
|
|
|
|
}
|
|
|
|
}
|
2006-06-10 03:20:56 +08:00
|
|
|
netif_tx_unlock(dev);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
dev_put(dev);
|
|
|
|
}
|
|
|
|
|
|
|
|
void __netdev_watchdog_up(struct net_device *dev)
|
|
|
|
{
|
|
|
|
if (dev->tx_timeout) {
|
|
|
|
if (dev->watchdog_timeo <= 0)
|
|
|
|
dev->watchdog_timeo = 5*HZ;
|
2007-06-01 12:28:44 +08:00
|
|
|
if (!mod_timer(&dev->watchdog_timer,
|
|
|
|
round_jiffies(jiffies + dev->watchdog_timeo)))
|
2005-04-17 06:20:36 +08:00
|
|
|
dev_hold(dev);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
static void dev_watchdog_up(struct net_device *dev)
|
|
|
|
{
|
|
|
|
__netdev_watchdog_up(dev);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void dev_watchdog_down(struct net_device *dev)
|
|
|
|
{
|
2006-06-10 03:20:56 +08:00
|
|
|
netif_tx_lock_bh(dev);
|
2005-04-17 06:20:36 +08:00
|
|
|
if (del_timer(&dev->watchdog_timer))
|
2006-03-21 14:32:28 +08:00
|
|
|
dev_put(dev);
|
2006-06-10 03:20:56 +08:00
|
|
|
netif_tx_unlock_bh(dev);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
[NET]: Make NAPI polling independent of struct net_device objects.
Several devices have multiple independant RX queues per net
device, and some have a single interrupt doorbell for several
queues.
In either case, it's easier to support layouts like that if the
structure representing the poll is independant from the net
device itself.
The signature of the ->poll() call back goes from:
int foo_poll(struct net_device *dev, int *budget)
to
int foo_poll(struct napi_struct *napi, int budget)
The caller is returned the number of RX packets processed (or
the number of "NAPI credits" consumed if you want to get
abstract). The callee no longer messes around bumping
dev->quota, *budget, etc. because that is all handled in the
caller upon return.
The napi_struct is to be embedded in the device driver private data
structures.
Furthermore, it is the driver's responsibility to disable all NAPI
instances in it's ->stop() device close handler. Since the
napi_struct is privatized into the driver's private data structures,
only the driver knows how to get at all of the napi_struct instances
it may have per-device.
With lots of help and suggestions from Rusty Russell, Roland Dreier,
Michael Chan, Jeff Garzik, and Jamal Hadi Salim.
Bug fixes from Thomas Graf, Roland Dreier, Peter Zijlstra,
Joseph Fannin, Scott Wood, Hans J. Koch, and Michael Chan.
[ Ported to current tree and all drivers converted. Integrated
Stephen's follow-on kerneldoc additions, and restored poll_list
handling to the old style to fix mutual exclusion issues. -DaveM ]
Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-10-04 07:41:36 +08:00
|
|
|
/**
|
|
|
|
* netif_carrier_on - set carrier
|
|
|
|
* @dev: network device
|
|
|
|
*
|
|
|
|
* Device has detected that carrier.
|
|
|
|
*/
|
2005-08-12 06:32:53 +08:00
|
|
|
void netif_carrier_on(struct net_device *dev)
|
|
|
|
{
|
2007-10-18 14:26:43 +08:00
|
|
|
if (test_and_clear_bit(__LINK_STATE_NOCARRIER, &dev->state)) {
|
2005-08-12 06:32:53 +08:00
|
|
|
linkwatch_fire_event(dev);
|
2007-10-18 14:26:43 +08:00
|
|
|
if (netif_running(dev))
|
|
|
|
__netdev_watchdog_up(dev);
|
|
|
|
}
|
2005-08-12 06:32:53 +08:00
|
|
|
}
|
2008-01-23 14:10:23 +08:00
|
|
|
EXPORT_SYMBOL(netif_carrier_on);
|
2005-08-12 06:32:53 +08:00
|
|
|
|
[NET]: Make NAPI polling independent of struct net_device objects.
Several devices have multiple independant RX queues per net
device, and some have a single interrupt doorbell for several
queues.
In either case, it's easier to support layouts like that if the
structure representing the poll is independant from the net
device itself.
The signature of the ->poll() call back goes from:
int foo_poll(struct net_device *dev, int *budget)
to
int foo_poll(struct napi_struct *napi, int budget)
The caller is returned the number of RX packets processed (or
the number of "NAPI credits" consumed if you want to get
abstract). The callee no longer messes around bumping
dev->quota, *budget, etc. because that is all handled in the
caller upon return.
The napi_struct is to be embedded in the device driver private data
structures.
Furthermore, it is the driver's responsibility to disable all NAPI
instances in it's ->stop() device close handler. Since the
napi_struct is privatized into the driver's private data structures,
only the driver knows how to get at all of the napi_struct instances
it may have per-device.
With lots of help and suggestions from Rusty Russell, Roland Dreier,
Michael Chan, Jeff Garzik, and Jamal Hadi Salim.
Bug fixes from Thomas Graf, Roland Dreier, Peter Zijlstra,
Joseph Fannin, Scott Wood, Hans J. Koch, and Michael Chan.
[ Ported to current tree and all drivers converted. Integrated
Stephen's follow-on kerneldoc additions, and restored poll_list
handling to the old style to fix mutual exclusion issues. -DaveM ]
Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
2007-10-04 07:41:36 +08:00
|
|
|
/**
|
|
|
|
* netif_carrier_off - clear carrier
|
|
|
|
* @dev: network device
|
|
|
|
*
|
|
|
|
* Device has detected loss of carrier.
|
|
|
|
*/
|
2005-08-12 06:32:53 +08:00
|
|
|
void netif_carrier_off(struct net_device *dev)
|
|
|
|
{
|
|
|
|
if (!test_and_set_bit(__LINK_STATE_NOCARRIER, &dev->state))
|
|
|
|
linkwatch_fire_event(dev);
|
|
|
|
}
|
2008-01-23 14:10:23 +08:00
|
|
|
EXPORT_SYMBOL(netif_carrier_off);
|
2005-08-12 06:32:53 +08:00
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
/* "NOOP" scheduler: the best scheduler, recommended for all interfaces
|
|
|
|
under all circumstances. It is difficult to invent anything faster or
|
|
|
|
cheaper.
|
|
|
|
*/
|
|
|
|
|
2005-06-19 13:59:08 +08:00
|
|
|
static int noop_enqueue(struct sk_buff *skb, struct Qdisc * qdisc)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
|
|
|
kfree_skb(skb);
|
|
|
|
return NET_XMIT_CN;
|
|
|
|
}
|
|
|
|
|
2005-06-19 13:59:08 +08:00
|
|
|
static struct sk_buff *noop_dequeue(struct Qdisc * qdisc)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2005-06-19 13:59:08 +08:00
|
|
|
static int noop_requeue(struct sk_buff *skb, struct Qdisc* qdisc)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
|
|
|
if (net_ratelimit())
|
2005-06-19 13:59:08 +08:00
|
|
|
printk(KERN_DEBUG "%s deferred output. It is buggy.\n",
|
|
|
|
skb->dev->name);
|
2005-04-17 06:20:36 +08:00
|
|
|
kfree_skb(skb);
|
|
|
|
return NET_XMIT_CN;
|
|
|
|
}
|
|
|
|
|
2007-11-14 17:44:41 +08:00
|
|
|
struct Qdisc_ops noop_qdisc_ops __read_mostly = {
|
2005-04-17 06:20:36 +08:00
|
|
|
.id = "noop",
|
|
|
|
.priv_size = 0,
|
|
|
|
.enqueue = noop_enqueue,
|
|
|
|
.dequeue = noop_dequeue,
|
|
|
|
.requeue = noop_requeue,
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
};
|
|
|
|
|
|
|
|
struct Qdisc noop_qdisc = {
|
|
|
|
.enqueue = noop_enqueue,
|
|
|
|
.dequeue = noop_dequeue,
|
|
|
|
.flags = TCQ_F_BUILTIN,
|
2007-02-09 22:25:16 +08:00
|
|
|
.ops = &noop_qdisc_ops,
|
2005-04-17 06:20:36 +08:00
|
|
|
.list = LIST_HEAD_INIT(noop_qdisc.list),
|
|
|
|
};
|
2008-01-23 14:10:23 +08:00
|
|
|
EXPORT_SYMBOL(noop_qdisc);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2007-11-14 17:44:41 +08:00
|
|
|
static struct Qdisc_ops noqueue_qdisc_ops __read_mostly = {
|
2005-04-17 06:20:36 +08:00
|
|
|
.id = "noqueue",
|
|
|
|
.priv_size = 0,
|
|
|
|
.enqueue = noop_enqueue,
|
|
|
|
.dequeue = noop_dequeue,
|
|
|
|
.requeue = noop_requeue,
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
};
|
|
|
|
|
|
|
|
static struct Qdisc noqueue_qdisc = {
|
|
|
|
.enqueue = NULL,
|
|
|
|
.dequeue = noop_dequeue,
|
|
|
|
.flags = TCQ_F_BUILTIN,
|
|
|
|
.ops = &noqueue_qdisc_ops,
|
|
|
|
.list = LIST_HEAD_INIT(noqueue_qdisc.list),
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
|
|
static const u8 prio2band[TC_PRIO_MAX+1] =
|
|
|
|
{ 1, 2, 2, 2, 1, 2, 0, 0 , 1, 1, 1, 1, 1, 1, 1, 1 };
|
|
|
|
|
|
|
|
/* 3-band FIFO queue: old style, but should be a bit faster than
|
|
|
|
generic prio+fifo combination.
|
|
|
|
*/
|
|
|
|
|
2005-06-19 13:58:53 +08:00
|
|
|
#define PFIFO_FAST_BANDS 3
|
|
|
|
|
2005-06-19 13:58:35 +08:00
|
|
|
static inline struct sk_buff_head *prio2list(struct sk_buff *skb,
|
|
|
|
struct Qdisc *qdisc)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
|
|
|
struct sk_buff_head *list = qdisc_priv(qdisc);
|
2005-06-19 13:58:35 +08:00
|
|
|
return list + prio2band[skb->priority & TC_PRIO_MAX];
|
|
|
|
}
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2005-06-19 13:58:53 +08:00
|
|
|
static int pfifo_fast_enqueue(struct sk_buff *skb, struct Qdisc* qdisc)
|
2005-06-19 13:58:35 +08:00
|
|
|
{
|
|
|
|
struct sk_buff_head *list = prio2list(skb, qdisc);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2008-07-09 08:06:30 +08:00
|
|
|
if (skb_queue_len(list) < qdisc_dev(qdisc)->tx_queue_len) {
|
2005-04-17 06:20:36 +08:00
|
|
|
qdisc->q.qlen++;
|
2005-06-19 13:58:15 +08:00
|
|
|
return __qdisc_enqueue_tail(skb, qdisc, list);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
2005-06-19 13:58:15 +08:00
|
|
|
|
|
|
|
return qdisc_drop(skb, qdisc);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
2005-06-19 13:58:53 +08:00
|
|
|
static struct sk_buff *pfifo_fast_dequeue(struct Qdisc* qdisc)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
|
|
|
int prio;
|
|
|
|
struct sk_buff_head *list = qdisc_priv(qdisc);
|
|
|
|
|
2005-07-19 04:30:53 +08:00
|
|
|
for (prio = 0; prio < PFIFO_FAST_BANDS; prio++) {
|
|
|
|
if (!skb_queue_empty(list + prio)) {
|
2005-04-17 06:20:36 +08:00
|
|
|
qdisc->q.qlen--;
|
2005-07-19 04:30:53 +08:00
|
|
|
return __qdisc_dequeue_head(qdisc, list + prio);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
}
|
2005-06-19 13:58:53 +08:00
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2005-06-19 13:58:53 +08:00
|
|
|
static int pfifo_fast_requeue(struct sk_buff *skb, struct Qdisc* qdisc)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
|
|
|
qdisc->q.qlen++;
|
2005-06-19 13:58:35 +08:00
|
|
|
return __qdisc_requeue(skb, qdisc, prio2list(skb, qdisc));
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
2005-06-19 13:58:53 +08:00
|
|
|
static void pfifo_fast_reset(struct Qdisc* qdisc)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
|
|
|
int prio;
|
|
|
|
struct sk_buff_head *list = qdisc_priv(qdisc);
|
|
|
|
|
2005-06-19 13:58:53 +08:00
|
|
|
for (prio = 0; prio < PFIFO_FAST_BANDS; prio++)
|
2005-06-19 13:58:15 +08:00
|
|
|
__qdisc_reset_queue(qdisc, list + prio);
|
|
|
|
|
|
|
|
qdisc->qstats.backlog = 0;
|
2005-04-17 06:20:36 +08:00
|
|
|
qdisc->q.qlen = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int pfifo_fast_dump(struct Qdisc *qdisc, struct sk_buff *skb)
|
|
|
|
{
|
2005-06-19 13:58:53 +08:00
|
|
|
struct tc_prio_qopt opt = { .bands = PFIFO_FAST_BANDS };
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
memcpy(&opt.priomap, prio2band, TC_PRIO_MAX+1);
|
2008-01-23 14:11:17 +08:00
|
|
|
NLA_PUT(skb, TCA_OPTIONS, sizeof(opt), &opt);
|
2005-04-17 06:20:36 +08:00
|
|
|
return skb->len;
|
|
|
|
|
2008-01-23 14:11:17 +08:00
|
|
|
nla_put_failure:
|
2005-04-17 06:20:36 +08:00
|
|
|
return -1;
|
|
|
|
}
|
|
|
|
|
2008-01-23 14:11:17 +08:00
|
|
|
static int pfifo_fast_init(struct Qdisc *qdisc, struct nlattr *opt)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2005-06-19 13:58:53 +08:00
|
|
|
int prio;
|
2005-04-17 06:20:36 +08:00
|
|
|
struct sk_buff_head *list = qdisc_priv(qdisc);
|
|
|
|
|
2005-06-19 13:58:53 +08:00
|
|
|
for (prio = 0; prio < PFIFO_FAST_BANDS; prio++)
|
|
|
|
skb_queue_head_init(list + prio);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2007-11-14 17:44:41 +08:00
|
|
|
static struct Qdisc_ops pfifo_fast_ops __read_mostly = {
|
2005-04-17 06:20:36 +08:00
|
|
|
.id = "pfifo_fast",
|
2005-06-19 13:58:53 +08:00
|
|
|
.priv_size = PFIFO_FAST_BANDS * sizeof(struct sk_buff_head),
|
2005-04-17 06:20:36 +08:00
|
|
|
.enqueue = pfifo_fast_enqueue,
|
|
|
|
.dequeue = pfifo_fast_dequeue,
|
|
|
|
.requeue = pfifo_fast_requeue,
|
|
|
|
.init = pfifo_fast_init,
|
|
|
|
.reset = pfifo_fast_reset,
|
|
|
|
.dump = pfifo_fast_dump,
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
};
|
|
|
|
|
2008-07-09 08:06:30 +08:00
|
|
|
struct Qdisc *qdisc_alloc(struct netdev_queue *dev_queue,
|
2008-07-09 07:55:56 +08:00
|
|
|
struct Qdisc_ops *ops)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
|
|
|
void *p;
|
|
|
|
struct Qdisc *sch;
|
2005-07-06 05:15:09 +08:00
|
|
|
unsigned int size;
|
|
|
|
int err = -ENOBUFS;
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
/* ensure that the Qdisc and the private data are 32-byte aligned */
|
2005-07-06 05:15:09 +08:00
|
|
|
size = QDISC_ALIGN(sizeof(*sch));
|
|
|
|
size += ops->priv_size + (QDISC_ALIGNTO - 1);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2006-07-22 05:51:30 +08:00
|
|
|
p = kzalloc(size, GFP_KERNEL);
|
2005-04-17 06:20:36 +08:00
|
|
|
if (!p)
|
2005-07-06 05:15:09 +08:00
|
|
|
goto errout;
|
|
|
|
sch = (struct Qdisc *) QDISC_ALIGN((unsigned long) p);
|
|
|
|
sch->padded = (char *) sch - (char *) p;
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
INIT_LIST_HEAD(&sch->list);
|
|
|
|
skb_queue_head_init(&sch->q);
|
|
|
|
sch->ops = ops;
|
|
|
|
sch->enqueue = ops->enqueue;
|
|
|
|
sch->dequeue = ops->dequeue;
|
2008-07-09 07:55:56 +08:00
|
|
|
sch->dev_queue = dev_queue;
|
2008-07-09 08:06:30 +08:00
|
|
|
dev_hold(qdisc_dev(sch));
|
2005-04-17 06:20:36 +08:00
|
|
|
atomic_set(&sch->refcnt, 1);
|
2005-07-06 05:15:09 +08:00
|
|
|
|
|
|
|
return sch;
|
|
|
|
errout:
|
2008-06-28 10:51:35 +08:00
|
|
|
return ERR_PTR(err);
|
2005-07-06 05:15:09 +08:00
|
|
|
}
|
|
|
|
|
2008-07-09 07:55:56 +08:00
|
|
|
struct Qdisc * qdisc_create_dflt(struct net_device *dev,
|
|
|
|
struct netdev_queue *dev_queue,
|
|
|
|
struct Qdisc_ops *ops,
|
2006-11-30 09:35:18 +08:00
|
|
|
unsigned int parentid)
|
2005-07-06 05:15:09 +08:00
|
|
|
{
|
|
|
|
struct Qdisc *sch;
|
2007-02-09 22:25:16 +08:00
|
|
|
|
2008-07-09 08:06:30 +08:00
|
|
|
sch = qdisc_alloc(dev_queue, ops);
|
2005-07-06 05:15:09 +08:00
|
|
|
if (IS_ERR(sch))
|
|
|
|
goto errout;
|
2006-11-30 09:35:18 +08:00
|
|
|
sch->parent = parentid;
|
2005-07-06 05:15:09 +08:00
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
if (!ops->init || ops->init(sch, NULL) == 0)
|
|
|
|
return sch;
|
|
|
|
|
2005-08-24 01:12:44 +08:00
|
|
|
qdisc_destroy(sch);
|
2005-07-06 05:15:09 +08:00
|
|
|
errout:
|
2005-04-17 06:20:36 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
2008-01-23 14:10:23 +08:00
|
|
|
EXPORT_SYMBOL(qdisc_create_dflt);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2008-07-09 08:18:23 +08:00
|
|
|
/* Under queue->lock and BH! */
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
void qdisc_reset(struct Qdisc *qdisc)
|
|
|
|
{
|
2007-11-14 17:44:41 +08:00
|
|
|
const struct Qdisc_ops *ops = qdisc->ops;
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
if (ops->reset)
|
|
|
|
ops->reset(qdisc);
|
|
|
|
}
|
2008-01-23 14:10:23 +08:00
|
|
|
EXPORT_SYMBOL(qdisc_reset);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2007-02-09 22:25:16 +08:00
|
|
|
/* this is the rcu callback function to clean up a qdisc when there
|
2005-04-17 06:20:36 +08:00
|
|
|
* are no further references to it */
|
|
|
|
|
|
|
|
static void __qdisc_destroy(struct rcu_head *head)
|
|
|
|
{
|
|
|
|
struct Qdisc *qdisc = container_of(head, struct Qdisc, q_rcu);
|
|
|
|
kfree((char *) qdisc - qdisc->padded);
|
|
|
|
}
|
|
|
|
|
2008-07-09 08:18:23 +08:00
|
|
|
/* Under queue->lock and BH! */
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
void qdisc_destroy(struct Qdisc *qdisc)
|
|
|
|
{
|
2007-11-14 17:44:41 +08:00
|
|
|
const struct Qdisc_ops *ops = qdisc->ops;
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
if (qdisc->flags & TCQ_F_BUILTIN ||
|
[NET_SCHED]: Fix fallout from dev->qdisc RCU change
The move of qdisc destruction to a rcu callback broke locking in the
entire qdisc layer by invalidating previously valid assumptions about
the context in which changes to the qdisc tree occur.
The two assumptions were:
- since changes only happen in process context, read_lock doesn't need
bottem half protection. Now invalid since destruction of inner qdiscs,
classifiers, actions and estimators happens in the RCU callback unless
they're manually deleted, resulting in dead-locks when read_lock in
process context is interrupted by write_lock_bh in bottem half context.
- since changes only happen under the RTNL, no additional locking is
necessary for data not used during packet processing (f.e. u32_list).
Again, since destruction now happens in the RCU callback, this assumption
is not valid anymore, causing races while using this data, which can
result in corruption or use-after-free.
Instead of "fixing" this by disabling bottem halfs everywhere and adding
new locks/refcounting, this patch makes these assumptions valid again by
moving destruction back to process context. Since only the dev->qdisc
pointer is protected by RCU, but ->enqueue and the qdisc tree are still
protected by dev->qdisc_lock, destruction of the tree can be performed
immediately and only the final free needs to happen in the rcu callback
to make sure dev_queue_xmit doesn't access already freed memory.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
2006-09-28 07:45:45 +08:00
|
|
|
!atomic_dec_and_test(&qdisc->refcnt))
|
2005-04-17 06:20:36 +08:00
|
|
|
return;
|
|
|
|
|
[NET_SCHED]: Fix fallout from dev->qdisc RCU change
The move of qdisc destruction to a rcu callback broke locking in the
entire qdisc layer by invalidating previously valid assumptions about
the context in which changes to the qdisc tree occur.
The two assumptions were:
- since changes only happen in process context, read_lock doesn't need
bottem half protection. Now invalid since destruction of inner qdiscs,
classifiers, actions and estimators happens in the RCU callback unless
they're manually deleted, resulting in dead-locks when read_lock in
process context is interrupted by write_lock_bh in bottem half context.
- since changes only happen under the RTNL, no additional locking is
necessary for data not used during packet processing (f.e. u32_list).
Again, since destruction now happens in the RCU callback, this assumption
is not valid anymore, causing races while using this data, which can
result in corruption or use-after-free.
Instead of "fixing" this by disabling bottem halfs everywhere and adding
new locks/refcounting, this patch makes these assumptions valid again by
moving destruction back to process context. Since only the dev->qdisc
pointer is protected by RCU, but ->enqueue and the qdisc tree are still
protected by dev->qdisc_lock, destruction of the tree can be performed
immediately and only the final free needs to happen in the rcu callback
to make sure dev_queue_xmit doesn't access already freed memory.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
2006-09-28 07:45:45 +08:00
|
|
|
list_del(&qdisc->list);
|
|
|
|
gen_kill_estimator(&qdisc->bstats, &qdisc->rate_est);
|
|
|
|
if (ops->reset)
|
|
|
|
ops->reset(qdisc);
|
|
|
|
if (ops->destroy)
|
|
|
|
ops->destroy(qdisc);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
[NET_SCHED]: Fix fallout from dev->qdisc RCU change
The move of qdisc destruction to a rcu callback broke locking in the
entire qdisc layer by invalidating previously valid assumptions about
the context in which changes to the qdisc tree occur.
The two assumptions were:
- since changes only happen in process context, read_lock doesn't need
bottem half protection. Now invalid since destruction of inner qdiscs,
classifiers, actions and estimators happens in the RCU callback unless
they're manually deleted, resulting in dead-locks when read_lock in
process context is interrupted by write_lock_bh in bottem half context.
- since changes only happen under the RTNL, no additional locking is
necessary for data not used during packet processing (f.e. u32_list).
Again, since destruction now happens in the RCU callback, this assumption
is not valid anymore, causing races while using this data, which can
result in corruption or use-after-free.
Instead of "fixing" this by disabling bottem halfs everywhere and adding
new locks/refcounting, this patch makes these assumptions valid again by
moving destruction back to process context. Since only the dev->qdisc
pointer is protected by RCU, but ->enqueue and the qdisc tree are still
protected by dev->qdisc_lock, destruction of the tree can be performed
immediately and only the final free needs to happen in the rcu callback
to make sure dev_queue_xmit doesn't access already freed memory.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
2006-09-28 07:45:45 +08:00
|
|
|
module_put(ops->owner);
|
2008-07-09 08:06:30 +08:00
|
|
|
dev_put(qdisc_dev(qdisc));
|
2005-04-17 06:20:36 +08:00
|
|
|
call_rcu(&qdisc->q_rcu, __qdisc_destroy);
|
|
|
|
}
|
2008-01-23 14:10:23 +08:00
|
|
|
EXPORT_SYMBOL(qdisc_destroy);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2008-07-17 15:34:19 +08:00
|
|
|
static bool dev_all_qdisc_sleeping_noop(struct net_device *dev)
|
|
|
|
{
|
|
|
|
unsigned int i;
|
|
|
|
|
|
|
|
for (i = 0; i < dev->num_tx_queues; i++) {
|
|
|
|
struct netdev_queue *txq = netdev_get_tx_queue(dev, i);
|
|
|
|
|
|
|
|
if (txq->qdisc_sleeping != &noop_qdisc)
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void attach_one_default_qdisc(struct net_device *dev,
|
|
|
|
struct netdev_queue *dev_queue,
|
|
|
|
void *_unused)
|
|
|
|
{
|
|
|
|
struct Qdisc *qdisc;
|
|
|
|
|
|
|
|
if (dev->tx_queue_len) {
|
|
|
|
qdisc = qdisc_create_dflt(dev, dev_queue,
|
|
|
|
&pfifo_fast_ops, TC_H_ROOT);
|
|
|
|
if (!qdisc) {
|
|
|
|
printk(KERN_INFO "%s: activation failed\n", dev->name);
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
list_add_tail(&qdisc->list, &dev_queue->qdisc_list);
|
|
|
|
} else {
|
|
|
|
qdisc = &noqueue_qdisc;
|
|
|
|
}
|
|
|
|
dev_queue->qdisc_sleeping = qdisc;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void transition_one_qdisc(struct net_device *dev,
|
|
|
|
struct netdev_queue *dev_queue,
|
|
|
|
void *_need_watchdog)
|
|
|
|
{
|
|
|
|
int *need_watchdog_p = _need_watchdog;
|
|
|
|
|
|
|
|
spin_lock_bh(&dev_queue->lock);
|
|
|
|
rcu_assign_pointer(dev_queue->qdisc, dev_queue->qdisc_sleeping);
|
|
|
|
if (dev_queue->qdisc != &noqueue_qdisc)
|
|
|
|
*need_watchdog_p = 1;
|
|
|
|
spin_unlock_bh(&dev_queue->lock);
|
|
|
|
}
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
void dev_activate(struct net_device *dev)
|
|
|
|
{
|
2008-07-17 15:34:19 +08:00
|
|
|
int need_watchdog;
|
2008-07-09 08:42:10 +08:00
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
/* No queueing discipline is attached to device;
|
|
|
|
create default one i.e. pfifo_fast for devices,
|
|
|
|
which need queueing and noqueue_qdisc for
|
|
|
|
virtual interfaces
|
|
|
|
*/
|
|
|
|
|
2008-07-17 15:34:19 +08:00
|
|
|
if (dev_all_qdisc_sleeping_noop(dev))
|
|
|
|
netdev_for_each_tx_queue(dev, attach_one_default_qdisc, NULL);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2005-05-04 07:18:52 +08:00
|
|
|
if (!netif_carrier_ok(dev))
|
|
|
|
/* Delay activation until next carrier-on event */
|
|
|
|
return;
|
|
|
|
|
2008-07-17 15:34:19 +08:00
|
|
|
need_watchdog = 0;
|
|
|
|
netdev_for_each_tx_queue(dev, transition_one_qdisc, &need_watchdog);
|
|
|
|
|
|
|
|
if (need_watchdog) {
|
2005-04-17 06:20:36 +08:00
|
|
|
dev->trans_start = jiffies;
|
|
|
|
dev_watchdog_up(dev);
|
|
|
|
}
|
2008-07-09 08:42:10 +08:00
|
|
|
}
|
|
|
|
|
2008-07-17 15:34:19 +08:00
|
|
|
static void dev_deactivate_queue(struct net_device *dev,
|
|
|
|
struct netdev_queue *dev_queue,
|
|
|
|
void *_qdisc_default)
|
2008-07-09 08:42:10 +08:00
|
|
|
{
|
2008-07-17 15:34:19 +08:00
|
|
|
struct Qdisc *qdisc_default = _qdisc_default;
|
2008-07-09 14:10:33 +08:00
|
|
|
struct Qdisc *qdisc;
|
|
|
|
struct sk_buff *skb;
|
|
|
|
|
|
|
|
spin_lock_bh(&dev_queue->lock);
|
2008-07-09 08:42:10 +08:00
|
|
|
|
2008-07-09 14:10:33 +08:00
|
|
|
qdisc = dev_queue->qdisc;
|
2008-07-09 08:42:10 +08:00
|
|
|
if (qdisc) {
|
|
|
|
dev_queue->qdisc = qdisc_default;
|
|
|
|
qdisc_reset(qdisc);
|
|
|
|
}
|
2008-07-09 14:10:33 +08:00
|
|
|
skb = dev_queue->gso_skb;
|
|
|
|
dev_queue->gso_skb = NULL;
|
|
|
|
|
|
|
|
spin_unlock_bh(&dev_queue->lock);
|
|
|
|
|
|
|
|
kfree_skb(skb);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
2008-07-17 15:34:19 +08:00
|
|
|
static bool some_qdisc_is_running(struct net_device *dev, int lock)
|
|
|
|
{
|
|
|
|
unsigned int i;
|
|
|
|
|
|
|
|
for (i = 0; i < dev->num_tx_queues; i++) {
|
|
|
|
struct netdev_queue *dev_queue;
|
|
|
|
int val;
|
|
|
|
|
|
|
|
dev_queue = netdev_get_tx_queue(dev, i);
|
|
|
|
|
|
|
|
if (lock)
|
|
|
|
spin_lock_bh(&dev_queue->lock);
|
|
|
|
|
|
|
|
val = test_bit(__QUEUE_STATE_QDISC_RUNNING, &dev_queue->state);
|
|
|
|
|
|
|
|
if (lock)
|
|
|
|
spin_unlock_bh(&dev_queue->lock);
|
|
|
|
|
|
|
|
if (val)
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
void dev_deactivate(struct net_device *dev)
|
|
|
|
{
|
2008-07-17 15:34:19 +08:00
|
|
|
bool running;
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2008-07-17 15:34:19 +08:00
|
|
|
netdev_for_each_tx_queue(dev, dev_deactivate_queue, &noop_qdisc);
|
2007-05-11 05:12:47 +08:00
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
dev_watchdog_down(dev);
|
|
|
|
|
2007-10-19 13:37:58 +08:00
|
|
|
/* Wait for outstanding qdisc-less dev_queue_xmit calls. */
|
2006-06-22 17:28:18 +08:00
|
|
|
synchronize_rcu();
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2006-06-22 17:28:18 +08:00
|
|
|
/* Wait for outstanding qdisc_run calls. */
|
2007-10-19 13:37:58 +08:00
|
|
|
do {
|
2008-07-17 15:34:19 +08:00
|
|
|
while (some_qdisc_is_running(dev, 0))
|
2007-10-19 13:37:58 +08:00
|
|
|
yield();
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Double-check inside queue lock to ensure that all effects
|
|
|
|
* of the queue run are visible when we return.
|
|
|
|
*/
|
2008-07-17 15:34:19 +08:00
|
|
|
running = some_qdisc_is_running(dev, 1);
|
2007-10-19 13:37:58 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* The running flag should never be set at this point because
|
|
|
|
* we've already set dev->qdisc to noop_qdisc *inside* the same
|
|
|
|
* pair of spin locks. That is, if any qdisc_run starts after
|
|
|
|
* our initial test it should see the noop_qdisc and then
|
|
|
|
* clear the RUNNING bit before dropping the queue lock. So
|
|
|
|
* if it is set here then we've found a bug.
|
|
|
|
*/
|
|
|
|
} while (WARN_ON_ONCE(running));
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
2008-07-09 08:42:10 +08:00
|
|
|
static void dev_init_scheduler_queue(struct net_device *dev,
|
|
|
|
struct netdev_queue *dev_queue,
|
2008-07-17 15:34:19 +08:00
|
|
|
void *_qdisc)
|
2008-07-09 08:42:10 +08:00
|
|
|
{
|
2008-07-17 15:34:19 +08:00
|
|
|
struct Qdisc *qdisc = _qdisc;
|
|
|
|
|
2008-07-09 08:42:10 +08:00
|
|
|
dev_queue->qdisc = qdisc;
|
|
|
|
dev_queue->qdisc_sleeping = qdisc;
|
|
|
|
INIT_LIST_HEAD(&dev_queue->qdisc_list);
|
|
|
|
}
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
void dev_init_scheduler(struct net_device *dev)
|
|
|
|
{
|
|
|
|
qdisc_lock_tree(dev);
|
2008-07-17 15:34:19 +08:00
|
|
|
netdev_for_each_tx_queue(dev, dev_init_scheduler_queue, &noop_qdisc);
|
2008-07-09 08:42:10 +08:00
|
|
|
dev_init_scheduler_queue(dev, &dev->rx_queue, NULL);
|
2005-04-17 06:20:36 +08:00
|
|
|
qdisc_unlock_tree(dev);
|
|
|
|
|
2008-01-24 13:20:07 +08:00
|
|
|
setup_timer(&dev->watchdog_timer, dev_watchdog, (unsigned long)dev);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
2008-07-17 15:34:19 +08:00
|
|
|
static void shutdown_scheduler_queue(struct net_device *dev,
|
|
|
|
struct netdev_queue *dev_queue,
|
|
|
|
void *_qdisc_default)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2008-07-09 08:42:10 +08:00
|
|
|
struct Qdisc *qdisc = dev_queue->qdisc_sleeping;
|
2008-07-17 15:34:19 +08:00
|
|
|
struct Qdisc *qdisc_default = _qdisc_default;
|
2008-07-09 08:42:10 +08:00
|
|
|
|
|
|
|
if (qdisc) {
|
|
|
|
dev_queue->qdisc = qdisc_default;
|
|
|
|
dev_queue->qdisc_sleeping = qdisc_default;
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
qdisc_destroy(qdisc);
|
2007-02-09 22:25:16 +08:00
|
|
|
}
|
2008-07-09 08:42:10 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
void dev_shutdown(struct net_device *dev)
|
|
|
|
{
|
|
|
|
qdisc_lock_tree(dev);
|
2008-07-17 15:34:19 +08:00
|
|
|
netdev_for_each_tx_queue(dev, shutdown_scheduler_queue, &noop_qdisc);
|
|
|
|
shutdown_scheduler_queue(dev, &dev->rx_queue, NULL);
|
2005-04-17 06:20:36 +08:00
|
|
|
BUG_TRAP(!timer_pending(&dev->watchdog_timer));
|
|
|
|
qdisc_unlock_tree(dev);
|
|
|
|
}
|