acpid initscript speedup

[Alexander Butenko]

Binary package hint: acpid

in order to build list of loaded modules we getting this information from /proc/modules instead of running extra external command

got rid of find (1) and grep (1) in favor of checking files with '-f '.

using insmod instead of modprobe because acpi modules dont have any depends.

[Daniel Hahler (blueyed)]

Thank you for your patch, which looks interesting.

Can you please attach it as a unified patch ("diff -u")? This makes it more robust against other code changes, so it can be reviewed later (instead of not applying anymore)

[Alexander Butenko (avb)]

patch speedup a process for a while, but ideal way is to get rid of insmod here and just compile acpi stuff in kernel. We are not gaining nothing having them as modules once we are not dealing with apm stuff anymore.

[Daniel Hahler (blueyed)]

Thank you for updating the patch.

I'm marking the bug as triaged and hopefully someone who can make more sense out of it than me gets around incorporating it.

[Chris Coulson (chrisccoulson)]

Subscribed sponsors

[Martin Mai (mrkanister-deactivatedaccount-deactivatedaccount)]

Here is a debdiff that fixes it for Jaunty.

[Colin Watson (cjwatson)]

Martin, there is really no need to "reformat" plain 'diff -u' patches as debdiffs; they can be sponsored perfectly well without this. (Any developer worth their salt can apply a patch and add a changelog entry.)

If you do reformat a patch as a debdiff, you must credit the original contributor! It's not fair to take their work and not mention them.

[Martin Mai (mrkanister-deactivatedaccount-deactivatedaccount)]

Sorry. I heard about it being easier for uploaders, if the patch already exists as a debdiff.

[Colin Watson (cjwatson)]

That's an old myth; it doesn't. (It might make five seconds difference, perhaps, but certainly not enough to justify the effort.)

[Scott James Remnant (Canonical) (canonical-scott)]

Sorry, missed this one being assigned to me - will look in karmic

[Scott James Remnant (Canonical) (canonical-scott)]

Instead we dropped all that and ported acpid to use upstart