Hi Jason, Am 25.08.23 um 13:26 schrieb Jason A. Donenfeld:
Hi Stefan, On Fri, Aug 25, 2023 at 01:14:55PM +0200, Stefan Wahren wrote:Hi, i didn't find the time to fix the performance regression in bcm2835-rng which affects Raspberry Pi 0 - 3, so report it at least. AFAIK the first report about this issue was here [1] and identified the offending commit: 96cb9d055445 ("hwrng: bcm2835 - use hwrng_msleep() instead of cpu_relax()") #regzbot introduced: 96cb9d055445 I was able to reproduce this issue with a Raspberry Pi 3 B+ on Linux 6.5-rc6 (arm64/defconfig). Before: time sudo dd if=/dev/hwrng of=/dev/urandom count=1 bs=4096 status=none real 3m29,002s user 0m0,018s sys 0m0,054sThat's not surprising. But also, does it matter? That script has *always* been wrong. Writing to /dev/urandom like that has *never* ensured that those bytes are taken into account immediately after. It's just not how that interface works. So any assumptions based on that are bogus, and that line effectively does nothing. Fortunately, however, the kernel itself incorporates hwrng output into the rng pool, so you don't need to think about doing it yourself. So go ahead and remove that line from your script.
Thanks for your explanation. Unfortunately this isn't my script. I'm just a former BCM2835 maintainer and interested that more user stick to the mainline kernel instead of the vendor ones. I will try to report the script owner.
Now as far as the "regression" goes, we've made an already broken userspace script take 3 minutes longer than usual, but it still does eventually complete, so it's not making boot impossible or something. How this relates to the "don't break userspace" rule might be a matter of opinion. If you think it does, maybe send a patch to Herbert reducing that sleep from 1000 to 100 and stating why with my background above, and see if he agrees it's worth fixing. Or, if removing that line from your scripts is good enough for you, that's also fine by me.
Now i agree that the provided example isn't the proper way to handle /dev/urandom. Unfortunately most of the Raspberry Pi users doesn't care about such details. In their eyes the mainline kernel is "broken" and this is one argument to go back to the vendor ones. Beside of the /dev/urandom abuse, i'm not convinced that sleeping for 1 sec is a good choice. This HRNG IP is used in embedded devices (e.g. Router) and mostly without any other good source of entropy. So i think it's worth to define a reasonable value. Best regards
Jason
