xfs-masters
[Top] [All Lists]

[xfs-masters] Re: [PATCH] Allow lazy unmapping by taking extra page refe

To: David Chinner <dgc@xxxxxxx>
Subject: [xfs-masters] Re: [PATCH] Allow lazy unmapping by taking extra page references V2
From: Jeremy Fitzhardinge <jeremy@xxxxxxxx>
Date: Tue, 23 Oct 2007 22:08:47 -0700
Cc: Andi Kleen <andi@xxxxxxxxxxxxxx>, dean gaudet <dean@xxxxxxxxxx>, Nick Piggin <nickpiggin@xxxxxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Morten@xxxxxxx, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, Bøgeskov <xen-users@xxxxxxxxxxxxxxxxxx>, xfs@xxxxxxxxxxx, xfs-masters@xxxxxxxxxxx, Mark Williamson <mark.williamson@xxxxxxxxxxxx>
In-reply-to: <20071024043619.GE66820511@xxxxxxx>
References: <alpine.DEB.0.9999.0710212015430.2320@xxxxxxxxxxxxxxxxxxx> <471C1A61.1010001@xxxxxxxx> <p73sl439s7k.fsf@xxxxxxxxxxxxxx> <471CEEB4.9040807@xxxxxxxx> <20071022190740.GA1695@xxxxxxxxxxxxxxxxxx> <20071022223224.GC995458@xxxxxxx> <20071022233514.GA9057@xxxxxxxxxxxxxxxxxx> <20071023003641.GF995458@xxxxxxx> <20071023070414.GM66820511@xxxxxxx> <20071023093035.GB24536@xxxxxxxxxxxxxxxxxx> <20071024043619.GE66820511@xxxxxxx>
Reply-to: xfs-masters@xxxxxxxxxxx
Sender: xfs-masters-bounce@xxxxxxxxxxx
User-agent: Thunderbird 2.0.0.5 (X11/20070727)
David Chinner wrote:
> Allow lazy unmapping of vmap()d regions be taking a reference
> on each page in the region being vmap()ed. The page references
> get released after the region is vunmap()ed, thereby ensuring
> we don't leave stray mappings on freed pages that could lead to
> problems pages being reallocated with incorrect mappings
> associated with them.
>
> Tested with XFS filesystems with 64k directory blocks with
> dirstress and XFSQA.
>
> Version 2:
>   - drop the release function stuff and just call put_page()
>     as it falls down to the same code in all calling cases.
>   

This doesn't apply cleanly to the current git tree.

    J


<Prev in Thread] Current Thread [Next in Thread>