All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kay Sievers <kay.sievers@vrfy.org>
To: David Woodhouse <dwmw2@infradead.org>
Cc: Marco d'Itri <md@linux.it>, Harald Hoyer <harald@redhat.com>,
	linux-hotplug@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: udev can't name PS3's network devices correctly
Date: Mon, 14 Apr 2008 14:52:11 +0200	[thread overview]
Message-ID: <1208177531.2572.26.camel@lov.site> (raw)
In-Reply-To: <1208175549.31695.67.camel@pmac.infradead.org>

On Mon, 2008-04-14 at 13:19 +0100, David Woodhouse wrote:
> On Mon, 2008-04-14 at 14:03 +0200, Kay Sievers wrote:
> > 
> > >  One thing I don't understand: Don't we already emit a KERNEL== criterion
> > >  in the case where there is already a udev rule 'reserving' the name that
> > >  the kernel used for the current device? Why is that one OK, and why only
> > >  in that case? This patch just makes it consistent.
> > 
> > Yes, we do that in the recent udev versions. We only make sure we keep
> > the enumeration across the same basename, not across different device
> > names.
> 
> Right, but you _only_ do that when you're asked to create a rule for a
> device where the kernel's name matches a pre-existing rule.

We do it for all new rules, regardless of pre-existing rules with that
name. It just does not add KERNEL, if some external tool tried to
overwrite the kernel name, we write a rule for.

Not sure why Fedora rules are mangling INTERFACE_NAME here:
  https://bugzilla.redhat.com/show_bug.cgi?id=440568#c4

> You don't do it consistently --

Right, we will do that, if there are no other issues coming up with the
general approach of adding KERNEL matches.

> and Marco was objecting to a patch which just made it
> happen consistently.

Right, I'm just waiting for Marco to comment on this.

> That patch on its own would be sufficient to fix the PS3 and Libertas
> problems, since the kernel uses a different basename for the logical
> devices we want to disambiguate. (It would also fix the clash between
> wlan0 and wmaster0 on mac80211 interfaces).

Sure, that's why we have it in every default rule.

> We probably also want to do the dev_id thing, but maybe not for PS3 and
> Libertas (since it affects their IPv6 addressing too).

It can just be in all rules, like the ATTR{type} match, right?

Thanks,
Kay


  reply	other threads:[~2008-04-14 12:52 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <ftdb3v$tls$1@ger.gmane.org>
     [not found] ` <20080407143805.GA9492@bongo.bofh.it>
2008-04-14 10:08   ` udev can't name PS3's network devices correctly David Woodhouse
2008-04-14 11:11     ` David Woodhouse
2008-04-14 11:55       ` Kay Sievers
2008-04-14 12:32         ` [PATCH] Expose netdevice dev_id through sysfs David Woodhouse
2008-04-20  1:33           ` David Miller
2008-04-20  5:21             ` Andrey Borzenkov
2008-04-20  5:32               ` David Miller
2008-04-20 10:50             ` David Woodhouse
2008-04-20 10:55               ` David Miller
2008-04-20 11:12                 ` David Woodhouse
2008-04-20 11:14                   ` David Miller
2008-04-20 11:22         ` David Woodhouse
2008-04-27 17:37         ` udev can't name PS3's network devices correctly David Woodhouse
2008-04-27 18:28           ` Kay Sievers
2008-04-14 12:03     ` Kay Sievers
2008-04-14 12:19       ` David Woodhouse
2008-04-14 12:52         ` Kay Sievers [this message]
2008-04-14 13:16           ` David Woodhouse
2008-04-14 12:51     ` Marco d'Itri
2008-04-14 13:38       ` David Woodhouse

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=1208177531.2572.26.camel@lov.site \
    --to=kay.sievers@vrfy.org \
    --cc=dwmw2@infradead.org \
    --cc=harald@redhat.com \
    --cc=linux-hotplug@vger.kernel.org \
    --cc=md@linux.it \
    --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.