CPL should be determined by code segment's DPL, not data segment's.
This solves a problem migrating a QEMU guest without PV-on-HVM drivers
Signed-off-by: Robert Plillips <rphillips@xxxxxxxxxxxxxxx>
Signed-off-by: Ben Guthro <bguthro@xxxxxxxxxxxxxxxx>

diff -r f76f151a7431 xen/arch/x86/hvm/svm/svm.c
--- a/xen/arch/x86/hvm/svm/svm.c
+++ b/xen/arch/x86/hvm/svm/svm.c
@@ -557,7 +557,7 @@ static void svm_set_segment_register(str
 break;
 case x86_seg_ss:
 memcpy(&vmcb->ss, reg, sizeof(*reg));
- vmcb->cpl = vmcb->ss.attr.fields.dpl;
+ vmcb->cpl = vmcb->cs.attr.fields.dpl;
 break;
 case x86_seg_tr:
 memcpy(&vmcb->tr, reg, sizeof(*reg));

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