From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1946838AbXD3WHe (ORCPT ); Mon, 30 Apr 2007 18:07:34 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1946852AbXD3WHd (ORCPT ); Mon, 30 Apr 2007 18:07:33 -0400 Received: from agminet01.oracle.com ([141.146.126.228]:24554 "EHLO agminet01.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1946854AbXD3WHc (ORCPT ); Mon, 30 Apr 2007 18:07:32 -0400 Date: Mon, 30 Apr 2007 15:04:03 -0700 From: Bill Irwin To: Christoph Hellwig , Andi Kleen , Alan Cox , David Chinner , Zan Lynx , Adrian Bunk , Linux Kernel Cc: William Irwin Subject: Re: [2/6] add config option to vmalloc stacks (was: Re: [-mm patch] i386: enable 4k stacks by default) Message-ID: <20070430220403.GF26598@holomorphy.com> Mail-Followup-To: Bill Irwin , Christoph Hellwig , Andi Kleen , Alan Cox , David Chinner , Zan Lynx , Adrian Bunk , Linux Kernel References: <20070430035838.GC77450368@melbourne.sgi.com> <20070430091754.24df88df@the-village.bc.nu> <20070430104806.GA14944@infradead.org> <20070430173819.GC19966@holomorphy.com> <20070430174310.GE19966@holomorphy.com> <20070430181104.GB14739@infradead.org> <20070430190952.GP31925@holomorphy.com> <20070430191511.GA25318@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20070430191511.GA25318@infradead.org> User-Agent: Mutt/1.5.11 X-Brightmail-Tracker: AAAAAQAAAAI= X-Brightmail-Tracker: AAAAAQAAAAI= X-Whitelist: TRUE X-Whitelist: TRUE Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Apr 30, 2007 at 08:15:11PM +0100, Christoph Hellwig wrote: > So if you want to invest some time into getting this into mergeable > shape I'd suggest you redo the patch series in the following way: > patch 1: dynamic allocated irq stacks > patch 2: make irqstacks unconditional, but allow selecting 4/8k stacks > patch 3: introduce a DEBUG_STACK option that enables vmalloc'ed stacks > and adds the __pa check. I've not heard anything back on this from freitag, so I've redone the patch series precisely as you've outlined above. I'm taking the updated patch series through a battery of testboots before posting for merging purposes. -- wli