Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757864AbYBESRv (ORCPT ); Tue, 5 Feb 2008 13:17:51 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753356AbYBESRn (ORCPT ); Tue, 5 Feb 2008 13:17:43 -0500 Received: from relay2.sgi.com ([192.48.171.30]:55276 "EHLO relay.sgi.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752318AbYBESRm (ORCPT ); Tue, 5 Feb 2008 13:17:42 -0500 Date: Tue, 5 Feb 2008 10:17:41 -0800 (PST) From: Christoph Lameter X-X-Sender: clameter@schroedinger.engr.sgi.com To: Andrea Arcangeli cc: Robin Holt , Avi Kivity , Izik Eidus , kvm-devel@lists.sourceforge.net, Peter Zijlstra , steiner@sgi.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, daniel.blueman@quadrics.com Subject: Re: [PATCH] mmu notifiers #v5 In-Reply-To: <20080205180802.GE7441@v2.random> Message-ID: References: <20080131234101.GS7185@v2.random> <20080201120955.GX7185@v2.random> <20080203021704.GC7185@v2.random> <20080205052525.GD7441@v2.random> <20080205180802.GE7441@v2.random> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1498 Lines: 33 On Tue, 5 Feb 2008, Andrea Arcangeli wrote: > given I never allow a coherency-loss between two threads that will > read/write to two different physical pages for the same virtual > adddress in remap_file_pages). The other approach will not have any remote ptes at that point. Why would there be a coherency issue? > In performance terms with your patch before GRU can run follow_page it > has to take a mm-wide global mutex where each thread in all cpus will > have to take it. That will trash on >4-way when the tlb misses start No. It only has to lock the affected range. Remote page faults can occur while another part of the address space is being invalidated. The complexity of locking is up to the user of the mmu notifier. A simple implementation is satisfactory for the GRU right now. Should it become a problem then the lock granularity can be refined without changing the API. > > "conversion of some page in pages"? A proposal to defer the freeing of the > > pages until after the pte_unlock? > > There can be many tricks to optimize page in pages, but again munmap > and do_exit aren't the interesting path to optimzie, nor for GRU nor > for KVM so it doesn't matter right now. Still not sure what we are talking about here. -- 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/