Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1491958 > unrolled thread

Re: [v2] RANDOM: ATH9K RNG delivers zero bits of entropy

Started byKalle Valo <kvalo@qca.qualcomm.com>
First post2016-09-27 16:50 +0200
Last post2016-09-27 17:20 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [v2] RANDOM: ATH9K RNG delivers zero bits of entropy Kalle Valo <kvalo@qca.qualcomm.com> - 2016-09-27 16:50 +0200
    Re: [v2] RANDOM: ATH9K RNG delivers zero bits of entropy Stephan Mueller <smueller@chronox.de> - 2016-09-27 17:20 +0200

#1491958 — Re: [v2] RANDOM: ATH9K RNG delivers zero bits of entropy

FromKalle Valo <kvalo@qca.qualcomm.com>
Date2016-09-27 16:50 +0200
SubjectRe: [v2] RANDOM: ATH9K RNG delivers zero bits of entropy
Message-ID<sm1RL-2fQ-11@gated-at.bofh.it>
Stephan Mueller <smueller@chronox.de> wrote:
> The ATH9K driver implements an RNG which is completely bypassing the
> standard Linux HW generator logic.
> 
> The RNG may or may not deliver entropy. Considering the conservative
> approach in treating entropy with respect to non-auditable sources, this
> patch changes the delivered entropy value to zero. The RNG still feeds
> data into the input_pool but it is assumed to have no entropy.
> 
> When the ATH9K RNG changes to use the HW RNG framework, it may re-enable
> the entropy estimation considering that a user can change that value at
> boot and runtime.
> 
> Reviewed-by: Jason Cooper <jason@lakedaemon.net>
> Signed-off-by: Stephan Mueller <smueller@chronox.de>

Based on the discussion I'm dropping this patch. But the discussion was
hard to follow so please let me know if I misunderstood.

Patch set to Rejected.

-- 
https://patchwork.kernel.org/patch/9266265/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

[toc] | [next] | [standalone]


#1491980

FromStephan Mueller <smueller@chronox.de>
Date2016-09-27 17:20 +0200
Message-ID<sm2kO-2EY-9@gated-at.bofh.it>
In reply to#1491958
Am Dienstag, 27. September 2016, 16:44:16 CEST schrieb Kalle Valo:

Hi Kalle,

> Stephan Mueller <smueller@chronox.de> wrote:
> > The ATH9K driver implements an RNG which is completely bypassing the
> > standard Linux HW generator logic.
> > 
> > The RNG may or may not deliver entropy. Considering the conservative
> > approach in treating entropy with respect to non-auditable sources, this
> > patch changes the delivered entropy value to zero. The RNG still feeds
> > data into the input_pool but it is assumed to have no entropy.
> > 
> > When the ATH9K RNG changes to use the HW RNG framework, it may re-enable
> > the entropy estimation considering that a user can change that value at
> > boot and runtime.
> > 
> > Reviewed-by: Jason Cooper <jason@lakedaemon.net>
> > Signed-off-by: Stephan Mueller <smueller@chronox.de>
> 
> Based on the discussion I'm dropping this patch. But the discussion was
> hard to follow so please let me know if I misunderstood.

I guess the rejection is appropriate, but something needs to be done: 
add_hwgenerator_randomness should not be used in this scenario.
> 
> Patch set to Rejected.



Ciao
Stephan

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web