From: Shaohua Li <shaohua.li@intel.com>
Date: Fri, 11 Nov 2011 00:33:48 +0000 (+0800)
Subject: slub: use correct parameter to add a page to partial list tail
X-Git-Tag: MMI-PSA29.97-13-9~17949^2~3
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=f64ae042d94d376b54e7a343d93c48561e9d2e16;p=GitHub%2FMotorolaMobilityLLC%2Fkernel-slsi.git

slub: use correct parameter to add a page to partial list tail

unfreeze_partials() needs add the page to partial list tail, since such page
hasn't too many free objects. We now explictly use DEACTIVATE_TO_TAIL for this,
while DEACTIVATE_TO_TAIL != 1. This will cause performance regression (eg, more
lock contention in node->list_lock) without below fix.

Signed-off-by: Shaohua Li <shaohua.li@intel.com>
Acked-by: Christoph Lameter <cl@linux.com>
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
---

diff --git a/mm/slub.c b/mm/slub.c
index 7d2a996c307e..60e16c43f88c 100644
--- a/mm/slub.c
+++ b/mm/slub.c
@@ -1904,7 +1904,8 @@ static void unfreeze_partials(struct kmem_cache *s)
 				if (l == M_PARTIAL)
 					remove_partial(n, page);
 				else
-					add_partial(n, page, 1);
+					add_partial(n, page,
+						DEACTIVATE_TO_TAIL);
 
 				l = m;
 			}