Re: [PATCH] udev+ifrename integration

[Jean Tourrilhes <jt@hpl.hp.com>]

On Fri, Mar 09, 2007 at 11:57:49AM +0000, Scott James Remnant wrote:
> On Thu, 2007-03-08 at 10:15 -0800, Jean Tourrilhes wrote:
> > 
> > Yes, but your patch does not seem to be in the upstream
> > udev. Why is that ?
> > 
> It's been submitted upstream, but not accepted; Kay tends towards
> conservatism until things are proved to be needed.

	And my personal bias is modularity. All users should not have
to bear the cost of a selected few, but the selected few should be
able to do it.

> > And does you patch support all features of ifrename, or just a
> > select few ?
> > 
> Select few.
> 
> We explicitly do not want to support all of the features of ifrename,
> because it caused us nothing but headaches in the past.
> 
> All we want is for interface names to be consistent across reboots, and
> for there to be no race conditions between the interface being added and
> renamed; the patch we use is sufficient to do that.

	Actually, udev does a much better job at that than ifrename,
as ifrename does not autogenerate persistent rules.

> When we used to call-out ifrename from hotplug, we had incredible
> numbers of problems due to ifrename mis-features and race conditions
> inherent with not controlling the interface renaming ourselves.

	That was more a hotplug issue than a ifrename issue. And
actually, the other big culprit there was ifupdown and its stupid
locking strategy (thank god this is now fixed).

> Scott

	Jean

-------------------------------------------------------------------------
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