-----BEGIN PGP SIGNED MESSAGE-----
Hash: RIPEMD160

Dan Nicholson wrote:
> On 8/15/07, Kevin Day <[EMAIL PROTECTED]> wrote:
>> All this patch does is change the in kernel default to 
>> /sbin/udevtrigger from /sbin/hotplug.
>> This way there should be no need to echo '/sbin/udevtrigger' >>
>> /proc/sys/kernel/hotplug
> 
> But we `echo "" > /proc/sys/kernel/hotplug' so the kernel won't 
> execute anything on uevents. It would be a bad idea to execute 
> udevtrigger for every event processed. It would be better if you just
> changed it to be "". Unless I'm missing something.

Nope, you're not missing anything -- the current LFS udev script echos
nothing into /proc/sys/kernel/hotplug before starting udevd; see line 77
of lfs/init.d/udev in the trunk/bootscripts repo.

(It could also echo the empty string into /sys/kernel/uevent_helper,
since that's the newer file to use.  The other may end up going away in
a future kernel.)

And udevtrigger is *completely* the wrong program to be using for that
anyway -- udevtrigger creates fake uevents for *all* kobjects, not just
the "current" kobject.  You'll end up with O(N^2) uevents, instead of
plain old O(N), because for every uevent, you'll re-run the events for
all the old kobjects also.  Bad, bad idea...  ;-)

I would have no problem with setting it to "", though.  Well, as long as
nobody uses the hotplug program in their initramfs (and nobody should
be, now that udevtrigger exists, but I don't know for sure).
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGw3n4S5vET1Wea5wRA/MgAKC7R1NKYvfm8n1s3lDOQ4ZjRilURQCfT5qP
PYPPUGeAngPue8LgXIfoZJA=
=SaVG
-----END PGP SIGNATURE-----
-- 
http://linuxfromscratch.org/mailman/listinfo/patches
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page

Reply via email to