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] Remove physmap page upon granttab xfer

To: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Remove physmap page upon granttab xfer
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Thu, 29 May 2008 11:15:31 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, Espen Skoglund <espen.skoglund@xxxxxxxxxxxxx>, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 29 May 2008 03:16:11 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C4643F9C.215ED%keir.fraser@xxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcjBdDM8cb89yC1nEd28TQAX8io7RQAALleq
Thread-topic: [Xen-devel] [PATCH] Remove physmap page upon granttab xfer
User-agent: Microsoft-Entourage/11.4.0.080122


On 29/5/08 11:10, "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx> wrote:

>> Thank you for explanation.
>> The patch breaks ia64 granttab xfer breakage so that it have to be fixed.
>> However the operation is now obsoleted one, so I don't want to make
>> big effort to fix.
>> What do you think about the following patch?
> 
> How does grant transfer work on ia64? Looking at netback I don't even see
> how the 'hole' left by transfer of the old page gets filled by mapping in a
> new page.

Oh, I see you do it steal_page(). Okay, I think best to make the
remove_page() call ifndef ia64 then.

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel