All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Hemminger <shemminger@linux-foundation.org>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Patrick McHardy <kaber@trash.net>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	David Miller <davem@davemloft.net>,
	akpm@linux-foundation.org, dim@openvz.org,
	netdev@vger.kernel.org, jgarzik@pobox.com,
	containers@lists.osdl.org, kuznet@ms2.inr.ac.ru,
	greearb@candelatech.com
Subject: Re: [PATCH] net: Add etun driver
Date: Wed, 11 Apr 2007 09:16:06 -0700	[thread overview]
Message-ID: <20070411091606.04d73b31@freekitty> (raw)
In-Reply-To: <1176239819.10381.10.camel@johannes.berg>

On Tue, 10 Apr 2007 23:16:59 +0200
Johannes Berg <johannes@sipsolutions.net> wrote:

> On Tue, 2007-04-10 at 02:06 +0200, Patrick McHardy wrote:
> 
> > Same way as the current RTM_SETLINK message works, but with creating
> > a new link in advance. It works fine in other subsystems, so I don't
> > see why it would in this case as well. Some subsystems do it in an
> > atomic fashion (network schedulers for example), some first create
> > the object, then configure it (network classifiers in the non-compat
> > cases). In the network device case I suppose the later should work
> > fine since a device needs to be set UP in a second action before
> > it really does anything.
> 
> Looking at br_netlink.c it seems that this sort of contradicts why
> generic netlink was done, now all the sudden everything that wants to
> create new links need its own netlink protocol number, no?
> 
> johannes

Bridging is different since there was already a bridge protocol number assigned,
there was no point in doing generic netlink.


---
Stephen Hemminger <shemminger@linux-foundation.org>

  parent reply	other threads:[~2007-04-11 16:17 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-06 20:43 [PATCH] net: Add etun driver Eric W. Biederman
2007-04-06 20:34 ` Stephen Hemminger
2007-04-06 21:38   ` Ben Greear
2007-04-06 20:54     ` Stephen Hemminger
2007-04-06 22:01       ` Ben Greear
2007-04-07  2:51   ` Eric W. Biederman
2007-04-09 16:37     ` Johannes Berg
2007-04-09 16:58       ` Patrick McHardy
2007-04-09 18:14         ` Ben Greear
2007-04-09 18:37           ` David Miller
2007-04-09 18:48             ` Ben Greear
2007-04-09 18:45           ` Patrick McHardy
2007-04-09 18:44         ` David Miller
2007-04-09 19:35           ` Eric W. Biederman
2007-04-09 19:48             ` Jeff Garzik
2007-04-09 20:03             ` Johannes Berg
2007-04-09 20:11             ` Patrick McHardy
2007-04-09 20:29               ` Johannes Berg
2007-04-10  0:06                 ` Patrick McHardy
2007-04-10  5:47                   ` Johannes Berg
2007-04-10  6:08                     ` Eric W. Biederman
2007-04-10  6:18                       ` Johannes Berg
2007-04-10  7:52                     ` Patrick McHardy
2007-04-10  9:18                       ` Johannes Berg
2007-04-10  9:52                         ` Patrick McHardy
2007-04-10 10:27                           ` Johannes Berg
2007-04-10 10:46                             ` Patrick McHardy
2007-04-10 11:02                               ` Johannes Berg
2007-04-10 11:09                                 ` Patrick McHardy
2007-04-10 11:16                                   ` Jeff Garzik
2007-04-10 11:24                                   ` Johannes Berg
2007-04-10 12:05                                     ` Patrick McHardy
2007-04-10 13:44                                       ` John W. Linville
2007-04-10 13:48                                       ` Johannes Berg
2007-04-10 21:16                   ` Johannes Berg
2007-04-11 16:15                     ` Patrick McHardy
2007-04-11 16:43                       ` Johannes Berg
2007-04-11 16:52                         ` Patrick McHardy
2007-04-11 16:59                           ` Johannes Berg
2007-04-11 16:16                     ` Stephen Hemminger [this message]
2007-04-06 20:57 ` Roland Dreier
2007-04-07  2:08   ` Eric W. Biederman
2007-04-06 21:20 ` Ben Greear
2007-04-07  2:06   ` Eric W. Biederman
2007-04-07  3:31     ` Ben Greear
2007-04-07  5:37       ` Eric W. Biederman
2007-04-07  7:31         ` 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=20070411091606.04d73b31@freekitty \
    --to=shemminger@linux-foundation.org \
    --cc=akpm@linux-foundation.org \
    --cc=containers@lists.osdl.org \
    --cc=davem@davemloft.net \
    --cc=dim@openvz.org \
    --cc=ebiederm@xmission.com \
    --cc=greearb@candelatech.com \
    --cc=jgarzik@pobox.com \
    --cc=johannes@sipsolutions.net \
    --cc=kaber@trash.net \
    --cc=kuznet@ms2.inr.ac.ru \
    --cc=netdev@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.