[Pkg-acpi-devel] acpid module loading

Michael Meskes meskes at debian.org
Thu Oct 30 08:39:32 UTC 2008


On Wed, Oct 29, 2008 at 05:34:27PM +0100, Loïc Minier wrote:
>  He suggested improved the wording in the default file, and I noticed
>  additional issues in the init script to enhance / fix it further.

Do your changes fix 502704? Or do you think they might fix it, I take it you
don't have any feedback from Derrick either.

>  - The list of modules which are listed by default in MODULES is
>    probably obsolete: these modules exist, but what about new modules,
>    or modules which might be added in the future?  e.g. bay, dock, sbs
>    etc.

They might have to be added.

>  - Using a list of modules as the default doesn't play particularly nice
>    with people who have custom kernels who possibly lack these modules;
>    of course I could check whether the modules before loading it, but
>    that would also hide errors of listing bogus or obsolete modules and
>    seems generally wrong.  A better default might be no module.
>  - The all setting is awful; it includes platform specific modules.
>    Beside, we completely paper over other interesting modules in other
>    directories.

Agreed. We already ran into problems with all. That's why it is not the default.

>  - This is all more shell which gets run before acpid is started.
> 
>  As a result, I'd like to propose dropping module loading entirely from
>  acpid.init and relying on udev to load modules based on what's present
>  on the system and not supported by the kernel (current behavior anyway:
>  the modules are already loaded when the acpid init script starts!).

Haven't tried but sounds logical to me. Big question is do we really want to
make an update like this for Lenny? If not we should wait until we have fixed
502704 IMO.

>  Another open point is why do people get acpid start errors on boot and
>  not on invoke-rc.d acpid start after boot.  No clue here.  One guess
>  could be that acpid starts too early, but this seems weird to me.  If
>  that's the case though, I think we should patch acpid to wait for stuff
>  to appear instead of failing.  I've requested more info to people
>  seeing this for now.

So this is #502704 I guess. Or do you have more reports like this?

Michael
-- 
Michael Meskes
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
Michael at BorussiaFan dot De, Meskes at (Debian|Postgresql) dot Org
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes at jabber.org
Go VfL Borussia! Go SF 49ers! Use Debian GNU/Linux! Use PostgreSQL!



More information about the Pkg-acpi-devel mailing list