diff options
| author | Jarek Poplawski <jarkao2@gmail.com> | 2008-08-27 11:22:07 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-08-27 11:22:07 +0200 |
| commit | f7a54c13c7b072d9426bd5cec1cdb8306df5ef55 (patch) | |
| tree | 12134183d09f0826e0488fd281e6c0b830270d37 /net/sched/sch_generic.c | |
| parent | pkt_sched: Fix dev_graft_qdisc() locking (diff) | |
| download | linux-f7a54c13c7b072d9426bd5cec1cdb8306df5ef55.tar.xz linux-f7a54c13c7b072d9426bd5cec1cdb8306df5ef55.zip | |
pkt_sched: Use rcu_assign_pointer() to change dev_queue->qdisc
These pointers are RCU protected, so proper primitives should be used.
Signed-off-by: Jarek Poplawski <jarkao2@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/sch_generic.c')
| -rw-r--r-- | net/sched/sch_generic.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/sched/sch_generic.c b/net/sched/sch_generic.c index 5f0ade7806a7..9634091ee2f0 100644 --- a/net/sched/sch_generic.c +++ b/net/sched/sch_generic.c @@ -634,7 +634,7 @@ static void dev_deactivate_queue(struct net_device *dev, if (!(qdisc->flags & TCQ_F_BUILTIN)) set_bit(__QDISC_STATE_DEACTIVATED, &qdisc->state); - dev_queue->qdisc = qdisc_default; + rcu_assign_pointer(dev_queue->qdisc, qdisc_default); qdisc_reset(qdisc); spin_unlock_bh(qdisc_lock(qdisc)); @@ -709,7 +709,7 @@ static void shutdown_scheduler_queue(struct net_device *dev, struct Qdisc *qdisc_default = _qdisc_default; if (qdisc) { - dev_queue->qdisc = qdisc_default; + rcu_assign_pointer(dev_queue->qdisc, qdisc_default); dev_queue->qdisc_sleeping = qdisc_default; qdisc_destroy(qdisc); |