diff options
| author | Jarek Poplawski <jarkao2@gmail.com> | 2009-09-15 05:53:07 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-09-15 05:53:07 -0400 |
| commit | 926e61b7c44db83013159ac2f74bccd451607b5a (patch) | |
| tree | 512b532e22d4374948e0d149902304edfbef7e25 /include/net | |
| parent | ca519274d537706b6fb1e3e91238d34a23320584 (diff) | |
pkt_sched: Fix tx queue selection in tc_modify_qdisc
After the recent mq change there is the new select_queue qdisc class
method used in tc_modify_qdisc, but it works OK only for direct child
qdiscs of mq qdisc. Grandchildren always get the first tx queue, which
would give wrong qdisc_root etc. results (e.g. for sch_htb as child of
sch_prio). This patch fixes it by using parent's dev_queue for such
grandchildren qdiscs. The select_queue method's return type is changed
BTW.
With feedback from: Patrick McHardy <kaber@trash.net>
Signed-off-by: Jarek Poplawski <jarkao2@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net')
| -rw-r--r-- | include/net/sch_generic.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/net/sch_generic.h b/include/net/sch_generic.h index 88eb9de095de..c33180dd42b4 100644 --- a/include/net/sch_generic.h +++ b/include/net/sch_generic.h | |||
| @@ -81,7 +81,7 @@ struct Qdisc | |||
| 81 | struct Qdisc_class_ops | 81 | struct Qdisc_class_ops |
| 82 | { | 82 | { |
| 83 | /* Child qdisc manipulation */ | 83 | /* Child qdisc manipulation */ |
| 84 | unsigned int (*select_queue)(struct Qdisc *, struct tcmsg *); | 84 | struct netdev_queue * (*select_queue)(struct Qdisc *, struct tcmsg *); |
| 85 | int (*graft)(struct Qdisc *, unsigned long cl, | 85 | int (*graft)(struct Qdisc *, unsigned long cl, |
| 86 | struct Qdisc *, struct Qdisc **); | 86 | struct Qdisc *, struct Qdisc **); |
| 87 | struct Qdisc * (*leaf)(struct Qdisc *, unsigned long cl); | 87 | struct Qdisc * (*leaf)(struct Qdisc *, unsigned long cl); |
