diff options
Diffstat (limited to 'net/sched/sch_htb.c')
-rw-r--r-- | net/sched/sch_htb.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/net/sched/sch_htb.c b/net/sched/sch_htb.c index c8ca54cc26b0..ee48457eaa4a 100644 --- a/net/sched/sch_htb.c +++ b/net/sched/sch_htb.c | |||
@@ -1039,11 +1039,12 @@ static int htb_init(struct Qdisc *sch, struct nlattr *opt) | |||
1039 | 1039 | ||
1040 | static int htb_dump(struct Qdisc *sch, struct sk_buff *skb) | 1040 | static int htb_dump(struct Qdisc *sch, struct sk_buff *skb) |
1041 | { | 1041 | { |
1042 | spinlock_t *root_lock = qdisc_root_lock(sch); | ||
1042 | struct htb_sched *q = qdisc_priv(sch); | 1043 | struct htb_sched *q = qdisc_priv(sch); |
1043 | struct nlattr *nest; | 1044 | struct nlattr *nest; |
1044 | struct tc_htb_glob gopt; | 1045 | struct tc_htb_glob gopt; |
1045 | 1046 | ||
1046 | spin_lock_bh(&sch->dev_queue->lock); | 1047 | spin_lock_bh(root_lock); |
1047 | 1048 | ||
1048 | gopt.direct_pkts = q->direct_pkts; | 1049 | gopt.direct_pkts = q->direct_pkts; |
1049 | gopt.version = HTB_VER; | 1050 | gopt.version = HTB_VER; |
@@ -1057,11 +1058,11 @@ static int htb_dump(struct Qdisc *sch, struct sk_buff *skb) | |||
1057 | NLA_PUT(skb, TCA_HTB_INIT, sizeof(gopt), &gopt); | 1058 | NLA_PUT(skb, TCA_HTB_INIT, sizeof(gopt), &gopt); |
1058 | nla_nest_end(skb, nest); | 1059 | nla_nest_end(skb, nest); |
1059 | 1060 | ||
1060 | spin_unlock_bh(&sch->dev_queue->lock); | 1061 | spin_unlock_bh(root_lock); |
1061 | return skb->len; | 1062 | return skb->len; |
1062 | 1063 | ||
1063 | nla_put_failure: | 1064 | nla_put_failure: |
1064 | spin_unlock_bh(&sch->dev_queue->lock); | 1065 | spin_unlock_bh(root_lock); |
1065 | nla_nest_cancel(skb, nest); | 1066 | nla_nest_cancel(skb, nest); |
1066 | return -1; | 1067 | return -1; |
1067 | } | 1068 | } |
@@ -1070,10 +1071,11 @@ static int htb_dump_class(struct Qdisc *sch, unsigned long arg, | |||
1070 | struct sk_buff *skb, struct tcmsg *tcm) | 1071 | struct sk_buff *skb, struct tcmsg *tcm) |
1071 | { | 1072 | { |
1072 | struct htb_class *cl = (struct htb_class *)arg; | 1073 | struct htb_class *cl = (struct htb_class *)arg; |
1074 | spinlock_t *root_lock = qdisc_root_lock(sch); | ||
1073 | struct nlattr *nest; | 1075 | struct nlattr *nest; |
1074 | struct tc_htb_opt opt; | 1076 | struct tc_htb_opt opt; |
1075 | 1077 | ||
1076 | spin_lock_bh(&sch->dev_queue->lock); | 1078 | spin_lock_bh(root_lock); |
1077 | tcm->tcm_parent = cl->parent ? cl->parent->common.classid : TC_H_ROOT; | 1079 | tcm->tcm_parent = cl->parent ? cl->parent->common.classid : TC_H_ROOT; |
1078 | tcm->tcm_handle = cl->common.classid; | 1080 | tcm->tcm_handle = cl->common.classid; |
1079 | if (!cl->level && cl->un.leaf.q) | 1081 | if (!cl->level && cl->un.leaf.q) |
@@ -1095,11 +1097,11 @@ static int htb_dump_class(struct Qdisc *sch, unsigned long arg, | |||
1095 | NLA_PUT(skb, TCA_HTB_PARMS, sizeof(opt), &opt); | 1097 | NLA_PUT(skb, TCA_HTB_PARMS, sizeof(opt), &opt); |
1096 | 1098 | ||
1097 | nla_nest_end(skb, nest); | 1099 | nla_nest_end(skb, nest); |
1098 | spin_unlock_bh(&sch->dev_queue->lock); | 1100 | spin_unlock_bh(root_lock); |
1099 | return skb->len; | 1101 | return skb->len; |
1100 | 1102 | ||
1101 | nla_put_failure: | 1103 | nla_put_failure: |
1102 | spin_unlock_bh(&sch->dev_queue->lock); | 1104 | spin_unlock_bh(root_lock); |
1103 | nla_nest_cancel(skb, nest); | 1105 | nla_nest_cancel(skb, nest); |
1104 | return -1; | 1106 | return -1; |
1105 | } | 1107 | } |
@@ -1365,7 +1367,7 @@ static int htb_change_class(struct Qdisc *sch, u32 classid, | |||
1365 | goto failure; | 1367 | goto failure; |
1366 | 1368 | ||
1367 | gen_new_estimator(&cl->bstats, &cl->rate_est, | 1369 | gen_new_estimator(&cl->bstats, &cl->rate_est, |
1368 | &sch->dev_queue->lock, | 1370 | qdisc_root_lock(sch), |
1369 | tca[TCA_RATE] ? : &est.nla); | 1371 | tca[TCA_RATE] ? : &est.nla); |
1370 | cl->refcnt = 1; | 1372 | cl->refcnt = 1; |
1371 | cl->children = 0; | 1373 | cl->children = 0; |
@@ -1420,7 +1422,7 @@ static int htb_change_class(struct Qdisc *sch, u32 classid, | |||
1420 | } else { | 1422 | } else { |
1421 | if (tca[TCA_RATE]) | 1423 | if (tca[TCA_RATE]) |
1422 | gen_replace_estimator(&cl->bstats, &cl->rate_est, | 1424 | gen_replace_estimator(&cl->bstats, &cl->rate_est, |
1423 | &sch->dev_queue->lock, | 1425 | qdisc_root_lock(sch), |
1424 | tca[TCA_RATE]); | 1426 | tca[TCA_RATE]); |
1425 | sch_tree_lock(sch); | 1427 | sch_tree_lock(sch); |
1426 | } | 1428 | } |