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 | org.kernel.vger.linux-hams,dev.linux.lists.iommu,dev.linux.lists.loongarch,org.infradead.lists.linux-riscv,org.kernel.vger.linux-alpha,org.kernel.vger.linux-crypto,org.kernel.vger.linux-ext4,org.kernel.vger.linux-fbdev,org.kernel.vger.linux-kernel,org.kernel.vger.linux-m68k,org.kernel.vger.linux-openrisc,org.kernel.vger.linux-parisc,org.kernel.vger.linux-s390,org.kernel.vger.linux-wireless,org.kernel.vger.netdev,org.kernel.vger.sparclinux,org.kvack.linux-mm,org.ozlabs.lists.linuxppc-dev |
|---|---|
| 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.