From nobody Mon Feb 9 22:37:58 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1503409409146185.630027946087; Tue, 22 Aug 2017 06:43:29 -0700 (PDT) Received: from localhost ([::1]:47045 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dk9Sp-0007gZ-WF for importer@patchew.org; Tue, 22 Aug 2017 09:43:28 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53712) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dk9Ez-000469-VM for qemu-devel@nongnu.org; Tue, 22 Aug 2017 09:29:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dk9Ez-0003mU-4N for qemu-devel@nongnu.org; Tue, 22 Aug 2017 09:29:09 -0400 Received: from mx1.redhat.com ([209.132.183.28]:37262) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dk9Ey-0003mH-R9 for qemu-devel@nongnu.org; Tue, 22 Aug 2017 09:29:09 -0400 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 77AD1C058EBD for ; Tue, 22 Aug 2017 13:23:39 +0000 (UTC) Received: from localhost (ovpn-112-43.ams2.redhat.com [10.36.112.43]) by smtp.corp.redhat.com (Postfix) with ESMTP id F221E617A3; Tue, 22 Aug 2017 13:23:36 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 77AD1C058EBD Authentication-Results: ext-mx08.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx08.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=marcandre.lureau@redhat.com From: =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= To: qemu-devel@nongnu.org Date: Tue, 22 Aug 2017 15:22:08 +0200 Message-Id: <20170822132255.23945-8-marcandre.lureau@redhat.com> In-Reply-To: <20170822132255.23945-1-marcandre.lureau@redhat.com> References: <20170822132255.23945-1-marcandre.lureau@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.32]); Tue, 22 Aug 2017 13:23:39 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [PATCH v2 07/54] tpm: simplify driver registration & lookup X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" Make be_drivers[n] match order of TpmType enum. Use qapi_enum_parse() in tpm_get_backend_driver. Signed-off-by: Marc-Andr=C3=A9 Lureau Reviewed-by: Markus Armbruster --- include/sysemu/tpm_backend.h | 2 +- tpm.c | 42 ++++++++++++----------------------------= -- 2 files changed, 13 insertions(+), 31 deletions(-) diff --git a/include/sysemu/tpm_backend.h b/include/sysemu/tpm_backend.h index b58f52d39f..1d21c6b19b 100644 --- a/include/sysemu/tpm_backend.h +++ b/include/sysemu/tpm_backend.h @@ -227,6 +227,6 @@ TPMBackend *qemu_find_tpm(const char *id); =20 const TPMDriverOps *tpm_get_backend_driver(const char *type); int tpm_register_model(enum TpmModel model); -int tpm_register_driver(const TPMDriverOps *tdo); +void tpm_register_driver(const TPMDriverOps *tdo); =20 #endif diff --git a/tpm.c b/tpm.c index c3b731b3f2..f175661bfe 100644 --- a/tpm.c +++ b/tpm.c @@ -26,9 +26,8 @@ static QLIST_HEAD(, TPMBackend) tpm_backends =3D =20 =20 #define TPM_MAX_MODELS 1 -#define TPM_MAX_DRIVERS 1 =20 -static TPMDriverOps const *be_drivers[TPM_MAX_DRIVERS] =3D { +static TPMDriverOps const *be_drivers[TPM_TYPE__MAX] =3D { NULL, }; =20 @@ -64,31 +63,18 @@ static bool tpm_model_is_registered(enum TpmModel model) =20 const TPMDriverOps *tpm_get_backend_driver(const char *type) { - int i; + int i =3D qapi_enum_parse(TpmType_lookup, type, TPM_TYPE__MAX, -1, NUL= L); =20 - for (i =3D 0; i < TPM_MAX_DRIVERS && be_drivers[i] !=3D NULL; i++) { - if (!strcmp(TpmType_lookup[be_drivers[i]->type], type)) { - return be_drivers[i]; - } - } - - return NULL; + return i >=3D 0 ? be_drivers[i] : NULL; } =20 #ifdef CONFIG_TPM =20 -int tpm_register_driver(const TPMDriverOps *tdo) +void tpm_register_driver(const TPMDriverOps *tdo) { - int i; + assert(!be_drivers[tdo->type]); =20 - for (i =3D 0; i < TPM_MAX_DRIVERS; i++) { - if (!be_drivers[i]) { - be_drivers[i] =3D tdo; - return 0; - } - } - error_report("Could not register TPM driver"); - return 1; + be_drivers[tdo->type] =3D tdo; } =20 /* @@ -101,9 +87,12 @@ static void tpm_display_backend_drivers(void) =20 fprintf(stderr, "Supported TPM types (choose only one):\n"); =20 - for (i =3D 0; i < TPM_MAX_DRIVERS && be_drivers[i] !=3D NULL; i++) { + for (i =3D 0; i < TPM_TYPE__MAX; i++) { + if (be_drivers[i] =3D=3D NULL) { + continue; + } fprintf(stderr, "%12s %s\n", - qapi_enum_lookup(TpmType_lookup, be_drivers[i]->type), + qapi_enum_lookup(TpmType_lookup, i), be_drivers[i]->desc()); } fprintf(stderr, "\n"); @@ -241,14 +230,7 @@ int tpm_config_parse(QemuOptsList *opts_list, const ch= ar *optarg) =20 static const TPMDriverOps *tpm_driver_find_by_type(enum TpmType type) { - int i; - - for (i =3D 0; i < TPM_MAX_DRIVERS && be_drivers[i] !=3D NULL; i++) { - if (be_drivers[i]->type =3D=3D type) { - return be_drivers[i]; - } - } - return NULL; + return be_drivers[type]; } =20 static TPMInfo *qmp_query_tpm_inst(TPMBackend *drv) --=20 2.14.1.146.gd35faa819