summaryrefslogtreecommitdiff
path: root/net
diff options
context:
space:
mode:
authorDavid S. Miller <davem@davemloft.net>2008-07-16 07:14:35 +0400
committerDavid S. Miller <davem@davemloft.net>2008-07-18 06:21:18 +0400
commitd3b753db7c4f1f37a98b51974d484fda5d86dab5 (patch)
tree6596288854d9626f935ddf9c014471a4c38b5c74 /net
parentb4c21639ab0f6df07ab7624a8c2f974936708ae5 (diff)
downloadlinux-d3b753db7c4f1f37a98b51974d484fda5d86dab5.tar.xz
pkt_sched: Move gso_skb into Qdisc.
We liberate any dangling gso_skb during qdisc destruction. It really only matters for the root qdisc. But when qdiscs can be shared by multiple netdev_queue objects, we can't have the gso_skb in the netdev_queue any more. Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/sched/sch_generic.c19
1 files changed, 9 insertions, 10 deletions
diff --git a/net/sched/sch_generic.c b/net/sched/sch_generic.c
index 2f575b9017d1..2bd75befa066 100644
--- a/net/sched/sch_generic.c
+++ b/net/sched/sch_generic.c
@@ -77,7 +77,7 @@ static inline int dev_requeue_skb(struct sk_buff *skb,
struct Qdisc *q)
{
if (unlikely(skb->next))
- dev_queue->gso_skb = skb;
+ q->gso_skb = skb;
else
q->ops->requeue(skb, q);
@@ -85,13 +85,12 @@ static inline int dev_requeue_skb(struct sk_buff *skb,
return 0;
}
-static inline struct sk_buff *dequeue_skb(struct netdev_queue *dev_queue,
- struct Qdisc *q)
+static inline struct sk_buff *dequeue_skb(struct Qdisc *q)
{
struct sk_buff *skb;
- if ((skb = dev_queue->gso_skb))
- dev_queue->gso_skb = NULL;
+ if ((skb = q->gso_skb))
+ q->gso_skb = NULL;
else
skb = q->dequeue(q);
@@ -155,10 +154,9 @@ static inline int qdisc_restart(struct netdev_queue *txq)
struct sk_buff *skb;
/* Dequeue packet */
- if (unlikely((skb = dequeue_skb(txq, q)) == NULL))
+ if (unlikely((skb = dequeue_skb(q)) == NULL))
return 0;
-
/* And release queue */
spin_unlock(&txq->lock);
@@ -643,8 +641,8 @@ static void dev_deactivate_queue(struct net_device *dev,
void *_qdisc_default)
{
struct Qdisc *qdisc_default = _qdisc_default;
+ struct sk_buff *skb = NULL;
struct Qdisc *qdisc;
- struct sk_buff *skb;
spin_lock_bh(&dev_queue->lock);
@@ -652,9 +650,10 @@ static void dev_deactivate_queue(struct net_device *dev,
if (qdisc) {
dev_queue->qdisc = qdisc_default;
qdisc_reset(qdisc);
+
+ skb = qdisc->gso_skb;
+ qdisc->gso_skb = NULL;
}
- skb = dev_queue->gso_skb;
- dev_queue->gso_skb = NULL;
spin_unlock_bh(&dev_queue->lock);