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] [PATCH] Strange problem with virtual aliases

To: "Xu, Anthony" <anthony.xu@xxxxxxxxx>
Subject: Re: [Xen-ia64-devel] [PATCH] Strange problem with virtual aliases
From: Matthew Chapman <matthewc@xxxxxxxxxxxxxxx>
Date: Thu, 30 Nov 2006 16:57:40 +1100
Cc: xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Wed, 29 Nov 2006 21:57:36 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <51CFAB8CB6883745AE7B93B3E084EBE207DD79@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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>
References: <51CFAB8CB6883745AE7B93B3E084EBE207DD79@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.11
On Thu, Nov 30, 2006 at 01:11:51PM +0800, Xu, Anthony wrote:
> 
> I found following code segment in function vcpu_itc_no_srlz,
> If inserted into VHPT, the pagesize is forced to PAGE_SHIFT.

I have compiled Xen with PAGE_SHIFT=12, otherwise yes 4KB pages would
not work.  But I consider that a limitation not a bug :)

However at the bottom of that function (vcpu_itc_no_srlz) there is:

  vcpu_set_tr_entry(&PSCBX(vcpu,dtlb),mp_pte,ps<<2,vaddr);

Assume itc.d inserts a large page (say 16MB).  mp_pte.ppn is the
original value passed to itc.d - the bottom of the large page.  vaddr is
the original ifa - maybe in the middle of the large page.  However, ps
has been forced to PAGE_SHIFT.  The result is a bad mapping in the
1-entry DTLB (&PSCBX(vcpu,dtlb)).  That is what my patch fixes.

Matt

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

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