[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 22/34] i386: Hyper-V VMBus ACPI DSDT entry
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [RFC PATCH 22/34] i386: Hyper-V VMBus ACPI DSDT entry |
Date: |
Wed, 7 Feb 2018 12:05:23 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 |
On 06/02/2018 21:30, Roman Kagan wrote:
> From: Andrey Smetanin <address@hidden>
>
> Guest OS uses ACPI to discover vmbus presence. Add a corresponding
> entry to DSDT in case vmbus has been enabled.
>
> Experimentally Windows guests were found to require this entry to
> include two IRQ resources, so this patch adds two semi-arbitrarily
> chosen ones (7 and 13). This results, in particular, in parallel port
> conflicting with vmbus.
>
> TODO: discover and use spare IRQs to avoid conflicts.
Could they be level-triggered IRQs? If so you could reuse 10 and 11
which are used by PCI INTx. Otherwise, the parallel port conflict is
not a huge deal.
> Signed-off-by: Evgeny Yakovlev <address@hidden>
Somewhat messy SoB chain. :)
Paolo
> Signed-off-by: Roman Kagan <address@hidden>
> ---
> hw/i386/acpi-build.c | 42 ++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 42 insertions(+)
>
> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
> index ed78c4ed9f..6f8cd3eb41 100644
> --- a/hw/i386/acpi-build.c
> +++ b/hw/i386/acpi-build.c
> @@ -46,6 +46,7 @@
> #include "sysemu/tpm_backend.h"
> #include "hw/timer/mc146818rtc_regs.h"
> #include "sysemu/numa.h"
> +#include "hw/vmbus/vmbus.h"
>
> /* Supported chipsets: */
> #include "hw/acpi/piix4.h"
> @@ -1317,6 +1318,43 @@ static Aml *build_com_device_aml(uint8_t uid)
> return dev;
> }
>
> +static Aml *build_vmbus_device_aml(void)
> +{
> + Aml *dev;
> + Aml *method;
> + Aml *crs;
> +
> + dev = aml_device("VMBS");
> + aml_append(dev, aml_name_decl("STA", aml_int(0xF)));
> + aml_append(dev, aml_name_decl("_HID", aml_string("VMBus")));
> + aml_append(dev, aml_name_decl("_UID", aml_int(0x0)));
> + aml_append(dev, aml_name_decl("_DDN", aml_string("VMBUS")));
> +
> + method = aml_method("_DIS", 0, AML_NOTSERIALIZED);
> + aml_append(method, aml_store(aml_and(aml_name("STA"), aml_int(0xD),
> NULL),
> + aml_name("STA")));
> + aml_append(dev, method);
> +
> + method = aml_method("_PS0", 0, AML_NOTSERIALIZED);
> + aml_append(method, aml_store(aml_or(aml_name("STA"), aml_int(0xF), NULL),
> + aml_name("STA")));
> + aml_append(dev, method);
> +
> + method = aml_method("_STA", 0, AML_NOTSERIALIZED);
> + aml_append(method, aml_store(aml_name("STA"), aml_local(0)));
> + aml_append(method, aml_return(aml_local(0)));
> + aml_append(dev, method);
> +
> + aml_append(dev, aml_name_decl("_PS3", aml_int(0x0)));
> +
> + crs = aml_resource_template();
> + aml_append(crs, aml_irq_no_flags(7));
> + aml_append(crs, aml_irq_no_flags(13));
> + aml_append(dev, aml_name_decl("_CRS", crs));
> +
> + return dev;
> +}
> +
> static void build_isa_devices_aml(Aml *table)
> {
> ISADevice *fdc = pc_find_fdc0();
> @@ -1343,6 +1381,10 @@ static void build_isa_devices_aml(Aml *table)
> build_acpi_ipmi_devices(scope, BUS(obj));
> }
>
> + if (vmbus_exists()) {
> + aml_append(scope, build_vmbus_device_aml());
> + }
> +
> aml_append(table, scope);
> }
>
>
[Qemu-devel] [RFC PATCH 23/34] i386: en/disable vmbus by a machine property, Roman Kagan, 2018/02/06
[Qemu-devel] [RFC PATCH 26/34] hv-scsi: limit the number of requests per notification, Roman Kagan, 2018/02/06
[Qemu-devel] [RFC PATCH 24/34] scsi: add Hyper-V/VMBus SCSI protocol definitions, Roman Kagan, 2018/02/06
[Qemu-devel] [RFC PATCH 20/34] vmbus: vmbus implementation, Roman Kagan, 2018/02/06