This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH] xenpaging: call pageout policy function in xenpa

To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xenpaging: call pageout policy function in xenpaging_evict_page
From: Patrick Colp <pjcolp@xxxxxxxxx>
Date: Tue, 24 Aug 2010 10:30:58 -0700
Cc: Olaf Hering <olaf@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 24 Aug 2010 10:31:45 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <19571.64485.747279.591829@xxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20100824093825.GA31663@xxxxxxxxx> <19571.64485.747279.591829@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Yes, this patch is correct.

Acked-by: Patrick Colp <pjcolp@xxxxxxxxx>


On 24 August 2010 10:05, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> wrote:
> Olaf Hering writes ("[Xen-devel] [PATCH] xenpaging: call pageout policy 
> function in xenpaging_evict_page"):
>> Notify policy about a page that was just paged out to disk.
>> Up to now the code called the opposite function, which clears the
>> (xenpaging internal) reference bit, instead of setting it and marking
>> the page as gone.
>> Signed-off-by: Olaf Hering <olaf@xxxxxxxxx>
> This looks plausible but as I don't really understand the xenpaging
> code I'd like an ack from Patrick.  Patrick ?
> Thanks,
> Ian.

Xen-devel mailing list

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