From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.120]) by mx.groups.io with SMTP id smtpd.web12.6010.1597307700678252046 for ; Thu, 13 Aug 2020 01:35:00 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=XE88byXB; spf=pass (domain: redhat.com, ip: 207.211.31.120, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1597307699; 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=GHSCytXPTGaTZFnLfoszRx5vYfZmJ9s4ikNCz+wIUH8=; b=XE88byXBpsGixgqrY0hdxRWUC4/BHUTGnowfZHbYQu/EDXuhI0xfxVGZ354rGwUO+uybN8 TduyHD9iBh9TnpeNsqkPduq/SViuW+TlqOP8fu1qX7DitkIEJkNRHQ2vMoHjxU1II+G9UK WnSY7byw7ywalx/iFB8DgN21bX5Qxzw= 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-461-sf3AVuzoNlSKuym2QxqoTQ-1; Thu, 13 Aug 2020 04:34:57 -0400 X-MC-Unique: sf3AVuzoNlSKuym2QxqoTQ-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 805C98712D9; Thu, 13 Aug 2020 08:34:56 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-114-59.ams2.redhat.com [10.36.114.59]) by smtp.corp.redhat.com (Postfix) with ESMTP id E4B44648B1; Thu, 13 Aug 2020 08:34:54 +0000 (UTC) Subject: Re: [PATCH v6 3/5] OvmfPkg: Add RngLib based on TimerLib for Crypto To: matthewfcarlson@gmail.com, devel@edk2.groups.io Cc: Jordan Justen , Ard Biesheuvel , Anthony Perard , Julien Grall References: <20200812224338.287-1-matthewfcarlson@gmail.com> <20200812224338.287-4-matthewfcarlson@gmail.com> From: "Laszlo Ersek" Message-ID: <2c518937-8a0d-127f-ccf9-9258b31bef38@redhat.com> Date: Thu, 13 Aug 2020 10:34:53 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <20200812224338.287-4-matthewfcarlson@gmail.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Hi Matthew, On 08/13/20 00:43, matthewfcarlson@gmail.com wrote: > From: Matthew Carlson > > Updates the DSC's for Ovmf based platforms to add a RngLib that uses the > TimerLib. This is due to a later change that adds TimerLib as a dependency > for OpenSSL. The TimerLib based RngLib mimics the behavior of OpenSSL > previously and it is recommended to switch to a better source of > entropy than the system's performance counter. > > Ref: https://github.com/tianocore/edk2/pull/845 > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871 > > Cc: Jordan Justen > Cc: Laszlo Ersek > Cc: Ard Biesheuvel > Cc: Anthony Perard > Cc: Julien Grall > Signed-off-by: Matthew Carlson > Reviewed-by: Laszlo Ersek I think you must have copied my R-b from a mailing list archive (on the web) into this commit message, and not from an email of mine. That's because my email address is truncated above, similarly to how the archives display email addresses (for fighting spam). Please don't repost the series just for this; the maintainer that merges this series can (and should) fix up this wart just in time. However, if a v7 becomes necessary, please do refresh my email address above. For convenience, I'll repeat my R-b here: Reviewed-by: Laszlo Ersek Thanks! Laszlo > --- > OvmfPkg/Bhyve/BhyvePkgX64.dsc | 1 + > OvmfPkg/OvmfPkgIa32.dsc | 1 + > OvmfPkg/OvmfPkgIa32X64.dsc | 1 + > OvmfPkg/OvmfPkgX64.dsc | 1 + > OvmfPkg/OvmfXen.dsc | 1 + > 5 files changed, 5 insertions(+) > > diff --git a/OvmfPkg/Bhyve/BhyvePkgX64.dsc b/OvmfPkg/Bhyve/BhyvePkgX64.dsc > index 8eba48d109a3..4b1fbb361b28 100644 > --- a/OvmfPkg/Bhyve/BhyvePkgX64.dsc > +++ b/OvmfPkg/Bhyve/BhyvePkgX64.dsc > @@ -185,6 +185,7 @@ > !else > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf > !endif > + RngLib|MdePkg/Library/BaseRngLibTimerLib/BaseRngLibTimerLib.inf > > !if $(SECURE_BOOT_ENABLE) == TRUE > PlatformSecureLib|OvmfPkg/Bhyve/Library/PlatformSecureLib/PlatformSecureLib.inf > diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc > index 9178ffeb71cb..fc1c8014eba1 100644 > --- a/OvmfPkg/OvmfPkgIa32.dsc > +++ b/OvmfPkg/OvmfPkgIa32.dsc > @@ -189,6 +189,7 @@ > !else > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf > !endif > + RngLib|MdePkg/Library/BaseRngLibTimerLib/BaseRngLibTimerLib.inf > > !if $(SECURE_BOOT_ENABLE) == TRUE > PlatformSecureLib|OvmfPkg/Library/PlatformSecureLib/PlatformSecureLib.inf > diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc > index a665f78f0dc7..2365dc2fa98d 100644 > --- a/OvmfPkg/OvmfPkgIa32X64.dsc > +++ b/OvmfPkg/OvmfPkgIa32X64.dsc > @@ -193,6 +193,7 @@ > !else > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf > !endif > + RngLib|MdePkg/Library/BaseRngLibTimerLib/BaseRngLibTimerLib.inf > > !if $(SECURE_BOOT_ENABLE) == TRUE > PlatformSecureLib|OvmfPkg/Library/PlatformSecureLib/PlatformSecureLib.inf > diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc > index 17f345acf4ee..6992cfd98b70 100644 > --- a/OvmfPkg/OvmfPkgX64.dsc > +++ b/OvmfPkg/OvmfPkgX64.dsc > @@ -193,6 +193,7 @@ > !else > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf > !endif > + RngLib|MdePkg/Library/BaseRngLibTimerLib/BaseRngLibTimerLib.inf > > !if $(SECURE_BOOT_ENABLE) == TRUE > PlatformSecureLib|OvmfPkg/Library/PlatformSecureLib/PlatformSecureLib.inf > diff --git a/OvmfPkg/OvmfXen.dsc b/OvmfPkg/OvmfXen.dsc > index 782803cb2787..416f81f06a04 100644 > --- a/OvmfPkg/OvmfXen.dsc > +++ b/OvmfPkg/OvmfXen.dsc > @@ -179,6 +179,7 @@ > !else > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf > !endif > + RngLib|MdePkg/Library/BaseRngLibTimerLib/BaseRngLibTimerLib.inf > > AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLibNull.inf > VarCheckLib|MdeModulePkg/Library/VarCheckLib/VarCheckLib.inf >