|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [Qemu-devel] [PATCH for-3.2 v3 02/14] qom: make interface types abstract
On Wed, 7 Nov 2018 16:36:40 +0400
Marc-André Lureau <marcandre.lureau@xxxxxxxxxx> wrote:
> Interfaces don't have instance, let's make the interface type really
> abstract to avoid confusion.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
Reviewed-by: Igor Mammedov <imammedo@xxxxxxxxxx>
> ---
> include/hw/acpi/acpi_dev_interface.h | 6 +-----
> include/hw/arm/linux-boot-if.h | 5 +----
> include/hw/fw-path-provider.h | 4 +---
> include/hw/hotplug.h | 6 +-----
> include/hw/intc/intc.h | 4 +---
> include/hw/ipmi/ipmi.h | 4 +---
> include/hw/isa/isa.h | 4 ----
> include/hw/mem/memory-device.h | 4 +---
> include/hw/nmi.h | 4 +---
> include/hw/stream.h | 4 +---
> include/hw/timer/m48t59.h | 4 +---
> include/qom/object_interfaces.h | 6 +-----
> include/sysemu/tpm.h | 4 +---
> target/arm/idau.h | 4 +---
> tests/check-qom-interface.c | 4 +---
> 15 files changed, 14 insertions(+), 53 deletions(-)
>
> diff --git a/include/hw/acpi/acpi_dev_interface.h
> b/include/hw/acpi/acpi_dev_interface.h
> index dabf4c4fc9..43ff119179 100644
> --- a/include/hw/acpi/acpi_dev_interface.h
> +++ b/include/hw/acpi/acpi_dev_interface.h
> @@ -25,11 +25,7 @@ typedef enum {
> INTERFACE_CHECK(AcpiDeviceIf, (obj), \
> TYPE_ACPI_DEVICE_IF)
>
> -
> -typedef struct AcpiDeviceIf {
> - /* <private> */
> - Object Parent;
> -} AcpiDeviceIf;
> +typedef struct AcpiDeviceIf AcpiDeviceIf;
>
> void acpi_send_event(DeviceState *dev, AcpiEventStatusBits event);
>
> diff --git a/include/hw/arm/linux-boot-if.h b/include/hw/arm/linux-boot-if.h
> index aba4479a14..7bbdfd1cc6 100644
> --- a/include/hw/arm/linux-boot-if.h
> +++ b/include/hw/arm/linux-boot-if.h
> @@ -16,10 +16,7 @@
> #define ARM_LINUX_BOOT_IF(obj) \
> INTERFACE_CHECK(ARMLinuxBootIf, (obj), TYPE_ARM_LINUX_BOOT_IF)
>
> -typedef struct ARMLinuxBootIf {
> - /*< private >*/
> - Object parent_obj;
> -} ARMLinuxBootIf;
> +typedef struct ARMLinuxBootIf ARMLinuxBootIf;
>
> typedef struct ARMLinuxBootIfClass {
> /*< private >*/
> diff --git a/include/hw/fw-path-provider.h b/include/hw/fw-path-provider.h
> index 050cb05d92..5df893a3d8 100644
> --- a/include/hw/fw-path-provider.h
> +++ b/include/hw/fw-path-provider.h
> @@ -30,9 +30,7 @@
> #define FW_PATH_PROVIDER(obj) \
> INTERFACE_CHECK(FWPathProvider, (obj), TYPE_FW_PATH_PROVIDER)
>
> -typedef struct FWPathProvider {
> - Object parent_obj;
> -} FWPathProvider;
> +typedef struct FWPathProvider FWPathProvider;
>
> typedef struct FWPathProviderClass {
> InterfaceClass parent_class;
> diff --git a/include/hw/hotplug.h b/include/hw/hotplug.h
> index 1a0516a479..6321e292fd 100644
> --- a/include/hw/hotplug.h
> +++ b/include/hw/hotplug.h
> @@ -23,11 +23,7 @@
> #define HOTPLUG_HANDLER(obj) \
> INTERFACE_CHECK(HotplugHandler, (obj), TYPE_HOTPLUG_HANDLER)
>
> -
> -typedef struct HotplugHandler {
> - /* <private> */
> - Object Parent;
> -} HotplugHandler;
> +typedef struct HotplugHandler HotplugHandler;
>
> /**
> * hotplug_fn:
> diff --git a/include/hw/intc/intc.h b/include/hw/intc/intc.h
> index 27d9828943..fb3e8e621f 100644
> --- a/include/hw/intc/intc.h
> +++ b/include/hw/intc/intc.h
> @@ -15,9 +15,7 @@
> INTERFACE_CHECK(InterruptStatsProvider, (obj), \
> TYPE_INTERRUPT_STATS_PROVIDER)
>
> -typedef struct InterruptStatsProvider {
> - Object parent;
> -} InterruptStatsProvider;
> +typedef struct InterruptStatsProvider InterruptStatsProvider;
>
> typedef struct InterruptStatsProviderClass {
> InterfaceClass parent;
> diff --git a/include/hw/ipmi/ipmi.h b/include/hw/ipmi/ipmi.h
> index 0affe5a4d8..99661d2bf0 100644
> --- a/include/hw/ipmi/ipmi.h
> +++ b/include/hw/ipmi/ipmi.h
> @@ -114,9 +114,7 @@ uint32_t ipmi_next_uuid(void);
> #define IPMI_INTERFACE_GET_CLASS(class) \
> OBJECT_GET_CLASS(IPMIInterfaceClass, (class), TYPE_IPMI_INTERFACE)
>
> -typedef struct IPMIInterface {
> - Object parent;
> -} IPMIInterface;
> +typedef struct IPMIInterface IPMIInterface;
>
> typedef struct IPMIInterfaceClass {
> InterfaceClass parent;
> diff --git a/include/hw/isa/isa.h b/include/hw/isa/isa.h
> index b9dbab24b4..e62ac91c19 100644
> --- a/include/hw/isa/isa.h
> +++ b/include/hw/isa/isa.h
> @@ -43,10 +43,6 @@ static inline uint16_t applesmc_port(void)
> #define ISADMA(obj) \
> INTERFACE_CHECK(IsaDma, (obj), TYPE_ISADMA)
>
> -struct IsaDma {
> - Object parent;
> -};
> -
> typedef enum {
> ISADMA_TRANSFER_VERIFY,
> ISADMA_TRANSFER_READ,
> diff --git a/include/hw/mem/memory-device.h b/include/hw/mem/memory-device.h
> index e904e194d5..0293a96abb 100644
> --- a/include/hw/mem/memory-device.h
> +++ b/include/hw/mem/memory-device.h
> @@ -25,9 +25,7 @@
> #define MEMORY_DEVICE(obj) \
> INTERFACE_CHECK(MemoryDeviceState, (obj), TYPE_MEMORY_DEVICE)
>
> -typedef struct MemoryDeviceState {
> - Object parent_obj;
> -} MemoryDeviceState;
> +typedef struct MemoryDeviceState MemoryDeviceState;
>
> /**
> * MemoryDeviceClass:
> diff --git a/include/hw/nmi.h b/include/hw/nmi.h
> index d092c684a1..ad857f3832 100644
> --- a/include/hw/nmi.h
> +++ b/include/hw/nmi.h
> @@ -34,9 +34,7 @@
> #define NMI(obj) \
> INTERFACE_CHECK(NMI, (obj), TYPE_NMI)
>
> -typedef struct NMIState {
> - Object parent_obj;
> -} NMIState;
> +typedef struct NMIState NMIState;
>
> typedef struct NMIClass {
> InterfaceClass parent_class;
> diff --git a/include/hw/stream.h b/include/hw/stream.h
> index c370ba0c66..15774f07ab 100644
> --- a/include/hw/stream.h
> +++ b/include/hw/stream.h
> @@ -14,9 +14,7 @@
> #define STREAM_SLAVE(obj) \
> INTERFACE_CHECK(StreamSlave, (obj), TYPE_STREAM_SLAVE)
>
> -typedef struct StreamSlave {
> - Object Parent;
> -} StreamSlave;
> +typedef struct StreamSlave StreamSlave;
>
> typedef void (*StreamCanPushNotifyFn)(void *opaque);
>
> diff --git a/include/hw/timer/m48t59.h b/include/hw/timer/m48t59.h
> index db5e43a8da..6f8db04fce 100644
> --- a/include/hw/timer/m48t59.h
> +++ b/include/hw/timer/m48t59.h
> @@ -13,9 +13,7 @@
> #define NVRAM(obj) \
> INTERFACE_CHECK(Nvram, (obj), TYPE_NVRAM)
>
> -typedef struct Nvram {
> - Object parent;
> -} Nvram;
> +typedef struct Nvram Nvram;
>
> typedef struct NvramClass {
> InterfaceClass parent;
> diff --git a/include/qom/object_interfaces.h b/include/qom/object_interfaces.h
> index 4d513fb329..652a16d2ba 100644
> --- a/include/qom/object_interfaces.h
> +++ b/include/qom/object_interfaces.h
> @@ -16,11 +16,7 @@
> INTERFACE_CHECK(UserCreatable, (obj), \
> TYPE_USER_CREATABLE)
>
> -
> -typedef struct UserCreatable {
> - /* <private> */
> - Object Parent;
> -} UserCreatable;
> +typedef struct UserCreatable UserCreatable;
>
> /**
> * UserCreatableClass:
> diff --git a/include/sysemu/tpm.h b/include/sysemu/tpm.h
> index 17a97ed77a..5b541a71c8 100644
> --- a/include/sysemu/tpm.h
> +++ b/include/sysemu/tpm.h
> @@ -33,9 +33,7 @@ typedef enum TPMVersion {
> #define TPM_IF(obj) \
> INTERFACE_CHECK(TPMIf, (obj), TYPE_TPM_IF)
>
> -typedef struct TPMIf {
> - Object parent_obj;
> -} TPMIf;
> +typedef struct TPMIf TPMIf;
>
> typedef struct TPMIfClass {
> InterfaceClass parent_class;
> diff --git a/target/arm/idau.h b/target/arm/idau.h
> index cac27b95fa..7c0e4e3776 100644
> --- a/target/arm/idau.h
> +++ b/target/arm/idau.h
> @@ -38,9 +38,7 @@
> #define IDAU_INTERFACE_GET_CLASS(obj) \
> OBJECT_GET_CLASS(IDAUInterfaceClass, (obj), TYPE_IDAU_INTERFACE)
>
> -typedef struct IDAUInterface {
> - Object parent;
> -} IDAUInterface;
> +typedef struct IDAUInterface IDAUInterface;
>
> #define IREGION_NOTVALID -1
>
> diff --git a/tests/check-qom-interface.c b/tests/check-qom-interface.c
> index f87c9aaa8a..2177f0dce5 100644
> --- a/tests/check-qom-interface.c
> +++ b/tests/check-qom-interface.c
> @@ -23,9 +23,7 @@
> #define TEST_IF(obj) \
> INTERFACE_CHECK(TestIf, (obj), TYPE_TEST_IF)
>
> -typedef struct TestIf {
> - Object parent_obj;
> -} TestIf;
> +typedef struct TestIf TestIf;
>
> typedef struct TestIfClass {
> InterfaceClass parent_class;
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |