Xen 
 
Home About Xen.org Xen Xen Summit Wiki Mailing List Bug Tracker Xen Downloads
 
   
 

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