mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Kay Sievers" <kay.sievers@vrfy.org>
To: "Bill Nottingham" <notting@redhat.com>
Cc: linux-kernel@vger.kernel.org, "Greg KH" <greg@kroah.com>
Subject: Re: new sysfs layout and ethernet device names
Date: Thu, 22 Mar 2007 09:47:34 +0100	[thread overview]
Message-ID: <3ae72650703220147j42db8f79s734d4755c2fa43fd@mail.gmail.com> (raw)
In-Reply-To: <20070320170102.GA16341@nostromo.devel.redhat.com>

On 3/20/07, Bill Nottingham <notting@redhat.com> wrote:
> I was fiddling with the 'new' (no CONFIG_SYSFS_DEPRECATED) layout
> and ethernet device names, and noticed that the new layout effectively
> restricts the availability of certain device names.
>
> By making a directory for the ethernet device name in the parent
> device, you no longer can use any name that is used in sysfs for
> the parent device type - for PCI devices this would be 'resource0',
> 'power', 'enable', etc. For USB, it would be a different set
> of names that are no longer available.

Yeah, we heard, people want to name their network interfaces "irq". :)

> If you *do* try to use one of these names, the rename will succeed...
> partway. The link in /sys/class/net is renamed, the directory is
> not (as it obviously can't rename on top of whatever is already there.)
> Various networking tools then break in assorted ways due to the
> naming disconnect.
>
> Going back to the deprecated layout makes these names available again -
> it's possible (although not necessarily likely) that the new layout
> will break someone's device configuration if they upgrade kernels,
> even if the rest of their tools are updated for the new layout.

There will be a directory at the bus-device with the name of the
class, that glues together the bus-devices and the class-devices in
the unified tree:
  http://git.kernel.org/?p=linux/kernel/git/gregkh/patches.git;a=blob;f=driver/driver-core-fix-namespace-issue-with-devices-assigned-to-classes.patch;hb=HEAD

So the network-interface(s) will be in their own directory "net",
below the bus-device, and don't conflict with the existing attributes.

Thanks,
Kay

  reply	other threads:[~2007-03-22  8:47 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-20 17:01 Bill Nottingham
2007-03-22  8:47 ` Kay Sievers [this message]
2007-03-28  3:17   ` Bill Nottingham
2007-03-28  4:10     ` Greg KH
2007-03-29  0:41       ` Bill Nottingham
2007-03-30  2:31         ` Greg KH
2007-03-30  3:29           ` Bill Nottingham
2007-03-30  5:26             ` Greg KH

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=3ae72650703220147j42db8f79s734d4755c2fa43fd@mail.gmail.com \
    --to=kay.sievers@vrfy.org \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=notting@redhat.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

all inboxes | Powered by JetHome®