From: "Yao, Jiewen" <jiewen.yao@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
"quic_llindhol@quicinc.com" <quic_llindhol@quicinc.com>
Cc: "pierre.gondois@arm.com" <pierre.gondois@arm.com>,
Sami Mujawar <sami.mujawar@arm.com>,
Ard Biesheuvel <ardb+tianocore@kernel.org>,
"Rebecca Cran" <rebecca@bsdio.com>,
"Kinney, Michael D" <michael.d.kinney@intel.com>,
"Gao, Liming" <gaoliming@byosoft.com.cn>,
"Wang, Jian J" <jian.j.wang@intel.com>
Subject: Re: [edk2-devel] [PATCH v8 06/19] MdePkg/TrngLib: Definition for TRNG library class interface
Date: Wed, 26 Oct 2022 14:06:44 +0000 [thread overview]
Message-ID: <MW4PR11MB5872CFC0DB569F0676E7E0208C309@MW4PR11MB5872.namprd11.prod.outlook.com> (raw)
In-Reply-To: <Y1k8dk0JOm02pita@qc-i7.hemma.eciton.net>
Oh, I did not realize we are planning to deprecate ArmPkg.
Is there a Bugzilla I can refer to?
If so, I don't think MdeModulePkg is better, because we usually say: industry standard can be in MdePkg. With this justification, I would suggest to name this library class to be ArmTrngLib.
In the library header, I recommend to say: it follows "Arm True Random Number Generator Firmware, Interface 1.0", but not NIST SP800-90A/B/C. The latter is quite misleading, because there is not GetVersion/GetUUID in NIST document at all.
Thank you
Yao Jiewen
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Leif
> Lindholm
> Sent: Wednesday, October 26, 2022 9:56 PM
> To: Yao, Jiewen <jiewen.yao@intel.com>
> Cc: devel@edk2.groups.io; pierre.gondois@arm.com; Sami Mujawar
> <sami.mujawar@arm.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>;
> Rebecca Cran <rebecca@bsdio.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>;
> Wang, Jian J <jian.j.wang@intel.com>
> Subject: Re: [edk2-devel] [PATCH v8 06/19] MdePkg/TrngLib: Definition for
> TRNG library class interface
>
> Hi Jiewen,
>
> There could be an argument for moving it to MdeModulePkg, but there is
> no argument for it moving to ArmPkg.
>
> ArmPkg does not have any reason to exist other than that it has not
> yet been properly integrated in the core packages. Which clearly is
> becoming more urgent, as I seem to be raising this point even more
> frequently these days.
>
> Best Regards,
>
> Leif
>
> On Wed, Oct 26, 2022 at 13:25:54 +0000, Yao, Jiewen wrote:
> > Hi
> > This API is very ARM platform specific.
> >
> > I don't see any interface like GetVersion or GetUUID is defined in 800-
> 90A/90B/90C.
> > What it is following is "Arm True Random Number Generator Firmware,
> Interface 1.0". As such, I feel uncomfortable to define TrngLib in MdePkg.
> >
> > I have raised my concern before. My recommend is to move ArmPkg, for
> example ArmTrngLib.
> >
> >
> >
> > Thank you
> > Yao Jiewen
> >
> >
> > > -----Original Message-----
> > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > PierreGondois
> > > Sent: Tuesday, October 18, 2022 9:21 PM
> > > To: devel@edk2.groups.io
> > > Cc: Sami Mujawar <sami.mujawar@arm.com>; Leif Lindholm
> > > <quic_llindhol@quicinc.com>; Ard Biesheuvel
> > > <ardb+tianocore@kernel.org>; Rebecca Cran <rebecca@bsdio.com>;
> Kinney,
> > > Michael D <michael.d.kinney@intel.com>; Gao, Liming
> > > <gaoliming@byosoft.com.cn>; Yao, Jiewen <jiewen.yao@intel.com>;
> Wang,
> > > Jian J <jian.j.wang@intel.com>
> > > Subject: [edk2-devel] [PATCH v8 06/19] MdePkg/TrngLib: Definition for
> > > TRNG library class interface
> > >
> > > From: Sami Mujawar <sami.mujawar@arm.com>
> > >
> > > Bugzilla: 3668 (https://bugzilla.tianocore.org/show_bug.cgi?id=3668)
> > >
> > > The NIST Special Publications 800-90A, 800-90B and 800-90C
> > > provide recommendations for random number generation. The
> > > NIST 800-90C, Recommendation for Random Bit Generator (RBG)
> > > Constructions, defines the GetEntropy() interface that is
> > > used to access the entropy source. The GetEntropy() interface
> > > is further used by Deterministic Random Bit Generators (DRBG)
> > > to generate random numbers.
> > >
> > > The True Random Number Generator (TRNG) library defines an
> > > interface to access the entropy source on a platform. Some
> > > platforms/architectures may provide access to the entropy
> > > using a firmware interface. In such cases the TRNG library
> > > shall be used to provide an abstraction.
> > >
> > > Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
> > > ---
> > > MdePkg/Include/Library/TrngLib.h | 103
> > > +++++++++++++++++++++++++++++++
> > > MdePkg/MdePkg.dec | 5 ++
> > > 2 files changed, 108 insertions(+)
> > > create mode 100644 MdePkg/Include/Library/TrngLib.h
> > >
> > > diff --git a/MdePkg/Include/Library/TrngLib.h
> > > b/MdePkg/Include/Library/TrngLib.h
> > > new file mode 100644
> > > index 000000000000..535fd53f4305
> > > --- /dev/null
> > > +++ b/MdePkg/Include/Library/TrngLib.h
> > > @@ -0,0 +1,103 @@
> > > +/** @file
> > > + TRNG interface library definitions.
> > > +
> > > + Copyright (c) 2021 - 2022, Arm Limited. All rights reserved.<BR>
> > > +
> > > + SPDX-License-Identifier: BSD-2-Clause-Patent
> > > +
> > > + @par Reference(s):
> > > + - [1] NIST Special Publication 800-90B, Recommendation for the
> Entropy
> > > + Sources Used for Random Bit Generation.
> > > + (https://csrc.nist.gov/publications/detail/sp/800-90b/final)
> > > +
> > > + @par Glossary:
> > > + - TRNG - True Random Number Generator
> > > +**/
> > > +
> > > +#ifndef TRNG_LIB_H_
> > > +#define TRNG_LIB_H_
> > > +
> > > +/** Get the version of the TRNG backend.
> > > +
> > > + A TRNG may be implemented by the system firmware, in which case
> this
> > > + function shall return the version of the TRNG backend.
> > > + The implementation must return NOT_SUPPORTED if a Back end is
> not
> > > present.
> > > +
> > > + @param [out] MajorRevision Major revision.
> > > + @param [out] MinorRevision Minor revision.
> > > +
> > > + @retval RETURN_SUCCESS The function completed
> successfully.
> > > + @retval RETURN_INVALID_PARAMETER Invalid parameter.
> > > + @retval RETURN_UNSUPPORTED Backend not present.
> > > +**/
> > > +RETURN_STATUS
> > > +EFIAPI
> > > +GetTrngVersion (
> > > + OUT UINT16 *MajorRevision,
> > > + OUT UINT16 *MinorRevision
> > > + );
> > > +
> > > +/** Get the UUID of the TRNG backend.
> > > +
> > > + A TRNG may be implemented by the system firmware, in which case
> this
> > > + function shall return the UUID of the TRNG backend.
> > > + Returning the TRNG UUID is optional and if not implemented,
> > > RETURN_UNSUPPORTED
> > > + shall be returned.
> > > +
> > > + Note: The caller must not rely on the returned UUID as a trustworthy
> > > TRNG
> > > + Back end identity
> > > +
> > > + @param [out] Guid UUID of the TRNG backend.
> > > +
> > > + @retval RETURN_SUCCESS The function completed
> successfully.
> > > + @retval RETURN_INVALID_PARAMETER Invalid parameter.
> > > + @retval RETURN_UNSUPPORTED Function not implemented.
> > > +**/
> > > +RETURN_STATUS
> > > +EFIAPI
> > > +GetTrngUuid (
> > > + OUT GUID *Guid
> > > + );
> > > +
> > > +/** Returns maximum number of entropy bits that can be returned in
> a
> > > single
> > > + call.
> > > +
> > > + @return Returns the maximum number of Entropy bits that can be
> > > returned
> > > + in a single call to GetTrngEntropy().
> > > +**/
> > > +UINTN
> > > +EFIAPI
> > > +GetTrngMaxSupportedEntropyBits (
> > > + VOID
> > > + );
> > > +
> > > +/** Returns N bits of conditioned entropy.
> > > +
> > > + See [1] Section 2.3.1 GetEntropy: An Interface to the Entropy Source
> > > + GetEntropy
> > > + Input:
> > > + bits_of_entropy: the requested amount of entropy
> > > + Output:
> > > + entropy_bitstring: The string that provides the requested entropy.
> > > + status: A Boolean value that is TRUE if the request has been
> satisfied,
> > > + and is FALSE otherwise.
> > > +
> > > + @param [in] EntropyBits Number of entropy bits requested.
> > > + @param [in] BufferSize Size of the Buffer in bytes.
> > > + @param [out] Buffer Buffer to return the entropy bits.
> > > +
> > > + @retval RETURN_SUCCESS The function completed
> successfully.
> > > + @retval RETURN_INVALID_PARAMETER Invalid parameter.
> > > + @retval RETURN_UNSUPPORTED Function not implemented.
> > > + @retval RETURN_BAD_BUFFER_SIZE Buffer size is too small.
> > > + @retval RETURN_NOT_READY No Entropy available.
> > > +**/
> > > +RETURN_STATUS
> > > +EFIAPI
> > > +GetTrngEntropy (
> > > + IN UINTN EntropyBits,
> > > + IN UINTN BufferSize,
> > > + OUT UINT8 *Buffer
> > > + );
> > > +
> > > +#endif // TRNG_LIB_H_
> > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec
> > > index 4c81cbd75ab2..0a7859fc1d3f 100644
> > > --- a/MdePkg/MdePkg.dec
> > > +++ b/MdePkg/MdePkg.dec
> > > @@ -8,6 +8,7 @@
> > > # Portions copyright (c) 2008 - 2009, Apple Inc. All rights reserved.<BR>
> > > # (C) Copyright 2016 - 2021 Hewlett Packard Enterprise Development
> > > LP<BR>
> > > # Copyright (c) 2022, Loongson Technology Corporation Limited. All
> rights
> > > reserved.<BR>
> > > +# Copyright (c) 2021 - 2022, Arm Limited. All rights reserved.<BR>
> > > #
> > > # SPDX-License-Identifier: BSD-2-Clause-Patent
> > > #
> > > @@ -279,6 +280,10 @@ [LibraryClasses]
> > > ## @libraryclass Provides function for SMM CPU Rendezvous Library.
> > > SmmCpuRendezvousLib|Include/Library/SmmCpuRendezvousLib.h
> > >
> > > + ## @libraryclass Provides services to generate Entropy using a TRNG.
> > > + #
> > > + TrngLib|Include/Library/TrngLib.h
> > > +
> > > [LibraryClasses.IA32, LibraryClasses.X64, LibraryClasses.AARCH64]
> > > ## @libraryclass Provides services to generate random number.
> > > #
> > > --
> > > 2.25.1
> > >
> > >
> > >
> > > -=-=-=-=-=-=
> > > Groups.io Links: You receive all messages sent to this group.
> > > View/Reply Online (#95322):
> > > https://edk2.groups.io/g/devel/message/95322
> > > Mute This Topic: https://groups.io/mt/94407745/1772286
> > > Group Owner: devel+owner@edk2.groups.io
> > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> [jiewen.yao@intel.com]
> > > -=-=-=-=-=-=
> > >
> >
>
>
>
>
next prev parent reply other threads:[~2022-10-26 14:06 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-18 13:20 [PATCH v8 00/19] Add Raw algorithm support using Arm FW-TRNG interface PierreGondois
2022-10-18 13:20 ` [PATCH v8 01/19] ArmPkg: PCD to select conduit for monitor calls PierreGondois
2022-10-18 13:20 ` [PATCH v8 02/19] ArmPkg/ArmMonitorLib: Definition for ArmMonitorLib library class PierreGondois
2022-10-18 13:20 ` [PATCH v8 03/19] ArmPkg/ArmMonitorLib: Add ArmMonitorLib PierreGondois
2022-10-18 13:20 ` [PATCH v8 04/19] ArmPkg: Sort HVC/SMC section alphbetically in ArmPkg.dsc PierreGondois
2022-10-18 13:20 ` [PATCH v8 05/19] ArmPkg/ArmHvcLibNull: Add NULL instance of ArmHvcLib PierreGondois
2022-10-18 13:20 ` [PATCH v8 06/19] MdePkg/TrngLib: Definition for TRNG library class interface PierreGondois
2022-10-26 13:25 ` [edk2-devel] " Yao, Jiewen
2022-10-26 13:56 ` Leif Lindholm
2022-10-26 14:06 ` Yao, Jiewen [this message]
2022-10-26 15:44 ` Leif Lindholm
2022-10-18 13:20 ` [PATCH v8 07/19] MdePkg/TrngLib: Add NULL instance of TRNG Library PierreGondois
2022-10-18 13:20 ` [PATCH v8 08/19] ArmPkg: Add FID definitions for Firmware TRNG PierreGondois
2022-10-20 8:55 ` Leif Lindholm
2022-10-18 13:20 ` [PATCH v8 09/19] ArmPkg/TrngLib: Add Arm Firmware TRNG library PierreGondois
2022-10-18 13:20 ` [PATCH v8 10/19] SecurityPkg/RngDxe: Rename RdRandGenerateEntropy to generic name PierreGondois
2022-10-18 13:20 ` [PATCH v8 11/19] SecurityPkg/RngDxe: Replace Pcd with Sp80090Ctr256Guid PierreGondois
2022-10-18 13:20 ` [PATCH v8 12/19] SecurityPkg/RngDxe: Remove ArchGetSupportedRngAlgorithms() PierreGondois
2022-10-18 13:20 ` [PATCH v8 13/19] SecurityPkg/RngDxe: Documentation/include/parameter cleanup PierreGondois
2022-10-18 13:20 ` [PATCH v8 14/19] SecurityPkg/RngDxe: Check before advertising Cpu Rng algo PierreGondois
2022-10-18 13:20 ` [PATCH v8 15/19] SecurityPkg/RngDxe: Add AArch64 RawAlgorithm support through TrngLib PierreGondois
2022-10-18 13:20 ` [PATCH v8 16/19] SecurityPkg/RngDxe: Add debug warning for NULL PcdCpuRngSupportedAlgorithm PierreGondois
2022-10-18 13:20 ` [PATCH v8 17/19] SecurityPkg/RngDxe: Rename AArch64/RngDxe.c PierreGondois
2022-10-18 13:20 ` [PATCH v8 18/19] SecurityPkg/RngDxe: Add Arm support of RngDxe PierreGondois
2022-10-18 13:20 ` [PATCH v8 19/19] ArmVirtPkg: Kvmtool: Add RNG support using FW-TRNG interface PierreGondois
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=MW4PR11MB5872CFC0DB569F0676E7E0208C309@MW4PR11MB5872.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