This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


[Xen-devel] RE: [PATCH] remove _PS0 from the DSDT

To: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-devel] RE: [PATCH] remove _PS0 from the DSDT
From: "Kay, Allen M" <allen.m.kay@xxxxxxxxx>
Date: Fri, 25 Feb 2011 16:02:22 -0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxx>, "Jean.Guyader@xxxxxxxxxx" <Jean.Guyader@xxxxxxxxxx>
Delivery-date: Fri, 25 Feb 2011 16:03:24 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <alpine.DEB.2.00.1102231405070.17894@kaball-desktop>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <alpine.DEB.2.00.1102231405070.17894@kaball-desktop>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcvTZH5j//N6dhgeToWbQNbFWEBPvwB4zC5g
Thread-topic: [PATCH] remove _PS0 from the DSDT
Hi Stefano,

I just tried this patch booting Win7-64bit in following two configurations:

    1) Passthrough two NIC devices - onboard + PCIe E1000
    2) Passthrough SNB IGD + USB + audio + NIC

In both cases, Windows failed to boot complaining about BIOS is not ACPI 

If I don't passthrough any devices, Windows can boot successfully.


-----Original Message-----
From: Stefano Stabellini [mailto:stefano.stabellini@xxxxxxxxxxxxx] 
Sent: Wednesday, February 23, 2011 6:18 AM
To: xen-devel@xxxxxxxxxxxxxxxxxxx
Cc: Kay, Allen M; Ian Campbell; Jean.Guyader@xxxxxxxxxx
Subject: [PATCH] remove _PS0 from the DSDT

This patch removes all the _PS0 entries from the DSDT.
The reason for removing them is that if a passthrough device doesn't
have power management capabilities declared in its pci config space but
_PS0 is declared in the DSDT a Linux HVM guest gets confused and cannot
set the device to D0.  In order to reproduce this problem you can try to
assign a VF to a Linux HVM guest, the result is a guest kernel crash as
reported in this bug:
I tested this patch with Linux and Windows guests and it looks like it
is not breaking anything (guest S3 in particular is working).
However I am afraid it could cause unwanted side effects, specifically
it could break gfx passthrough of Intel GPUs (I haven't tested this on xen
4.1 but it looks like it would break gfx passthrough on xen 3.4).
Could anyone from Intel confirm whether this patch is correct or might
cause any problems?

Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>

diff -r 13a7d1f7f62c tools/firmware/hvmloader/acpi/mk_dsdt.c
--- a/tools/firmware/hvmloader/acpi/mk_dsdt.c   Mon Feb 21 09:11:57 2011 +0000
+++ b/tools/firmware/hvmloader/acpi/mk_dsdt.c   Wed Feb 23 12:27:35 2011 +0000
@@ -252,7 +252,6 @@ int main(int argc, char **argv)
      *  _EJ0: eject a device
      *  _STA: return a device's status, e.g. enabled or removed
      * Other methods are optional: 
-     *  _PS0/3: put them here for debug purpose
      * Eject button would generate a general-purpose event, then the
      * control method for this event uses Notify() to inform OSPM which
@@ -271,10 +270,6 @@ int main(int argc, char **argv)
         stmt("Name", "_ADR, 0x%08x", ((slot & ~7) << 13) | (slot & 7));
         /* _SUN == dev */
         stmt("Name", "_SUN, 0x%08x", slot >> 3);
-        push_block("Method", "_PS0, 0");
-        stmt("Store", "0x%02x, \\_GPE.DPT1", slot);
-        stmt("Store", "0x80, \\_GPE.DPT2");
-        pop_block();
         push_block("Method", "_PS3, 0");
         stmt("Store", "0x%02x, \\_GPE.DPT1", slot);
         stmt("Store", "0x83, \\_GPE.DPT2");

Xen-devel mailing list