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: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, Tim Deegan <Tim.Deegan@xxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] Make get_page_from_l1e refcount correctly onforeign pagetables.
From: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Date: Fri, 15 May 2009 13:00:31 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 14 May 2009 22:01:11 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C631A1A3.B06C%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/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: <20090514082238.GI12377@xxxxxxxxxxxxxxxxxxxxx> <C631A1A3.B06C%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcnUbTLqQDQENtenT9K/ZSOZGFCvUAACHKZ1ACkM8JA=
Thread-topic: [Xen-devel] [PATCH] Make get_page_from_l1e refcount correctly onforeign pagetables.

Keir Fraser wrote:
> 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.

Sorry for slow response.
Yes, that patch is for page ofline as stated by Tim. However, because it is too 
later to 3.4 release, the whole patchset is not in the tree in the end. I will 
re-send the patch once the unstable tree is open again, and we can discuss it 
again at that time.

-- jyh

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