diff options
| author | Simon Arlott <simon@fire.lp0.eu> | 2010-05-10 13:31:11 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-05-16 10:10:15 +0400 |
| commit | e0f43752a942b7be1bc06b9fd74e20ae337c1cca (patch) | |
| tree | 892d0fbc2ecae39f2e5c4a7b711cbb988c8dc3f1 /net/bridge/br_if.c | |
| parent | 28a16c97963d3bc36a2c192859f6d8025ef2967a (diff) | |
| download | linux-e0f43752a942b7be1bc06b9fd74e20ae337c1cca.tar.xz | |
bridge: update sysfs link names if port device names have changed
Links for each port are created in sysfs using the device
name, but this could be changed after being added to the
bridge.
As well as being unable to remove interfaces after this
occurs (because userspace tools don't recognise the new
name, and the kernel won't recognise the old name), adding
another interface with the old name to the bridge will
cause an error trying to create the sysfs link.
This fixes the problem by listening for NETDEV_CHANGENAME
notifications and renaming the link.
https://bugzilla.kernel.org/show_bug.cgi?id=12743
Signed-off-by: Simon Arlott <simon@fire.lp0.eu>
Acked-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bridge/br_if.c')
| -rw-r--r-- | net/bridge/br_if.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/bridge/br_if.c b/net/bridge/br_if.c index 45f3f8871149..f25e3c92bd72 100644 --- a/net/bridge/br_if.c +++ b/net/bridge/br_if.c @@ -133,7 +133,7 @@ static void del_nbp(struct net_bridge_port *p) struct net_bridge *br = p->br; struct net_device *dev = p->dev; - sysfs_remove_link(br->ifobj, dev->name); + sysfs_remove_link(br->ifobj, p->sysfs_name); dev_set_promiscuity(dev, -1); |