From: Patrick McHardy <kaber@trash.net>
To: Jarek Poplawski <jarkao2@gmail.com>
Cc: Lennart Sorensen <lsorense@csclub.uwaterloo.ca>,
Stephen Hemminger <shemminger@vyatta.com>,
netdev@vger.kernel.org
Subject: Re: Vlan interface nuisance
Date: Tue, 03 Mar 2009 00:35:00 +0100 [thread overview]
Message-ID: <49AC6D24.1030605@trash.net> (raw)
In-Reply-To: <20090302223011.GA3388@ami.dom.local>
Jarek Poplawski wrote:
> On Mon, Mar 02, 2009 at 10:49:18PM +0100, Patrick McHardy wrote:
>> The binding is displayed when listing interfaces. This hole
>> argument is silly, if you want a particular name, just specify
>> it. The current naming schemes are entirely based on information
>> that you have to specify anyways.
>
> Just for the record, I don't agree with calling "this" argument silly
> just like it was with "that" argument. Actually, I think they are
> both so "right" that I've changed my mind and think it's great each
> tool does it differently...
I'm not sure whether I'm supposed to understand this, so I'm going
to respond with something useful unrelated to naming user resources
that every virtual device author should know (and that seems to be
not well known):
Every virtual device bound to a different device should set the
dev->iflink field to the ifindex of the device bound to. This
makes every device related netlink message include this relation.
If the binding is already known at device-creation time and
relevant for the virtuals device's existance, this must be done
in the ->init callback to make sure its already included in the
first netlink creation message to avoid inconsistent information.
This is the also the prefered way device bindings should be
signaled to the kernel, and at least iproute and libnl are
aware of this in both directions. And to get back to the main
point of this discussion:
# ip l l
4: vlan0@dummy0: <BROADCAST,NOARP> mtu 1500 qdisc noop state DOWN
link/ether 92:2a:4f:ae:dc:29 brd ff:ff:ff:ff:ff:ff
next prev parent reply other threads:[~2009-03-02 23:35 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-03-02 4:47 Vlan interface nuisance Stephen Hemminger
2009-03-02 4:57 ` David Miller
2009-03-02 5:05 ` Stephen Hemminger
2009-03-02 5:28 ` David Miller
2009-03-02 9:44 ` Patrick McHardy
2009-03-02 6:50 ` Herbert Xu
2009-03-02 7:15 ` Stephen Hemminger
2009-03-02 7:39 ` Herbert Xu
2009-03-02 9:45 ` Patrick McHardy
2009-03-02 9:57 ` David Miller
2009-03-02 17:20 ` Lennart Sorensen
2009-03-02 18:51 ` Jarek Poplawski
2009-03-02 18:56 ` Ben Greear
2009-03-02 19:06 ` Jarek Poplawski
2009-03-02 19:23 ` Denys Fedoryschenko
2009-03-02 20:24 ` Jarek Poplawski
2009-03-02 21:52 ` Patrick McHardy
2009-03-02 19:36 ` Lennart Sorensen
2009-03-02 20:07 ` Jarek Poplawski
2009-03-02 21:49 ` Patrick McHardy
2009-03-02 21:51 ` Stephen Hemminger
2009-03-02 21:54 ` Patrick McHardy
2009-03-02 22:30 ` Jarek Poplawski
2009-03-02 23:35 ` Patrick McHardy [this message]
2009-03-03 6:41 ` Jarek Poplawski
2009-03-03 6:53 ` Ben Greear
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=49AC6D24.1030605@trash.net \
--to=kaber@trash.net \
--cc=jarkao2@gmail.com \
--cc=lsorense@csclub.uwaterloo.ca \
--cc=netdev@vger.kernel.org \
--cc=shemminger@vyatta.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).