diff mbox series

[1/1] efi_loader: EFI_RNG_PROTOCOL

Message ID 20200214223441.4739-1-xypron.glpk@gmx.de
State Accepted
Commit 0e228853b520611a0ee6fe2b9bd7a0bc3f8c30fc
Headers show
Series [1/1] efi_loader: EFI_RNG_PROTOCOL | expand

Commit Message

Heinrich Schuchardt Feb. 14, 2020, 10:34 p.m. UTC
Do not use quotation marks for Kconfig help text.
Replace rng abbreviation by full words.

Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
---
 lib/efi_loader/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

--
2.25.0

Comments

Sughosh Ganu Feb. 18, 2020, 10:26 a.m. UTC | #1
On Sat, 15 Feb 2020 at 04:04, Heinrich Schuchardt <xypron.glpk at gmx.de>
wrote:

> Do not use quotation marks for Kconfig help text.
> Replace rng abbreviation by full words.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
>  lib/efi_loader/Kconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
> index a7afa3f29e..84a4a35d80 100644
> --- a/lib/efi_loader/Kconfig
> +++ b/lib/efi_loader/Kconfig
> @@ -127,7 +127,7 @@ config EFI_RNG_PROTOCOL
>         bool "EFI_RNG_PROTOCOL support"
>         depends on DM_RNG
>         help
> -         "Support for EFI_RNG_PROTOCOL implementation. Uses the rng
> -          device on the platform"
> +         Provide a EFI_RNG_PROTOCOL implementation using the hardware
> random
> +         number generator of the platform.
>

Should we mention the word hardware in the sentence. Can it simply be
stated as "random number generator". I know that the existing sentence too
uses "rng device" which has hardware connotations. But the protocol can
also be used with a pseudo random number generator as well.

-sughosh
Heinrich Schuchardt Feb. 18, 2020, 2:14 p.m. UTC | #2
On 2/18/20 11:26 AM, Sughosh Ganu wrote:
>
> On Sat, 15 Feb 2020 at 04:04, Heinrich Schuchardt <xypron.glpk at gmx.de
> <mailto:xypron.glpk at gmx.de>> wrote:
>
>     Do not use quotation marks for Kconfig help text.
>     Replace rng abbreviation by full words.
>
>     Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de
>     <mailto:xypron.glpk at gmx.de>>
>     ---
>      ?lib/efi_loader/Kconfig | 4 ++--
>      ?1 file changed, 2 insertions(+), 2 deletions(-)
>
>     diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
>     index a7afa3f29e..84a4a35d80 100644
>     --- a/lib/efi_loader/Kconfig
>     +++ b/lib/efi_loader/Kconfig
>     @@ -127,7 +127,7 @@ config EFI_RNG_PROTOCOL
>      ? ? ? ? bool "EFI_RNG_PROTOCOL support"
>      ? ? ? ? depends on DM_RNG
>      ? ? ? ? help
>     -? ? ? ? ?"Support for EFI_RNG_PROTOCOL implementation. Uses the rng
>     -? ? ? ? ? device on the platform"
>     +? ? ? ? ?Provide a EFI_RNG_PROTOCOL implementation using the
>     hardware random
>     +? ? ? ? ?number generator of the platform.
>
>
> Should we mention the word hardware in the sentence. Can it simply be
> stated as "random number generator". I know that the existing sentence
> too uses "rng device" which has hardware connotations. But the protocol
> can also be used with a pseudo random number generator as well.
>
> -sughosh

Chapter 37.5 Random Number Generator Protocol of the UEFI mentions that
if this protocol uses a deterministic RNG it must be based on a source
of raw entropy.

So I think "using the hardware random number generator of the platform"
is accurate.

Currently the sandbox RNG does not use a hardware random number
generator. We should consider using the host's getrandom() function
which is available both on Linux, FreeBSD, and OpenBSD.

Best regards

Heinrich
Sughosh Ganu Feb. 18, 2020, 4:39 p.m. UTC | #3
On Tue, 18 Feb 2020 at 19:44, Heinrich Schuchardt <xypron.glpk at gmx.de>
wrote:

>
>
> On 2/18/20 11:26 AM, Sughosh Ganu wrote:
> >
> > On Sat, 15 Feb 2020 at 04:04, Heinrich Schuchardt <xypron.glpk at gmx.de
> > <mailto:xypron.glpk at gmx.de>> wrote:
> >
> >     Do not use quotation marks for Kconfig help text.
> >     Replace rng abbreviation by full words.
> >
> >     Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de
> >     <mailto:xypron.glpk at gmx.de>>
> >     ---
> >       lib/efi_loader/Kconfig | 4 ++--
> >       1 file changed, 2 insertions(+), 2 deletions(-)
> >
> >     diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
> >     index a7afa3f29e..84a4a35d80 100644
> >     --- a/lib/efi_loader/Kconfig
> >     +++ b/lib/efi_loader/Kconfig
> >     @@ -127,7 +127,7 @@ config EFI_RNG_PROTOCOL
> >              bool "EFI_RNG_PROTOCOL support"
> >              depends on DM_RNG
> >              help
> >     -         "Support for EFI_RNG_PROTOCOL implementation. Uses the rng
> >     -          device on the platform"
> >     +         Provide a EFI_RNG_PROTOCOL implementation using the
> >     hardware random
> >     +         number generator of the platform.
> >
> >
> > Should we mention the word hardware in the sentence. Can it simply be
> > stated as "random number generator". I know that the existing sentence
> > too uses "rng device" which has hardware connotations. But the protocol
> > can also be used with a pseudo random number generator as well.
> >
> > -sughosh
>
> Chapter 37.5 Random Number Generator Protocol of the UEFI mentions that
> if this protocol uses a deterministic RNG it must be based on a source
> of raw entropy.
>
> So I think "using the hardware random number generator of the platform"
> is accurate.
>

Yes. Makes sense.

Acked-by: Sughosh Ganu <sughosh.ganu at linaro.org>

-sughosh
diff mbox series

Patch

diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
index a7afa3f29e..84a4a35d80 100644
--- a/lib/efi_loader/Kconfig
+++ b/lib/efi_loader/Kconfig
@@ -127,7 +127,7 @@  config EFI_RNG_PROTOCOL
 	bool "EFI_RNG_PROTOCOL support"
 	depends on DM_RNG
 	help
-	  "Support for EFI_RNG_PROTOCOL implementation. Uses the rng
-	   device on the platform"
+	  Provide a EFI_RNG_PROTOCOL implementation using the hardware random
+	  number generator of the platform.

 endif