diff mbox series

[v1,1/2] hw/i386: Make pit a property of common x86 base machine type

Message ID 20220111073528.1771552-2-xiaoyao.li@intel.com (mailing list archive)
State New, archived
Headers show
Series i386: Make PIT and PIC the property of common x86 base machine type | expand

Commit Message

Xiaoyao Li Jan. 11, 2022, 7:35 a.m. UTC
Both pc and microvm have pit property individually. Let's just make it
the property of common x86 base machine type.

Signed-off-by: Xiaoyao Li <xiaoyao.li@intel.com>
---
 hw/i386/microvm.c         | 27 +--------------------------
 hw/i386/pc.c              | 24 +++---------------------
 hw/i386/x86.c             | 25 +++++++++++++++++++++++++
 include/hw/i386/microvm.h |  2 --
 include/hw/i386/pc.h      |  2 --
 include/hw/i386/x86.h     |  2 ++
 6 files changed, 31 insertions(+), 51 deletions(-)

Comments

Sergio Lopez Jan. 11, 2022, 11:07 a.m. UTC | #1
On Tue, Jan 11, 2022 at 03:35:27PM +0800, Xiaoyao Li wrote:
> Both pc and microvm have pit property individually. Let's just make it
> the property of common x86 base machine type.
> 
> Signed-off-by: Xiaoyao Li <xiaoyao.li@intel.com>
> ---
>  hw/i386/microvm.c         | 27 +--------------------------
>  hw/i386/pc.c              | 24 +++---------------------
>  hw/i386/x86.c             | 25 +++++++++++++++++++++++++
>  include/hw/i386/microvm.h |  2 --
>  include/hw/i386/pc.h      |  2 --
>  include/hw/i386/x86.h     |  2 ++
>  6 files changed, 31 insertions(+), 51 deletions(-)

Reviewed-by: Sergio Lopez <slp@redhat.com>
Xiaoyao Li Jan. 12, 2022, 1:41 a.m. UTC | #2
+ Paolo

On 1/11/2022 3:35 PM, Xiaoyao Li wrote:
> Both pc and microvm have pit property individually. Let's just make it
> the property of common x86 base machine type.
> 
> Signed-off-by: Xiaoyao Li <xiaoyao.li@intel.com>
> ---
>   hw/i386/microvm.c         | 27 +--------------------------
>   hw/i386/pc.c              | 24 +++---------------------
>   hw/i386/x86.c             | 25 +++++++++++++++++++++++++
>   include/hw/i386/microvm.h |  2 --
>   include/hw/i386/pc.h      |  2 --
>   include/hw/i386/x86.h     |  2 ++
>   6 files changed, 31 insertions(+), 51 deletions(-)
> 
> diff --git a/hw/i386/microvm.c b/hw/i386/microvm.c
> index 4b3b1dd262f1..89b555a2f584 100644
> --- a/hw/i386/microvm.c
> +++ b/hw/i386/microvm.c
> @@ -257,7 +257,7 @@ static void microvm_devices_init(MicrovmMachineState *mms)
>           g_free(i8259);
>       }
>   
> -    if (mms->pit == ON_OFF_AUTO_ON || mms->pit == ON_OFF_AUTO_AUTO) {
> +    if (x86ms->pit == ON_OFF_AUTO_ON || x86ms->pit == ON_OFF_AUTO_AUTO) {
>           if (kvm_pit_in_kernel()) {
>               kvm_pit_init(isa_bus, 0x40);
>           } else {
> @@ -508,23 +508,6 @@ static void microvm_machine_set_pic(Object *obj, Visitor *v, const char *name,
>       visit_type_OnOffAuto(v, name, &mms->pic, errp);
>   }
>   
> -static void microvm_machine_get_pit(Object *obj, Visitor *v, const char *name,
> -                                    void *opaque, Error **errp)
> -{
> -    MicrovmMachineState *mms = MICROVM_MACHINE(obj);
> -    OnOffAuto pit = mms->pit;
> -
> -    visit_type_OnOffAuto(v, name, &pit, errp);
> -}
> -
> -static void microvm_machine_set_pit(Object *obj, Visitor *v, const char *name,
> -                                    void *opaque, Error **errp)
> -{
> -    MicrovmMachineState *mms = MICROVM_MACHINE(obj);
> -
> -    visit_type_OnOffAuto(v, name, &mms->pit, errp);
> -}
> -
>   static void microvm_machine_get_rtc(Object *obj, Visitor *v, const char *name,
>                                       void *opaque, Error **errp)
>   {
> @@ -650,7 +633,6 @@ static void microvm_machine_initfn(Object *obj)
>   
>       /* Configuration */
>       mms->pic = ON_OFF_AUTO_AUTO;
> -    mms->pit = ON_OFF_AUTO_AUTO;
>       mms->rtc = ON_OFF_AUTO_AUTO;
>       mms->pcie = ON_OFF_AUTO_AUTO;
>       mms->ioapic2 = ON_OFF_AUTO_AUTO;
> @@ -709,13 +691,6 @@ static void microvm_class_init(ObjectClass *oc, void *data)
>       object_class_property_set_description(oc, MICROVM_MACHINE_PIC,
>           "Enable i8259 PIC");
>   
> -    object_class_property_add(oc, MICROVM_MACHINE_PIT, "OnOffAuto",
> -                              microvm_machine_get_pit,
> -                              microvm_machine_set_pit,
> -                              NULL, NULL);
> -    object_class_property_set_description(oc, MICROVM_MACHINE_PIT,
> -        "Enable i8254 PIT");
> -
>       object_class_property_add(oc, MICROVM_MACHINE_RTC, "OnOffAuto",
>                                 microvm_machine_get_rtc,
>                                 microvm_machine_set_rtc,
> diff --git a/hw/i386/pc.c b/hw/i386/pc.c
> index c8696ac01e85..48ab4cf44012 100644
> --- a/hw/i386/pc.c
> +++ b/hw/i386/pc.c
> @@ -1071,6 +1071,7 @@ void pc_basic_device_init(struct PCMachineState *pcms,
>       ISADevice *pit = NULL;
>       MemoryRegion *ioport80_io = g_new(MemoryRegion, 1);
>       MemoryRegion *ioportF0_io = g_new(MemoryRegion, 1);
> +    X86MachineState *x86ms = X86_MACHINE(pcms);
>   
>       memory_region_init_io(ioport80_io, NULL, &ioport80_io_ops, NULL, "ioport80", 1);
>       memory_region_add_subregion(isa_bus->address_space_io, 0x80, ioport80_io);
> @@ -1115,7 +1116,8 @@ void pc_basic_device_init(struct PCMachineState *pcms,
>   
>       qemu_register_boot_set(pc_boot_set, *rtc_state);
>   
> -    if (!xen_enabled() && pcms->pit_enabled) {
> +    if (!xen_enabled() &&
> +        (x86ms->pit == ON_OFF_AUTO_AUTO || x86ms->pit == ON_OFF_AUTO_ON)) {
>           if (kvm_pit_in_kernel()) {
>               pit = kvm_pit_init(isa_bus, 0x40);
>           } else {
> @@ -1484,20 +1486,6 @@ static void pc_machine_set_sata(Object *obj, bool value, Error **errp)
>       pcms->sata_enabled = value;
>   }
>   
> -static bool pc_machine_get_pit(Object *obj, Error **errp)
> -{
> -    PCMachineState *pcms = PC_MACHINE(obj);
> -
> -    return pcms->pit_enabled;
> -}
> -
> -static void pc_machine_set_pit(Object *obj, bool value, Error **errp)
> -{
> -    PCMachineState *pcms = PC_MACHINE(obj);
> -
> -    pcms->pit_enabled = value;
> -}
> -
>   static bool pc_machine_get_hpet(Object *obj, Error **errp)
>   {
>       PCMachineState *pcms = PC_MACHINE(obj);
> @@ -1640,7 +1628,6 @@ static void pc_machine_initfn(Object *obj)
>       pcms->acpi_build_enabled = PC_MACHINE_GET_CLASS(pcms)->has_acpi_build;
>       pcms->smbus_enabled = true;
>       pcms->sata_enabled = true;
> -    pcms->pit_enabled = true;
>       pcms->max_fw_size = 8 * MiB;
>   #ifdef CONFIG_HPET
>       pcms->hpet_enabled = true;
> @@ -1767,11 +1754,6 @@ static void pc_machine_class_init(ObjectClass *oc, void *data)
>       object_class_property_set_description(oc, PC_MACHINE_SATA,
>           "Enable/disable Serial ATA bus");
>   
> -    object_class_property_add_bool(oc, PC_MACHINE_PIT,
> -        pc_machine_get_pit, pc_machine_set_pit);
> -    object_class_property_set_description(oc, PC_MACHINE_PIT,
> -        "Enable/disable Intel 8254 programmable interval timer emulation");
> -
>       object_class_property_add_bool(oc, "hpet",
>           pc_machine_get_hpet, pc_machine_set_hpet);
>       object_class_property_set_description(oc, "hpet",
> diff --git a/hw/i386/x86.c b/hw/i386/x86.c
> index b84840a1bb99..744a50937761 100644
> --- a/hw/i386/x86.c
> +++ b/hw/i386/x86.c
> @@ -1226,6 +1226,23 @@ static void x86_machine_set_acpi(Object *obj, Visitor *v, const char *name,
>       visit_type_OnOffAuto(v, name, &x86ms->acpi, errp);
>   }
>   
> +static void x86_machine_get_pit(Object *obj, Visitor *v, const char *name,
> +                                    void *opaque, Error **errp)
> +{
> +    X86MachineState *x86ms = X86_MACHINE(obj);
> +    OnOffAuto pit = x86ms->pit;
> +
> +    visit_type_OnOffAuto(v, name, &pit, errp);
> +}
> +
> +static void x86_machine_set_pit(Object *obj, Visitor *v, const char *name,
> +                                    void *opaque, Error **errp)
> +{
> +    X86MachineState *x86ms = X86_MACHINE(obj);;
> +
> +    visit_type_OnOffAuto(v, name, &x86ms->pit, errp);
> +}
> +
>   static char *x86_machine_get_oem_id(Object *obj, Error **errp)
>   {
>       X86MachineState *x86ms = X86_MACHINE(obj);
> @@ -1315,6 +1332,7 @@ static void x86_machine_initfn(Object *obj)
>   
>       x86ms->smm = ON_OFF_AUTO_AUTO;
>       x86ms->acpi = ON_OFF_AUTO_AUTO;
> +    x86ms->pit = ON_OFF_AUTO_AUTO;
>       x86ms->pci_irq_mask = ACPI_BUILD_PCI_IRQS;
>       x86ms->oem_id = g_strndup(ACPI_BUILD_APPNAME6, 6);
>       x86ms->oem_table_id = g_strndup(ACPI_BUILD_APPNAME8, 8);
> @@ -1347,6 +1365,13 @@ static void x86_machine_class_init(ObjectClass *oc, void *data)
>       object_class_property_set_description(oc, X86_MACHINE_ACPI,
>           "Enable ACPI");
>   
> +    object_class_property_add(oc, X86_MACHINE_PIT, "OnOffAuto",
> +                              x86_machine_get_pit,
> +                              x86_machine_set_pit,
> +                              NULL, NULL);
> +    object_class_property_set_description(oc, X86_MACHINE_PIT,
> +        "Enable i8254 PIT");
> +
>       object_class_property_add_str(oc, X86_MACHINE_OEM_ID,
>                                     x86_machine_get_oem_id,
>                                     x86_machine_set_oem_id);
> diff --git a/include/hw/i386/microvm.h b/include/hw/i386/microvm.h
> index efcbd926fd43..83f9ac0b2ad0 100644
> --- a/include/hw/i386/microvm.h
> +++ b/include/hw/i386/microvm.h
> @@ -67,7 +67,6 @@
>   #define PCIE_ECAM_SIZE        0x10000000
>   
>   /* Machine type options */
> -#define MICROVM_MACHINE_PIT                 "pit"
>   #define MICROVM_MACHINE_PIC                 "pic"
>   #define MICROVM_MACHINE_RTC                 "rtc"
>   #define MICROVM_MACHINE_PCIE                "pcie"
> @@ -87,7 +86,6 @@ struct MicrovmMachineState {
>   
>       /* Machine type options */
>       OnOffAuto pic;
> -    OnOffAuto pit;
>       OnOffAuto rtc;
>       OnOffAuto pcie;
>       OnOffAuto ioapic2;
> diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h
> index 9c9f4ac74810..01e84d5c4aa4 100644
> --- a/include/hw/i386/pc.h
> +++ b/include/hw/i386/pc.h
> @@ -46,7 +46,6 @@ typedef struct PCMachineState {
>       bool acpi_build_enabled;
>       bool smbus_enabled;
>       bool sata_enabled;
> -    bool pit_enabled;
>       bool hpet_enabled;
>       bool default_bus_bypass_iommu;
>       uint64_t max_fw_size;
> @@ -63,7 +62,6 @@ typedef struct PCMachineState {
>   #define PC_MACHINE_VMPORT           "vmport"
>   #define PC_MACHINE_SMBUS            "smbus"
>   #define PC_MACHINE_SATA             "sata"
> -#define PC_MACHINE_PIT              "pit"
>   #define PC_MACHINE_MAX_FW_SIZE      "max-fw-size"
>   #define PC_MACHINE_SMBIOS_EP        "smbios-entry-point-type"
>   
> diff --git a/include/hw/i386/x86.h b/include/hw/i386/x86.h
> index a145a303703f..f0168902356d 100644
> --- a/include/hw/i386/x86.h
> +++ b/include/hw/i386/x86.h
> @@ -67,6 +67,7 @@ struct X86MachineState {
>   
>       OnOffAuto smm;
>       OnOffAuto acpi;
> +    OnOffAuto pit;
>   
>       char *oem_id;
>       char *oem_table_id;
> @@ -86,6 +87,7 @@ struct X86MachineState {
>   
>   #define X86_MACHINE_SMM              "smm"
>   #define X86_MACHINE_ACPI             "acpi"
> +#define X86_MACHINE_PIT              "pit"
>   #define X86_MACHINE_OEM_ID           "x-oem-id"
>   #define X86_MACHINE_OEM_TABLE_ID     "x-oem-table-id"
>   #define X86_MACHINE_BUS_LOCK_RATELIMIT  "bus-lock-ratelimit"
diff mbox series

Patch

diff --git a/hw/i386/microvm.c b/hw/i386/microvm.c
index 4b3b1dd262f1..89b555a2f584 100644
--- a/hw/i386/microvm.c
+++ b/hw/i386/microvm.c
@@ -257,7 +257,7 @@  static void microvm_devices_init(MicrovmMachineState *mms)
         g_free(i8259);
     }
 
-    if (mms->pit == ON_OFF_AUTO_ON || mms->pit == ON_OFF_AUTO_AUTO) {
+    if (x86ms->pit == ON_OFF_AUTO_ON || x86ms->pit == ON_OFF_AUTO_AUTO) {
         if (kvm_pit_in_kernel()) {
             kvm_pit_init(isa_bus, 0x40);
         } else {
@@ -508,23 +508,6 @@  static void microvm_machine_set_pic(Object *obj, Visitor *v, const char *name,
     visit_type_OnOffAuto(v, name, &mms->pic, errp);
 }
 
-static void microvm_machine_get_pit(Object *obj, Visitor *v, const char *name,
-                                    void *opaque, Error **errp)
-{
-    MicrovmMachineState *mms = MICROVM_MACHINE(obj);
-    OnOffAuto pit = mms->pit;
-
-    visit_type_OnOffAuto(v, name, &pit, errp);
-}
-
-static void microvm_machine_set_pit(Object *obj, Visitor *v, const char *name,
-                                    void *opaque, Error **errp)
-{
-    MicrovmMachineState *mms = MICROVM_MACHINE(obj);
-
-    visit_type_OnOffAuto(v, name, &mms->pit, errp);
-}
-
 static void microvm_machine_get_rtc(Object *obj, Visitor *v, const char *name,
                                     void *opaque, Error **errp)
 {
@@ -650,7 +633,6 @@  static void microvm_machine_initfn(Object *obj)
 
     /* Configuration */
     mms->pic = ON_OFF_AUTO_AUTO;
-    mms->pit = ON_OFF_AUTO_AUTO;
     mms->rtc = ON_OFF_AUTO_AUTO;
     mms->pcie = ON_OFF_AUTO_AUTO;
     mms->ioapic2 = ON_OFF_AUTO_AUTO;
@@ -709,13 +691,6 @@  static void microvm_class_init(ObjectClass *oc, void *data)
     object_class_property_set_description(oc, MICROVM_MACHINE_PIC,
         "Enable i8259 PIC");
 
-    object_class_property_add(oc, MICROVM_MACHINE_PIT, "OnOffAuto",
-                              microvm_machine_get_pit,
-                              microvm_machine_set_pit,
-                              NULL, NULL);
-    object_class_property_set_description(oc, MICROVM_MACHINE_PIT,
-        "Enable i8254 PIT");
-
     object_class_property_add(oc, MICROVM_MACHINE_RTC, "OnOffAuto",
                               microvm_machine_get_rtc,
                               microvm_machine_set_rtc,
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index c8696ac01e85..48ab4cf44012 100644
--- a/hw/i386/pc.c
+++ b/hw/i386/pc.c
@@ -1071,6 +1071,7 @@  void pc_basic_device_init(struct PCMachineState *pcms,
     ISADevice *pit = NULL;
     MemoryRegion *ioport80_io = g_new(MemoryRegion, 1);
     MemoryRegion *ioportF0_io = g_new(MemoryRegion, 1);
+    X86MachineState *x86ms = X86_MACHINE(pcms);
 
     memory_region_init_io(ioport80_io, NULL, &ioport80_io_ops, NULL, "ioport80", 1);
     memory_region_add_subregion(isa_bus->address_space_io, 0x80, ioport80_io);
@@ -1115,7 +1116,8 @@  void pc_basic_device_init(struct PCMachineState *pcms,
 
     qemu_register_boot_set(pc_boot_set, *rtc_state);
 
-    if (!xen_enabled() && pcms->pit_enabled) {
+    if (!xen_enabled() &&
+        (x86ms->pit == ON_OFF_AUTO_AUTO || x86ms->pit == ON_OFF_AUTO_ON)) {
         if (kvm_pit_in_kernel()) {
             pit = kvm_pit_init(isa_bus, 0x40);
         } else {
@@ -1484,20 +1486,6 @@  static void pc_machine_set_sata(Object *obj, bool value, Error **errp)
     pcms->sata_enabled = value;
 }
 
-static bool pc_machine_get_pit(Object *obj, Error **errp)
-{
-    PCMachineState *pcms = PC_MACHINE(obj);
-
-    return pcms->pit_enabled;
-}
-
-static void pc_machine_set_pit(Object *obj, bool value, Error **errp)
-{
-    PCMachineState *pcms = PC_MACHINE(obj);
-
-    pcms->pit_enabled = value;
-}
-
 static bool pc_machine_get_hpet(Object *obj, Error **errp)
 {
     PCMachineState *pcms = PC_MACHINE(obj);
@@ -1640,7 +1628,6 @@  static void pc_machine_initfn(Object *obj)
     pcms->acpi_build_enabled = PC_MACHINE_GET_CLASS(pcms)->has_acpi_build;
     pcms->smbus_enabled = true;
     pcms->sata_enabled = true;
-    pcms->pit_enabled = true;
     pcms->max_fw_size = 8 * MiB;
 #ifdef CONFIG_HPET
     pcms->hpet_enabled = true;
@@ -1767,11 +1754,6 @@  static void pc_machine_class_init(ObjectClass *oc, void *data)
     object_class_property_set_description(oc, PC_MACHINE_SATA,
         "Enable/disable Serial ATA bus");
 
-    object_class_property_add_bool(oc, PC_MACHINE_PIT,
-        pc_machine_get_pit, pc_machine_set_pit);
-    object_class_property_set_description(oc, PC_MACHINE_PIT,
-        "Enable/disable Intel 8254 programmable interval timer emulation");
-
     object_class_property_add_bool(oc, "hpet",
         pc_machine_get_hpet, pc_machine_set_hpet);
     object_class_property_set_description(oc, "hpet",
diff --git a/hw/i386/x86.c b/hw/i386/x86.c
index b84840a1bb99..744a50937761 100644
--- a/hw/i386/x86.c
+++ b/hw/i386/x86.c
@@ -1226,6 +1226,23 @@  static void x86_machine_set_acpi(Object *obj, Visitor *v, const char *name,
     visit_type_OnOffAuto(v, name, &x86ms->acpi, errp);
 }
 
+static void x86_machine_get_pit(Object *obj, Visitor *v, const char *name,
+                                    void *opaque, Error **errp)
+{
+    X86MachineState *x86ms = X86_MACHINE(obj);
+    OnOffAuto pit = x86ms->pit;
+
+    visit_type_OnOffAuto(v, name, &pit, errp);
+}
+
+static void x86_machine_set_pit(Object *obj, Visitor *v, const char *name,
+                                    void *opaque, Error **errp)
+{
+    X86MachineState *x86ms = X86_MACHINE(obj);;
+
+    visit_type_OnOffAuto(v, name, &x86ms->pit, errp);
+}
+
 static char *x86_machine_get_oem_id(Object *obj, Error **errp)
 {
     X86MachineState *x86ms = X86_MACHINE(obj);
@@ -1315,6 +1332,7 @@  static void x86_machine_initfn(Object *obj)
 
     x86ms->smm = ON_OFF_AUTO_AUTO;
     x86ms->acpi = ON_OFF_AUTO_AUTO;
+    x86ms->pit = ON_OFF_AUTO_AUTO;
     x86ms->pci_irq_mask = ACPI_BUILD_PCI_IRQS;
     x86ms->oem_id = g_strndup(ACPI_BUILD_APPNAME6, 6);
     x86ms->oem_table_id = g_strndup(ACPI_BUILD_APPNAME8, 8);
@@ -1347,6 +1365,13 @@  static void x86_machine_class_init(ObjectClass *oc, void *data)
     object_class_property_set_description(oc, X86_MACHINE_ACPI,
         "Enable ACPI");
 
+    object_class_property_add(oc, X86_MACHINE_PIT, "OnOffAuto",
+                              x86_machine_get_pit,
+                              x86_machine_set_pit,
+                              NULL, NULL);
+    object_class_property_set_description(oc, X86_MACHINE_PIT,
+        "Enable i8254 PIT");
+
     object_class_property_add_str(oc, X86_MACHINE_OEM_ID,
                                   x86_machine_get_oem_id,
                                   x86_machine_set_oem_id);
diff --git a/include/hw/i386/microvm.h b/include/hw/i386/microvm.h
index efcbd926fd43..83f9ac0b2ad0 100644
--- a/include/hw/i386/microvm.h
+++ b/include/hw/i386/microvm.h
@@ -67,7 +67,6 @@ 
 #define PCIE_ECAM_SIZE        0x10000000
 
 /* Machine type options */
-#define MICROVM_MACHINE_PIT                 "pit"
 #define MICROVM_MACHINE_PIC                 "pic"
 #define MICROVM_MACHINE_RTC                 "rtc"
 #define MICROVM_MACHINE_PCIE                "pcie"
@@ -87,7 +86,6 @@  struct MicrovmMachineState {
 
     /* Machine type options */
     OnOffAuto pic;
-    OnOffAuto pit;
     OnOffAuto rtc;
     OnOffAuto pcie;
     OnOffAuto ioapic2;
diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h
index 9c9f4ac74810..01e84d5c4aa4 100644
--- a/include/hw/i386/pc.h
+++ b/include/hw/i386/pc.h
@@ -46,7 +46,6 @@  typedef struct PCMachineState {
     bool acpi_build_enabled;
     bool smbus_enabled;
     bool sata_enabled;
-    bool pit_enabled;
     bool hpet_enabled;
     bool default_bus_bypass_iommu;
     uint64_t max_fw_size;
@@ -63,7 +62,6 @@  typedef struct PCMachineState {
 #define PC_MACHINE_VMPORT           "vmport"
 #define PC_MACHINE_SMBUS            "smbus"
 #define PC_MACHINE_SATA             "sata"
-#define PC_MACHINE_PIT              "pit"
 #define PC_MACHINE_MAX_FW_SIZE      "max-fw-size"
 #define PC_MACHINE_SMBIOS_EP        "smbios-entry-point-type"
 
diff --git a/include/hw/i386/x86.h b/include/hw/i386/x86.h
index a145a303703f..f0168902356d 100644
--- a/include/hw/i386/x86.h
+++ b/include/hw/i386/x86.h
@@ -67,6 +67,7 @@  struct X86MachineState {
 
     OnOffAuto smm;
     OnOffAuto acpi;
+    OnOffAuto pit;
 
     char *oem_id;
     char *oem_table_id;
@@ -86,6 +87,7 @@  struct X86MachineState {
 
 #define X86_MACHINE_SMM              "smm"
 #define X86_MACHINE_ACPI             "acpi"
+#define X86_MACHINE_PIT              "pit"
 #define X86_MACHINE_OEM_ID           "x-oem-id"
 #define X86_MACHINE_OEM_TABLE_ID     "x-oem-table-id"
 #define X86_MACHINE_BUS_LOCK_RATELIMIT  "bus-lock-ratelimit"