WARNING - OLD ARCHIVES

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/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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>


Patrick


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
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

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