From: "Michael D Kinney" <michael.d.kinney@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
"Li, Yi1" <yi1.li@intel.com>,
"Kinney, Michael D" <michael.d.kinney@intel.com>
Subject: Re: [edk2-devel] [PATCH 3/4] CryptoPkg: Make EC source file config-able
Date: Mon, 18 Apr 2022 15:21:27 +0000 [thread overview]
Message-ID: <CO1PR11MB4929C8EC8882E1E5481DC134D2F39@CO1PR11MB4929.namprd11.prod.outlook.com> (raw)
In-Reply-To: <c2aaf2127e267ba2f2811d64f9a22392478d363c.1650286720.git.yi1.li@intel.com>
A new top level PCD should not be added.
There is already a structured PCD to enable/disable crypto features.
We take advantage of compiler optimizations to remove unused functions,
so filtering the source files using a PCD should not be required.
I want to make sure we use a consistent method to configure all
crypto related services.
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of yi1 li
> Sent: Monday, April 18, 2022 6:03 AM
> To: devel@edk2.groups.io
> Cc: Li, Yi1 <yi1.li@intel.com>
> Subject: [edk2-devel] [PATCH 3/4] CryptoPkg: Make EC source file config-able
>
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3679
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3828
>
> Use PCD gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled to config-able
> source files list in OpensslLib.inf and OpensslLibCrypto.inf.
> If PcdEcEnabled equals to FALSE, this file will not be compiled.
>
> Signed-off-by: yi1 li <yi1.li@intel.com>
> ---
> CryptoPkg/CryptoPkg.dec | 4 +
> .../Library/Include/openssl/opensslconf.h | 7 +-
> CryptoPkg/Library/OpensslLib/OpensslLib.inf | 95 ++++++++++---------
> .../Library/OpensslLib/OpensslLibCrypto.inf | 95 ++++++++++---------
> 4 files changed, 108 insertions(+), 93 deletions(-)
>
> diff --git a/CryptoPkg/CryptoPkg.dec b/CryptoPkg/CryptoPkg.dec
> index 5888941bab4c..ebec64050b71 100644
> --- a/CryptoPkg/CryptoPkg.dec
> +++ b/CryptoPkg/CryptoPkg.dec
> @@ -81,5 +81,9 @@
> # @ValidList 0x80000001 | 0x00000001, 0x00000002, 0x00000004, 0x00000008, 0x00000010
> gEfiCryptoPkgTokenSpaceGuid.PcdHashApiLibPolicy|0x00000002|UINT32|0x00000001
>
> + ## Enable/Disable the ECC feature in openssl library. The default is disabled.
> + # If ECC feature is disabled, all related source files will not be compiled.
> + gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled|FALSE|BOOLEAN|0x0000003
> +
> [UserExtensions.TianoCore."ExtraFiles"]
> CryptoPkgExtra.uni
> diff --git a/CryptoPkg/Library/Include/openssl/opensslconf.h b/CryptoPkg/Library/Include/openssl/opensslconf.h
> index 7ea976b2252e..1485b8c9f108 100644
> --- a/CryptoPkg/Library/Include/openssl/opensslconf.h
> +++ b/CryptoPkg/Library/Include/openssl/opensslconf.h
> @@ -9,7 +9,7 @@
> * in the file LICENSE in the source distribution or at
> * https://www.openssl.org/source/license.html
> */
> -
> +#include <Library/PcdLib.h>
> #include <openssl/opensslv.h>
>
> #ifdef __cplusplus
> @@ -55,6 +55,11 @@ extern "C" {
> #ifndef OPENSSL_NO_DSA
> #define OPENSSL_NO_DSA
> #endif
> +#if !FixedPcdGetBool (PcdEcEnabled)
> + #ifndef OPENSSL_NO_EC
> +#define OPENSSL_NO_EC
> + #endif
> +#endif
> #ifndef OPENSSL_NO_IDEA
> #define OPENSSL_NO_IDEA
> #endif
> diff --git a/CryptoPkg/Library/OpensslLib/OpensslLib.inf b/CryptoPkg/Library/OpensslLib/OpensslLib.inf
> index 1d67ed55e1b1..459ac4864a4e 100644
> --- a/CryptoPkg/Library/OpensslLib/OpensslLib.inf
> +++ b/CryptoPkg/Library/OpensslLib/OpensslLib.inf
> @@ -199,43 +199,43 @@
> $(OPENSSL_PATH)/crypto/dso/dso_vms.c
> $(OPENSSL_PATH)/crypto/dso/dso_win32.c
> $(OPENSSL_PATH)/crypto/ebcdic.c
> - $(OPENSSL_PATH)/crypto/ec/curve25519.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/f_impl.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/curve448.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/curve448_tables.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/eddsa.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/f_generic.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/scalar.c
> - $(OPENSSL_PATH)/crypto/ec/ec2_oct.c
> - $(OPENSSL_PATH)/crypto/ec/ec2_smpl.c
> - $(OPENSSL_PATH)/crypto/ec/ec_ameth.c
> - $(OPENSSL_PATH)/crypto/ec/ec_asn1.c
> - $(OPENSSL_PATH)/crypto/ec/ec_check.c
> - $(OPENSSL_PATH)/crypto/ec/ec_curve.c
> - $(OPENSSL_PATH)/crypto/ec/ec_cvt.c
> - $(OPENSSL_PATH)/crypto/ec/ec_err.c
> - $(OPENSSL_PATH)/crypto/ec/ec_key.c
> - $(OPENSSL_PATH)/crypto/ec/ec_kmeth.c
> - $(OPENSSL_PATH)/crypto/ec/ec_lib.c
> - $(OPENSSL_PATH)/crypto/ec/ec_mult.c
> - $(OPENSSL_PATH)/crypto/ec/ec_oct.c
> - $(OPENSSL_PATH)/crypto/ec/ec_pmeth.c
> - $(OPENSSL_PATH)/crypto/ec/ec_print.c
> - $(OPENSSL_PATH)/crypto/ec/ecdh_kdf.c
> - $(OPENSSL_PATH)/crypto/ec/ecdh_ossl.c
> - $(OPENSSL_PATH)/crypto/ec/ecdsa_ossl.c
> - $(OPENSSL_PATH)/crypto/ec/ecdsa_sign.c
> - $(OPENSSL_PATH)/crypto/ec/ecdsa_vrf.c
> - $(OPENSSL_PATH)/crypto/ec/eck_prn.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_mont.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nist.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nistp224.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nistp256.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nistp521.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nistputil.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_oct.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_smpl.c
> - $(OPENSSL_PATH)/crypto/ec/ecx_meth.c
> + $(OPENSSL_PATH)/crypto/ec/curve25519.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/f_impl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/curve448.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/curve448_tables.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/eddsa.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/f_generic.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/scalar.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec2_oct.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec2_smpl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_ameth.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_asn1.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_check.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_curve.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_cvt.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_err.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_key.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_kmeth.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_lib.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_mult.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_oct.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_pmeth.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_print.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdh_kdf.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdh_ossl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdsa_ossl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdsa_sign.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdsa_vrf.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/eck_prn.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_mont.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nist.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nistp224.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nistp256.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nistp521.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nistputil.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_oct.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_smpl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecx_meth.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> $(OPENSSL_PATH)/crypto/err/err.c
> $(OPENSSL_PATH)/crypto/err/err_prn.c
> $(OPENSSL_PATH)/crypto/evp/bio_b64.c
> @@ -533,15 +533,15 @@
> $(OPENSSL_PATH)/crypto/conf/conf_local.h
> $(OPENSSL_PATH)/crypto/dh/dh_local.h
> $(OPENSSL_PATH)/crypto/dso/dso_local.h
> - $(OPENSSL_PATH)/crypto/ec/ec_local.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/curve448_local.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/curve448utils.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/ed448.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/field.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/point_448.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/word.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/arch_intrinsics.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/f_impl.h
> + $(OPENSSL_PATH)/crypto/ec/ec_local.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/curve448_local.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/curve448utils.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/ed448.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/field.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/point_448.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/word.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/arch_intrinsics.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/f_impl.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> $(OPENSSL_PATH)/crypto/evp/evp_local.h
> $(OPENSSL_PATH)/crypto/hmac/hmac_local.h
> $(OPENSSL_PATH)/crypto/lhash/lhash_local.h
> @@ -633,6 +633,9 @@
> [LibraryClasses.ARM]
> ArmSoftFloatLib
>
> +[Pcd]
> + gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled ## CONSUMES
> +
> [BuildOptions]
> #
> # Disables the following Visual Studio compiler warnings brought by openssl source,
> diff --git a/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf b/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
> index 66ca5b1250c1..c9d69a368e3c 100644
> --- a/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
> +++ b/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf
> @@ -199,43 +199,43 @@
> $(OPENSSL_PATH)/crypto/dso/dso_vms.c
> $(OPENSSL_PATH)/crypto/dso/dso_win32.c
> $(OPENSSL_PATH)/crypto/ebcdic.c
> - $(OPENSSL_PATH)/crypto/ec/curve25519.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/f_impl.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/curve448.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/curve448_tables.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/eddsa.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/f_generic.c
> - $(OPENSSL_PATH)/crypto/ec/curve448/scalar.c
> - $(OPENSSL_PATH)/crypto/ec/ec2_oct.c
> - $(OPENSSL_PATH)/crypto/ec/ec2_smpl.c
> - $(OPENSSL_PATH)/crypto/ec/ec_ameth.c
> - $(OPENSSL_PATH)/crypto/ec/ec_asn1.c
> - $(OPENSSL_PATH)/crypto/ec/ec_check.c
> - $(OPENSSL_PATH)/crypto/ec/ec_curve.c
> - $(OPENSSL_PATH)/crypto/ec/ec_cvt.c
> - $(OPENSSL_PATH)/crypto/ec/ec_err.c
> - $(OPENSSL_PATH)/crypto/ec/ec_key.c
> - $(OPENSSL_PATH)/crypto/ec/ec_kmeth.c
> - $(OPENSSL_PATH)/crypto/ec/ec_lib.c
> - $(OPENSSL_PATH)/crypto/ec/ec_mult.c
> - $(OPENSSL_PATH)/crypto/ec/ec_oct.c
> - $(OPENSSL_PATH)/crypto/ec/ec_pmeth.c
> - $(OPENSSL_PATH)/crypto/ec/ec_print.c
> - $(OPENSSL_PATH)/crypto/ec/ecdh_kdf.c
> - $(OPENSSL_PATH)/crypto/ec/ecdh_ossl.c
> - $(OPENSSL_PATH)/crypto/ec/ecdsa_ossl.c
> - $(OPENSSL_PATH)/crypto/ec/ecdsa_sign.c
> - $(OPENSSL_PATH)/crypto/ec/ecdsa_vrf.c
> - $(OPENSSL_PATH)/crypto/ec/eck_prn.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_mont.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nist.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nistp224.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nistp256.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nistp521.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_nistputil.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_oct.c
> - $(OPENSSL_PATH)/crypto/ec/ecp_smpl.c
> - $(OPENSSL_PATH)/crypto/ec/ecx_meth.c
> + $(OPENSSL_PATH)/crypto/ec/curve25519.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/f_impl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/curve448.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/curve448_tables.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/eddsa.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/f_generic.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/scalar.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec2_oct.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec2_smpl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_ameth.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_asn1.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_check.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_curve.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_cvt.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_err.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_key.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_kmeth.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_lib.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_mult.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_oct.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_pmeth.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ec_print.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdh_kdf.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdh_ossl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdsa_ossl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdsa_sign.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecdsa_vrf.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/eck_prn.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_mont.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nist.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nistp224.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nistp256.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nistp521.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_nistputil.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_oct.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecp_smpl.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/ecx_meth.c |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> $(OPENSSL_PATH)/crypto/err/err.c
> $(OPENSSL_PATH)/crypto/err/err_prn.c
> $(OPENSSL_PATH)/crypto/evp/bio_b64.c
> @@ -533,15 +533,15 @@
> $(OPENSSL_PATH)/crypto/conf/conf_local.h
> $(OPENSSL_PATH)/crypto/dh/dh_local.h
> $(OPENSSL_PATH)/crypto/dso/dso_local.h
> - $(OPENSSL_PATH)/crypto/ec/ec_local.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/curve448_local.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/curve448utils.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/ed448.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/field.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/point_448.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/word.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/arch_intrinsics.h
> - $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/f_impl.h
> + $(OPENSSL_PATH)/crypto/ec/ec_local.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/curve448_local.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/curve448utils.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/ed448.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/field.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/point_448.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/word.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/arch_intrinsics.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> + $(OPENSSL_PATH)/crypto/ec/curve448/arch_32/f_impl.h |*|*|*|gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled
> $(OPENSSL_PATH)/crypto/evp/evp_local.h
> $(OPENSSL_PATH)/crypto/hmac/hmac_local.h
> $(OPENSSL_PATH)/crypto/lhash/lhash_local.h
> @@ -582,6 +582,9 @@
> [LibraryClasses.ARM]
> ArmSoftFloatLib
>
> +[Pcd]
> + gEfiCryptoPkgTokenSpaceGuid.PcdEcEnabled ## CONSUMES
> +
> [BuildOptions]
> #
> # Disables the following Visual Studio compiler warnings brought by openssl source,
> --
> 2.33.0.windows.2
>
>
>
>
>
next prev parent reply other threads:[~2022-04-18 15:21 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-18 13:03 [PATCH 0/4] !!TEST Fix warnings of EC feature yi1 li
2022-04-18 13:03 ` [PATCH 1/4] CryptoPkg: Add instrinsics to support building ECC on IA32 windows yi1 li
2022-04-18 15:24 ` [edk2-devel] " Michael D Kinney
2022-04-19 1:14 ` yi1 li
2022-04-19 0:51 ` Yao, Jiewen
2022-04-18 13:03 ` [PATCH 2/4] CryptoPkg: Reconfigure OpensslLib to add EC algorithms yi1 li
2022-04-18 13:03 ` [PATCH 3/4] CryptoPkg: Make EC source file config-able yi1 li
2022-04-18 15:21 ` Michael D Kinney [this message]
2022-04-25 15:52 ` [edk2-devel] " PierreGondois
2022-04-26 2:01 ` yi1 li
2022-04-26 8:14 ` PierreGondois
2022-04-26 8:33 ` Bob Feng
2022-04-18 13:03 ` [PATCH 4/4] CryptoPkg: Add PcdEcEnabled to Base/Pei/SmmCryptLib.inf yi1 li
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CO1PR11MB4929C8EC8882E1E5481DC134D2F39@CO1PR11MB4929.namprd11.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox