All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] crypto: caam: set hwrng quality level
@ 2019-02-13 14:49 Philipp Puschmann
  2019-02-19  6:57 ` Horia Geanta
  0 siblings, 1 reply; 2+ messages in thread
From: Philipp Puschmann @ 2019-02-13 14:49 UTC (permalink / raw)
  To: horia.geanta
  Cc: aymen.sghaier, herbert, davem, linux-crypto, linux-kernel,
	o.rempel, Philipp Puschmann

When quality is set to 0 this driver is not used as randomness source by
HWRNG framework at all. So set the quality and finally make this driver
work. The value of the quality was discussed before
( see https://patchwork.kernel.org/patch/9850669/ ) and it is still a rough
value.

Signed-off-by: Philipp Puschmann <philipp.puschmann@emlix.com>
---
 drivers/crypto/caam/caamrng.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/crypto/caam/caamrng.c b/drivers/crypto/caam/caamrng.c
index a387c8d49a62..b6fd60f7638e 100644
--- a/drivers/crypto/caam/caamrng.c
+++ b/drivers/crypto/caam/caamrng.c
@@ -294,6 +294,7 @@ static struct hwrng caam_rng = {
 	.name		= "rng-caam",
 	.cleanup	= caam_cleanup,
 	.read		= caam_read,
+	.quality	= 990,
 };
 
 static void __exit caam_rng_exit(void)
-- 
2.20.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] crypto: caam: set hwrng quality level
  2019-02-13 14:49 [PATCH] crypto: caam: set hwrng quality level Philipp Puschmann
@ 2019-02-19  6:57 ` Horia Geanta
  0 siblings, 0 replies; 2+ messages in thread
From: Horia Geanta @ 2019-02-19  6:57 UTC (permalink / raw)
  To: Philipp Puschmann
  Cc: Aymen Sghaier, herbert, davem, linux-crypto, linux-kernel, o.rempel

On 2/13/2019 4:49 PM, Philipp Puschmann wrote:
> When quality is set to 0 this driver is not used as randomness source by
> HWRNG framework at all. So set the quality and finally make this driver
> work. The value of the quality was discussed before
> ( see see https://patchwork.kernel.org/patch/9850669/ ) and it is still a rough
> value.
> 
In the same thread you mention, the conclusion is that caamrng should be
properly configured (TRNG instead of DRBG) before claiming to be a reliable
hwrng entropy source.

Horia

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-02-19  6:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-13 14:49 [PATCH] crypto: caam: set hwrng quality level Philipp Puschmann
2019-02-19  6:57 ` Horia Geanta

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.