Re: [PATCH] An initrd level patch for customized DSDT
On Wed, May 26, 2004 at 01:48:35PM +0200, Christoph Hellwig wrote:
> On Wed, May 26, 2004 at 01:42:47PM +0200, Francesco P. Lovergine wrote:
> > Package: kernel-image
> > Severity: wishlist
> > Tags: patch
> >
> > Please consider
> >
> > http://gaugusch.at/kernel.shtml
> >
> > as a possible stock kernel patch to correct optionally on-fly broken ACPI DSDTs.
> > A kernel-patch package could also be considered, but I see no reason to
> > not provide this optional overriding by default. A mkinitrd patch could
> > also be introduced to add the table to the initrd image.
>
> The patch is completely unacceptable as-is, e.g. putting ACPI-specific
> ifdefs into initramfs.c. The idea looks good and with some more work
> like using the firmware loader interface it should be mergeable
> upstream. So could you please talk to the patch authors and upstream,
> this isn't debian-specific in any way.
>
Ack, a more sane patch should load an alternative DSDT using an
alternative acpi module. I suspect that cannot be done at ACPI
initialization during first phase of boot, due to lack of disk access.
Dunno why none wrote a module to override the table instead of
playing with initrd or so.
The other patch available on the net is more bad, because it add the
local DSDT in the kernel sources. Bleah!
BTW, gaugusch patch is currently present in suse and mandrake kernels.
--
Francesco P. Lovergine
Reply to: