From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [205.139.110.120]) by mx.groups.io with SMTP id smtpd.web11.1133.1590084295501095929 for ; Thu, 21 May 2020 11:04:56 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=WPCUwuzF; spf=pass (domain: redhat.com, ip: 205.139.110.120, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1590084294; 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: in-reply-to:in-reply-to:references:references; bh=ljuZK8zLJGRp3X/KI/W0lRlBRs4SoImvX4t0kQL/lgA=; b=WPCUwuzFwDxafaG07s/JW2frouUGLzfqsGdmWCpzh/yuehheYEelLbR3TgXmVKCho9yTHl r3CJ0R+5npUiooXUhq4We6Zit9CkquHJ9JvtBlYmdnToW6CGhV4Qxgb1j3OSFWZZJ2JejU KT3w42bBPxa9dDBrCQQlqye3ectouM4= 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-242-4_hKbUqIOSyIm7ht9S10sw-1; Thu, 21 May 2020 14:04:48 -0400 X-MC-Unique: 4_hKbUqIOSyIm7ht9S10sw-1 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 mimecast-mx01.redhat.com (Postfix) with ESMTPS id 4F3D38018AB; Thu, 21 May 2020 18:04:47 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-113-225.ams2.redhat.com [10.36.113.225]) by smtp.corp.redhat.com (Postfix) with ESMTP id D6ABE5C1D0; Thu, 21 May 2020 18:04:39 +0000 (UTC) Subject: Re: [PATCH 1/3] OvmfPkg/Tcg2ConfigPei: clean up some lib class dependencies To: =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , edk2-devel-groups-io Cc: Ard Biesheuvel , Eric Auger , Jordan Justen , =?UTF-8?Q?Marc-Andr=c3=a9_Lureau?= , Simon Hardy , Stefan Berger References: <20200520225841.17793-1-lersek@redhat.com> <20200520225841.17793-2-lersek@redhat.com> From: "Laszlo Ersek" Message-ID: <495e4d6a-696f-4232-68e1-23a54249655e@redhat.com> Date: Thu, 21 May 2020 20:04:38 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Hi Phil, On 05/21/20 12:29, Philippe Mathieu-Daudé wrote: > On 5/21/20 12:58 AM, Laszlo Ersek wrote: >> Commit 89236992913f introduced an explicit Tpm12CommandLib dependency to >> Tcg2ConfigPei. >> >> In reality this lib class is not consumed by Tcg2ConfigPei at all (such a >> dependency is not even inherited from other lib instances). Simplify the >> module by dropping the superfluous dependency. >> >> (The Tpm12CommandLib class resolution that was also added in commit >> 89236992913f is not useless, at the platform build level: it is consumed >> by TcgPei and TcgDxe. Meaning that said Tpm12CommandLib resolution should >> have likely been a part of the subsequent patch in the original series, >> namely commit 6be54f15a0c9.) >> >> Commit 89236992913f also introduced SwapBytesXx() calls. Those functions >> are provided by BaseLib. Spell out the BaseLib dependency. >> >> Functionally, this patch is a no-op. >> >> Cc: Ard Biesheuvel >> Cc: Eric Auger >> Cc: Jordan Justen >> Cc: Marc-André Lureau >> Cc: Philippe Mathieu-Daudé >> Cc: Simon Hardy >> Cc: Stefan Berger >> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2728 >> Signed-off-by: Laszlo Ersek >> --- >>   OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPei.inf | 2 +- >>   OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c  | 2 +- >>   2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPei.inf >> b/OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPei.inf >> index 97c529c91d0b..b79d0a3fb912 100644 >> --- a/OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPei.inf >> +++ b/OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPei.inf >> @@ -30,9 +30,9 @@ [Packages] >>     [LibraryClasses] >>     PeimEntryPoint >> +  BaseLib >>     DebugLib >>     PeiServicesLib >> -  Tpm12CommandLib >>     Tpm12DeviceLib >>     Tpm2DeviceLib >>   diff --git a/OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c >> b/OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c >> index 5b5075bded92..44abd6c541f9 100644 >> --- a/OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c >> +++ b/OvmfPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c >> @@ -15,11 +15,11 @@ >>   #include >>     #include >> +#include >>   #include >>   #include >>   #include >>   #include >> -#include >>   #include >>     STATIC CONST EFI_PEI_PPI_DESCRIPTOR mTpmSelectedPpi = { >> > > Reviewed-by: Philippe Mathieu-Daude > I'm extremely sorry for missing your R-b's. When I read your email with msgid <60493dcc-0832-1fb3-f380-867410691672@redhat.com>, and consequently decided that I'd go ahead with the feedback provided thus far, I didn't realize you were going to review the series as well! I switched off my email refresh very soon after. Normally I let a series sit for at least a day (24 hours) on the list, so that everyone (in different timezones too) can at least state an intent (even in private) to review or test the series. I encourage other maintainers to do the same, and I've complained in the past when I would have liked to review or test a series but wasn't given enough time even to signal my interest. So this is totally my fault. My only excuse is that the feature freeze(s) are upon us, and fixing stuff feels "urgent". I wish we could append release candidates (RCs) "on-demand", like QEMU does. I'm sorry! And thank you for your review. Laszlo