cvs commit: src/usr.sbin/acpi/acpidump acpi.c acpidump.c
robert.moore at intel.com
Tue Oct 5 15:37:37 PDT 2004
I can add this next time I open up this interface.
I think there are a couple of other requests outstanding on
AcpiOsTableOverride(), perhaps in the SSDT area, I forget.
> -----Original Message-----
> From: Nate Lawson [mailto:nate at root.org]
> Sent: Tuesday, October 05, 2004 2:31 PM
> To: John Baldwin
> Cc: Takanori Watanabe; src-committers at FreeBSD.org;
cvs-src at FreeBSD.org;
> cvs-all at FreeBSD.org
> Subject: Re: cvs commit: src/usr.sbin/acpi/acpidump acpi.c acpidump.c
> Nate Lawson wrote:
> > John Baldwin wrote:
> >> I agree. It makes the user's lives a lot easier if they procedure
> >> remains:
> >> - acpidump > foo.asl
> >> - hack on foo.asl to fix it
> >> - compile foo.asl to foo.dsdt
> >> - load foo.dsdt in the loader
> >> and have it all just work. Actually, you know, we could make this
> >> work on our own without requiring ACPI-CA to change. You just need
> >> hack AcpiOsTableOverride() in OsdTable.c to return a pointer to an
> >> empty table for "SSDT" tables when the DSDT is overridden. In
> >> think that is probably where this behavior decision really belongs.
> > Good idea, I'll look into this. If feasible, I'll commit this and
> > revert the acpidump change.
> Ok, I've committed a tested version with this approach. The only
> annoying part is that setting NewTable to NULL means "don't override
> this one" so I had to supply an empty no-op table in place of the
> I backed out the acpidump changes. It would be better if ACPI-CA
> an option where returning AE_NO_TABLES or something would mean "ignore
> this table completely". Right now, you can only override or leave the
> default table.
More information about the cvs-src