[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/6] hw/arm/virt: Remove device tree restriction for virtio-i
From: |
Eric Auger |
Subject: |
Re: [PATCH 4/6] hw/arm/virt: Remove device tree restriction for virtio-iommu |
Date: |
Tue, 17 Aug 2021 15:42:22 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 |
Hi Jean,
On 8/10/21 10:45 AM, Jean-Philippe Brucker wrote:
> virtio-iommu is now supported with ACPI VIOT as well as device tree.
> Remove the restriction that prevents from instantiating a virtio-iommu
> device under ACPI.
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> hw/arm/virt.c | 10 ++--------
> hw/virtio/virtio-iommu-pci.c | 7 -------
> 2 files changed, 2 insertions(+), 15 deletions(-)
>
> diff --git a/hw/arm/virt.c b/hw/arm/virt.c
> index 81eda46b0b..b4598d3fe6 100644
> --- a/hw/arm/virt.c
> +++ b/hw/arm/virt.c
> @@ -2551,16 +2551,10 @@ static HotplugHandler
> *virt_machine_get_hotplug_handler(MachineState *machine,
> MachineClass *mc = MACHINE_GET_CLASS(machine);
>
> if (device_is_dynamic_sysbus(mc, dev) ||
> - (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM))) {
> + object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM) ||
> + object_dynamic_cast(OBJECT(dev), TYPE_VIRTIO_IOMMU_PCI)) {
> return HOTPLUG_HANDLER(machine);
> }
> - if (object_dynamic_cast(OBJECT(dev), TYPE_VIRTIO_IOMMU_PCI)) {
> - VirtMachineState *vms = VIRT_MACHINE(machine);
> -
> - if (!vms->bootinfo.firmware_loaded || !virt_is_acpi_enabled(vms)) {
> - return HOTPLUG_HANDLER(machine);
> - }
> - }
> return NULL;
> }
>
> diff --git a/hw/virtio/virtio-iommu-pci.c b/hw/virtio/virtio-iommu-pci.c
> index 770c286be7..f30eb16cbf 100644
> --- a/hw/virtio/virtio-iommu-pci.c
> +++ b/hw/virtio/virtio-iommu-pci.c
> @@ -48,16 +48,9 @@ static void virtio_iommu_pci_realize(VirtIOPCIProxy
> *vpci_dev, Error **errp)
> VirtIOIOMMU *s = VIRTIO_IOMMU(vdev);
>
> if (!qdev_get_machine_hotplug_handler(DEVICE(vpci_dev))) {
> - MachineClass *mc = MACHINE_GET_CLASS(qdev_get_machine());
> -
> - error_setg(errp,
> - "%s machine fails to create iommu-map device tree
> bindings",
> - mc->name);
> error_append_hint(errp,
> "Check your machine implements a hotplug handler "
> "for the virtio-iommu-pci device\n");
> - error_append_hint(errp, "Check the guest is booted without FW or
> with "
> - "-no-acpi\n");
We may check the vms->iommu is not already set to something else (to
VIRT_IOMMU_SMMUV3 for instance).
Thanks
Eric
> return;
> }
> for (int i = 0; i < s->nb_reserved_regions; i++) {
- [PATCH 0/6] virtio-iommu: Add ACPI support, Jean-Philippe Brucker, 2021/08/10
- [PATCH 1/6] acpi: Add VIOT structure definitions, Jean-Philippe Brucker, 2021/08/10
- [PATCH 3/6] hw/arm/virt-acpi-build: Add VIOT table for virtio-iommu, Jean-Philippe Brucker, 2021/08/10
- [PATCH 2/6] hw/acpi: Add VIOT table, Jean-Philippe Brucker, 2021/08/10
- [PATCH 4/6] hw/arm/virt: Remove device tree restriction for virtio-iommu, Jean-Philippe Brucker, 2021/08/10
- Re: [PATCH 4/6] hw/arm/virt: Remove device tree restriction for virtio-iommu,
Eric Auger <=
- [PATCH 5/6] pc: Add VIOT table for virtio-iommu, Jean-Philippe Brucker, 2021/08/10
- [PATCH 6/6] pc: Allow instantiating a virtio-iommu device, Jean-Philippe Brucker, 2021/08/10
- Re: [PATCH 0/6] virtio-iommu: Add ACPI support, Eric Auger, 2021/08/17