On Wed, 22 Oct 2014, Christoph Lameter wrote: > Somehow the two branches in __slab_alloc do the same. > Unify them. > > Signed-off-by: Christoph Lameter <cl@xxxxxxxxx> > > Index: linux/mm/slub.c > =================================================================== > --- linux.orig/mm/slub.c > +++ linux/mm/slub.c > @@ -2280,12 +2280,8 @@ redo: > if (node != NUMA_NO_NODE && !node_present_pages(node)) > searchnode = node_to_mem_node(node); > > - if (unlikely(!node_match(page, searchnode))) { > - stat(s, ALLOC_NODE_MISMATCH); > - deactivate_slab(s, page, c->freelist); > - c->page = NULL; > - c->freelist = NULL; > - goto new_slab; > + if (unlikely(!node_match(page, searchnode))) > + goto deactivate; > } That's not compiling at all due to the left over '}' ! And shouldn't you keep the stat(); call in that code path? Thanks, tglx -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>