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][SVM][1/2] fix SVM 64bit hv cores>0 reboot/hang i

To: "Woller, Thomas" <thomas.woller@xxxxxxx>
Subject: Re: [Xen-devel] [PATCH][SVM][1/2] fix SVM 64bit hv cores>0 reboot/hang issue
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Thu, 4 May 2006 11:10:00 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 04 May 2006 03:10:13 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <F35BBA9A7A4D684FBFCEA191F108E53914570650@xxxxxxxxxxxxxxxxx>
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: <F35BBA9A7A4D684FBFCEA191F108E53914570650@xxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

On 3 May 2006, at 22:17, Woller, Thomas wrote:

Now, cursory testing on VT boxes indicates that the 0x2B selectors are
also restored on VMX, but the VT microcode does not seem to validate
these values, and therefore does not cause a processor shutdown.

These registers are officially ignored in 64bit mode, and zero'ing them
out seems to be a solution that is functional, but we're unsure as to
the use of the __USER_DS 0x2B values in ds, es selectors in 64bit mode.

Zeroing them out is probably the best thing to do. Xen doesn't bother to change them while it executes because ds/es/ss are effectively unused in 64-bit mode. The fact they happen to contain __USER_DS is utterly unimportant.

You probably need to worry about fs and gs as well. They're not causing you problems now because guests don't usually set those registers to non-zero values, but if they ever do then you'll presumably have problems?

So, I think the patch is good but needs to zero fs/gs as well. I'll wait for further advice before applying this patch.

 -- Keir


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

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