commit c82f503dd5c3f0a01a9e63741f1f875652669867
author Marcel Apfelbaum <marcel@redhat.com> Mon Mar 07 21:14:37 2016 +0200
committer Michael S. Tsirkin <mst@redhat.com> Fri Mar 11 16:45:21 2016 +0200
tree 637e98c2ac3c39d3caf0d1ca048f9048defe2d95
parent 7335a95abd400127b46f3b641e6cd15dd11703fd

hw/acpi: fix Q35 support for legacy Windows OS

Legacy Windows operating systems like Windows XP and Windows 2003
require _DIS method to be present for all interrupt links.

PC machines already have a no-op implemented for GSI links, add
it also in Q35.

Signed-off-by: Marcel Apfelbaum <marcel@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>

hw/i386/acpi-build.c

diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index dbd0b93..0e32395 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -1556,6 +1556,12 @@
 
     aml_append(dev, aml_name_decl("_CRS", crs));
 
+    /*
+     * _DIS can be no-op because the interrupt cannot be disabled.
+     */
+    method = aml_method("_DIS", 0, AML_NOTSERIALIZED);
+    aml_append(dev, method);
+
     method = aml_method("_SRS", 1, AML_NOTSERIALIZED);
     aml_append(dev, method);