Re: [patch 14/38] slub: Use prandom instead of get_cycles()
"Vlastimil Babka (SUSE)" <[email protected]> Mon, 13 Apr 2026 15:45:59 +0200
| Newsgroups | gmane.linux.ports.ppc.embedded,gmane.linux.kernel,gmane.linux.kernel.mm,gmane.linux.network,gmane.linux.kernel.wireless.general,gmane.linux.kernel.cryptoapi,gmane.comp.file-systems.ext4,gmane.linux.hams,gmane.linux.ports.alpha,gmane.linux.ports.arm.kernel,gmane.linux.ports.parisc,gmane.linux.ports.riscv,gmane.linux.ports.sparc |
|---|---|
| Message-ID | <[email protected]> |
On 4/13/26 15:02, [email protected] wrote: > Harry wrote: >> [Resending after fixing broken email headers] >> >> On Fri, Apr 10, 2026 at 02:19:37PM +0200, Thomas Gleixner wrote: >> > The decision whether to scan remote nodes is based on a 'random' number >> > retrieved via get_cycles(). get_cycles() is about to be removed. >> > >> > There is already prandom state in the code, so use that instead. >> > >> > Signed-off-by: Thomas Gleixner <[email protected]> >> > Cc: Vlastimil Babka <[email protected]> >> > Cc: [email protected] >> > --- >> >> Acked-by: Harry Yoo (Oracle) <[email protected]> >> >> Is this for this merge window? I'd say it's not intended for 7.1 as it's not in -next and v1 was posted just before the merge window. >> This may conflict with upcoming changes on freelist shuffling [1] >> (not queued for slab/for-next yet though), but it should be easy to >> resolve. Indeed, it's a simple conflict. > > Hi Harry, > > Would you like me to wait for this patch to land linux-next and then > rebase and send v6 on top? Just send it now based same as previously so we can finish the reviews, and we'll deal with it after rc1.