diff options
author | Pavel Emelianov <xemul@openvz.org> | 2007-05-03 18:13:45 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2007-05-03 18:13:45 -0400 |
commit | 7562f876cd93800f2f8c89445f2a563590b24e09 (patch) | |
tree | 78a34c011af275efa0d55ba59c3bd49b771dd533 /net/sched | |
parent | 03fba0479600114f32d29eee74ca3eaa364606bf (diff) |
[NET]: Rework dev_base via list_head (v3)
Cleanup of dev_base list use, with the aim to simplify making device
list per-namespace. In almost every occasion, use of dev_base variable
and dev->next pointer could be easily replaced by for_each_netdev
loop. A few most complicated places were converted to using
first_netdev()/next_netdev().
Signed-off-by: Pavel Emelianov <xemul@openvz.org>
Acked-by: Kirill Korotaev <dev@openvz.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched')
-rw-r--r-- | net/sched/sch_api.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/net/sched/sch_api.c b/net/sched/sch_api.c index 8699e7006d80..bec600af03ca 100644 --- a/net/sched/sch_api.c +++ b/net/sched/sch_api.c | |||
@@ -894,9 +894,10 @@ static int tc_dump_qdisc(struct sk_buff *skb, struct netlink_callback *cb) | |||
894 | s_idx = cb->args[0]; | 894 | s_idx = cb->args[0]; |
895 | s_q_idx = q_idx = cb->args[1]; | 895 | s_q_idx = q_idx = cb->args[1]; |
896 | read_lock(&dev_base_lock); | 896 | read_lock(&dev_base_lock); |
897 | for (dev=dev_base, idx=0; dev; dev = dev->next, idx++) { | 897 | idx = 0; |
898 | for_each_netdev(dev) { | ||
898 | if (idx < s_idx) | 899 | if (idx < s_idx) |
899 | continue; | 900 | goto cont; |
900 | if (idx > s_idx) | 901 | if (idx > s_idx) |
901 | s_q_idx = 0; | 902 | s_q_idx = 0; |
902 | q_idx = 0; | 903 | q_idx = 0; |
@@ -910,6 +911,8 @@ static int tc_dump_qdisc(struct sk_buff *skb, struct netlink_callback *cb) | |||
910 | goto done; | 911 | goto done; |
911 | q_idx++; | 912 | q_idx++; |
912 | } | 913 | } |
914 | cont: | ||
915 | idx++; | ||
913 | } | 916 | } |
914 | 917 | ||
915 | done: | 918 | done: |