Re: [patch 14/38] slub: Use prandom instead of get_cycles()

[Vlastimil Babka (SUSE)]

On 4/13/26 15:02, hu.shengming@xxxxxxxxxx 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 <tglx@xxxxxxxxxx>
>> > Cc: Vlastimil Babka <vbabka@xxxxxxxxxx>
>> > Cc: linux-mm@xxxxxxxxx
>> > ---
>> 
>> Acked-by: Harry Yoo (Oracle) <harry@xxxxxxxxxx>
>> 
>> 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.