udev udevstart merge

udev udevstart merge

[Kay Sievers]

http://linuxusb.bkbits.net:8080/udev/cset@1.831?nav=index.html|ChangeSet@-2d

Isn't it pretty ugly to decide the behavior of a binary by its name?

+	if (strstr(argv[0], "udevstart"))
+		is_udevstart = 1;

And why you don't just link the necesssary objects to udevstart?
It is only run once on a booting system.

And at last, why don't we see patches like this on the list?

Kay


-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
_______________________________________________
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

Re: udev udevstart merge

[Greg KH]

On Tue, Aug 10, 2004 at 05:11:04PM +0200, Kay Sievers wrote:
> 
> http://linuxusb.bkbits.net:8080/udev/cset@1.831?nav=index.html|ChangeSet@-2d
> 
> Isn't it pretty ugly to decide the behavior of a binary by its name?
> 
> +	if (strstr(argv[0], "udevstart"))
> +		is_udevstart = 1;

It's the way programs like busybox works.

> And why you don't just link the necesssary objects to udevstart?
> It is only run once on a booting system.

The embedded people wanted to make things smaller.  Overall it is.

> And at last, why don't we see patches like this on the list?

Bug Tom about that one...

thanks,

greg k-h


-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
_______________________________________________
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

Re: udev udevstart merge

[Tom Rini]

On Tue, Aug 10, 2004 at 04:34:28PM -0700, Greg KH wrote:

> On Tue, Aug 10, 2004 at 05:11:04PM +0200, Kay Sievers wrote:
> > 
> > http://linuxusb.bkbits.net:8080/udev/cset@1.831?nav=index.html|ChangeSet@-2d
> > 
> > Isn't it pretty ugly to decide the behavior of a binary by its name?
> > 
> > +	if (strstr(argv[0], "udevstart"))
> > +		is_udevstart = 1;
> 
> It's the way programs like busybox works.
> 
> > And why you don't just link the necesssary objects to udevstart?
> > It is only run once on a booting system.
> 
> The embedded people wanted to make things smaller.  Overall it is.

And, faster.  While it's not a noticable difference on a desktop box, it
is on slower embedded ones.

> > And at last, why don't we see patches like this on the list?
> 
> Bug Tom about that one...

I didn't know there was a list, but I'm on it now.

-- 
Tom Rini
http://gate.crashing.org/~trini/


-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
_______________________________________________
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