On Thu, 2013-01-17 at 18:20 -0500, Steven Rostedt wrote: > object = c->freelist; > page = c->page; Hmm, having local_irq_restore() here is probably just as good, as object and page were grabbed together under it. It doesn't change the condition below any. /me updates patch. -- Steve > - if (unlikely(!object || !node_match(page, node))) > + > + new_object = !object || !node_match(page, node); > + local_irq_restore(flags); > + > + if (new_object) > object = __slab_alloc(s, gfpflags, node, addr, c); > > else { > -- 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>