All of lore.kernel.org
 help / color / mirror / Atom feed
From: dann frazier <dannf@hp.com>
To: Ben Hutchings <bhutchings@solarflare.com>
Cc: Narendra_K@Dell.com, netdev@vger.kernel.org,
	linux-hotplug@vger.kernel.org, Matt_Domsch@Dell.com,
	Jordan_Hargrave@Dell.com, Charles_Rose@Dell.com
Subject: Re: PATCH: Network Device Naming mechanism and policy
Date: Fri, 16 Oct 2009 21:40:24 +0000	[thread overview]
Message-ID: <20091016214024.GA10091@ldl.fc.hp.com> (raw)
In-Reply-To: <1255707193.2869.12.camel@achroite>

On Fri, Oct 16, 2009 at 04:33:13PM +0100, Ben Hutchings wrote:
> On Fri, 2009-10-16 at 09:20 -0600, dann frazier wrote:
> > On Fri, Oct 16, 2009 at 07:32:50PM +0530, Narendra_K@Dell.com wrote:
> [...]
> > > And how would the regular file look like in terms of holding ifindex of
> > > the interface, which can be passed to libnetdevname.
> > 
> > I can't think of anything we need to store in the regular file. If we
> > have the kernel name for the device, we can look up the ifindex in
> > /sys. Correct me if I'm wrong, but storing it ourselves seems
> > redundant.
> 
> But the name of a netdev can change whereas its ifindex never does.
> Identifying netdevs by name would require additional work to update the
> links when a netdev is renamed and would still be prone to race
> conditions.  This is why Narendra and Matt were proposing to store the
> ifindex in the node all along...

Matt, Ben and I talked about a few other possibilities on IRC.
The one I like the most at the moment is an idea Ben had to creat
dummy files named after the ifindex. Then, use symlinks for the kernel
name and the various by-$property subdirectories. This means the KOBJ
events will need to expose the ifindex.

I'm a novice at net programming, but I'm told that ifindex is
the information apps ultimately require here.

-- 
dann frazier


WARNING: multiple messages have this Message-ID (diff)
From: dann frazier <dannf@hp.com>
To: Ben Hutchings <bhutchings@solarflare.com>
Cc: Narendra_K@Dell.com, netdev@vger.kernel.org,
	linux-hotplug@vger.kernel.org, Matt_Domsch@Dell.com,
	Jordan_Hargrave@Dell.com, Charles_Rose@Dell.com
Subject: Re: PATCH: Network Device Naming mechanism and policy
Date: Fri, 16 Oct 2009 15:40:24 -0600	[thread overview]
Message-ID: <20091016214024.GA10091@ldl.fc.hp.com> (raw)
In-Reply-To: <1255707193.2869.12.camel@achroite>

On Fri, Oct 16, 2009 at 04:33:13PM +0100, Ben Hutchings wrote:
> On Fri, 2009-10-16 at 09:20 -0600, dann frazier wrote:
> > On Fri, Oct 16, 2009 at 07:32:50PM +0530, Narendra_K@Dell.com wrote:
> [...]
> > > And how would the regular file look like in terms of holding ifindex of
> > > the interface, which can be passed to libnetdevname.
> > 
> > I can't think of anything we need to store in the regular file. If we
> > have the kernel name for the device, we can look up the ifindex in
> > /sys. Correct me if I'm wrong, but storing it ourselves seems
> > redundant.
> 
> But the name of a netdev can change whereas its ifindex never does.
> Identifying netdevs by name would require additional work to update the
> links when a netdev is renamed and would still be prone to race
> conditions.  This is why Narendra and Matt were proposing to store the
> ifindex in the node all along...

Matt, Ben and I talked about a few other possibilities on IRC.
The one I like the most at the moment is an idea Ben had to creat
dummy files named after the ifindex. Then, use symlinks for the kernel
name and the various by-$property subdirectories. This means the KOBJ
events will need to expose the ifindex.

I'm a novice at net programming, but I'm told that ifindex is
the information apps ultimately require here.

-- 
dann frazier


  parent reply	other threads:[~2009-10-16 21:40 UTC|newest]

Thread overview: 241+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <EDA0A4495861324DA2618B4C45DCB3EE5894ED@blrx3m08.blr.amer.dell.com>
2009-10-09 14:00 ` PATCH: Network Device Naming mechanism and policy Narendra K
2009-10-09 14:00   ` Narendra K
2009-10-09 14:51   ` Matt Domsch
2009-10-09 14:51     ` Matt Domsch
2009-10-09 16:23     ` Bryan Kadzban
2009-10-09 16:23       ` Bryan Kadzban
2009-10-09 16:56       ` Marco d'Itri
2009-10-09 16:56         ` Marco d'Itri
2009-10-12 10:41     ` Scott James Remnant
2009-10-12 10:41       ` Scott James Remnant
2009-10-12 11:31       ` Ben Hutchings
2009-10-12 11:31         ` Ben Hutchings
2009-10-12 17:37       ` Bill Nottingham
2009-10-12 17:37         ` Bill Nottingham
2009-10-13 18:06         ` Dan Williams
2009-10-13 18:06           ` Dan Williams
2009-10-13 18:53           ` Ben Hutchings
2009-10-13 18:53             ` Ben Hutchings
2009-10-13 19:53             ` John W. Linville
2009-10-13 19:53               ` John W. Linville
2009-10-09 16:36   ` Greg KH
2009-10-09 16:36     ` Greg KH
2009-10-09 17:17     ` Matt Domsch
2009-10-09 17:17       ` Matt Domsch
2009-10-09 17:22       ` Greg KH
2009-10-09 17:22         ` Greg KH
2009-10-09 21:09   ` Matt Domsch
2009-10-09 21:09     ` Matt Domsch
2009-10-10  2:44     ` Stephen Hemminger
2009-10-10  2:44       ` Stephen Hemminger
2009-10-10  4:40       ` Matt Domsch
2009-10-10  4:40         ` Matt Domsch
2009-10-10  5:23         ` Greg KH
2009-10-10  5:23           ` Greg KH
2009-10-10  8:17           ` Sujit K M
2009-10-10  8:29             ` Sujit K M
2009-10-10 16:27             ` Greg KH
2009-10-10 16:27               ` Greg KH
2009-10-10 19:00               ` Ben Hutchings
2009-10-10 19:00                 ` Ben Hutchings
2009-10-10 21:10                 ` Greg KH
2009-10-10 21:10                   ` Greg KH
2009-10-10 12:47           ` Matt Domsch
2009-10-10 12:47             ` Matt Domsch
2009-10-10 16:25             ` Greg KH
2009-10-10 16:25               ` Greg KH
2009-10-10 17:34               ` Bryan Kadzban
2009-10-10 17:34                 ` Bryan Kadzban
2009-10-10 21:13                 ` Greg KH
2009-10-10 21:13                   ` Greg KH
2009-10-12  6:21                   ` Bryan Kadzban
2009-10-12  6:21                     ` Bryan Kadzban
2009-10-12 16:19                     ` Bryan Kadzban
2009-10-12 16:19                       ` Bryan Kadzban
2009-10-11 16:40               ` David Zeuthen
2009-10-11 16:40                 ` David Zeuthen
2009-10-11 18:47                 ` Greg KH
2009-10-11 18:47                   ` Greg KH
2009-10-10 18:11           ` Bill Fink
2009-10-10 18:11             ` Bill Fink
2009-10-10 18:35             ` Kay Sievers
2009-10-10 18:35               ` Kay Sievers
2009-10-11 21:10           ` Rob Townley
2009-10-11 21:10             ` Rob Townley
2009-10-11 23:04             ` Matt Domsch
2009-10-11 23:04               ` Matt Domsch
2009-10-12  3:00             ` Greg KH
2009-10-12  3:00               ` Greg KH
2009-10-12 18:35               ` Rob Townley
2009-10-12 18:35                 ` Rob Townley
2009-10-12 18:44                 ` Matt Domsch
2009-10-12 18:44                   ` Matt Domsch
2009-10-12 17:45           ` Bill Nottingham
2009-10-12 17:45             ` Bill Nottingham
2009-10-12 17:55             ` Greg KH
2009-10-12 17:55               ` Greg KH
2009-10-12 18:07               ` Bill Nottingham
2009-10-12 18:07                 ` Bill Nottingham
2009-10-12 18:15                 ` Greg KH
2009-10-12 18:15                   ` Greg KH
2009-10-10 18:32         ` Stephen Hemminger
2009-10-10 18:32           ` Stephen Hemminger
2009-10-10 21:06           ` Greg KH
2009-10-10 21:06             ` Greg KH
2009-10-13 18:02             ` Dan Williams
2009-10-13 18:02               ` Dan Williams
2009-10-13 18:53               ` Narendra_K
2009-10-13 18:56                 ` Narendra_K
2009-10-12  7:30           ` Kurt Van Dijck
2009-10-12  7:30             ` Kurt Van Dijck
2009-10-11  0:37         ` Marco d'Itri
2009-10-11  0:37           ` Marco d'Itri
2009-10-13 15:08   ` dann frazier
2009-10-13 15:08     ` dann frazier
2009-10-13 17:13     ` Narendra_K
2009-10-13 17:25       ` Narendra_K
2009-10-13 17:36       ` dann frazier
2009-10-13 17:36         ` dann frazier
2009-10-16  0:32         ` dann frazier
2009-10-16  0:32           ` dann frazier
2009-10-16 14:02           ` Narendra_K
2009-10-16 14:14             ` Narendra_K
2009-10-16 15:20             ` dann frazier
2009-10-16 15:20               ` dann frazier
2009-10-16 15:33               ` Ben Hutchings
2009-10-16 15:33                 ` Ben Hutchings
2009-10-16 15:41                 ` dann frazier
2009-10-16 15:41                   ` dann frazier
2009-10-16 21:40                 ` dann frazier [this message]
2009-10-16 21:40                   ` dann frazier
2009-10-19 11:30                   ` Narendra_K
2009-10-19 11:42                     ` Narendra_K
2009-10-19 16:14                     ` Bryan Kadzban
2009-10-19 16:14                       ` Bryan Kadzban
2009-11-04 14:23                       ` Narendra_K
2009-11-04 14:35                         ` Narendra_K
2009-11-06  8:49                         ` Marco d'Itri
2009-11-06  8:49                           ` Marco d'Itri
2009-11-06 22:06                           ` Matt Domsch
2009-11-06 22:06                             ` Matt Domsch
2009-11-06 22:35                             ` Marco d'Itri
2009-11-06 22:35                               ` Marco d'Itri
2009-11-06 23:17                               ` dann frazier
2009-11-06 23:17                                 ` dann frazier
2009-11-09 14:41                               ` Narendra_K
2009-11-09 14:53                                 ` Narendra_K
2009-11-10 17:23                                 ` Stephen Hemminger
2009-11-10 17:23                                   ` Stephen Hemminger
2009-11-11  6:31                                   ` Narendra_K
2009-11-11  6:43                                     ` Narendra_K
2009-11-06 22:05                         ` Domsch, Matt
2009-11-06 22:05                           ` Domsch, Matt
2009-10-22  6:36                   ` [PATCH] udev: create empty regular files to represent net dann frazier
2009-10-22  6:36                     ` [PATCH] udev: create empty regular files to represent net interfaces dann frazier
2009-10-27 20:55                     ` Matt Domsch
2009-10-27 20:55                       ` Matt Domsch
2009-10-28  8:23                       ` [PATCH] udev: create empty regular files to represent net Kay Sievers
2009-10-28  8:23                         ` [PATCH] udev: create empty regular files to represent net interfaces Kay Sievers
2009-10-28 13:03                         ` Matt Domsch
2009-10-28 13:03                           ` Matt Domsch
2009-10-28 15:09                           ` [PATCH] udev: create empty regular files to represent net dann frazier
2009-10-28 15:09                             ` [PATCH] udev: create empty regular files to represent net interfaces dann frazier
2009-10-28 16:09                             ` Jordan_Hargrave
2009-10-28 16:09                               ` Jordan_Hargrave
2009-10-28 16:09                             ` Jordan_Hargrave
2009-10-28 16:09                               ` Jordan_Hargrave
2009-10-28 16:11                               ` [PATCH] udev: create empty regular files to represent net Greg KH
2009-10-28 16:11                                 ` [PATCH] udev: create empty regular files to represent net interfaces Greg KH
2009-10-28 13:06                         ` [PATCH] udev: create empty regular files to represent net Ben Hutchings
2009-10-28 13:06                           ` [PATCH] udev: create empty regular files to represent net interfaces Ben Hutchings
2009-10-28 19:15                         ` Narendra_K
2009-10-28 19:27                           ` Narendra_K
2009-10-29 13:11                         ` Matt Domsch
2009-10-29 13:11                           ` Matt Domsch
2009-10-29 14:25                           ` [PATCH] udev: create empty regular files to represent net Greg KH
2009-10-29 14:25                             ` [PATCH] udev: create empty regular files to represent net interfaces Greg KH
2009-10-29 16:44                             ` Narendra_K
2009-10-29 16:56                               ` Narendra_K
2009-10-29 16:52                               ` [PATCH] udev: create empty regular files to represent net Greg KH
2009-10-29 16:52                                 ` [PATCH] udev: create empty regular files to represent net interfaces Greg KH
2009-10-29 17:22                                 ` [PATCH] udev: create empty regular files to represent netinterfaces Narendra_K
2009-10-29 17:34                                   ` Narendra_K
2009-10-29 17:50                                   ` [PATCH] udev: create empty regular files to represent dann frazier
2009-10-29 17:50                                     ` [PATCH] udev: create empty regular files to represent netinterfaces dann frazier
2009-10-29 16:49                             ` [PATCH] udev: create empty regular files to represent net Ben Hutchings
2009-10-29 16:49                               ` [PATCH] udev: create empty regular files to represent net interfaces Ben Hutchings
2009-10-29 16:55                               ` [PATCH] udev: create empty regular files to represent net Greg KH
2009-10-29 16:55                                 ` [PATCH] udev: create empty regular files to represent net interfaces Greg KH
2009-10-29 17:12                                 ` [PATCH] udev: create empty regular files to represent net Ben Hutchings
2009-10-29 17:12                                   ` [PATCH] udev: create empty regular files to represent net interfaces Ben Hutchings
2009-10-29 17:20                                   ` [PATCH] udev: create empty regular files to represent net Greg KH
2009-10-29 17:20                                     ` [PATCH] udev: create empty regular files to represent net interfaces Greg KH
2009-10-29 17:46                             ` [PATCH] udev: create empty regular files to represent net dann frazier
2009-10-29 17:46                               ` [PATCH] udev: create empty regular files to represent net interfaces dann frazier
2009-10-30  3:30                               ` [PATCH] udev: create empty regular files to represent net Marco d'Itri
2009-10-30  3:30                                 ` [PATCH] udev: create empty regular files to represent net interfaces Marco d'Itri
2009-10-30  5:38                                 ` [PATCH] udev: create empty regular files to represent net dann frazier
2009-10-30  5:38                                   ` [PATCH] udev: create empty regular files to represent net interfaces dann frazier
2009-10-30  6:22                                   ` [PATCH] udev: create empty regular files to represent net Marco d'Itri
2009-10-30  6:22                                     ` [PATCH] udev: create empty regular files to represent net interfaces Marco d'Itri
2009-10-30 15:00                                     ` [PATCH] udev: create empty regular files to represent net dann frazier
2009-10-30 15:00                                       ` [PATCH] udev: create empty regular files to represent net interfaces dann frazier
2009-10-30 15:13                               ` Narendra_K
2009-10-30 15:25                                 ` Narendra_K
2009-10-30 16:08                                 ` [PATCH] udev: create empty regular files to represent net dann frazier
2009-10-30 16:08                                   ` [PATCH] udev: create empty regular files to represent net interfaces dann frazier
2009-10-30 16:53                                   ` [PATCH] udev: create empty regular files to represent netinterfaces Narendra_K
2009-10-30 16:54                                     ` Narendra_K
2009-10-30 17:05                                     ` [PATCH] udev: create empty regular files to represent dann frazier
2009-10-30 17:05                                       ` [PATCH] udev: create empty regular files to represent netinterfaces dann frazier
2009-10-30 17:10                                   ` [PATCH] udev: create empty regular files to represent net interfaces Matt Domsch
2009-10-30 17:10                                     ` Matt Domsch
2009-10-30 17:13                                     ` [PATCH] udev: create empty regular files to represent net Greg KH
2009-10-30 17:13                                       ` [PATCH] udev: create empty regular files to represent net interfaces Greg KH
2009-10-30  7:45                         ` Hannes Reinecke
2009-10-30  7:45                           ` Hannes Reinecke
2009-10-30 16:22                           ` Bryan Kadzban
2009-10-30 16:22                             ` Bryan Kadzban
2009-10-30 16:34                             ` [PATCH] udev: create empty regular files to represent net Stephen Hemminger
2009-10-30 16:34                               ` [PATCH] udev: create empty regular files to represent net interfaces Stephen Hemminger
2009-10-13 19:51       ` PATCH: Network Device Naming mechanism and policy Greg KH
2009-10-13 19:51         ` Greg KH
2009-10-13 20:00         ` Jordan_Hargrave
2009-10-13 20:00           ` Jordan_Hargrave
2009-10-13 20:19           ` Greg KH
2009-10-13 20:19             ` Greg KH
2009-10-13 22:05             ` Matt Domsch
2009-10-13 22:05               ` Matt Domsch
2009-10-13 22:08             ` dann frazier
2009-10-13 22:08               ` dann frazier
     [not found] <EDA0A4495861324DA2618B4C45DCB3EE58964E@blrx3m08.blr.amer.dell.com>
2009-10-28 13:06 ` Narendra K
2009-10-28 13:06   ` Narendra K
     [not found] <EDA0A4495861324DA2618B4C45DCB3EE589541@blrx3m08.blr.amer.dell.com>
2009-10-12 18:47 ` Narendra K
2009-10-12 18:47   ` Narendra K
2009-10-12 19:09   ` Greg KH
2009-10-12 19:09     ` Greg KH
2009-10-12 19:41     ` Karl O. Pinc
2009-10-12 19:41       ` Karl O. Pinc
2009-10-13 18:17       ` Dan Williams
2009-10-13 18:17         ` Dan Williams
2009-10-13 18:56         ` Ben Hutchings
2009-10-13 18:56           ` Ben Hutchings
2009-10-12 19:48     ` Matt Domsch
2009-10-12 19:48       ` Matt Domsch
     [not found] <EDA0A4495861324DA2618B4C45DCB3EE58953F@blrx3m08.blr.amer.dell.com>
2009-10-12 18:07 ` Narendra K
2009-10-12 18:07   ` Narendra K
     [not found] <EDA0A4495861324DA2618B4C45DCB3EE5894F6@blrx3m08.blr.amer.dell.com>
2009-10-09 16:04 ` Narendra K
2009-10-09 16:04   ` Narendra K
2009-10-09 16:12   ` Stephen Hemminger
2009-10-09 16:12     ` Stephen Hemminger
2009-10-09 16:25     ` Matt Domsch
2009-10-09 16:25       ` Matt Domsch
2009-08-05 19:50 Jordan_Hargrave
2009-08-19 18:56 ` PATCH: " Jordan_Hargrave
2009-08-19 18:56   ` Jordan_Hargrave
2009-08-19 19:26   ` Ben Hutchings
2009-08-19 19:26     ` Ben Hutchings
2009-08-19 19:40     ` Jordan_Hargrave
2009-08-19 19:40       ` Jordan_Hargrave
2009-08-20  4:41   ` Bryan Kadzban
2009-08-20  4:41     ` Bryan Kadzban
2009-10-12 17:54   ` Marco d'Itri

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=20091016214024.GA10091@ldl.fc.hp.com \
    --to=dannf@hp.com \
    --cc=Charles_Rose@Dell.com \
    --cc=Jordan_Hargrave@Dell.com \
    --cc=Matt_Domsch@Dell.com \
    --cc=Narendra_K@Dell.com \
    --cc=bhutchings@solarflare.com \
    --cc=linux-hotplug@vger.kernel.org \
    --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.