| Message ID | 20240529051539.71210-11-philmd@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | hw/i386: Remove deprecated pc-i440fx-2.0 -> 2.3 machines | expand |
On Wed, 29 May 2024 07:15:26 +0200 Philippe Mathieu-Daudé <philmd@linaro.org> wrote: > PCMachineClass::smbios_uuid_encoded was only used by the > pc-i440fx-2.1 machine, which got removed. It is now always > true, remove it. > > Reviewed-by: Thomas Huth <thuth@redhat.com> > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> > Reviewed-by: Zhao Liu <zhao1.liu@intel.com> Reviewed-by: Igor Mammedov <imammedo@redhat.com> > --- > include/hw/i386/pc.h | 1 - > hw/i386/fw_cfg.c | 3 +-- > hw/i386/pc.c | 1 - > 3 files changed, 1 insertion(+), 4 deletions(-) > > diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h > index db0f8e0e36..bbbf58bd42 100644 > --- a/include/hw/i386/pc.h > +++ b/include/hw/i386/pc.h > @@ -109,7 +109,6 @@ struct PCMachineClass { > /* SMBIOS compat: */ > bool smbios_defaults; > bool smbios_legacy_mode; > - bool smbios_uuid_encoded; > SmbiosEntryPointType default_smbios_ep_type; > > /* RAM / address space compat: */ > diff --git a/hw/i386/fw_cfg.c b/hw/i386/fw_cfg.c > index 6e0d9945d0..f9e8af3bf5 100644 > --- a/hw/i386/fw_cfg.c > +++ b/hw/i386/fw_cfg.c > @@ -63,8 +63,7 @@ void fw_cfg_build_smbios(PCMachineState *pcms, FWCfgState *fw_cfg, > > if (pcmc->smbios_defaults) { > /* These values are guest ABI, do not change */ > - smbios_set_defaults("QEMU", mc->desc, mc->name, > - pcmc->smbios_uuid_encoded); > + smbios_set_defaults("QEMU", mc->desc, mc->name, true); > } > > /* tell smbios about cpuid version and features */ > diff --git a/hw/i386/pc.c b/hw/i386/pc.c > index f27c9fd98c..4b2a29bf08 100644 > --- a/hw/i386/pc.c > +++ b/hw/i386/pc.c > @@ -1790,7 +1790,6 @@ static void pc_machine_class_init(ObjectClass *oc, void *data) > pcmc->has_acpi_build = true; > pcmc->rsdp_in_ram = true; > pcmc->smbios_defaults = true; > - pcmc->smbios_uuid_encoded = true; > pcmc->gigabyte_align = true; > pcmc->has_reserved_memory = true; > pcmc->enforce_aligned_dimm = true;
diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h index db0f8e0e36..bbbf58bd42 100644 --- a/include/hw/i386/pc.h +++ b/include/hw/i386/pc.h @@ -109,7 +109,6 @@ struct PCMachineClass { /* SMBIOS compat: */ bool smbios_defaults; bool smbios_legacy_mode; - bool smbios_uuid_encoded; SmbiosEntryPointType default_smbios_ep_type; /* RAM / address space compat: */ diff --git a/hw/i386/fw_cfg.c b/hw/i386/fw_cfg.c index 6e0d9945d0..f9e8af3bf5 100644 --- a/hw/i386/fw_cfg.c +++ b/hw/i386/fw_cfg.c @@ -63,8 +63,7 @@ void fw_cfg_build_smbios(PCMachineState *pcms, FWCfgState *fw_cfg, if (pcmc->smbios_defaults) { /* These values are guest ABI, do not change */ - smbios_set_defaults("QEMU", mc->desc, mc->name, - pcmc->smbios_uuid_encoded); + smbios_set_defaults("QEMU", mc->desc, mc->name, true); } /* tell smbios about cpuid version and features */ diff --git a/hw/i386/pc.c b/hw/i386/pc.c index f27c9fd98c..4b2a29bf08 100644 --- a/hw/i386/pc.c +++ b/hw/i386/pc.c @@ -1790,7 +1790,6 @@ static void pc_machine_class_init(ObjectClass *oc, void *data) pcmc->has_acpi_build = true; pcmc->rsdp_in_ram = true; pcmc->smbios_defaults = true; - pcmc->smbios_uuid_encoded = true; pcmc->gigabyte_align = true; pcmc->has_reserved_memory = true; pcmc->enforce_aligned_dimm = true;