Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-21 Thread Jarkko Sakkinen
On Thu, Oct 17, 2019 at 09:04:40PM +0300, Jarkko Sakkinen wrote: > On Wed, Oct 16, 2019 at 03:10:29PM -0400, James Bottomley wrote: > > On Wed, 2019-10-16 at 19:25 +0300, Jarkko Sakkinen wrote: > > > On Wed, Oct 16, 2019 at 08:34:12AM -0400, James Bottomley wrote: > > > > reversible ciphers are gen

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-18 Thread Janne Karhunen
On Wed, Oct 16, 2019 at 6:35 PM James Bottomley wrote: > > The documentation says that krng is suitable for key generation. > > Should the documentation changed to state that it is unsuitable? > > How do you get that from the argument above? The krng is about the > best we have in terms of unpre

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-17 Thread Jarkko Sakkinen
On Wed, Oct 16, 2019 at 03:10:29PM -0400, James Bottomley wrote: > On Wed, 2019-10-16 at 19:25 +0300, Jarkko Sakkinen wrote: > > On Wed, Oct 16, 2019 at 08:34:12AM -0400, James Bottomley wrote: > > > reversible ciphers are generally frowned upon in random number > > > generation, that's why the krn

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-17 Thread James Bottomley
On Thu, 2019-10-17 at 18:22 +0530, Sumit Garg wrote: > On Thu, 17 Oct 2019 at 00:40, James Bottomley > wrote: > > > > On Wed, 2019-10-16 at 19:25 +0300, Jarkko Sakkinen wrote: > > > On Wed, Oct 16, 2019 at 08:34:12AM -0400, James Bottomley wrote: > > > > reversible ciphers are generally frowned u

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-17 Thread Sumit Garg
On Thu, 17 Oct 2019 at 00:40, James Bottomley wrote: > > On Wed, 2019-10-16 at 19:25 +0300, Jarkko Sakkinen wrote: > > On Wed, Oct 16, 2019 at 08:34:12AM -0400, James Bottomley wrote: > > > reversible ciphers are generally frowned upon in random number > > > generation, that's why the krng uses ch

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-16 Thread James Bottomley
On Wed, 2019-10-16 at 19:25 +0300, Jarkko Sakkinen wrote: > On Wed, Oct 16, 2019 at 08:34:12AM -0400, James Bottomley wrote: > > reversible ciphers are generally frowned upon in random number > > generation, that's why the krng uses chacha20. In general I think > > we shouldn't try to code our own

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-16 Thread Jarkko Sakkinen
On Wed, Oct 16, 2019 at 08:34:12AM -0400, James Bottomley wrote: > reversible ciphers are generally frowned upon in random number > generation, that's why the krng uses chacha20. In general I think we > shouldn't try to code our own mixing and instead should get the krng to > do it for us using wh

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-16 Thread James Bottomley
On Wed, 2019-10-16 at 14:00 +0300, Jarkko Sakkinen wrote: > On Mon, Oct 14, 2019 at 12:29:57PM -0700, James Bottomley wrote: > > The job of the in-kernel rng is simply to produce a mixed entropy > > pool from which we can draw random numbers. The idea is that quite > > a few attackers have identif

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-16 Thread Jarkko Sakkinen
On Mon, Oct 14, 2019 at 12:29:57PM -0700, James Bottomley wrote: > The job of the in-kernel rng is simply to produce a mixed entropy pool > from which we can draw random numbers. The idea is that quite a few > attackers have identified the rng as being a weak point in the security > architecture o

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-14 Thread James Bottomley
> > Cc: Safford, David (GE Global Research, US) > > >; Mimi > > > Zohar ; linux-integr...@vger.kernel.org; > > > sta...@vger.kernel.org; open list:ASYMMETRIC KEYS > > > ; open list:CRYPTO API > > cry...@vger.kernel.org>; open list > > >

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-14 Thread Jarkko Sakkinen
an > > > Cc: Safford, David (GE Global Research, US) ; Mimi > > > Zohar ; linux-integr...@vger.kernel.org; > > > sta...@vger.kernel.org; open list:ASYMMETRIC KEYS > > > ; open list:CRYPTO API > > cry...@vger.kernel.org>; open list > > > Subje

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-14 Thread Jarkko Sakkinen
On Wed, Oct 09, 2019 at 08:09:29AM +, Pascal Van Leeuwen wrote: > There's certification and certification. Not all certificates are > created equally. But if it matches your specific requirements, why not. > There's a _lot_ of HW out there that's not x86 though ... > > And: is RDRAND certified

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-14 Thread Jarkko Sakkinen
ntegr...@vger.kernel.org; > > sta...@vger.kernel.org; open list:ASYMMETRIC KEYS > > ; open list:CRYPTO API > cry...@vger.kernel.org>; open list > > Subject: EXT: Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes() > > > > On Wed, Oct 09, 2019 at 02:49:35AM +

[PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-09 Thread Safford, David (GE Global Research, US)
ernel.org>; open list > Subject: EXT: Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes() > > On Wed, Oct 09, 2019 at 02:49:35AM +0300, Jarkko Sakkinen wrote: > > On Mon, Oct 07, 2019 at 06:13:01PM -0400, Ken Goldman wrote: > > > The TPM library specification

RE: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-09 Thread Pascal Van Leeuwen
ta...@vger.kernel.org; open > list:ASYMMETRIC KEYS ; open list:CRYPTO API cry...@vger.kernel.org>; open list > Subject: Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes() > > On Wed, Oct 09, 2019 at 10:33:15AM +0300, Jarkko Sakkinen wrote: > > On Wed, Oct 09, 2019 at 0

RE: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-09 Thread Pascal Van Leeuwen
ta...@vger.kernel.org; open > list:ASYMMETRIC KEYS ; open list:CRYPTO API cry...@vger.kernel.org>; open list > Subject: Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes() > > On Wed, Oct 09, 2019 at 02:53:39AM +0300, Jarkko Sakkinen wrote: > > On Wed, Oct 09, 2019 at 0

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-09 Thread Jarkko Sakkinen
On Wed, Oct 09, 2019 at 10:33:15AM +0300, Jarkko Sakkinen wrote: > On Wed, Oct 09, 2019 at 02:53:39AM +0300, Jarkko Sakkinen wrote: > > On Wed, Oct 09, 2019 at 02:49:35AM +0300, Jarkko Sakkinen wrote: > > > On Mon, Oct 07, 2019 at 06:13:01PM -0400, Ken Goldman wrote: > > > > The TPM library specifi

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-09 Thread Jarkko Sakkinen
On Wed, Oct 09, 2019 at 02:53:39AM +0300, Jarkko Sakkinen wrote: > On Wed, Oct 09, 2019 at 02:49:35AM +0300, Jarkko Sakkinen wrote: > > On Mon, Oct 07, 2019 at 06:13:01PM -0400, Ken Goldman wrote: > > > The TPM library specification states that the TPM must comply with NIST > > > SP800-90 A. > > >

RE: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-09 Thread Pascal Van Leeuwen
ta...@vger.kernel.org; open > list:ASYMMETRIC KEYS ; open list:CRYPTO API cry...@vger.kernel.org>; open list > Subject: Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes() > > On Wed, Oct 09, 2019 at 02:49:35AM +0300, Jarkko Sakkinen wrote: > > On Mon, Oct 07, 2019 at 06

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-08 Thread Jarkko Sakkinen
On Wed, Oct 09, 2019 at 02:49:35AM +0300, Jarkko Sakkinen wrote: > On Mon, Oct 07, 2019 at 06:13:01PM -0400, Ken Goldman wrote: > > The TPM library specification states that the TPM must comply with NIST > > SP800-90 A. > > > > https://trustedcomputinggroup.org/membership/certification/tpm-certifi

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-08 Thread Jarkko Sakkinen
On Mon, Oct 07, 2019 at 06:13:01PM -0400, Ken Goldman wrote: > The TPM library specification states that the TPM must comply with NIST > SP800-90 A. > > https://trustedcomputinggroup.org/membership/certification/tpm-certified-products/ > > shows that the TPMs get third party certification, Common

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-07 Thread Ken Goldman
The TPM library specification states that the TPM must comply with NIST SP800-90 A. https://trustedcomputinggroup.org/membership/certification/tpm-certified-products/ shows that the TPMs get third party certification, Common Criteria EAL 4+. While it's theoretically possible that an attacker c

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-07 Thread Mimi Zohar
On Mon, 2019-10-07 at 02:52 +0300, Jarkko Sakkinen wrote: > > With TEE coming in, TPM is not the only hardware measure anymore sealing > the keys and we don't want a mess where every hardware asset does their > own proprietary key generation. The proprietary technology should only > take care of t

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-07 Thread Janne Karhunen
On Thu, Oct 3, 2019 at 2:41 PM Jarkko Sakkinen wrote: > > At what point during boot is the kernel random pool available? Does > > this imply that you're planning on changing trusted keys as well? > > Well trusted keys *must* be changed to use it. It is not a choice > because using a proprietary

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-06 Thread Jarkko Sakkinen
On Fri, Oct 04, 2019 at 07:56:01PM +, Safford, David (GE Global Research, US) wrote: > > > From: linux-integrity-ow...@vger.kernel.org > ow...@vger.kernel.org> On Behalf Of Jarkko Sakkinen > > Sent: Friday, October 4, 2019 2:27 PM > > Subject: EXT: Re: [PATCH

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-06 Thread Jarkko Sakkinen
On Sat, Oct 05, 2019 at 08:38:53PM -0400, Mimi Zohar wrote: > On Fri, 2019-10-04 at 15:11 -0700, James Bottomley wrote: > > > + > > +/** > > + * tpm_get_random() - get random bytes influenced by the TPM's RNG > > + * @chip: a &struct tpm_chip instance, %NULL for the default chip > > + * @out: d

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-05 Thread Mimi Zohar
On Fri, 2019-10-04 at 15:11 -0700, James Bottomley wrote: > + > +/** > + * tpm_get_random() - get random bytes influenced by the TPM's RNG > + * @chip:a &struct tpm_chip instance, %NULL for the default chip > + * @out: destination buffer for the random bytes > + * @max: the max number

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread James Bottomley
On Fri, 2019-10-04 at 13:11 -0700, Jerry Snitselaar wrote: > On Fri Oct 04 19, Jerry Snitselaar wrote: > > On Fri Oct 04 19, James Bottomley wrote: > > > On Fri, 2019-10-04 at 11:33 -0700, Jerry Snitselaar wrote: > > > > On Fri Oct 04 19, James Bottomley wrote: > > > > > On Fri, 2019-10-04 at 21:22

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread Jerry Snitselaar
On Fri Oct 04 19, Jerry Snitselaar wrote: On Fri Oct 04 19, James Bottomley wrote: On Fri, 2019-10-04 at 11:33 -0700, Jerry Snitselaar wrote: On Fri Oct 04 19, James Bottomley wrote: On Fri, 2019-10-04 at 21:22 +0300, Jarkko Sakkinen wrote: > On Thu, Oct 03, 2019 at 04:59:37PM -0700, James Bot

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread Jerry Snitselaar
On Fri Oct 04 19, James Bottomley wrote: On Fri, 2019-10-04 at 11:33 -0700, Jerry Snitselaar wrote: On Fri Oct 04 19, James Bottomley wrote: > On Fri, 2019-10-04 at 21:22 +0300, Jarkko Sakkinen wrote: > > On Thu, Oct 03, 2019 at 04:59:37PM -0700, James Bottomley wrote: > > > I think the principl

[PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread Safford, David (GE Global Research, US)
> From: linux-integrity-ow...@vger.kernel.org ow...@vger.kernel.org> On Behalf Of Jarkko Sakkinen > Sent: Friday, October 4, 2019 2:27 PM > Subject: EXT: Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes() > > If you are able to call tpm_get_random(), the driver has

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread James Bottomley
On Fri, 2019-10-04 at 11:33 -0700, Jerry Snitselaar wrote: > On Fri Oct 04 19, James Bottomley wrote: > > On Fri, 2019-10-04 at 21:22 +0300, Jarkko Sakkinen wrote: > > > On Thu, Oct 03, 2019 at 04:59:37PM -0700, James Bottomley wrote: > > > > I think the principle of using multiple RNG sources for

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread Jerry Snitselaar
On Fri Oct 04 19, James Bottomley wrote: On Fri, 2019-10-04 at 21:22 +0300, Jarkko Sakkinen wrote: On Thu, Oct 03, 2019 at 04:59:37PM -0700, James Bottomley wrote: > I think the principle of using multiple RNG sources for strong keys > is a sound one, so could I propose a compromise: We have a

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread Jarkko Sakkinen
> > There are many good reasons for wanting the keys to be based on the > > TPM generator. As the source for the kernel random number generator > > itself says, some systems lack good randomness at startup, and systems > > should preserve and reload the pool across shutdown and startup. > > There

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread Jarkko Sakkinen
On Fri, Oct 04, 2019 at 01:26:58PM +, Safford, David (GE Global Research, US) wrote: > As the original author of trusted keys, let me make a few comments. > First, trusted keys were specifically implemented and *documented* to > use the TPM to both generate and seal keys. Its kernel documentat

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread James Bottomley
On Fri, 2019-10-04 at 21:22 +0300, Jarkko Sakkinen wrote: > On Thu, Oct 03, 2019 at 04:59:37PM -0700, James Bottomley wrote: > > I think the principle of using multiple RNG sources for strong keys > > is a sound one, so could I propose a compromise: We have a tpm > > subsystem random number genera

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread Jarkko Sakkinen
On Thu, Oct 03, 2019 at 04:59:37PM -0700, James Bottomley wrote: > I think the principle of using multiple RNG sources for strong keys is > a sound one, so could I propose a compromise: We have a tpm subsystem > random number generator that, when asked for random bytes first > extracts bytes fro

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread Jarkko Sakkinen
On Thu, Oct 03, 2019 at 06:08:11PM -0400, Mimi Zohar wrote: > > At the time when trusted keys was introduced I'd say that it was a wrong > > design decision and badly implemented code. But you are right in that as > > far that code is considered it would unfair to speak of a regression. > > > > as

[PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-04 Thread Safford, David (GE Global Research, US)
> From: Mimi Zohar > Sent: Thursday, October 3, 2019 2:54 PM > To: Jarkko Sakkinen ; Safford, David (GE > Subject: EXT: Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes() > > [Cc'ing David Safford] > > On Thu, 2019-10-03 at 20:58 +0300, Jarkko Sakkinen wrote

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread James Bottomley
On Thu, 2019-10-03 at 18:08 -0400, Mimi Zohar wrote: > On Fri, 2019-10-04 at 00:57 +0300, Jarkko Sakkinen wrote: > > On Fri, Oct 04, 2019 at 12:51:25AM +0300, Jarkko Sakkinen wrote: > > > On Thu, Oct 03, 2019 at 02:53:47PM -0400, Mimi Zohar wrote: > > > > [Cc'ing David Safford] > > > > > > > > On

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Jarkko Sakkinen
On Fri, Oct 04, 2019 at 12:57:43AM +0300, Jarkko Sakkinen wrote: > On Fri, Oct 04, 2019 at 12:51:25AM +0300, Jarkko Sakkinen wrote: > > On Thu, Oct 03, 2019 at 02:53:47PM -0400, Mimi Zohar wrote: > > > [Cc'ing David Safford] > > > > > > On Thu, 2019-10-03 at 20:58 +0300, Jarkko Sakkinen wrote: > >

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Mimi Zohar
On Fri, 2019-10-04 at 00:57 +0300, Jarkko Sakkinen wrote: > On Fri, Oct 04, 2019 at 12:51:25AM +0300, Jarkko Sakkinen wrote: > > On Thu, Oct 03, 2019 at 02:53:47PM -0400, Mimi Zohar wrote: > > > [Cc'ing David Safford] > > > > > > On Thu, 2019-10-03 at 20:58 +0300, Jarkko Sakkinen wrote: > > > > On

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Jarkko Sakkinen
On Fri, Oct 04, 2019 at 12:51:25AM +0300, Jarkko Sakkinen wrote: > On Thu, Oct 03, 2019 at 02:53:47PM -0400, Mimi Zohar wrote: > > [Cc'ing David Safford] > > > > On Thu, 2019-10-03 at 20:58 +0300, Jarkko Sakkinen wrote: > > > On Thu, Oct 03, 2019 at 09:02:32AM -0400, Mimi Zohar wrote: > > > > On T

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Jarkko Sakkinen
On Thu, Oct 03, 2019 at 02:53:47PM -0400, Mimi Zohar wrote: > [Cc'ing David Safford] > > On Thu, 2019-10-03 at 20:58 +0300, Jarkko Sakkinen wrote: > > On Thu, Oct 03, 2019 at 09:02:32AM -0400, Mimi Zohar wrote: > > > On Thu, 2019-10-03 at 14:41 +0300, Jarkko Sakkinen wrote: > > > > On Wed, Oct 02,

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Mimi Zohar
[Cc'ing David Safford] On Thu, 2019-10-03 at 20:58 +0300, Jarkko Sakkinen wrote: > On Thu, Oct 03, 2019 at 09:02:32AM -0400, Mimi Zohar wrote: > > On Thu, 2019-10-03 at 14:41 +0300, Jarkko Sakkinen wrote: > > > On Wed, Oct 02, 2019 at 10:00:19AM -0400, Mimi Zohar wrote: > > > > On Thu, 2019-09-26

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Jarkko Sakkinen
On Thu, Oct 03, 2019 at 09:02:01PM +0300, Jarkko Sakkinen wrote: > On Thu, Oct 03, 2019 at 09:02:32AM -0400, Mimi Zohar wrote: > > That isn't a valid justification for changing the original definition > > of trusted keys.  Just as the kernel supports different methods of > > implementing the same f

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Jarkko Sakkinen
On Thu, Oct 03, 2019 at 09:02:32AM -0400, Mimi Zohar wrote: > That isn't a valid justification for changing the original definition > of trusted keys.  Just as the kernel supports different methods of > implementing the same function on different architectures, trusted > keys will need to support d

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Jarkko Sakkinen
On Thu, Oct 03, 2019 at 09:02:32AM -0400, Mimi Zohar wrote: > On Thu, 2019-10-03 at 14:41 +0300, Jarkko Sakkinen wrote: > > On Wed, Oct 02, 2019 at 10:00:19AM -0400, Mimi Zohar wrote: > > > On Thu, 2019-09-26 at 20:16 +0300, Jarkko Sakkinen wrote: > > > > Only the kernel random pool should be used

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Mimi Zohar
On Thu, 2019-10-03 at 14:41 +0300, Jarkko Sakkinen wrote: > On Wed, Oct 02, 2019 at 10:00:19AM -0400, Mimi Zohar wrote: > > On Thu, 2019-09-26 at 20:16 +0300, Jarkko Sakkinen wrote: > > > Only the kernel random pool should be used for generating random numbers. > > > TPM contributes to that pool am

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Jarkko Sakkinen
On Thu, Oct 03, 2019 at 02:41:19PM +0300, Jarkko Sakkinen wrote: > On Wed, Oct 02, 2019 at 10:00:19AM -0400, Mimi Zohar wrote: > > On Thu, 2019-09-26 at 20:16 +0300, Jarkko Sakkinen wrote: > > > Only the kernel random pool should be used for generating random numbers. > > > TPM contributes to that

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-03 Thread Jarkko Sakkinen
On Wed, Oct 02, 2019 at 10:00:19AM -0400, Mimi Zohar wrote: > On Thu, 2019-09-26 at 20:16 +0300, Jarkko Sakkinen wrote: > > Only the kernel random pool should be used for generating random numbers. > > TPM contributes to that pool among the other sources of entropy. In here it > > is not, agreed, a

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-02 Thread Mimi Zohar
On Thu, 2019-09-26 at 20:16 +0300, Jarkko Sakkinen wrote: > Only the kernel random pool should be used for generating random numbers. > TPM contributes to that pool among the other sources of entropy. In here it > is not, agreed, absolutely critical because TPM is what is trusted anyway > but in or

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-10-01 Thread Jarkko Sakkinen
On Sat, Sep 28, 2019 at 11:05:59AM -0700, Jerry Snitselaar wrote: > On Thu Sep 26 19, Jarkko Sakkinen wrote: > > Only the kernel random pool should be used for generating random numbers. > > TPM contributes to that pool among the other sources of entropy. In here it > > is not, agreed, absolutely c

Re: [PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-09-28 Thread Jerry Snitselaar
On Thu Sep 26 19, Jarkko Sakkinen wrote: Only the kernel random pool should be used for generating random numbers. TPM contributes to that pool among the other sources of entropy. In here it is not, agreed, absolutely critical because TPM is what is trusted anyway but in order to remove tpm_get_r

[PATCH] KEYS: asym_tpm: Switch to get_random_bytes()

2019-09-26 Thread Jarkko Sakkinen
Only the kernel random pool should be used for generating random numbers. TPM contributes to that pool among the other sources of entropy. In here it is not, agreed, absolutely critical because TPM is what is trusted anyway but in order to remove tpm_get_random() we need to first remove all the cal