Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262976AbVCDTVy (ORCPT ); Fri, 4 Mar 2005 14:21:54 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S263020AbVCDTRX (ORCPT ); Fri, 4 Mar 2005 14:17:23 -0500 Received: from bay-bridge.veritas.com ([143.127.3.10]:20162 "EHLO MTVMIME01.enterprise.veritas.com") by vger.kernel.org with ESMTP id S262984AbVCDTJS (ORCPT ); Fri, 4 Mar 2005 14:09:18 -0500 Date: Fri, 4 Mar 2005 19:08:30 +0000 (GMT) From: Hugh Dickins X-X-Sender: hugh@goblin.wat.veritas.com To: Christoph Lameter cc: Andrew Morton , linux-kernel@vger.kernel.org, linux-ia64@vger.kernel.org Subject: Re: Page fault scalability patch V18: Drop first acquisition of ptl In-Reply-To: Message-ID: References: <20050302174507.7991af94.akpm@osdl.org> <20050302185508.4cd2f618.akpm@osdl.org> <20050302201425.2b994195.akpm@osdl.org> <20050302205612.451d220b.akpm@osdl.org> <20050302222008.4910eb7b.akpm@osdl.org> <20050303132011.7c80033d.akpm@osdl.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1048 Lines: 23 On Fri, 4 Mar 2005, Christoph Lameter wrote: > On Fri, 4 Mar 2005, Hugh Dickins wrote: > > Nacked for the same reason as just given to earlier version. Ugly too. > > Ok. Then we could still get back the also ugly solution in the earlier > patchsets that acquired the spinlock separately before getting to > do_wp_page (also no need for the separate patch anymore). Patch is > then shorter too. Maybe. I should make it clear that I simply haven't examined the recent incarnations of your patch, was just commenting on an issue I could comment on quickly without needing to find time to think. So, I just want to make clear, this absence of Nack doesn't mean Ack: I remain uneasy with it all, waiting to see some architecture maintainers come along with a clear "Yes, this is how it should be". Hugh - 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/