From patchwork Tue Dec 8 21:20:23 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Eduardo Habkost X-Patchwork-Id: 1413062 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=nongnu.org (client-ip=209.51.188.17; helo=lists.gnu.org; envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=MNF35G/z; dkim-atps=neutral Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 4CrDpw0jBrz9sSn for ; Wed, 9 Dec 2020 09:08:56 +1100 (AEDT) Received: from localhost ([::1]:56430 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kmlAE-0001j2-26 for incoming@patchwork.ozlabs.org; Tue, 08 Dec 2020 17:08:54 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:44880) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kmkPg-00064e-6g for qemu-devel@nongnu.org; Tue, 08 Dec 2020 16:20:48 -0500 Received: from us-smtp-delivery-124.mimecast.com ([63.128.21.124]:57780) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1kmkPc-0005ki-6J for qemu-devel@nongnu.org; Tue, 08 Dec 2020 16:20:47 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1607462441; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=GwpIEFbHR59Oa4qhxqvT3X738JeKDGXi+IAILCkKjZ0=; b=MNF35G/z8fh82YVwS4zURfzm8wy9UtU22obTBPcL7w3VVlCiNWRoM9gYC/787OhfuAYHYE JYO1RN3qJtZ7+5S/Au7BAcE5FNnJtLV3jkZobai+MwvPCtGHRMFCMzJXXZ76qFdJHPfqyR Ip0LeKWfjCk1GJHDUJQoddYiCzXqxic= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-526-CMoRmnxVMCOIoG9VU4tguw-1; Tue, 08 Dec 2020 16:20:40 -0500 X-MC-Unique: CMoRmnxVMCOIoG9VU4tguw-1 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 92776107B471 for ; Tue, 8 Dec 2020 21:20:34 +0000 (UTC) Received: from localhost (ovpn-120-147.rdu2.redhat.com [10.10.120.147]) by smtp.corp.redhat.com (Postfix) with ESMTP id 1C29F5D9DD; Tue, 8 Dec 2020 21:20:33 +0000 (UTC) From: Eduardo Habkost To: qemu-devel@nongnu.org Subject: [PATCH] smbios: entry-point-type option Date: Tue, 8 Dec 2020 16:20:23 -0500 Message-Id: <20201208212023.1560846-1-ehabkost@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=ehabkost@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Received-SPF: pass client-ip=63.128.21.124; envelope-from=ehabkost@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Igor Mammedov , =?utf-8?q?Philippe_Mathieu-Daud?= =?utf-8?q?=C3=A9?= , Laszlo Ersek , "Daniel P. Berrange" , "Michael S. Tsirkin" Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: "Qemu-devel" Add command-line option that lets the SMBIOS entry point type to be configured. SMBIOS 3.0 support is necessary to allow us to support more than 720 VCPUs in x86_64, due to SMBIOS 2.1 table size limits. Note that it's still up to firmware to decide whether to generate SMBIOS 2.1 and/or 3.0 entry points for the guest, using the information contained in etc/smbios/smbios-anchor. OVMF, for example, is able to generate both entry points, depending on the value of PcdSmbiosEntryPointProvideMethod. The SMBIOS 3.0 entry point won't be enabled by default because it is not supported yet by Seabios. This may be changed once Seabios starts supporting SMBIOS 3.0 entry points. Signed-off-by: Eduardo Habkost --- Laszlo, Philippe: do you know how exactly the value of PcdSmbiosEntryPointProvideMethod is chosen when running OVMF? --- hw/smbios/smbios.c | 39 +++++++++++++++++++++++++++++++++++++-- qemu-options.hx | 16 +++++++++++++++- 2 files changed, 52 insertions(+), 3 deletions(-) diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c index 6a3d39793b..ed5cf7fa9a 100644 --- a/hw/smbios/smbios.c +++ b/hw/smbios/smbios.c @@ -62,6 +62,7 @@ size_t smbios_tables_len; unsigned smbios_table_max; unsigned smbios_table_cnt; static SmbiosEntryPointType smbios_ep_type = SMBIOS_ENTRY_POINT_21; +static bool smbios_ep_type_set; static SmbiosEntryPoint ep; @@ -139,6 +140,15 @@ static const QemuOptDesc qemu_smbios_file_opts[] = { { /* end of list */ } }; +static const QemuOptDesc qemu_smbios_entry_point_opts[] = { + { + .name = "entry-point-type", + .type = QEMU_OPT_STRING, + .help = "SMBIOS Entry Point type (2.1 or 3.0)", + }, + { /* end of list */ } +}; + static const QemuOptDesc qemu_smbios_type0_opts[] = { { .name = "type", @@ -797,7 +807,9 @@ void smbios_set_defaults(const char *manufacturer, const char *product, smbios_have_defaults = true; smbios_legacy = legacy_mode; smbios_uuid_encoded = uuid_encoded; - smbios_ep_type = ep_type; + if (!smbios_ep_type_set) { + smbios_ep_type = ep_type; + } /* drop unwanted version of command-line file blob(s) */ if (smbios_legacy) { @@ -1232,5 +1244,28 @@ void smbios_entry_add(QemuOpts *opts, Error **errp) } } - error_setg(errp, "Must specify type= or file="); + val = qemu_opt_get(opts, "entry-point-type"); + if (val) { + if (!qemu_opts_validate(opts, qemu_smbios_entry_point_opts, errp)) { + return; + } + + if (smbios_ep_type_set) { + error_setg(errp, "entry-point-type can't be set twice"); + return; + } + + if (!strcmp(val, "2.1")) { + smbios_ep_type = SMBIOS_ENTRY_POINT_21; + smbios_ep_type_set = true; + } else if (!strcmp(val, "3.0")) { + smbios_ep_type = SMBIOS_ENTRY_POINT_30; + smbios_ep_type_set = true; + } else { + error_setg(errp, "Invalid entry point type: %s", val); + } + return; + } + + error_setg(errp, "Must specify type=, file=, or entry-point-type="); } diff --git a/qemu-options.hx b/qemu-options.hx index 104632ea34..d2a973f8a7 100644 --- a/qemu-options.hx +++ b/qemu-options.hx @@ -2294,7 +2294,9 @@ DEF("smbios", HAS_ARG, QEMU_OPTION_smbios, " specify SMBIOS type 11 fields\n" "-smbios type=17[,loc_pfx=str][,bank=str][,manufacturer=str][,serial=str]\n" " [,asset=str][,part=str][,speed=%d]\n" - " specify SMBIOS type 17 fields\n", + " specify SMBIOS type 17 fields\n" + "-smbios entry-point-type=2.1|3.0\n" + " specify SMBIOS entry point type\n", QEMU_ARCH_I386 | QEMU_ARCH_ARM) SRST ``-smbios file=binary`` @@ -2356,6 +2358,18 @@ SRST ``-smbios type=17[,loc_pfx=str][,bank=str][,manufacturer=str][,serial=str][,asset=str][,part=str][,speed=%d]`` Specify SMBIOS type 17 fields + +``-smbios entry-point-type=2.1|3.0`` + Specify SMBIOS entry point type + + SMBIOS 3.0 allows the total size of SMBIOS tables to be much + larger, but note that Seabios for x86 does not support SMBIOS + 3.0 (as of Seabios v1.14.0). + + Note that this option only affects the SMBIOS data exposed + through fw_cfg. It's still up to the firmware to generate + the actual SMBIOS entry point(s) seen by the guest operating + system. ERST DEFHEADING()