From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wright Subject: Re: [patch 15/26] Xen-paravirt_ops: Add apply_to_page_range() which applies a function to a pte range. Date: Fri, 16 Mar 2007 09:47:19 -0700 Message-ID: <20070316164719.GK10574@sequoia.sous-sol.org> References: <20070301232443.195603797@goop.org> <20070301232528.155002121@goop.org> <20070316091940.GJ23174@elte.hu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20070316091940.GJ23174@elte.hu> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ingo Molnar Cc: Zachary Amsden , Jeremy Fitzhardinge , xen-devel@lists.xensource.com, Ian Pratt , Andi Kleen , Rusty Russell , linux-kernel@vger.kernel.org, Chris Wright , virtualization@lists.osdl.org, Andrew Morton , Christoph Lameter , Christian Limpach List-Id: virtualization@lists.linuxfoundation.org * Ingo Molnar (mingo@elte.hu) wrote: > > * Jeremy Fitzhardinge wrote: > > > Add a new mm function apply_to_page_range() which applies a given > > function to every pte in a given virtual address range in a given mm > > structure. This is a generic alternative to cut-and-pasting the Linux > > idiomatic pagetable walking code in every place that a sequence of > > PTEs must be accessed. > > nice one! I suspect we could simplify some of the less > performance-critical open-coded pagetable walker loops in the kernel > with this? (i'm not even sure it's all that much of a performance loss > to pass a function pointer around - would be nice to convert say > mprotect() to this and see the performance difference?) We went through this before, and eventually got kick back to keep it Xen specific. There's still some effort for generalized walkers from the Gelato folks IIRC. And we can certainly adapt.