On Tue, Jun 04, 2019 at 07:44:48PM -0700, Roman Gushchin wrote: > Currently SLUB uses a work scheduled after an RCU grace period > to deactivate a non-root kmem_cache. This mechanism can be reused > for kmem_caches release, but requires generalization for SLAB > case. > > Introduce kmemcg_cache_deactivate() function, which calls > allocator-specific __kmem_cache_deactivate() and schedules > execution of __kmem_cache_deactivate_after_rcu() with all > necessary locks in a worker context after an rcu grace period. > > Here is the new calling scheme: > kmemcg_cache_deactivate() > __kmemcg_cache_deactivate() SLAB/SLUB-specific > kmemcg_rcufn() rcu > kmemcg_workfn() work > __kmemcg_cache_deactivate_after_rcu() SLAB/SLUB-specific > > instead of: > __kmemcg_cache_deactivate() SLAB/SLUB-specific > slab_deactivate_memcg_cache_rcu_sched() SLUB-only > kmemcg_rcufn() rcu > kmemcg_workfn() work > kmemcg_cache_deact_after_rcu() SLUB-only > > For consistency, all allocator-specific functions start with "__". > > Signed-off-by: Roman Gushchin <g...@fb.com>
Much easier to review after extracting renaming, thanks. Acked-by: Vladimir Davydov <vdavydov....@gmail.com>
