On Sun, Jul 10, 2022 at 12:48:04PM +0200, Hal Martin wrote:
>
> This patch adds support for SMBIOS type 8 to qemu.
>
> internal_reference: internal reference designator
> external_reference: external reference designator
> connector_type: hex value for port connector type (see SMBIOS 7.9.2)
> port_type: hex value for port type (see SMBIOS 7.9.3)
>
> After studying various vendor implementationsi (Dell, Lenovo, MSI),
> the value of internal connector type was hard-coded to 0x0 (None).
>
> Example usage:
> -smbios
> type=8,internal_reference=JUSB1,external_reference=USB1,connector_type=0x12,port_type=0x10
> \
> -smbios type=8,internal_reference=JAUD1,external_reference="Audio
> Jack",connector_type=0x1f,port_type=0x1d \
> -smbios
> type=8,internal_reference=LAN,external_reference=Ethernet,connector_type=0x0b,port_type=0x1f
> \
> -smbios
> type=8,internal_reference=PS2,external_reference=Mouse,connector_type=0x0f,port_type=0x0e
> \
> -smbios
> type=8,internal_reference=PS2,external_reference=Keyboard,connector_type=0x0f,port_type=0x0d
>
> Signed-off-by: Hal Martin <hal.mar...@gmail.com>
Looks ok overall. Some nits below
> ---
> hw/smbios/smbios.c | 65 ++++++++++++++++++++++++++++++++++++
> include/hw/firmware/smbios.h | 10 ++++++
> 2 files changed, 75 insertions(+)
>
> diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c
> index 60349ee402..11fe75ece0 100644
> --- a/hw/smbios/smbios.c
> +++ b/hw/smbios/smbios.c
> @@ -111,6 +111,13 @@ static struct {
> .processor_id = 0,
> };
>
> +struct type8_instance {
> + const char *internal_reference, *external_reference;
> + uint8_t connector_type, port_type;
> + QTAILQ_ENTRY(type8_instance) next;
> +};
> +static QTAILQ_HEAD(, type8_instance) type8 = QTAILQ_HEAD_INITIALIZER(type8);
> +
> static struct {
> size_t nvalues;
> char **values;
> @@ -136,6 +143,7 @@ static QEnumLookup type41_kind_lookup = {
> },
> .size = 10
> };
> +
> struct type41_instance {
> const char *designation, *pcidev;
> uint8_t instance, kind;
Don't add this empty line pls.
> @@ -337,6 +345,29 @@ static const QemuOptDesc qemu_smbios_type4_opts[] = {
> { /* end of list */ }
> };
>
> +static const QemuOptDesc qemu_smbios_type8_opts[] = {
> + {
> + .name = "internal_reference",
> + .type = QEMU_OPT_STRING,
> + .help = "internal reference designator",
> + },
> + {
> + .name = "external_reference",
> + .type = QEMU_OPT_STRING,
> + .help = "external reference designator",
> + },
> + {
> + .name = "connector_type",
> + .type = QEMU_OPT_NUMBER,
> + .help = "connector type",
> + },
> + {
> + .name = "port_type",
> + .type = QEMU_OPT_NUMBER,
> + .help = "port type",
> + },
> +};
> +
> static const QemuOptDesc qemu_smbios_type11_opts[] = {
> {
> .name = "value",
> @@ -718,6 +749,27 @@ static void smbios_build_type_4_table(MachineState *ms,
> unsigned instance)
> smbios_type4_count++;
> }
>
> +static void smbios_build_type_8_table(void)
> +{
> + unsigned instance = 0;
> + struct type8_instance *t8;
> +
> + QTAILQ_FOREACH(t8, &type8, next) {
> +
no empty line here pls
> + SMBIOS_BUILD_TABLE_PRE(8, T0_BASE + instance, true);
> +
> + SMBIOS_TABLE_SET_STR(8, internal_reference_str,
> t8->internal_reference);
> + SMBIOS_TABLE_SET_STR(8, external_reference_str,
> t8->external_reference);
> + // most vendors seem to set this to None
/* comments in this form please */
> + t->internal_connector_type = 0x0;
> + t->external_connector_type = t8->connector_type;
> + t->port_type = t8->port_type;
> +
> + SMBIOS_BUILD_TABLE_POST;
> + instance++;
> + }
> +}
> +
> static void smbios_build_type_11_table(void)
> {
> char count_str[128];
> @@ -1030,6 +1082,7 @@ void smbios_get_tables(MachineState *ms,
> smbios_build_type_4_table(ms, i);
> }
>
> + smbios_build_type_8_table();
> smbios_build_type_11_table();
>
> #define MAX_DIMM_SZ (16 * GiB)
> @@ -1346,6 +1399,18 @@ void smbios_entry_add(QemuOpts *opts, Error **errp)
> UINT16_MAX);
> }
> return;
> + case 8:
> + if (!qemu_opts_validate(opts, qemu_smbios_type8_opts, errp)) {
> + return;
> + }
> + struct type8_instance *t;
> + t = g_new0(struct type8_instance, 1);
> + save_opt(&t->internal_reference, opts, "internal_reference");
> + save_opt(&t->external_reference, opts, "external_reference");
> + t->connector_type = qemu_opt_get_number(opts, "connector_type",
> 0);
> + t->port_type = qemu_opt_get_number(opts, "port_type", 0);
> + QTAILQ_INSERT_TAIL(&type8, t, next);
> + return;
> case 11:
> if (!qemu_opts_validate(opts, qemu_smbios_type11_opts, errp)) {
> return;
> diff --git a/include/hw/firmware/smbios.h b/include/hw/firmware/smbios.h
> index 4b7ad77a44..e7d386f7c8 100644
> --- a/include/hw/firmware/smbios.h
> +++ b/include/hw/firmware/smbios.h
> @@ -189,6 +189,16 @@ struct smbios_type_4 {
> uint16_t processor_family2;
> } QEMU_PACKED;
>
> +/* SMBIOS type 8 - Port Connector Information */
> +struct smbios_type_8 {
> + struct smbios_structure_header header;
> + uint8_t internal_reference_str;
> + uint8_t internal_connector_type;
> + uint8_t external_reference_str;
> + uint8_t external_connector_type;
> + uint8_t port_type;
> +} QEMU_PACKED;
> +
> /* SMBIOS type 11 - OEM strings */
> struct smbios_type_11 {
> struct smbios_structure_header header;
> --
> 2.36.1
