summaryrefslogtreecommitdiffstats
path: root/mm/slub.c
diff options
context:
space:
mode:
authorShaohua Li <shaohua.li@intel.com>2011-11-11 14:54:14 +0800
committerPekka Enberg <penberg@kernel.org>2011-12-13 22:27:09 +0200
commitb13683d1cc14d1dd30b8e20f3ebea3f814ad029f (patch)
tree29384daa693218952473ee9a41dab2045926c914 /mm/slub.c
parent8f1e33daeda6cd89753f9e77d174805a6f21db09 (diff)
downloadop-kernel-dev-b13683d1cc14d1dd30b8e20f3ebea3f814ad029f.zip
op-kernel-dev-b13683d1cc14d1dd30b8e20f3ebea3f814ad029f.tar.gz
slub: add missed accounting
With per-cpu partial list, slab is added to partial list first and then moved to node list. The __slab_free() code path for add/remove_partial is almost deprecated(except for slub debug). But we forget to account add/remove_partial when move per-cpu partial pages to node list, so the statistics for such events are always 0. Add corresponding accounting. This is against the patch "slub: use correct parameter to add a page to partial list tail" Acked-by: Christoph Lameter <cl@linux.com> Signed-off-by: Shaohua Li <shaohua.li@intel.com> Signed-off-by: Pekka Enberg <penberg@kernel.org>
Diffstat (limited to 'mm/slub.c')
-rw-r--r--mm/slub.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/mm/slub.c b/mm/slub.c
index 4056d29..8284a20 100644
--- a/mm/slub.c
+++ b/mm/slub.c
@@ -1901,11 +1901,14 @@ static void unfreeze_partials(struct kmem_cache *s)
}
if (l != m) {
- if (l == M_PARTIAL)
+ if (l == M_PARTIAL) {
remove_partial(n, page);
- else
+ stat(s, FREE_REMOVE_PARTIAL);
+ } else {
add_partial(n, page,
DEACTIVATE_TO_TAIL);
+ stat(s, FREE_ADD_PARTIAL);
+ }
l = m;
}
OpenPOWER on IntegriCloud