Re: [PATCH v5 6/9] slub: Delay freezing of partial slabs

[Vlastimil Babka]

On 11/22/23 15:28, Chengming Zhou wrote:
> 
> Yes, it looks so. There are some background services on the 128 CPUs machine.
> Although "stress-ng --rawpkt 128 --rawpkt-ops 100000000" has so much regression,
> I tried other less contented testcases:
> 
> 1. stress-ng --rawpkt 64 --rawpkt-ops 100000000
> 2. perf bench sched messaging -g 5 -t -l 100000
> 
> The performance numbers of this atomic version are pretty much the same.
> 
> So this atomic version should be good in most cases IMHO.

OK will fold the fix using full atomic version.

>>> And I also tested the atomic-optional version like below, found the
>>> performance numbers are much stable.
>> 
>> This gets rather ugly and fragile so I'd maybe rather go back to the
>> __unused field approach :/
>> 
> 
> Agree. If we don't want this atomic version, the __unused field approach
> seems better.
> 
> Thanks!
>