On Fri, 12 Aug 2022, Vlastimil Babka wrote: > All alloc and free operations on debug caches are now serialized by > n->list_lock, so we can remove slab_lock() usage in validate_slab() > and list_slab_objects() as those also happen under n->list_lock. > > Note the usage in list_slab_objects() could happen even on non-debug > caches, but only during cache shutdown time so there should not be any > parallel freeing activity anymore. Except for buggy slab users, but in > that case the slab_lock() would not help against the common cmpxchg > based fast paths (in non-debug caches) anyway. > > Also adjust documentation comments accordingly. > > Suggested-by: Hyeonggon Yoo <42.hyeyoo@xxxxxxxxx> > Signed-off-by: Vlastimil Babka <vbabka@xxxxxxx> Acked-by: David Rientjes <rientjes@xxxxxxxxxx>
