On Wed, Aug 27, 2014 at 06:37:33PM -0500, Christoph Lameter wrote: > One minor nit. Otherwise > > Acked-by: Christoph Lameter <cl@xxxxxxxxx> > > On Thu, 21 Aug 2014, Joonsoo Kim wrote: > > > @@ -2041,56 +1982,63 @@ static size_t calculate_slab_order(struct kmem_cache *cachep, > > return left_over; > > } > > > > +static int alloc_kmem_cache_cpus(struct kmem_cache *cachep, int entries, > > + int batchcount) > > +{ > > + cachep->cpu_cache = __alloc_kmem_cache_cpus(cachep, entries, > > + batchcount); > > + if (!cachep->cpu_cache) > > + return 1; > > + > > + return 0; > > +} > > Do we really need this trivial function? It doesnt do anything useful as > far as I can tell. Hello, You are right. I will remove it in next spin. Thanks. -- 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>