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] Make get_page_from_l1e refcount correctly onfor

To: Tim Deegan <Tim.Deegan@xxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Make get_page_from_l1e refcount correctly onforeign pagetables.
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Thu, 14 May 2009 10:23:31 +0100
Cc: Yunhong Jiang <yunhong.jiang@xxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 14 May 2009 02:23:59 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20090514082238.GI12377@xxxxxxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcnUbTLqQDQENtenT9K/ZSOZGFCvUAACHKZ1
Thread-topic: [Xen-devel] [PATCH] Make get_page_from_l1e refcount correctly onforeign pagetables.
User-agent: Microsoft-Entourage/12.17.0.090302
On 14/05/2009 09:22, "Tim Deegan" <Tim.Deegan@xxxxxxxxxx> wrote:

>> Perhaps the whole thing gets more convoluted because of c/s 19383, which
>> added a vcpu parameter for no apparent reason (current is used for that
>> everywhere afaict).
> 
> That parameter is used so that dom0 can modify a PV domU's pagetables
> using the MMU ops (so dom0 tools can offline a suspect page without
> domU's help).  In that case, I think the current patch actually fixes a
> latent bug in refcounting foreign mappings from the target domU to a
> third (HVM) domain, that was missed in cs 19383.
> 
> This is all pretty confusing, though, and Keir may correct me on any or
> all of it. :)

I wish I could. I need a damp towel wrapped round my head to understand the
details of that code now. We may need to step back and refactor and rethink
after 3.4. Hence my reluctance to mess with it now, as I fear introducing
bugs more significant than latent stubdom issues.

 -- Keir



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