Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933476AbaLKKT6 (ORCPT ); Thu, 11 Dec 2014 05:19:58 -0500 Received: from mx1.redhat.com ([209.132.183.28]:47924 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750762AbaLKKTz (ORCPT ); Thu, 11 Dec 2014 05:19:55 -0500 Date: Thu, 11 Dec 2014 11:18:59 +0100 From: Jesper Dangaard Brouer To: Christoph Lameter Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-api@vger.kernel.org, Eric Dumazet , "David S. Miller" , Hannes Frederic Sowa , Alexander Duyck , Alexei Starovoitov , "Paul E. McKenney" , Mathieu Desnoyers , Steven Rostedt , brouer@redhat.com Subject: Re: [RFC PATCH 0/3] Faster than SLAB caching of SKBs with qmempool (backed by alf_queue) Message-ID: <20141211111859.21e23e90@redhat.com> In-Reply-To: References: <20141210033902.2114.68658.stgit@ahduyck-vm-fedora20> <20141210141332.31779.56391.stgit@dragon> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 10 Dec 2014 13:51:32 -0600 (CST) Christoph Lameter wrote: > On Wed, 10 Dec 2014, Jesper Dangaard Brouer wrote: > > > One of the building blocks for achieving this speedup is a cmpxchg > > based Lock-Free queue that supports bulking, named alf_queue for > > Array-based Lock-Free queue. By bulking elements (pointers) from the > > queue, the cost of the cmpxchg (approx 8 ns) is amortized over several > > elements. > > This is a bit of an issue since the design of the SLUB allocator is such > that you should pick up an object, apply some processing and then take the > next one. The fetching of an object warms up the first cacheline and this > is tied into the way free objects are linked in SLUB. > > So a bulk fetch from SLUB will not that effective and cause the touching > of many cachelines if we are dealing with just a few objects. If we are > looking at whole slab pages with all objects then SLUB can be effective > since we do not have to build up the linked pointer structure in each > page. SLAB has a different architecture there and a bulk fetch there is > possible without touching objects even for small sets since the freelist > management is separate from the objects. > > If you do this bulking then you will later access cache cold objects? > Doesnt that negate the benefit that you gain? Or are these objects written > to by hardware and therefore by necessity cache cold? Cache warmup is a concern, but perhaps it's the callers responsibility to prefetch for their use-case. For qmempool I do have patches that prefetch elems when going from the sharedq to the localq (per CPU), but I didn't see much gain, and I could prove my point (of being faster than slab) without it. And I would use/need the slab bulk interface to add elems to sharedq which I consider semi-cache cold. > We could provide a faster bulk alloc/free function. > > int kmem_cache_alloc_array(struct kmem_cache *s, gfp_t flags, > size_t objects, void **array) I like it :-) > and this could be optimized by each slab allocator to provide fast > population of objects in that array. We then assume that the number of > objects is in the hundreds or so right? I'm already seeing a benefit with 16 packets alloc/free "bulking". On RX we have a "budget" of 64 packets/descriptors (taken from the NIC RX ring) that need SKBs. On TX packets are put into the TX ring, and later at TX completion the TX ring is cleaned up, as many as 256 (as e.g. in the ixgbe driver). Scientific articles on userspace networking (like netmap) report that they need at least 8 packet bulking to see wirespeed 10G at 64 bytes. > The corresponding free function > > void kmem_cache_free_array(struct kmem_cache *s, > size_t objects, void **array) > > > I think the queue management of the array can be improved by using a > similar technique as used the SLUB allocator using the cmpxchg_local. > cmpxchg_local is much faster than a full cmpxchg and we are operating on > per cpu structures anyways. So the overhead could still be reduced. I think you missed that the per cpu localq is already not using cmpxchg (it is a SPSC queue). The sharedq (MPMC queue) does need and use the locked cmpxchg. -- Best regards, Jesper Dangaard Brouer MSc.CS, Sr. Network Kernel Developer at Red Hat Author of http://www.iptv-analyzer.org LinkedIn: http://www.linkedin.com/in/brouer -- 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/