On (Mon) 14 Jul 2014 [20:50:06], Herbert Xu wrote:> On Thu, Jul 10, 2014 at 03:42:33PM +0530, Amit Shah wrote: > > v3: > > - Kees Cook pointed out a weird side-effect: devices which have > > ->init() registered get their randomness added to the system each > > time they're switched in, but devices that don't have the init > > callback don't contribute to system randomness more than once. The > > weirdness is resolved here by using the randomness each time > > hwrng_init() is attempted, irrespective of the existence of the > > device's ->init() callback. > > All applied to crypto. Thanks!Thanks, Herbert. I didn't mention it, but pls queue it up for 3.16. Amit
On Tue, Jul 15, 2014 at 10:10:28AM +0530, Amit Shah wrote:> On (Mon) 14 Jul 2014 [20:50:06], Herbert Xu wrote: > > On Thu, Jul 10, 2014 at 03:42:33PM +0530, Amit Shah wrote: > > > v3: > > > - Kees Cook pointed out a weird side-effect: devices which have > > > ->init() registered get their randomness added to the system each > > > time they're switched in, but devices that don't have the init > > > callback don't contribute to system randomness more than once. The > > > weirdness is resolved here by using the randomness each time > > > hwrng_init() is attempted, irrespective of the existence of the > > > device's ->init() callback. > > > > All applied to crypto. Thanks! > > Thanks, Herbert. I didn't mention it, but pls queue it up for 3.16.Yes that's why it's in crypto as opposed to cryptodev. Cheers, -- Email: Herbert Xu <herbert at gondor.apana.org.au> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
On (Tue) 15 Jul 2014 [12:45:56], Herbert Xu wrote:> On Tue, Jul 15, 2014 at 10:10:28AM +0530, Amit Shah wrote: > > On (Mon) 14 Jul 2014 [20:50:06], Herbert Xu wrote: > > > On Thu, Jul 10, 2014 at 03:42:33PM +0530, Amit Shah wrote: > > > > v3: > > > > - Kees Cook pointed out a weird side-effect: devices which have > > > > ->init() registered get their randomness added to the system each > > > > time they're switched in, but devices that don't have the init > > > > callback don't contribute to system randomness more than once. The > > > > weirdness is resolved here by using the randomness each time > > > > hwrng_init() is attempted, irrespective of the existence of the > > > > device's ->init() callback. > > > > > > All applied to crypto. Thanks! > > > > Thanks, Herbert. I didn't mention it, but pls queue it up for 3.16. > > Yes that's why it's in crypto as opposed to cryptodev.Ah; thanks! Amit
Maybe Matching Threads
- [PATCH v3 0/2] hwrng, virtio-rng: init-time fixes
- [PATCH v3 0/2] hwrng, virtio-rng: init-time fixes
- [PATCH v3 0/2] hwrng, virtio-rng: init-time fixes
- [PATCH 1/2] hwrng: don't fetch rng from sources without init
- [PATCH 1/2] hwrng: don't fetch rng from sources without init