All of lore.kernel.org
 help / color / mirror / Atom feed
* [merged] slub-search-partial-list-on-numa_mem_id-instead-of-numa_node_id.patch removed from -mm tree
@ 2014-06-09 19:27 akpm
  0 siblings, 0 replies; only message in thread
From: akpm @ 2014-06-09 19:27 UTC (permalink / raw)
  To: mm-commits, rientjes, penberg, nacc, liwanp, hanpt, cl, iamjoonsoo.kim

Subject: [merged] slub-search-partial-list-on-numa_mem_id-instead-of-numa_node_id.patch removed from -mm tree
To: iamjoonsoo.kim@lge.com,cl@linux.com,hanpt@linux.vnet.ibm.com,liwanp@linux.vnet.ibm.com,nacc@linux.vnet.ibm.com,penberg@kernel.org,rientjes@google.com,mm-commits@vger.kernel.org
From: akpm@linux-foundation.org
Date: Mon, 09 Jun 2014 12:27:57 -0700


The patch titled
     Subject: slub: search partial list on numa_mem_id(), instead of numa_node_id()
has been removed from the -mm tree.  Its filename was
     slub-search-partial-list-on-numa_mem_id-instead-of-numa_node_id.patch

This patch was dropped because it was merged into mainline or a subsystem tree

------------------------------------------------------
From: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Subject: slub: search partial list on numa_mem_id(), instead of numa_node_id()

Currently, if allocation constraint to node is NUMA_NO_NODE, we search a
partial slab on numa_node_id() node.  This doesn't work properly on a
system having memoryless nodes, since it can have no memory on that node
so there must be no partial slab on that node.

On that node, page allocation always falls back to numa_mem_id() first. 
So searching a partial slab on numa_node_id() in that case is the proper
solution for the memoryless node case.

Signed-off-by: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Acked-by: Nishanth Aravamudan <nacc@linux.vnet.ibm.com>
Acked-by: David Rientjes <rientjes@google.com>
Acked-by: Christoph Lameter <cl@linux.com>
Cc: Pekka Enberg <penberg@kernel.org>
Cc: Wanpeng Li <liwanp@linux.vnet.ibm.com>
Cc: Han Pingtian <hanpt@linux.vnet.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 mm/slub.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff -puN mm/slub.c~slub-search-partial-list-on-numa_mem_id-instead-of-numa_node_id mm/slub.c
--- a/mm/slub.c~slub-search-partial-list-on-numa_mem_id-instead-of-numa_node_id
+++ a/mm/slub.c
@@ -1726,7 +1726,7 @@ static void *get_partial(struct kmem_cac
 		struct kmem_cache_cpu *c)
 {
 	void *object;
-	int searchnode = (node == NUMA_NO_NODE) ? numa_node_id() : node;
+	int searchnode = (node == NUMA_NO_NODE) ? numa_mem_id() : node;
 
 	object = get_partial_node(s, get_node(s, searchnode), c, flags);
 	if (object || node != NUMA_NO_NODE)
_

Patches currently in -mm which might be from iamjoonsoo.kim@lge.com are

origin.patch
mm-compactionc-isolate_freepages_block-small-tuneup.patch
linux-next.patch
page-owners-correct-page-order-when-to-free-page.patch


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2014-06-09 19:27 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-09 19:27 [merged] slub-search-partial-list-on-numa_mem_id-instead-of-numa_node_id.patch removed from -mm tree akpm

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.