Hi Matthew, On 08/13/20 00:43, matthewfcarl...@gmail.com wrote: > From: Matthew Carlson <mac...@microsoft.com> > > 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 <jordan.l.jus...@intel.com> > Cc: Laszlo Ersek <ler...@redhat.com> > Cc: Ard Biesheuvel <ard.biesheu...@arm.com> > Cc: Anthony Perard <anthony.per...@citrix.com> > Cc: Julien Grall <jul...@xen.org> > Signed-off-by: Matthew Carlson <matthewfcarl...@gmail.com> > Reviewed-by: Laszlo Ersek <lersek@...>
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 <ler...@redhat.com> 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 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#64145): https://edk2.groups.io/g/devel/message/64145 Mute This Topic: https://groups.io/mt/76157321/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
