Re: [Qemu-devel] [PULL 1/5] acpi-build: append description for non-hotplug

[Gabriel L. Somlo]

On Wed, Feb 19, 2014 at 03:50:22PM +0200, Michael S. Tsirkin wrote:
> On Mon, Feb 17, 2014 at 09:51:39AM -0500, Gabriel L. Somlo wrote:
> > 
> > With this latest version of your patch, I crash during OS X boot with
> > "unable to find driver for this 
> > platform:\"ACPI\".\n"@/SourceCache/xnu/xnu-2050.48.12/iokit/Kernel/IOPlatformExpert.cpp:1514"
> > 
> > Your original patch (slightly doctored since it no longer applies cleanly
> > to the current qemu git master) is included below, and still works for me.
> 
> Any chance below helps on top?

Nope, sorry, I'm getting the same error :(

> Another alternative is that DSDT referencing
> SSDT does not work for apple.
> I hope it's not that, that would be annoying...

I'm probably misunderstanding this in a major way, but when I try the
following, just for grins:

diff --git a/hw/i386/acpi-dsdt-hpet.dsl b/hw/i386/acpi-dsdt-hpet.dsl
index 44961b8..d4614e0 100644
--- a/hw/i386/acpi-dsdt-hpet.dsl
+++ b/hw/i386/acpi-dsdt-hpet.dsl
@@ -36,6 +36,9 @@ Scope(\_SB) {
             If (LOr(LEqual(Local1, 0), LGreater(Local1, 100000000)))
{
                 Return (0x0)
             }
+            If (LEqual(\_SB.FOBR, 0x12)) {
+                Return (0x0)
+            }
             Return (0x0F)
         }
         Name(_CRS, ResourceTemplate() {
diff --git a/hw/i386/ssdt-misc.dsl b/hw/i386/ssdt-misc.dsl
index a4484b8..67d853a 100644
--- a/hw/i386/ssdt-misc.dsl
+++ b/hw/i386/ssdt-misc.dsl
@@ -22,6 +22,10 @@ DefinitionBlock ("ssdt-misc.aml", "SSDT", 0x01,
"BXPC", "BXSS
  * PCI memory ranges
  ****************************************************************/
 
+    Scope(\_SB) {
+       Name(FOBR, 0x12)
+    }
+
     Scope(\) {
        ACPI_EXTRACT_NAME_DWORD_CONST acpi_pci32_start
        Name(P0S, 0x12345678)
---

I get iasl compile errors, never even make it to the point where
OS X would get a chance to be upset with me... :)

Thanks,
--Gabriel

> commit 12b48c660b8316b1e8ff633eb9f3f34bd4b78284
> Author: Michael S. Tsirkin <address@hidden>
> Date:   Wed Feb 19 15:47:03 2014 +0200
> 
>     acpi-build: drop _SUN for non hotpluggable slots
>     
>     Not needed there, let's not add it.
>     
>     Signed-off-by: Michael S. Tsirkin <address@hidden>
> 
> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
> index 5b0bb5a..226e59e 100644
> --- a/hw/i386/acpi-build.c
> +++ b/hw/i386/acpi-build.c
> @@ -644,19 +644,16 @@ static inline char acpi_get_hex(uint32_t val)
>  #define ACPI_PCIHP_AML (ssdp_pcihp_aml + *ssdt_pcihp_start)
>  
>  #define ACPI_PCINOHP_OFFSET_HEX (*ssdt_pcinohp_name - *ssdt_pcinohp_start + 
> 1)
> -#define ACPI_PCINOHP_OFFSET_ID (*ssdt_pcinohp_id - *ssdt_pcinohp_start)
>  #define ACPI_PCINOHP_OFFSET_ADR (*ssdt_pcinohp_adr - *ssdt_pcinohp_start)
>  #define ACPI_PCINOHP_SIZEOF (*ssdt_pcinohp_end - *ssdt_pcinohp_start)
>  #define ACPI_PCINOHP_AML (ssdp_pcihp_aml + *ssdt_pcinohp_start)
>  
>  #define ACPI_PCIVGA_OFFSET_HEX (*ssdt_pcivga_name - *ssdt_pcivga_start + 1)
> -#define ACPI_PCIVGA_OFFSET_ID (*ssdt_pcivga_id - *ssdt_pcivga_start)
>  #define ACPI_PCIVGA_OFFSET_ADR (*ssdt_pcivga_adr - *ssdt_pcivga_start)
>  #define ACPI_PCIVGA_SIZEOF (*ssdt_pcivga_end - *ssdt_pcivga_start)
>  #define ACPI_PCIVGA_AML (ssdp_pcihp_aml + *ssdt_pcivga_start)
>  
>  #define ACPI_PCIQXL_OFFSET_HEX (*ssdt_pciqxl_name - *ssdt_pciqxl_start + 1)
> -#define ACPI_PCIQXL_OFFSET_ID (*ssdt_pciqxl_id - *ssdt_pciqxl_start)
>  #define ACPI_PCIQXL_OFFSET_ADR (*ssdt_pciqxl_adr - *ssdt_pciqxl_start)
>  #define ACPI_PCIQXL_SIZEOF (*ssdt_pciqxl_end - *ssdt_pciqxl_start)
>  #define ACPI_PCIQXL_AML (ssdp_pcihp_aml + *ssdt_pciqxl_start)
> @@ -701,7 +698,6 @@ static void patch_pcinohp(int slot, uint8_t *ssdt_ptr)
>  
>      ssdt_ptr[ACPI_PCINOHP_OFFSET_HEX] = acpi_get_hex(devfn >> 4);
>      ssdt_ptr[ACPI_PCINOHP_OFFSET_HEX + 1] = acpi_get_hex(devfn);
> -    ssdt_ptr[ACPI_PCINOHP_OFFSET_ID] = slot;
>      ssdt_ptr[ACPI_PCINOHP_OFFSET_ADR + 2] = slot;
>  }
>  
> @@ -711,7 +707,6 @@ static void patch_pcivga(int slot, uint8_t *ssdt_ptr)
>  
>      ssdt_ptr[ACPI_PCIVGA_OFFSET_HEX] = acpi_get_hex(devfn >> 4);
>      ssdt_ptr[ACPI_PCIVGA_OFFSET_HEX + 1] = acpi_get_hex(devfn);
> -    ssdt_ptr[ACPI_PCIVGA_OFFSET_ID] = slot;
>      ssdt_ptr[ACPI_PCIVGA_OFFSET_ADR + 2] = slot;
>  }
>  
> @@ -721,7 +716,6 @@ static void patch_pciqxl(int slot, uint8_t *ssdt_ptr)
>  
>      ssdt_ptr[ACPI_PCIQXL_OFFSET_HEX] = acpi_get_hex(devfn >> 4);
>      ssdt_ptr[ACPI_PCIQXL_OFFSET_HEX + 1] = acpi_get_hex(devfn);
> -    ssdt_ptr[ACPI_PCIQXL_OFFSET_ID] = slot;
>      ssdt_ptr[ACPI_PCIQXL_OFFSET_ADR + 2] = slot;
>  }
>  
> diff --git a/hw/i386/ssdt-pcihp.dsl b/hw/i386/ssdt-pcihp.dsl
> index 69a0228..ac91c05 100644
> --- a/hw/i386/ssdt-pcihp.dsl
> +++ b/hw/i386/ssdt-pcihp.dsl
> @@ -53,8 +53,6 @@ DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", 
> "BXSSDTPCIHP", 0x1)
>          // Extract the offsets of the device name, address dword and the slot
>          // name byte - we fill them in for each device.
>          Device(SBB) {
> -            ACPI_EXTRACT_NAME_BYTE_CONST ssdt_pcinohp_id
> -            Name(_SUN, 0xAA)
>              ACPI_EXTRACT_NAME_DWORD_CONST ssdt_pcinohp_adr
>              Name(_ADR, 0xAA0000)
>          }
> @@ -66,8 +64,6 @@ DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", 
> "BXSSDTPCIHP", 0x1)
>          // Extract the offsets of the device name, address dword and the slot
>          // name byte - we fill them in for each device.
>          Device(SCC) {
> -            ACPI_EXTRACT_NAME_BYTE_CONST ssdt_pcivga_id
> -            Name(_SUN, 0xAA)
>              ACPI_EXTRACT_NAME_DWORD_CONST ssdt_pcivga_adr
>              Name(_ADR, 0xAA0000)
>              Method(_S1D, 0, NotSerialized) {
> @@ -88,8 +84,6 @@ DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", 
> "BXSSDTPCIHP", 0x1)
>          // Extract the offsets of the device name, address dword and the slot
>          // name byte - we fill them in for each device.
>          Device(SDD) {
> -            ACPI_EXTRACT_NAME_BYTE_CONST ssdt_pciqxl_id
> -            Name(_SUN, 0xAA)
>              ACPI_EXTRACT_NAME_DWORD_CONST ssdt_pciqxl_adr
>              Name(_ADR, 0xAA0000)
>              Method(_S1D, 0, NotSerialized) {