On 4/9/25 3:50 AM, Harry Yoo wrote: > On Mon, Mar 17, 2025 at 03:33:04PM +0100, Vlastimil Babka wrote: > > Hmm this hunk in v3 is fine, but on your slub-percpu-shaves-v4r0 branch > it's calling local_unlock() twice. Probably a rebase error?
Yeah, thanks a lot for catching that! I've just pushed https://git.kernel.org/pub/scm/linux/kernel/git/vbabka/linux.git/log/?h=slub-percpu-sheaves-v4r1 with this fixed, and fixups for 2/8 to the points you made, plus a proper strict_numa handling in 2/8, and an extra patch for better NUMA locality > Otherwise looks good to me. > > When you address this, please feel free to add: > > Reviewed-by: Harry Yoo <harry....@oracle.com> > > Thanks! >
