| Message ID | 20090301210541.4585c199@nehalam |
|---|---|
| State | Rejected, archived |
| Delegated to: | David Miller |
| Headers | show |
From: Stephen Hemminger <shemminger@vyatta.com> Date: Sun, 1 Mar 2009 21:05:41 -0800 > On Sun, 01 Mar 2009 20:57:02 -0800 (PST) > David Miller <davem@davemloft.net> wrote: > > > From: Stephen Hemminger <shemminger@vyatta.com> > > Date: Sun, 1 Mar 2009 20:47:31 -0800 > > > > > Why is interface created through netlink named 'vlan0' and > > > interface created through old vconfig called 'ethX.YY'. > > > Seems like the interface should be consistent. > > > > I think the netlink scheme is saner, and the idea is to > > transition people over to that. > > > > We can't change what vconfig uses, so what Patrick decided > > to do is pretty sane if you ask me. > > This is missing: You can just say you don't think the behavior is intentional.... :-/ -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Stephen Hemminger <shemminger@vyatta.com> wrote: > > Subject: vlan: names should be consistent > > Vlan's created via netlink should be the same as vlan's created > via older interface. This patches changes is to be consistent. > > One could argue this is an API change (it changes behaviour of netlink > interface), but I argue this was a design bug when the netlink interface > to VLAN was added. > > Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> I don't see the point of this. The new interface is available through ip(8) only, vconfig remains unchanged. So the user is completely aware that the name may change as he's using a new command. In any case, ip(8) allows you to set the name to whatever you want at creation time. So for those nostalgics you can still use ethX.Y through ip. Cheers,
Herbert Xu wrote: > Stephen Hemminger <shemminger@vyatta.com> wrote: > >> Subject: vlan: names should be consistent >> >> Vlan's created via netlink should be the same as vlan's created >> via older interface. This patches changes is to be consistent. >> >> One could argue this is an API change (it changes behaviour of netlink >> interface), but I argue this was a design bug when the netlink interface >> to VLAN was added. >> >> Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> >> > > I don't see the point of this. The new interface is available > through ip(8) only, vconfig remains unchanged. So the user is > completely aware that the name may change as he's using a new > command. > > In any case, ip(8) allows you to set the name to whatever you > want at creation time. So for those nostalgics you can still > use ethX.Y through ip. > > Cheers, > Yes but their is no indication as to which interface was just created. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Stephen Hemminger <stephen.hemminger@vyatta.com> wrote: > >> In any case, ip(8) allows you to set the name to whatever you >> want at creation time. So for those nostalgics you can still >> use ethX.Y through ip. >> > Yes but their is no indication as to which interface was just created. You should be able to set the name at creation time. The default is only used if you didn't specify an explicit name. Cheers,
David Miller wrote: > From: Stephen Hemminger <shemminger@vyatta.com> > Date: Sun, 1 Mar 2009 21:05:41 -0800 > >> On Sun, 01 Mar 2009 20:57:02 -0800 (PST) >> David Miller <davem@davemloft.net> wrote: >> >>> From: Stephen Hemminger <shemminger@vyatta.com> >>> Date: Sun, 1 Mar 2009 20:47:31 -0800 >>> >>>> Why is interface created through netlink named 'vlan0' and >>>> interface created through old vconfig called 'ethX.YY'. >>>> Seems like the interface should be consistent. >>> I think the netlink scheme is saner, and the idea is to >>> transition people over to that. >>> >>> We can't change what vconfig uses, so what Patrick decided >>> to do is pretty sane if you ask me. >> This is missing: > > You can just say you don't think the behavior is > intentional.... :-/ Indeed, it was intentional not to have these IMO useless naming schemes when you can simply supply the name of your choice when creating the interface. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Herbert Xu wrote: > Stephen Hemminger <stephen.hemminger@vyatta.com> wrote: >>> In any case, ip(8) allows you to set the name to whatever you >>> want at creation time. So for those nostalgics you can still >>> use ethX.Y through ip. >>> >> Yes but their is no indication as to which interface was just created. > > You should be able to set the name at creation time. > > The default is only used if you didn't specify an explicit name. Indeed. The kernel also sends notifications, so iproute could print the generated name if none was specified. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Patrick McHardy <kaber@trash.net> Date: Mon, 02 Mar 2009 10:45:43 +0100 > Herbert Xu wrote: > > Stephen Hemminger <stephen.hemminger@vyatta.com> wrote: > >>> In any case, ip(8) allows you to set the name to whatever you > >>> want at creation time. So for those nostalgics you can still > >>> use ethX.Y through ip. > >>> > >> Yes but their is no indication as to which interface was just created. > > You should be able to set the name at creation time. > > The default is only used if you didn't specify an explicit name. > > Indeed. The kernel also sends notifications, so iproute could print > the generated name if none was specified. I considered this issue closed 6 emails ago. I don't know why Stephen keeps persisting. :-) -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
--- a/net/8021q/vlan.c 2009-03-01 20:54:15.906263870 -0800 +++ b/net/8021q/vlan.c 2009-03-01 20:58:34.290763830 -0800 @@ -292,24 +292,11 @@ out_free_group: return err; } -/* Attach a VLAN device to a mac address (ie Ethernet Card). - * Returns 0 if the device was created or a negative error code otherwise. - */ -static int register_vlan_device(struct net_device *real_dev, u16 vlan_id) +void vlan_name(char *name, const struct net_device *real_dev, u16 vlan_id) { - struct net_device *new_dev; - struct net *net = dev_net(real_dev); - struct vlan_net *vn = net_generic(net, vlan_net_id); - char name[IFNAMSIZ]; - int err; - - if (vlan_id >= VLAN_VID_MASK) - return -ERANGE; - - err = vlan_check_real_dev(real_dev, vlan_id); - if (err < 0) - return err; - + const struct net *net = dev_net(real_dev); + const struct vlan_net *vn = net_generic(net, vlan_net_id); + /* Gotta set up the fields for the device. */ switch (vn->name_type) { case VLAN_NAME_TYPE_RAW_PLUS_VID: @@ -336,6 +323,25 @@ static int register_vlan_device(struct n snprintf(name, IFNAMSIZ, "vlan%.4i", vlan_id); } +} + +/* Attach a VLAN device to a mac address (ie Ethernet Card). + * Returns 0 if the device was created or a negative error code otherwise. + */ +static int register_vlan_device(struct net_device *real_dev, u16 vlan_id) +{ + struct net_device *new_dev; + char name[IFNAMSIZ]; + int err; + + if (vlan_id >= VLAN_VID_MASK) + return -ERANGE; + + err = vlan_check_real_dev(real_dev, vlan_id); + if (err < 0) + return err; + + vlan_name(name, real_dev, vlan_id); new_dev = alloc_netdev(sizeof(struct vlan_dev_info), name, vlan_setup); --- a/net/8021q/vlan.h 2009-03-01 20:57:49.766013840 -0800 +++ b/net/8021q/vlan.h 2009-03-01 20:58:33.490013558 -0800 @@ -80,6 +80,7 @@ int vlan_dev_change_flags(const struct n void vlan_dev_get_realdev_name(const struct net_device *dev, char *result); int vlan_check_real_dev(struct net_device *real_dev, u16 vlan_id); +void vlan_name(char *name, const struct net_device *real_dev, u16 vlan_id); void vlan_setup(struct net_device *dev); int register_vlan_dev(struct net_device *dev); void unregister_vlan_dev(struct net_device *dev); --- a/net/8021q/vlan_netlink.c 2009-03-01 20:51:14.943013010 -0800 +++ b/net/8021q/vlan_netlink.c 2009-03-01 20:57:37.448624035 -0800 @@ -106,6 +106,7 @@ static int vlan_newlink(struct net_devic struct vlan_dev_info *vlan = vlan_dev_info(dev); struct net_device *real_dev; int err; + char name[IFNAMSIZ]; if (!data[IFLA_VLAN_ID]) return -EINVAL; @@ -129,6 +130,11 @@ static int vlan_newlink(struct net_devic else if (dev->mtu > real_dev->mtu) return -EINVAL; + vlan_name(name, real_dev, vlan->vlan_id); + err = dev_change_name(dev, name); + if (err < 0) + return err; + err = vlan_changelink(dev, tb, data); if (err < 0) return err;