Hi Thomas, On Mon, Apr 11, 2022 at 10:18 AM Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
diff --git a/arch/m68k/include/asm/timex.h b/arch/m68k/include/asm/timex.h index 6a21d9358280..5351b10e1b18 100644 --- a/arch/m68k/include/asm/timex.h +++ b/arch/m68k/include/asm/timex.h @@ -35,7 +35,7 @@ static inline unsigned long random_get_entropy(void) { if (mach_random_get_entropy) return mach_random_get_entropy(); - return 0; + return ktime_read_raw_clock();I'd rather do something like this in a common header: unsigned long random_get_entropy_fallback(void); and use random_get_entropy_fallback() in the architecture specific files. That way you can encapsulate the fallback implementation in the random code and if it turns out that ktime_read_raw_clock() is a stupid idea or someone has a better idea then you have to change exactly one place and not patch the whole tree again.
Absolutely. That's a good idea. I'll do that for v3. Jason
