* [PATCH v2] hw/smbios: Add table 4 parameter, "processor-id"
@ 2022-01-24 20:11 Patrick Venture
2022-01-25 13:35 ` Igor Mammedov
0 siblings, 1 reply; 3+ messages in thread
From: Patrick Venture @ 2022-01-24 20:11 UTC (permalink / raw)
To: mst, imammedo, ani
Cc: qemu-devel, Patrick Venture, Peter Foley, Titus Rwantare
This parameter is to be used in the processor_id lower 32-bit entry in
the type 4 table. The upper 32-bits represent the features for the CPU.
This patch leaves those as 0 when the lower 32-bits are set.
This parameter is set as optional and if left will use the values from
the CPU model.
This enables hiding the host information from the guest and allowing AMD
VMs to run pretending to be Intel for some userspace software concerns.
Reviewed-by: Peter Foley <pefoley@google.com>
Reviewed-by: Titus Rwantare <titusr@google.com>
Signed-off-by: Patrick Venture <venture@google.com>
---
v2: Added to SRST options list, upgraded to full 64-bit value.
---
hw/smbios/smbios.c | 19 ++++++++++++++++---
qemu-options.hx | 3 ++-
2 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c
index 7397e56737..385b69b0c9 100644
--- a/hw/smbios/smbios.c
+++ b/hw/smbios/smbios.c
@@ -104,9 +104,11 @@ static struct {
const char *sock_pfx, *manufacturer, *version, *serial, *asset, *part;
uint64_t max_speed;
uint64_t current_speed;
+ uint64_t processor_id;
} type4 = {
.max_speed = DEFAULT_CPU_SPEED,
- .current_speed = DEFAULT_CPU_SPEED
+ .current_speed = DEFAULT_CPU_SPEED,
+ .processor_id = 0,
};
static struct {
@@ -327,6 +329,10 @@ static const QemuOptDesc qemu_smbios_type4_opts[] = {
.name = "part",
.type = QEMU_OPT_STRING,
.help = "part number",
+ }, {
+ .name = "processor-id",
+ .type = QEMU_OPT_NUMBER,
+ .help = "processor id",
},
{ /* end of list */ }
};
@@ -669,8 +675,13 @@ static void smbios_build_type_4_table(MachineState *ms, unsigned instance)
t->processor_type = 0x03; /* CPU */
t->processor_family = 0x01; /* Other */
SMBIOS_TABLE_SET_STR(4, processor_manufacturer_str, type4.manufacturer);
- t->processor_id[0] = cpu_to_le32(smbios_cpuid_version);
- t->processor_id[1] = cpu_to_le32(smbios_cpuid_features);
+ if (type4.processor_id == 0) {
+ t->processor_id[0] = cpu_to_le32(smbios_cpuid_version);
+ t->processor_id[1] = cpu_to_le32(smbios_cpuid_features);
+ } else {
+ t->processor_id[0] = cpu_to_le32((uint32_t)type4.processor_id);
+ t->processor_id[1] = cpu_to_le32(type4.processor_id >> 32);
+ }
SMBIOS_TABLE_SET_STR(4, processor_version_str, type4.version);
t->voltage = 0;
t->external_clock = cpu_to_le16(0); /* Unknown */
@@ -1292,6 +1303,8 @@ void smbios_entry_add(QemuOpts *opts, Error **errp)
save_opt(&type4.serial, opts, "serial");
save_opt(&type4.asset, opts, "asset");
save_opt(&type4.part, opts, "part");
+ /* If the value is 0, it will take the value from the CPU model. */
+ type4.processor_id = qemu_opt_get_number(opts, "processor-id", 0);
type4.max_speed = qemu_opt_get_number(opts, "max-speed",
DEFAULT_CPU_SPEED);
type4.current_speed = qemu_opt_get_number(opts, "current-speed",
diff --git a/qemu-options.hx b/qemu-options.hx
index ec90505d84..6256417935 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -2527,6 +2527,7 @@ DEF("smbios", HAS_ARG, QEMU_OPTION_smbios,
" specify SMBIOS type 3 fields\n"
"-smbios type=4[,sock_pfx=str][,manufacturer=str][,version=str][,serial=str]\n"
" [,asset=str][,part=str][,max-speed=%d][,current-speed=%d]\n"
+ " [,processor-id=%d]\n"
" specify SMBIOS type 4 fields\n"
"-smbios type=11[,value=str][,path=filename]\n"
" specify SMBIOS type 11 fields\n"
@@ -2552,7 +2553,7 @@ SRST
``-smbios type=3[,manufacturer=str][,version=str][,serial=str][,asset=str][,sku=str]``
Specify SMBIOS type 3 fields
-``-smbios type=4[,sock_pfx=str][,manufacturer=str][,version=str][,serial=str][,asset=str][,part=str]``
+``-smbios type=4[,sock_pfx=str][,manufacturer=str][,version=str][,serial=str][,asset=str][,part=str][,processor-id=%d]``
Specify SMBIOS type 4 fields
``-smbios type=11[,value=str][,path=filename]``
--
2.35.0.rc0.227.g00780c9af4-goog
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] hw/smbios: Add table 4 parameter, "processor-id"
2022-01-24 20:11 [PATCH v2] hw/smbios: Add table 4 parameter, "processor-id" Patrick Venture
@ 2022-01-25 13:35 ` Igor Mammedov
2022-01-25 16:26 ` Patrick Venture
0 siblings, 1 reply; 3+ messages in thread
From: Igor Mammedov @ 2022-01-25 13:35 UTC (permalink / raw)
To: Patrick Venture; +Cc: ani, Titus Rwantare, Peter Foley, qemu-devel, mst
On Mon, 24 Jan 2022 12:11:51 -0800
Patrick Venture <venture@google.com> wrote:
> This parameter is to be used in the processor_id lower 32-bit entry in
> the type 4 table. The upper 32-bits represent the features for the CPU.
> This patch leaves those as 0 when the lower 32-bits are set.
Did you forget to update commit message ^^^^ ?
Other than that patch looks good to me,
so with commit message fixed:
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
> This parameter is set as optional and if left will use the values from
> the CPU model.
>
> This enables hiding the host information from the guest and allowing AMD
> VMs to run pretending to be Intel for some userspace software concerns.
>
> Reviewed-by: Peter Foley <pefoley@google.com>
> Reviewed-by: Titus Rwantare <titusr@google.com>
> Signed-off-by: Patrick Venture <venture@google.com>
> ---
> v2: Added to SRST options list, upgraded to full 64-bit value.
> ---
> hw/smbios/smbios.c | 19 ++++++++++++++++---
> qemu-options.hx | 3 ++-
> 2 files changed, 18 insertions(+), 4 deletions(-)
>
> diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c
> index 7397e56737..385b69b0c9 100644
> --- a/hw/smbios/smbios.c
> +++ b/hw/smbios/smbios.c
> @@ -104,9 +104,11 @@ static struct {
> const char *sock_pfx, *manufacturer, *version, *serial, *asset, *part;
> uint64_t max_speed;
> uint64_t current_speed;
> + uint64_t processor_id;
> } type4 = {
> .max_speed = DEFAULT_CPU_SPEED,
> - .current_speed = DEFAULT_CPU_SPEED
> + .current_speed = DEFAULT_CPU_SPEED,
> + .processor_id = 0,
> };
>
> static struct {
> @@ -327,6 +329,10 @@ static const QemuOptDesc qemu_smbios_type4_opts[] = {
> .name = "part",
> .type = QEMU_OPT_STRING,
> .help = "part number",
> + }, {
> + .name = "processor-id",
> + .type = QEMU_OPT_NUMBER,
> + .help = "processor id",
> },
> { /* end of list */ }
> };
> @@ -669,8 +675,13 @@ static void smbios_build_type_4_table(MachineState *ms, unsigned instance)
> t->processor_type = 0x03; /* CPU */
> t->processor_family = 0x01; /* Other */
> SMBIOS_TABLE_SET_STR(4, processor_manufacturer_str, type4.manufacturer);
> - t->processor_id[0] = cpu_to_le32(smbios_cpuid_version);
> - t->processor_id[1] = cpu_to_le32(smbios_cpuid_features);
> + if (type4.processor_id == 0) {
> + t->processor_id[0] = cpu_to_le32(smbios_cpuid_version);
> + t->processor_id[1] = cpu_to_le32(smbios_cpuid_features);
> + } else {
> + t->processor_id[0] = cpu_to_le32((uint32_t)type4.processor_id);
> + t->processor_id[1] = cpu_to_le32(type4.processor_id >> 32);
> + }
> SMBIOS_TABLE_SET_STR(4, processor_version_str, type4.version);
> t->voltage = 0;
> t->external_clock = cpu_to_le16(0); /* Unknown */
> @@ -1292,6 +1303,8 @@ void smbios_entry_add(QemuOpts *opts, Error **errp)
> save_opt(&type4.serial, opts, "serial");
> save_opt(&type4.asset, opts, "asset");
> save_opt(&type4.part, opts, "part");
> + /* If the value is 0, it will take the value from the CPU model. */
> + type4.processor_id = qemu_opt_get_number(opts, "processor-id", 0);
> type4.max_speed = qemu_opt_get_number(opts, "max-speed",
> DEFAULT_CPU_SPEED);
> type4.current_speed = qemu_opt_get_number(opts, "current-speed",
> diff --git a/qemu-options.hx b/qemu-options.hx
> index ec90505d84..6256417935 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -2527,6 +2527,7 @@ DEF("smbios", HAS_ARG, QEMU_OPTION_smbios,
> " specify SMBIOS type 3 fields\n"
> "-smbios type=4[,sock_pfx=str][,manufacturer=str][,version=str][,serial=str]\n"
> " [,asset=str][,part=str][,max-speed=%d][,current-speed=%d]\n"
> + " [,processor-id=%d]\n"
> " specify SMBIOS type 4 fields\n"
> "-smbios type=11[,value=str][,path=filename]\n"
> " specify SMBIOS type 11 fields\n"
> @@ -2552,7 +2553,7 @@ SRST
> ``-smbios type=3[,manufacturer=str][,version=str][,serial=str][,asset=str][,sku=str]``
> Specify SMBIOS type 3 fields
>
> -``-smbios type=4[,sock_pfx=str][,manufacturer=str][,version=str][,serial=str][,asset=str][,part=str]``
> +``-smbios type=4[,sock_pfx=str][,manufacturer=str][,version=str][,serial=str][,asset=str][,part=str][,processor-id=%d]``
> Specify SMBIOS type 4 fields
>
> ``-smbios type=11[,value=str][,path=filename]``
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] hw/smbios: Add table 4 parameter, "processor-id"
2022-01-25 13:35 ` Igor Mammedov
@ 2022-01-25 16:26 ` Patrick Venture
0 siblings, 0 replies; 3+ messages in thread
From: Patrick Venture @ 2022-01-25 16:26 UTC (permalink / raw)
To: Igor Mammedov; +Cc: mst, ani, QEMU Developers, Peter Foley, Titus Rwantare
[-- Attachment #1: Type: text/plain, Size: 4933 bytes --]
On Tue, Jan 25, 2022 at 5:35 AM Igor Mammedov <imammedo@redhat.com> wrote:
> On Mon, 24 Jan 2022 12:11:51 -0800
> Patrick Venture <venture@google.com> wrote:
>
> > This parameter is to be used in the processor_id lower 32-bit entry in
> > the type 4 table. The upper 32-bits represent the features for the CPU.
> > This patch leaves those as 0 when the lower 32-bits are set.
>
> Did you forget to update commit message ^^^^ ?
>
I did. Oops :) I will send a quick v3 with that fix.
>
> Other than that patch looks good to me,
> so with commit message fixed:
>
> Reviewed-by: Igor Mammedov <imammedo@redhat.com>
>
> > This parameter is set as optional and if left will use the values from
> > the CPU model.
> >
> > This enables hiding the host information from the guest and allowing AMD
> > VMs to run pretending to be Intel for some userspace software concerns.
> >
> > Reviewed-by: Peter Foley <pefoley@google.com>
> > Reviewed-by: Titus Rwantare <titusr@google.com>
> > Signed-off-by: Patrick Venture <venture@google.com>
> > ---
> > v2: Added to SRST options list, upgraded to full 64-bit value.
> > ---
> > hw/smbios/smbios.c | 19 ++++++++++++++++---
> > qemu-options.hx | 3 ++-
> > 2 files changed, 18 insertions(+), 4 deletions(-)
> >
> > diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c
> > index 7397e56737..385b69b0c9 100644
> > --- a/hw/smbios/smbios.c
> > +++ b/hw/smbios/smbios.c
> > @@ -104,9 +104,11 @@ static struct {
> > const char *sock_pfx, *manufacturer, *version, *serial, *asset,
> *part;
> > uint64_t max_speed;
> > uint64_t current_speed;
> > + uint64_t processor_id;
> > } type4 = {
> > .max_speed = DEFAULT_CPU_SPEED,
> > - .current_speed = DEFAULT_CPU_SPEED
> > + .current_speed = DEFAULT_CPU_SPEED,
> > + .processor_id = 0,
> > };
> >
> > static struct {
> > @@ -327,6 +329,10 @@ static const QemuOptDesc qemu_smbios_type4_opts[] =
> {
> > .name = "part",
> > .type = QEMU_OPT_STRING,
> > .help = "part number",
> > + }, {
> > + .name = "processor-id",
> > + .type = QEMU_OPT_NUMBER,
> > + .help = "processor id",
> > },
> > { /* end of list */ }
> > };
> > @@ -669,8 +675,13 @@ static void smbios_build_type_4_table(MachineState
> *ms, unsigned instance)
> > t->processor_type = 0x03; /* CPU */
> > t->processor_family = 0x01; /* Other */
> > SMBIOS_TABLE_SET_STR(4, processor_manufacturer_str,
> type4.manufacturer);
> > - t->processor_id[0] = cpu_to_le32(smbios_cpuid_version);
> > - t->processor_id[1] = cpu_to_le32(smbios_cpuid_features);
> > + if (type4.processor_id == 0) {
> > + t->processor_id[0] = cpu_to_le32(smbios_cpuid_version);
> > + t->processor_id[1] = cpu_to_le32(smbios_cpuid_features);
> > + } else {
> > + t->processor_id[0] = cpu_to_le32((uint32_t)type4.processor_id);
> > + t->processor_id[1] = cpu_to_le32(type4.processor_id >> 32);
> > + }
> > SMBIOS_TABLE_SET_STR(4, processor_version_str, type4.version);
> > t->voltage = 0;
> > t->external_clock = cpu_to_le16(0); /* Unknown */
> > @@ -1292,6 +1303,8 @@ void smbios_entry_add(QemuOpts *opts, Error **errp)
> > save_opt(&type4.serial, opts, "serial");
> > save_opt(&type4.asset, opts, "asset");
> > save_opt(&type4.part, opts, "part");
> > + /* If the value is 0, it will take the value from the CPU
> model. */
> > + type4.processor_id = qemu_opt_get_number(opts,
> "processor-id", 0);
> > type4.max_speed = qemu_opt_get_number(opts, "max-speed",
> > DEFAULT_CPU_SPEED);
> > type4.current_speed = qemu_opt_get_number(opts,
> "current-speed",
> > diff --git a/qemu-options.hx b/qemu-options.hx
> > index ec90505d84..6256417935 100644
> > --- a/qemu-options.hx
> > +++ b/qemu-options.hx
> > @@ -2527,6 +2527,7 @@ DEF("smbios", HAS_ARG, QEMU_OPTION_smbios,
> > " specify SMBIOS type 3 fields\n"
> > "-smbios
> type=4[,sock_pfx=str][,manufacturer=str][,version=str][,serial=str]\n"
> > "
> [,asset=str][,part=str][,max-speed=%d][,current-speed=%d]\n"
> > + " [,processor-id=%d]\n"
> > " specify SMBIOS type 4 fields\n"
> > "-smbios type=11[,value=str][,path=filename]\n"
> > " specify SMBIOS type 11 fields\n"
> > @@ -2552,7 +2553,7 @@ SRST
> > ``-smbios
> type=3[,manufacturer=str][,version=str][,serial=str][,asset=str][,sku=str]``
> > Specify SMBIOS type 3 fields
> >
> > -``-smbios
> type=4[,sock_pfx=str][,manufacturer=str][,version=str][,serial=str][,asset=str][,part=str]``
> > +``-smbios
> type=4[,sock_pfx=str][,manufacturer=str][,version=str][,serial=str][,asset=str][,part=str][,processor-id=%d]``
> > Specify SMBIOS type 4 fields
> >
> > ``-smbios type=11[,value=str][,path=filename]``
>
>
[-- Attachment #2: Type: text/html, Size: 6793 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-01-25 16:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-01-24 20:11 [PATCH v2] hw/smbios: Add table 4 parameter, "processor-id" Patrick Venture
2022-01-25 13:35 ` Igor Mammedov
2022-01-25 16:26 ` Patrick Venture
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).