Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1763024AbXFRL6n (ORCPT ); Mon, 18 Jun 2007 07:58:43 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1761679AbXFRL6g (ORCPT ); Mon, 18 Jun 2007 07:58:36 -0400 Received: from nf-out-0910.google.com ([64.233.182.184]:48822 "EHLO nf-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761266AbXFRL6f (ORCPT ); Mon, 18 Jun 2007 07:58:35 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:user-agent:mime-version:to:cc:subject:references:in-reply-to:content-type:content-transfer-encoding:from; b=AYmATDgzIjV7shOnx7bybKMXAoW0V4rxrSSfPrcU3rBAY7tBVHiQcUF5gDbi+orBWYprtCzhppcSdKuuWLsj9Tn5V0yhPAHjinsAVC0zLaMuGy841tM7qk4N7769IH6pyWzOd+y/3ioDBvk3dklsKSuxWks8Fe4S1vesxhKxdBc= Message-ID: <46767346.2040108@googlemail.com> Date: Mon, 18 Jun 2007 13:57:58 +0200 User-Agent: Thunderbird 2.0.0.0 (X11/20070419) MIME-Version: 1.0 To: clameter@sgi.com CC: akpm@linux-foundation.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, Pekka Enberg , suresh.b.siddha@intel.com Subject: Re: [patch 00/26] Current slab allocator / SLUB patch queue References: <20070618095838.238615343@sgi.com> In-Reply-To: <20070618095838.238615343@sgi.com> Content-Type: text/plain; charset=ISO-8859-2; format=flowed Content-Transfer-Encoding: 8bit From: Michal Piotrowski Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3448 Lines: 102 Hi, clameter@sgi.com pisze: > These contain the following groups of patches: > > 1. Slab allocator code consolidation and fixing of inconsistencies > > This makes ZERO_SIZE_PTR generic so that it works in all > slab allocators. > > It adds __GFP_ZERO support to all slab allocators and > cleans up the zeroing in the slabs and provides modifications > to remove explicit zeroing following kmalloc_node and > kmem_cache_alloc_node calls. > > 2. SLUB improvements > > Inline some small functions to reduce code size. Some more memory > optimizations using CONFIG_SLUB_DEBUG. Changes to handling of the > slub_lock and an optimization of runtime determination of kmalloc slabs > (replaces ilog2 patch that failed with gcc 3.3 on powerpc). > > 3. Slab defragmentation > > This is V3 of the patchset with the one fix for the locking problem that > showed up during testing. > > 4. Performance optimizations > > These patches have a long history since the early drafts of SLUB. The > problem with these patches is that they require the touching of additional > cachelines (only for read) and SLUB was designed for minimal cacheline > touching. In doing so we may be able to remove cacheline bouncing in > particular for remote alloc/ free situations where I have had reports of > issues that I was not able to confirm for lack of specificity. The tradeoffs > here are not clear. Certainly the larger cacheline footprint will hurt the > casual slab user somewhat but it will benefit processes that perform these > local/remote alloc/free operations. > > I'd appreciate if someone could evaluate these. > > The complete patchset against 2.6.22-rc4-mm2 is available at > > http://ftp.kernel.org/pub/linux/kernel/people/christoph/slub/2.6.22-rc4-mm2 > > Tested on > > x86_64 SMP > x86_64 NUMA emulation > IA64 emulator > Altix 64p/128G NUMA system. > Altix 8p/6G asymmetric NUMA system. > > Testcase: #! /bin/sh for i in `find /sys/ -type f` do echo "wy?wietlam $i" sudo cat $i > /dev/null # sleep 1s done Result: [ 212.247759] WARNING: at lib/vsprintf.c:280 vsnprintf() [ 212.253263] [] dump_trace+0x63/0x1eb [ 212.259042] [] show_trace_log_lvl+0x1a/0x2f [ 212.266672] [] show_trace+0x12/0x14 [ 212.271622] [] dump_stack+0x16/0x18 [ 212.276663] [] vsnprintf+0x6b/0x48c [ 212.281325] [] scnprintf+0x20/0x2d [ 212.286707] [] bitmap_scnlistprintf+0xa8/0xec [ 212.292508] [] list_locations+0x24c/0x2a2 [ 212.298241] [] alloc_calls_show+0x1f/0x26 [ 212.303459] [] slab_attr_show+0x1c/0x20 [ 212.309469] [] sysfs_read_file+0x94/0x105 [ 212.315519] [] vfs_read+0xcf/0x158 [ 212.320215] [] sys_read+0x3d/0x72 [ 212.327539] [] syscall_call+0x7/0xb [ 212.332203] [] 0xb7f74410 [ 212.336229] ======================= Unfortunately, I don't know which file was cat'ed http://www.stardust.webpages.pl/files/tbf/bitis-gabonica/2.6.22-rc4-mm2-slub/slub-config http://www.stardust.webpages.pl/files/tbf/bitis-gabonica/2.6.22-rc4-mm2-slub/slub-dmesg Regards, Michal -- LOG http://www.stardust.webpages.pl/log/ - 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/