Commit 17af2bce authored by Marc Angel's avatar Marc Angel Committed by David S. Miller

macvtap: add namespace support to the sysfs device class

When creating macvtaps that are expected to have the same ifindex
in different network namespaces, only the first one will succeed.
The others will fail with a sysfs_warn_dup warning due to them trying
to create the following sysfs link (with 'NN' the ifindex of macvtapX):

/sys/class/macvtap/tapNN -> /sys/devices/virtual/net/macvtapX/tapNN

This is reproducible by running the following commands:

ip netns add ns1
ip netns add ns2
ip link add veth0 type veth peer name veth1
ip link set veth0 netns ns1
ip link set veth1 netns ns2
ip netns exec ns1 ip l add link veth0 macvtap0 type macvtap
ip netns exec ns2 ip l add link veth1 macvtap1 type macvtap

The last command will fail with "RTNETLINK answers: File exists" (along
with the kernel warning) but retrying it will work because the ifindex
was incremented.

The 'net' device class is isolated between network namespaces so each
one has its own hierarchy of net devices.
This isn't the case for the 'macvtap' device class.
The problem occurs half-way through the netdev registration, when
`macvtap_device_event` is called-back to create the 'tapNN' macvtap
class device under the 'macvtapX' net class device.

This patch adds namespace support to the 'macvtap' device class so
that /sys/class/macvtap is no longer shared between net namespaces.

However, making the macvtap sysfs class namespace-aware has the side
effect of changing /sys/devices/virtual/net/macvtapX/tapNN  into
/sys/devices/virtual/net/macvtapX/macvtap/tapNN.

This is due to Commit 24b1442d ("Driver-core: Always create class
directories for classses that support namespaces") and the fact that
class devices supporting namespaces are really not supposed to be placed
directly under other class devices.

To avoid breaking userland, a tapNN symlink pointing to macvtap/tapNN is
created inside the macvtapX directory.
Signed-off-by: default avatarMarc Angel <marc@arista.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 47dcc20a
...@@ -129,7 +129,18 @@ static DEFINE_MUTEX(minor_lock); ...@@ -129,7 +129,18 @@ static DEFINE_MUTEX(minor_lock);
static DEFINE_IDR(minor_idr); static DEFINE_IDR(minor_idr);
#define GOODCOPY_LEN 128 #define GOODCOPY_LEN 128
static struct class *macvtap_class; static const void *macvtap_net_namespace(struct device *d)
{
struct net_device *dev = to_net_dev(d->parent);
return dev_net(dev);
}
static struct class macvtap_class = {
.name = "macvtap",
.owner = THIS_MODULE,
.ns_type = &net_ns_type_operations,
.namespace = macvtap_net_namespace,
};
static struct cdev macvtap_cdev; static struct cdev macvtap_cdev;
static const struct proto_ops macvtap_socket_ops; static const struct proto_ops macvtap_socket_ops;
...@@ -1278,10 +1289,12 @@ static int macvtap_device_event(struct notifier_block *unused, ...@@ -1278,10 +1289,12 @@ static int macvtap_device_event(struct notifier_block *unused,
struct device *classdev; struct device *classdev;
dev_t devt; dev_t devt;
int err; int err;
char tap_name[IFNAMSIZ];
if (dev->rtnl_link_ops != &macvtap_link_ops) if (dev->rtnl_link_ops != &macvtap_link_ops)
return NOTIFY_DONE; return NOTIFY_DONE;
snprintf(tap_name, IFNAMSIZ, "tap%d", dev->ifindex);
vlan = netdev_priv(dev); vlan = netdev_priv(dev);
switch (event) { switch (event) {
...@@ -1295,19 +1308,24 @@ static int macvtap_device_event(struct notifier_block *unused, ...@@ -1295,19 +1308,24 @@ static int macvtap_device_event(struct notifier_block *unused,
return notifier_from_errno(err); return notifier_from_errno(err);
devt = MKDEV(MAJOR(macvtap_major), vlan->minor); devt = MKDEV(MAJOR(macvtap_major), vlan->minor);
classdev = device_create(macvtap_class, &dev->dev, devt, classdev = device_create(&macvtap_class, &dev->dev, devt,
dev, "tap%d", dev->ifindex); dev, tap_name);
if (IS_ERR(classdev)) { if (IS_ERR(classdev)) {
macvtap_free_minor(vlan); macvtap_free_minor(vlan);
return notifier_from_errno(PTR_ERR(classdev)); return notifier_from_errno(PTR_ERR(classdev));
} }
err = sysfs_create_link(&dev->dev.kobj, &classdev->kobj,
tap_name);
if (err)
return notifier_from_errno(err);
break; break;
case NETDEV_UNREGISTER: case NETDEV_UNREGISTER:
/* vlan->minor == 0 if NETDEV_REGISTER above failed */ /* vlan->minor == 0 if NETDEV_REGISTER above failed */
if (vlan->minor == 0) if (vlan->minor == 0)
break; break;
sysfs_remove_link(&dev->dev.kobj, tap_name);
devt = MKDEV(MAJOR(macvtap_major), vlan->minor); devt = MKDEV(MAJOR(macvtap_major), vlan->minor);
device_destroy(macvtap_class, devt); device_destroy(&macvtap_class, devt);
macvtap_free_minor(vlan); macvtap_free_minor(vlan);
break; break;
} }
...@@ -1333,11 +1351,9 @@ static int macvtap_init(void) ...@@ -1333,11 +1351,9 @@ static int macvtap_init(void)
if (err) if (err)
goto out2; goto out2;
macvtap_class = class_create(THIS_MODULE, "macvtap"); err = class_register(&macvtap_class);
if (IS_ERR(macvtap_class)) { if (err)
err = PTR_ERR(macvtap_class);
goto out3; goto out3;
}
err = register_netdevice_notifier(&macvtap_notifier_block); err = register_netdevice_notifier(&macvtap_notifier_block);
if (err) if (err)
...@@ -1352,7 +1368,7 @@ static int macvtap_init(void) ...@@ -1352,7 +1368,7 @@ static int macvtap_init(void)
out5: out5:
unregister_netdevice_notifier(&macvtap_notifier_block); unregister_netdevice_notifier(&macvtap_notifier_block);
out4: out4:
class_unregister(macvtap_class); class_unregister(&macvtap_class);
out3: out3:
cdev_del(&macvtap_cdev); cdev_del(&macvtap_cdev);
out2: out2:
...@@ -1366,7 +1382,7 @@ static void macvtap_exit(void) ...@@ -1366,7 +1382,7 @@ static void macvtap_exit(void)
{ {
rtnl_link_unregister(&macvtap_link_ops); rtnl_link_unregister(&macvtap_link_ops);
unregister_netdevice_notifier(&macvtap_notifier_block); unregister_netdevice_notifier(&macvtap_notifier_block);
class_unregister(macvtap_class); class_unregister(&macvtap_class);
cdev_del(&macvtap_cdev); cdev_del(&macvtap_cdev);
unregister_chrdev_region(macvtap_major, MACVTAP_NUM_DEVS); unregister_chrdev_region(macvtap_major, MACVTAP_NUM_DEVS);
idr_destroy(&minor_idr); idr_destroy(&minor_idr);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment