Re: [PATCH] udev+ifrename integration

[Kay Sievers <kay.sievers@vrfy.org>]

On Thu, 2007-03-15 at 16:27 -0700, Jean Tourrilhes wrote:
> On Mon, Mar 12, 2007 at 05:51:04PM +0100, Kay Sievers wrote:
> > On Mon, 2007-03-12 at 09:07 -0700, Jean Tourrilhes wrote:
> > 
> > > Maybe a sample of rules would help me.
> > 
> > SUBSYSTEM="net", ACTION="add", IMPORT="/sbin/ifrename --export -i %k"
> > 
> > > 	Second, I don't see how this would prevent subsequent naming
> > > rules to apply.
> > 
> > We can make NAME="" work, to prevent other NAME rules from matching. For
> > now NAME="%k" may already work.
> 
> 	Thanks for the help, I'm still learning about udev. It all
> make sense now.
> 	I've tested udev with your patch (see below) and the following
> udev rule and confirm that it works properly.

Nice!

> ---------------------------------
> SUBSYSTEM="net", ACTION="add", IMPORT="/sbin/ifrename -u -i %k", NAME:="%k"
> ---------------------------------

Current udev git should also prevent any later naming rule to match,
with NAME="" now.

> 	If the interface is specified in /etc/iftab, the renaming is
> done and further rules ignored. If the interface is not in /etc/iftab,
> or if /etc/iftab does not exist, no renaming is done and further rule
> do apply. And it works properly at boot time. This allow smooth
> integration.
> 	I've released a new version of ifrename that fixes some
> devpath bugs.

Sounds good.

> 	Also, with the additional fix in the attached patch, you can
> use udevtest like this :
> 
> ---------------------------------
> ENV{UDEV_LOG}="6", SUBSYSTEM="net", ACTION="add", IMPORT="/sbin/ifrename -D -V -u -i %k", NAME:="%k"
> ---------------------------------

I've applied the udevtest part, the other part I already committed
yesterday.

Thanks,
Kay


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CIDÞVDEV
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel