Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751482AbbBMCdU (ORCPT ); Thu, 12 Feb 2015 21:33:20 -0500 Received: from LGEMRELSE6Q.lge.com ([156.147.1.121]:52192 "EHLO lgemrelse6q.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750973AbbBMCdT (ORCPT ); Thu, 12 Feb 2015 21:33:19 -0500 X-Original-SENDERIP: 10.177.222.153 X-Original-MAILFROM: iamjoonsoo.kim@lge.com Date: Fri, 13 Feb 2015 11:35:34 +0900 From: Joonsoo Kim To: David Rientjes Cc: Christoph Lameter , akpm@linuxfoundation.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, penberg@kernel.org, iamjoonsoo@lge.com, Jesper Dangaard Brouer Subject: Re: [PATCH 1/3] Slab infrastructure for array operations Message-ID: <20150213023534.GA6592@js1304-P5Q-DELUXE> References: <20150210194804.288708936@linux.com> <20150210194811.787556326@linux.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1431 Lines: 31 On Wed, Feb 11, 2015 at 12:18:07PM -0800, David Rientjes wrote: > On Wed, 11 Feb 2015, Christoph Lameter wrote: > > > > This patch is referencing functions that don't exist and can do so since > > > it's not compiled, but I think this belongs in the next patch. I also > > > think that this particular implementation may be slub-specific so I would > > > have expected just a call to an allocator-defined > > > __kmem_cache_alloc_array() here with i = __kmem_cache_alloc_array(). > > > > The implementation is generic and can be used in the same way for SLAB. > > SLOB does not have these types of object though. > > > > Ok, I didn't know if the slab implementation would follow the same format > with the same callbacks or whether this would need to be cleaned up later. Hello, Christoph. I also think that this implementation is slub-specific. For example, in slab case, it is always better to access local cpu cache first than page allocator since slab doesn't use list to manage free objects and there is no cache line overhead like as slub. I think that, in kmem_cache_alloc_array(), just call to allocator-defined __kmem_cache_alloc_array() is better approach. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/