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 <m...@redhat.com>
> 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 <m...@redhat.com>
>
> 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) {
