[SeaBIOS] seabios fails to build with current IASL

Kevin O'Connor kevin at koconnor.net
Sun Jul 15 16:21:01 CEST 2018


On Mon, Jul 02, 2018 at 10:35:30AM -0400, Kevin O'Connor wrote:
> On Mon, Jul 02, 2018 at 09:10:34AM +0200, Gerd Hoffmann wrote:
> > The prebuild tables are there exactly to sidestep any iasl version
> > dependencies.  We had iasl trouble in the past (IIRC that was newer
> > iasl generating aml opcodes not supported by older windows versions),
> > and keeping up the tables which are legacy only and not expected to
> > ever change didn't look like being worth it.  So we added tables build
> > with a known-good iasl version to the repo.  I think you can find the
> > full story in the commit log.
> > 
> > If you don't want use the prebuilt tables I think the sane options are
> > (a) use a older iasl compiler, or (b) flip ACPI_DSDT to "n" and drop
> > support for qemu 1.3 + older.
> 
> I agree with the above.  That said, the patch is trivial enough that I
> don't think there's any harm in committing it.

I confirmed that older versions of iasl were truncating the table
names to 8 characters, and thus the patch has no negative impact.  So,
I went ahead and committed it.

-Kevin


> diff --git a/src/fw/ssdt-misc.dsl b/src/fw/ssdt-misc.dsl
> index acc850e..d1d2c9e 100644
> --- a/src/fw/ssdt-misc.dsl
> +++ b/src/fw/ssdt-misc.dsl
> @@ -1,6 +1,6 @@
>  ACPI_EXTRACT_ALL_CODE ssdp_misc_aml
> 
> -DefinitionBlock ("ssdt-misc.aml", "SSDT", 0x01, "BXPC", "BXSSDTSUSP", 0x1)
> +DefinitionBlock ("ssdt-misc.aml", "SSDT", 0x01, "BXPC", "BXSSDTSU", 0x1)
>  {
> 
>  /****************************************************************
> diff --git a/src/fw/ssdt-pcihp.dsl b/src/fw/ssdt-pcihp.dsl
> index cb24c11..518a5eb 100644
> --- a/src/fw/ssdt-pcihp.dsl
> +++ b/src/fw/ssdt-pcihp.dsl
> @@ -1,6 +1,6 @@
>  ACPI_EXTRACT_ALL_CODE ssdp_pcihp_aml
> 
> -DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", "BXSSDTPCIHP", 0x1)
> +DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", "BXSSDTPC", 0x1)
>  {
> 
>  /****************************************************************
> 



More information about the SeaBIOS mailing list