On Thu, 3 Sep 2009, Eric Dumazet wrote: > Christoph Lameter a ?crit : > > On Thu, 3 Sep 2009, Eric Dumazet wrote: > > > >> on a SLAB_DESTROY_BY_RCU cache, there is no need to try to optimize this > >> rcu_barrier() call, unless we want superfast reboot/halt sequences... > > > > I stilll think that the action to quiesce rcu is something that the caller > > of kmem_cache_destroy must take care of. > > Do you mean : > > if (kmem_cache_shrink(s) == 0) { > rcu_barrier(); > kmem_cache_destroy_no_rcu_barrier(s); > } else { > kmem_cache_destroy_with_rcu_barrier_because_SLAB_DESTROY_BY_RCU_cache(s); > } > > What would be the point ? The above is port of slub? I mean that (in this case) the net subsystem would have to deal with RCU quietness before disposing of the slab cache. There may be multiple ways of dealing with RCU. The RCU barrier may be unnecessary for future uses. Typically one would expect that all deferred handling of structures must be complete for correctness before disposing of the whole cache. > [PATCH] slub: fix slab_pad_check() Acked-by: Christoph Lameter <cl@xxxxxxxxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html