linux.git/net/core/rtnetlink.c, branch v2.6.22

[RTNETLINK]: ifindex 0 does not exist

2007-06-07T20:40:11+00:00

ifindex == 0 does not exist and implies we should do a lookup by name if
one was given.

Signed-off-by: Patrick McHardy 
Signed-off-by: David S. Miller

[NETLINK]: Mark netlink policies const

2007-06-07T20:40:10+00:00

Signed-off-by: Patrick McHardy 
Signed-off-by: David S. Miller

[RTNETLINK]: Remove remains of wireless extensions over rtnetlink

2007-05-23T00:00:49+00:00

Remove some unused variables and function arguments related to the
recently removed wireless extensions over rtnetlink.

Signed-off-by: Patrick McHardy 
Acked-by: Johannes Berg 
Signed-off-by: David S. Miller

[RTNETLINK]: Allow changing of subsets of netdevice flags in rtnl_setlink

2007-05-23T00:00:01+00:00

rtnl_setlink doesn't allow to change subsets of the flags, just to override
the set entirely by a new one. This means that for simply setting a device
up or down userspace first needs to query the current flags, change it and
send the changed flags back, which is racy and needlessly complicated.

Mask the flags using ifi_change since this is what it is intended for.
For backwards compatibility treat ifi_change == 0 as ~0 (even though it
seems quite unlikely that anyone has been using this so far).

Signed-off-by: Patrick McHardy 
Signed-off-by: David S. Miller

[NET]: Rework dev_base via list_head (v3)

2007-05-03T22:13:45+00:00

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 
Acked-by: Kirill Korotaev 
Signed-off-by: David S. Miller

[NETLINK]: Possible cleanups.

2007-04-26T07:57:41+00:00

- make the following needlessly global variables static:
  - core/rtnetlink.c: struct rtnl_msg_handlers[]
  - netfilter/nf_conntrack_proto.c: struct nf_ct_protos[]
- make the following needlessly global functions static:
  - core/rtnetlink.c: rtnl_dump_all()
  - netlink/af_netlink.c: netlink_queue_skip()

Signed-off-by: Adrian Bunk 
Signed-off-by: Andrew Morton 
Signed-off-by: David S. Miller

[WIRELESS]: Remove wext over netlink.

2007-04-26T05:29:42+00:00

As scheduled, this patch removes the pointless wext over netlink code.

Signed-off-by: Johannes Berg 
Signed-off-by: John W. Linville 
Signed-off-by: David S. Miller

[NET]: cleanup extra semicolons

2007-04-26T05:29:24+00:00

Spring cleaning time...

There seems to be a lot of places in the network code that have
extra bogus semicolons after conditionals.  Most commonly is a
bogus semicolon after: switch() { }

Signed-off-by: Stephen Hemminger 
Signed-off-by: David S. Miller

[RTNETLINK]: Remove unnecessary locking in dump callbacks

2007-04-26T05:29:05+00:00

Since we're now holding the rtnl during the entire dump operation, we can
remove additional locking for rtnl protected data. This patch does that
for all simple cases (dev_base_lock for dev_base walking, RCU protection
for FIB rule dumping).

Signed-off-by: Patrick McHardy 
Signed-off-by: David S. Miller

[RTNETLINK]: Hold rtnl_mutex during netlink dump callbacks

2007-04-26T05:29:04+00:00

Hold rtnl_mutex during the entire netlink dump operation. This allows
to simplify locking in the dump callbacks, since they can now rely on
that no concurrent changes happen.

Signed-off-by: Patrick McHardy 
Signed-off-by: David S. Miller