[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH] hw/ppc: Remove the deprecated spapr-pci-vfio-host
From: |
Alexey Kardashevskiy |
Subject: |
Re: [Qemu-ppc] [PATCH] hw/ppc: Remove the deprecated spapr-pci-vfio-host-bridge device |
Date: |
Tue, 19 Dec 2017 17:42:07 +1100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 19/12/17 04:35, Thomas Huth wrote:
> It's a deprecated dummy device since QEMU v2.6.0. That should have
> been enough time to allow the users to update their scripts in case
> they still use it, so let's remove this legacy code now.
>
> Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Alexey Kardashevskiy <address@hidden>
> ---
> hw/ppc/spapr_pci_vfio.c | 47
> -----------------------------------------------
> qemu-doc.texi | 5 -----
> scripts/device-crash-test | 1 -
> 3 files changed, 53 deletions(-)
>
> diff --git a/hw/ppc/spapr_pci_vfio.c b/hw/ppc/spapr_pci_vfio.c
> index 8448e0b..053efb0 100644
> --- a/hw/ppc/spapr_pci_vfio.c
> +++ b/hw/ppc/spapr_pci_vfio.c
> @@ -29,31 +29,6 @@
> #include "qemu/error-report.h"
> #include "sysemu/qtest.h"
>
> -#define TYPE_SPAPR_PCI_VFIO_HOST_BRIDGE "spapr-pci-vfio-host-bridge"
> -
> -#define SPAPR_PCI_VFIO_HOST_BRIDGE(obj) \
> - OBJECT_CHECK(sPAPRPHBVFIOState, (obj), TYPE_SPAPR_PCI_VFIO_HOST_BRIDGE)
> -
> -typedef struct sPAPRPHBVFIOState sPAPRPHBVFIOState;
> -
> -struct sPAPRPHBVFIOState {
> - sPAPRPHBState phb;
> -
> - int32_t iommugroupid;
> -};
> -
> -static Property spapr_phb_vfio_properties[] = {
> - DEFINE_PROP_INT32("iommu", sPAPRPHBVFIOState, iommugroupid, -1),
> - DEFINE_PROP_END_OF_LIST(),
> -};
> -
> -static void spapr_phb_vfio_instance_init(Object *obj)
> -{
> - if (!qtest_enabled()) {
> - error_report("spapr-pci-vfio-host-bridge is deprecated");
> - }
> -}
> -
> bool spapr_phb_eeh_available(sPAPRPHBState *sphb)
> {
> return vfio_eeh_as_ok(&sphb->iommu_as);
> @@ -218,25 +193,3 @@ int spapr_phb_vfio_eeh_configure(sPAPRPHBState *sphb)
>
> return RTAS_OUT_SUCCESS;
> }
> -
> -static void spapr_phb_vfio_class_init(ObjectClass *klass, void *data)
> -{
> - DeviceClass *dc = DEVICE_CLASS(klass);
> -
> - dc->props = spapr_phb_vfio_properties;
> -}
> -
> -static const TypeInfo spapr_phb_vfio_info = {
> - .name = TYPE_SPAPR_PCI_VFIO_HOST_BRIDGE,
> - .parent = TYPE_SPAPR_PCI_HOST_BRIDGE,
> - .instance_size = sizeof(sPAPRPHBVFIOState),
> - .instance_init = spapr_phb_vfio_instance_init,
> - .class_init = spapr_phb_vfio_class_init,
> -};
> -
> -static void spapr_pci_vfio_register_types(void)
> -{
> - type_register_static(&spapr_phb_vfio_info);
> -}
> -
> -type_init(spapr_pci_vfio_register_types)
> diff --git a/qemu-doc.texi b/qemu-doc.texi
> index f7317df..14d818d 100644
> --- a/qemu-doc.texi
> +++ b/qemu-doc.texi
> @@ -2533,11 +2533,6 @@ The ``host_net_remove'' command is replaced by the
> ``netdev_del'' command.
> The ``ivshmem'' device type is replaced by either the ``ivshmem-plain''
> or ``ivshmem-doorbell`` device types.
>
> address@hidden spapr-pci-vfio-host-bridge (since 2.6.0)
> -
> -The ``spapr-pci-vfio-host-bridge'' device type is replaced by
> -the ``spapr-pci-host-bridge'' device type.
> -
> @section System emulator machines
>
> @subsection Xilinx EP108 (since 2.11.0)
> diff --git a/scripts/device-crash-test b/scripts/device-crash-test
> index 1aca55f..7417177 100755
> --- a/scripts/device-crash-test
> +++ b/scripts/device-crash-test
> @@ -119,7 +119,6 @@ ERROR_WHITELIST = [
> {'device':'scsi-generic', 'expected':True}, # drive property
> not set
> {'device':'scsi-hd', 'expected':True}, # drive property
> not set
> {'device':'spapr-pci-host-bridge', 'expected':True}, # BUID not
> specified for PHB
> - {'device':'spapr-pci-vfio-host-bridge', 'expected':True}, # BUID not
> specified for PHB
> {'device':'spapr-rng', 'expected':True}, # spapr-rng needs
> an RNG backend!
> {'device':'spapr-vty', 'expected':True}, # chardev
> property not set
> {'device':'tpm-tis', 'expected':True}, # tpm_tis:
> backend driver with id (null) could not be found
>
--
Alexey