svm: Reported SS.DPL must equal CPL, as this is assumed by generic HVM

code despite the fact that AMD processors do not always maintain this
invariant.

Based on a bug report and proposed patch by Ben Guthro and Robert
Phillips of Virtual Iron.

Signed-off-by: Keir Fraser <keir.fraser@citrix.com>
xen-unstable changeset:   17235:c4de138175712f661e0a5bbe35b8834711db63b7
xen-unstable date:        Tue Mar 18 15:26:35 2008 +0000

diff --git a/xen/arch/x86/hvm/svm/svm.c b/xen/arch/x86/hvm/svm/svm.c
index 9afe5887c16614754abe49c1721cbdedf3775d99..9b1b91003b6f95b3f411592d8931d71a64f74cfc 100644 (file)
--- a/xen/arch/x86/hvm/svm/svm.c
+++ b/xen/arch/x86/hvm/svm/svm.c
@@ -755,6 +755,7 @@ static void svm_get_segment_register(struct vcpu *v, enum x86_segment seg,
         break;
     case x86_seg_ss:
         memcpy(reg, &vmcb->ss, sizeof(*reg));
+        reg->attr.fields.dpl = vmcb->cpl;
         break;
     case x86_seg_tr:
         svm_sync_vmcb(v);