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-ia64-devel

RE: [Xen-ia64-devel] alt_itlb_miss?

To: "Masaki Kanno" <kanno.masaki@xxxxxxxxxxxxxx>
Subject: RE: [Xen-ia64-devel] alt_itlb_miss?
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Mon, 24 Apr 2006 09:42:51 +0800
Cc: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>, xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Sun, 23 Apr 2006 18:43:08 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcZlMmom9/VGhrkMTIWvIlFF16FFzgCCwZiA
Thread-topic: [Xen-ia64-devel] alt_itlb_miss?
>From: Masaki Kanno [mailto:kanno.masaki@xxxxxxxxxxxxxx]
>Sent: 2006年4月21日 18:56
>>>
>>>Hi Kan,
>>>
>>>   Thanks, this looks like exactly what we need.  If there are no
>other
>>>comments, please send me this patch w/ a Signed-off-by and we can
>get
>>>it
>>>in tree.  BTW, glad to hear you're working on the FPSWA issue and
>are
>>>making good progress!  Thanks,
>>>
>>>     Alex
>>
>>Seems OK. One small comment is that we may also remove
>>FORCE_CRASH completely since the assumption to add that
>>check doesn't exist now. Actually VHPT_CCHAIN_LOOKUP
>>already makes check upon VMM area to decide whether jumping
>>to alt_itlb_miss handler. In this case, simply removing
>>FORCE_CRASH line can also work. :-)
>
>If alt_itlb_fault occurred, we need ifa checking and FORCE_CRASH,
>don't we?
>Therefore I don't need to change my patch, do I?
>

The check is already made before jumping to alt_itlb_miss. 
Also architecturally there's no limitation to prevent uncacheable 
instruction falling into that category. So I think there's no need 
for existence of FORCE_CRASH there, right? :-)

Thanks,
Kevin

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