Messages in this thread | | | From | Thomas Gleixner <> | Subject | Re: [PATCH] random: make random_get_entropy() return an unsigned long | Date | Sun, 10 Apr 2022 11:45:16 +0200 |
| |
On Fri, Apr 08 2022 at 19:53, Jason A. Donenfeld wrote: > Some implementations were returning type `unsigned long`, while others > that fell back to get_cycles() were implicitly returning a `cycles_t` or > an untyped constant int literal. That makes for weird and confusing > code, and basically all code in the kernel already handled it like it > was an `unsigned long`. I recently tried to handle it as the largest > type it could be, a `cycles_t`, but doing so doesn't really help with > much. Instead let's just make random_get_entropy() return an unsigned > long all the time. This also matches the commonly used > `arch_get_random_long()` function, so now RDRAND and RDTSC return the > same sized integer, which means one can fallback to the other more > gracefully.
Acked-by: Thomas Gleixner <tglx@linutronix.de>
| |