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] A question about guest_walk_tables

To: "Tian, Kevin" <kevin.tian@xxxxxxxxx>, "Tim Deegan" <Tim.Deegan@xxxxxxxxxx>
Subject: RE: [Xen-devel] A question about guest_walk_tables
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Thu, 6 Dec 2007 15:06:25 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 05 Dec 2007 23:07:32 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <D470B4E54465E3469E2ABBC5AFAC390F024D8CED@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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>
References: <D470B4E54465E3469E2ABBC5AFAC390F024D8CEA@xxxxxxxxxxxxxxxxxxxxxxxxxxxx><20071205103444.GB7121@xxxxxxxxxxxxxxxxxxxxx> <D470B4E54465E3469E2ABBC5AFAC390F024D8CED@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acg3KoVNc3owlGDvTVS3+0T+4OFw6QAfk4sQAAtfP1A=
Thread-topic: [Xen-devel] A question about guest_walk_tables
>From: Tian, Kevin
>Sent: 2007年12月6日 9:41
>
>>From: Tim Deegan [mailto:Tim.Deegan@xxxxxxxxxx] 
>>Sent: 2007年12月5日 18:35
>>
>>At 18:22 +0800 on 05 Dec (1196878922), Tian, Kevin wrote:
>>> Hi, Tim,
>>>     Just a curious question. Any reason why sh_page_fault can't
>>> benefit vtlb_lookup to skip heavy-weight guest_walk_tables, 
>>like other
>>> places like sh_gva_to_gfn?
>>
>>For cases when we go on to make the shadow pagetables, we 
>need the full
>>walk available so we know which MFNs to shadow.
>>
>>I don't know whether it would be worth adding a vtlb lookup for the
>>real-fault case; the fast-path code for not-present and MMIO catches a
>>lot of genuine faults already.  We should definitely add the 
>>walk result
>>to the vTLB in any case.
>>
>>Thanks,
>>
>>Tim.
>>
>
>Understand. fast-path should be enough without vtlb help. 
>
>Thanks,
>Kevin
>

BTW, is it cleaner to move vtlb_insert into guest_walk_tables, which
is more natural and save the effort to update all invocation points?

Thanks,
Kevin 

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

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