On Thu, 3 May 2018, Matthew Wilcox wrote: > OK. Do you want the conversion of slub to using slub_freelist and slub_list > as part of this patch series as well, then? Not sure if that is needed. Dont like allocator specific names. > Oh, and what do you want to do about cache_from_obj() in mm/slab.h? > That relies on having slab_cache be in the same location in struct > page as slub_cache. Maybe something like this? > > page = virt_to_head_page(x); > #ifdef CONFIG_SLUB > cachep = page->slub_cache; > #else > cachep = page->slab_cache; > #endif > if (slab_equal_or_root(cachep, s)) > return cachep; Name the field "cache" instead of sl?b_cache?