[PATCH edk2-platforms 0/5] Add RngLibNull for comming OpensslLib changes

[PATCH edk2-platforms 0/5] Add RngLibNull for comming OpensslLib changes

[Wang, Jian J]

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871

It's a prerequisite of removing dependency of TimerLib from OpensslLib.
RngLib will be used instead. To avoid build failure, add the null
instance of it in advance.

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Kelly Steele <kelly.steele@intel.com>
Cc: Dandan Bi <dandan.bi@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Zailiang Sun <zailiang.sun@intel.com>
Cc: Yi Qian <yi.qian@intel.com>
Cc: Marcin Wojtas <mw@semihalf.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Leif Lindholm <leif.lindholm@linaro.org>

Jian J Wang (5):
  QuarkPlatformPkg: add null version of RngLib
  UserInterfaceFeaturePkg: add null version RngLib
  Vlv2TbltDevicePkg: add null version RngLib
  Armada7k8k: add null version RngLib
  MiscPlatforms: add null version RngLib

 Platform/AMD/OverdriveBoard/OverdriveBoard.dsc                   | 1 +
 Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc                     | 1 +
 Platform/Intel/QuarkPlatformPkg/Quark.dsc                        | 1 +
 Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc                     | 1 +
 .../Intel/UserInterfaceFeaturePkg/UserInterfaceFeaturePkg.dsc    | 1 +
 Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc             | 1 +
 Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc              | 1 +
 Platform/LeMaker/CelloBoard/CelloBoard.dsc                       | 1 +
 Platform/RaspberryPi/RPi3/RPi3.dsc                               | 1 +
 Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc             | 1 +
 Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc     | 1 +
 Silicon/Hisilicon/Hisilicon.dsc.inc                              | 1 +
 Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc                    | 1 +
 13 files changed, 13 insertions(+)

-- 
2.17.1.windows.2

[PATCH edk2-platforms 1/5] QuarkPlatformPkg: add null version of RngLib

[Wang, Jian J]

It's a prerequisite of removing dependency of TimerLib from OpensslLib.
RngLib will be used instead. To avoid build failure, add the null
instance of it in advance.

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Kelly Steele <kelly.steele@intel.com>
Signed-off-by: Jian J Wang <jian.j.wang@intel.com>
---
 Platform/Intel/QuarkPlatformPkg/Quark.dsc    | 1 +
 Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc | 1 +
 2 files changed, 2 insertions(+)

diff --git a/Platform/Intel/QuarkPlatformPkg/Quark.dsc b/Platform/Intel/QuarkPlatformPkg/Quark.dsc
index a02adb64e695..2469c6e37e9e 100644
--- a/Platform/Intel/QuarkPlatformPkg/Quark.dsc
+++ b/Platform/Intel/QuarkPlatformPkg/Quark.dsc
@@ -161,6 +161,7 @@ [LibraryClasses]
   FileHandleLib|MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.inf
 
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
 
diff --git a/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc b/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc
index 3dbf616c664d..7737a232e909 100644
--- a/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc
+++ b/Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc
@@ -153,6 +153,7 @@ [LibraryClasses]
   PlatformSecureLib|QuarkPlatformPkg/Library/PlatformSecureLib/PlatformSecureLib.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   TpmMeasurementLib|SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.inf
   AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf
 !else
-- 
2.17.1.windows.2

[PATCH edk2-platforms 2/5] UserInterfaceFeaturePkg: add null version RngLib

[Wang, Jian J]

It's a prerequisite of removing dependency of TimerLib from OpensslLib.
RngLib will be used instead. To avoid build failure, add the null
instance of it in advance.

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871
Cc: Dandan Bi <dandan.bi@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Signed-off-by: Jian J Wang <jian.j.wang@intel.com>
---
 .../Intel/UserInterfaceFeaturePkg/UserInterfaceFeaturePkg.dsc    | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Platform/Intel/UserInterfaceFeaturePkg/UserInterfaceFeaturePkg.dsc b/Platform/Intel/UserInterfaceFeaturePkg/UserInterfaceFeaturePkg.dsc
index 6c5f77b59bbf..dba1d68b1d4b 100644
--- a/Platform/Intel/UserInterfaceFeaturePkg/UserInterfaceFeaturePkg.dsc
+++ b/Platform/Intel/UserInterfaceFeaturePkg/UserInterfaceFeaturePkg.dsc
@@ -38,6 +38,7 @@ [LibraryClasses]
   DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   PlatformPasswordLib|UserInterfaceFeaturePkg/Library/PlatformPasswordLibNull/PlatformPasswordLibNull.inf
   UserPasswordLib|UserInterfaceFeaturePkg/Library/UserPasswordLib/UserPasswordLib.inf
 
-- 
2.17.1.windows.2

[PATCH edk2-platforms 3/5] Vlv2TbltDevicePkg: add null version RngLib

[Wang, Jian J]

It's a prerequisite of removing dependency of TimerLib from OpensslLib.
RngLib will be used instead. To avoid build failure, add the null
instance of it in advance.

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871
Cc: Liming Gao <liming.gao@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Zailiang Sun <zailiang.sun@intel.com>
Cc: Yi Qian <yi.qian@intel.com>
Signed-off-by: Jian J Wang <jian.j.wang@intel.com>
---
 Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 1 +
 Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc  | 1 +
 2 files changed, 2 insertions(+)

diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc
index 463b952e6582..3dac5f740750 100644
--- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc
+++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc
@@ -234,6 +234,7 @@ [LibraryClasses.common]
   SortLib|MdeModulePkg/Library/UefiSortLib/UefiSortLib.inf
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
 !if $(NETWORK_TLS_ENABLE) == TRUE
   TlsLib|CryptoPkg/Library/TlsLib/TlsLib.inf
diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc
index ee18b45c9712..d3a841484a38 100644
--- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc
+++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc
@@ -236,6 +236,7 @@ [LibraryClasses.common]
   SortLib|MdeModulePkg/Library/UefiSortLib/UefiSortLib.inf
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
 !if $(NETWORK_TLS_ENABLE) == TRUE
   TlsLib|CryptoPkg/Library/TlsLib/TlsLib.inf
-- 
2.17.1.windows.2

[PATCH edk2-platforms 4/5] Armada7k8k: add null version RngLib

[Wang, Jian J]

It's a prerequisite of removing dependency of TimerLib from OpensslLib.
RngLib will be used instead. To avoid build failure, add the null
instance of it in advance.

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871
Cc: Marcin Wojtas <mw@semihalf.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Jian J Wang <jian.j.wang@intel.com>
---
 Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc b/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
index 42f8bbba926c..b7cde5723bff 100644
--- a/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
+++ b/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
@@ -20,6 +20,7 @@ [LibraryClasses.common]
 
 !if $(CAPSULE_ENABLE)
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
   CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCapsuleLib.inf
-- 
2.17.1.windows.2

[PATCH edk2-platforms 5/5] MiscPlatforms: add null version RngLib

[Wang, Jian J]

It's a prerequisite of removing dependency of TimerLib from OpensslLib.
RngLib will be used instead. To avoid build failure, add the null
instance of it in advance.

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Jian J Wang <jian.j.wang@intel.com>
---
 Platform/AMD/OverdriveBoard/OverdriveBoard.dsc               | 1 +
 Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc                 | 1 +
 Platform/LeMaker/CelloBoard/CelloBoard.dsc                   | 1 +
 Platform/RaspberryPi/RPi3/RPi3.dsc                           | 1 +
 Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc         | 1 +
 Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc | 1 +
 Silicon/Hisilicon/Hisilicon.dsc.inc                          | 1 +
 7 files changed, 7 insertions(+)

diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
index 7369173cc125..904200c7776b 100644
--- a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
+++ b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
@@ -119,6 +119,7 @@ [LibraryClasses.common]
   PlatformFlashAccessLib|Silicon/AMD/Styx/Library/StyxPlatformFlashAccessLib/StyxPlatformFlashAccessLib.inf
 
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   TlsLib|CryptoPkg/Library/TlsLib/TlsLib.inf
 
   UefiBootManagerLib|MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
diff --git a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
index 0d543d2a61b1..39eb248ec515 100644
--- a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
+++ b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
@@ -138,6 +138,7 @@ [LibraryClasses.common]
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   VarCheckLib|MdeModulePkg/Library/VarCheckLib/VarCheckLib.inf
 
   CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
diff --git a/Platform/LeMaker/CelloBoard/CelloBoard.dsc b/Platform/LeMaker/CelloBoard/CelloBoard.dsc
index 9ed4e03b2400..31770c6e5949 100644
--- a/Platform/LeMaker/CelloBoard/CelloBoard.dsc
+++ b/Platform/LeMaker/CelloBoard/CelloBoard.dsc
@@ -139,6 +139,7 @@ [LibraryClasses.common]
 
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
 
 [LibraryClasses.common.SEC]
diff --git a/Platform/RaspberryPi/RPi3/RPi3.dsc b/Platform/RaspberryPi/RPi3/RPi3.dsc
index a0365c5cf606..1dfef2ba9704 100644
--- a/Platform/RaspberryPi/RPi3/RPi3.dsc
+++ b/Platform/RaspberryPi/RPi3/RPi3.dsc
@@ -123,6 +123,7 @@ [LibraryClasses.common]
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
 
   #
   # Uncomment (and comment out the next line) For RealView Debugger. The Standard IO window
diff --git a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
index a10e48ca07ea..6ff6be585483 100644
--- a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
+++ b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
@@ -103,6 +103,7 @@ [LibraryClasses]
 
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   TlsLib|CryptoPkg/Library/TlsLib/TlsLib.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
   TpmMeasurementLib|SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.inf
diff --git a/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc b/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
index ab1ab6f2de60..4333a0554a1d 100644
--- a/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
+++ b/Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc
@@ -119,6 +119,7 @@ [LibraryClasses.common]
 
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
 
   NorFlashInfoLib|EmbeddedPkg/Library/NorFlashInfoLib/NorFlashInfoLib.inf
diff --git a/Silicon/Hisilicon/Hisilicon.dsc.inc b/Silicon/Hisilicon/Hisilicon.dsc.inc
index 7bc965e97350..6c74b0076c4b 100644
--- a/Silicon/Hisilicon/Hisilicon.dsc.inc
+++ b/Silicon/Hisilicon/Hisilicon.dsc.inc
@@ -103,6 +103,7 @@ [LibraryClasses.common]
 
   CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCapsuleLib.inf
   OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf
+  RngLib|MdePkg/Library/BaseRngLibNull/BaseRngLibNull.inf
   IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
   BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf
   FmpAuthenticationLib|SecurityPkg/Library/FmpAuthenticationLibPkcs7/FmpAuthenticationLibPkcs7.inf
-- 
2.17.1.windows.2

Re: [PATCH edk2-platforms 0/5] Add RngLibNull for comming OpensslLib changes

[Leif Lindholm]

Hi Jian,

Can MdePkg/Library/BaseRngLibNull/ be pushed to edk2, so that we can
run some build tests?

Best Regards,

Leif

On Wed, Nov 13, 2019 at 02:11:40PM +0800, Jian J Wang wrote:
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1871
> 
> It's a prerequisite of removing dependency of TimerLib from OpensslLib.
> RngLib will be used instead. To avoid build failure, add the null
> instance of it in advance.
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Kelly Steele <kelly.steele@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Zailiang Sun <zailiang.sun@intel.com>
> Cc: Yi Qian <yi.qian@intel.com>
> Cc: Marcin Wojtas <mw@semihalf.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> 
> Jian J Wang (5):
>   QuarkPlatformPkg: add null version of RngLib
>   UserInterfaceFeaturePkg: add null version RngLib
>   Vlv2TbltDevicePkg: add null version RngLib
>   Armada7k8k: add null version RngLib
>   MiscPlatforms: add null version RngLib
> 
>  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc                   | 1 +
>  Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc                     | 1 +
>  Platform/Intel/QuarkPlatformPkg/Quark.dsc                        | 1 +
>  Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc                     | 1 +
>  .../Intel/UserInterfaceFeaturePkg/UserInterfaceFeaturePkg.dsc    | 1 +
>  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc             | 1 +
>  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc              | 1 +
>  Platform/LeMaker/CelloBoard/CelloBoard.dsc                       | 1 +
>  Platform/RaspberryPi/RPi3/RPi3.dsc                               | 1 +
>  Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc             | 1 +
>  Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc     | 1 +
>  Silicon/Hisilicon/Hisilicon.dsc.inc                              | 1 +
>  Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc                    | 1 +
>  13 files changed, 13 insertions(+)
> 
> -- 
> 2.17.1.windows.2
> 

Re: [edk2-devel] [PATCH edk2-platforms 0/5] Add RngLibNull for comming OpensslLib changes

[Michael D Kinney]

Leif,

If it helps for testing purposes, the PR for edk2 has been run through EDK II CI.

    https://github.com/tianocore/edk2/pull/168

The public branch with the new lib is at:

    https://github.com/jwang36/edk2/tree/add-rnglibnull-instance

The lib is added in a single commit:

    https://github.com/jwang36/edk2/commit/b9802a924e8e5d3f9f12fd97d347be48eccae95e

Mike


> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On
> Behalf Of Leif Lindholm
> Sent: Wednesday, November 13, 2019 10:19 AM
> To: Wang, Jian J <jian.j.wang@intel.com>
> Cc: devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kinney@intel.com>; Steele, Kelly
> <kelly.steele@intel.com>; Bi, Dandan
> <dandan.bi@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Sun, Zailiang
> <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>;
> Marcin Wojtas <mw@semihalf.com>; Ard Biesheuvel
> <ard.biesheuvel@linaro.org>
> Subject: Re: [edk2-devel] [PATCH edk2-platforms 0/5]
> Add RngLibNull for comming OpensslLib changes
> 
> Hi Jian,
> 
> Can MdePkg/Library/BaseRngLibNull/ be pushed to edk2,
> so that we can run some build tests?
> 
> Best Regards,
> 
> Leif
> 
> On Wed, Nov 13, 2019 at 02:11:40PM +0800, Jian J Wang
> wrote:
> > Ref:
> https://bugzilla.tianocore.org/show_bug.cgi?id=1871
> >
> > It's a prerequisite of removing dependency of
> TimerLib from OpensslLib.
> > RngLib will be used instead. To avoid build failure,
> add the null
> > instance of it in advance.
> >
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Kelly Steele <kelly.steele@intel.com>
> > Cc: Dandan Bi <dandan.bi@intel.com>
> > Cc: Liming Gao <liming.gao@intel.com>
> > Cc: Zailiang Sun <zailiang.sun@intel.com>
> > Cc: Yi Qian <yi.qian@intel.com>
> > Cc: Marcin Wojtas <mw@semihalf.com>
> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > Cc: Leif Lindholm <leif.lindholm@linaro.org>
> >
> > Jian J Wang (5):
> >   QuarkPlatformPkg: add null version of RngLib
> >   UserInterfaceFeaturePkg: add null version RngLib
> >   Vlv2TbltDevicePkg: add null version RngLib
> >   Armada7k8k: add null version RngLib
> >   MiscPlatforms: add null version RngLib
> >
> >  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> | 1 +
> >  Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
> | 1 +
> >  Platform/Intel/QuarkPlatformPkg/Quark.dsc
> | 1 +
> >  Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc
> | 1 +
> >
> .../Intel/UserInterfaceFeaturePkg/UserInterfaceFeatureP
> kg.dsc    | 1 +
> >  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc
> | 1 +
> >  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc
> | 1 +
> >  Platform/LeMaker/CelloBoard/CelloBoard.dsc
> | 1 +
> >  Platform/RaspberryPi/RPi3/RPi3.dsc
> | 1 +
> >  Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
> | 1 +
> >
> Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoar
> d.dsc     | 1 +
> >  Silicon/Hisilicon/Hisilicon.dsc.inc
> | 1 +
> >  Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> | 1 +
> >  13 files changed, 13 insertions(+)
> >
> > --
> > 2.17.1.windows.2
> >
> 
> 

Re: [edk2-devel] [PATCH edk2-platforms 0/5] Add RngLibNull for comming OpensslLib changes

[Wang, Jian J]

It's pushed at c9af866cdd6214d65c1539e7008f6a7255316c80

Regards,
Jian

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Thursday, November 14, 2019 2:48 AM
> To: devel@edk2.groups.io; leif.lindholm@linaro.org; Wang, Jian J
> <jian.j.wang@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: Steele, Kelly <kelly.steele@intel.com>; Bi, Dandan <dandan.bi@intel.com>;
> Gao, Liming <liming.gao@intel.com>; Sun, Zailiang <zailiang.sun@intel.com>;
> Qian, Yi <yi.qian@intel.com>; Marcin Wojtas <mw@semihalf.com>; Ard
> Biesheuvel <ard.biesheuvel@linaro.org>
> Subject: RE: [edk2-devel] [PATCH edk2-platforms 0/5] Add RngLibNull for
> comming OpensslLib changes
> 
> Leif,
> 
> If it helps for testing purposes, the PR for edk2 has been run through EDK II CI.
> 
>     https://github.com/tianocore/edk2/pull/168
> 
> The public branch with the new lib is at:
> 
>     https://github.com/jwang36/edk2/tree/add-rnglibnull-instance
> 
> The lib is added in a single commit:
> 
> 
> https://github.com/jwang36/edk2/commit/b9802a924e8e5d3f9f12fd97d347be
> 48eccae95e
> 
> Mike
> 
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On
> > Behalf Of Leif Lindholm
> > Sent: Wednesday, November 13, 2019 10:19 AM
> > To: Wang, Jian J <jian.j.wang@intel.com>
> > Cc: devel@edk2.groups.io; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Steele, Kelly
> > <kelly.steele@intel.com>; Bi, Dandan
> > <dandan.bi@intel.com>; Gao, Liming
> > <liming.gao@intel.com>; Sun, Zailiang
> > <zailiang.sun@intel.com>; Qian, Yi <yi.qian@intel.com>;
> > Marcin Wojtas <mw@semihalf.com>; Ard Biesheuvel
> > <ard.biesheuvel@linaro.org>
> > Subject: Re: [edk2-devel] [PATCH edk2-platforms 0/5]
> > Add RngLibNull for comming OpensslLib changes
> >
> > Hi Jian,
> >
> > Can MdePkg/Library/BaseRngLibNull/ be pushed to edk2,
> > so that we can run some build tests?
> >
> > Best Regards,
> >
> > Leif
> >
> > On Wed, Nov 13, 2019 at 02:11:40PM +0800, Jian J Wang
> > wrote:
> > > Ref:
> > https://bugzilla.tianocore.org/show_bug.cgi?id=1871
> > >
> > > It's a prerequisite of removing dependency of
> > TimerLib from OpensslLib.
> > > RngLib will be used instead. To avoid build failure,
> > add the null
> > > instance of it in advance.
> > >
> > > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > > Cc: Kelly Steele <kelly.steele@intel.com>
> > > Cc: Dandan Bi <dandan.bi@intel.com>
> > > Cc: Liming Gao <liming.gao@intel.com>
> > > Cc: Zailiang Sun <zailiang.sun@intel.com>
> > > Cc: Yi Qian <yi.qian@intel.com>
> > > Cc: Marcin Wojtas <mw@semihalf.com>
> > > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > > Cc: Leif Lindholm <leif.lindholm@linaro.org>
> > >
> > > Jian J Wang (5):
> > >   QuarkPlatformPkg: add null version of RngLib
> > >   UserInterfaceFeaturePkg: add null version RngLib
> > >   Vlv2TbltDevicePkg: add null version RngLib
> > >   Armada7k8k: add null version RngLib
> > >   MiscPlatforms: add null version RngLib
> > >
> > >  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> > | 1 +
> > >  Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
> > | 1 +
> > >  Platform/Intel/QuarkPlatformPkg/Quark.dsc
> > | 1 +
> > >  Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc
> > | 1 +
> > >
> > .../Intel/UserInterfaceFeaturePkg/UserInterfaceFeatureP
> > kg.dsc    | 1 +
> > >  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc
> > | 1 +
> > >  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc
> > | 1 +
> > >  Platform/LeMaker/CelloBoard/CelloBoard.dsc
> > | 1 +
> > >  Platform/RaspberryPi/RPi3/RPi3.dsc
> > | 1 +
> > >  Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc
> > | 1 +
> > >
> > Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoar
> > d.dsc     | 1 +
> > >  Silicon/Hisilicon/Hisilicon.dsc.inc
> > | 1 +
> > >  Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
> > | 1 +
> > >  13 files changed, 13 insertions(+)
> > >
> > > --
> > > 2.17.1.windows.2
> > >
> >
> >