OSSTest has shown an assertion failure:
http://logs.test-lab.xenproject.org/osstest/logs/153906/test-xtf-amd64-amd64-1/serial-rimava1.log
These assertions were never appropriate, as they rule out legal (and, as it
turns out, sensible perf-wise) inputs based on an expectation of how the sole
caller would behave.
Fixes: ad0fd291c5 ("x86/pv: Rewrite segment context switching from scratch")
---
CC: Jan Beulich <jbeul...@suse.com>
CC: Roger Pau Monné <roger....@citrix.com>
CC: Wei Liu <w...@xen.org>
---
xen/arch/x86/hvm/svm/svm.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/xen/arch/x86/hvm/svm/svm.c b/xen/arch/x86/hvm/svm/svm.c
index af584ff5d1..eaacbcb668 100644
--- a/xen/arch/x86/hvm/svm/svm.c
+++ b/xen/arch/x86/hvm/svm/svm.c
@@ -1559,13 +1559,11 @@ bool svm_load_segs(unsigned int ldt_ents, unsigned long
ldt_base,
vmcb->ldtr.base = ldt_base;
}
- ASSERT(!(fs_sel & ~3));
vmcb->fs.sel = fs_sel;
vmcb->fs.attr = 0;
vmcb->fs.limit = 0;
vmcb->fs.base = fs_base;
- ASSERT(!(gs_sel & ~3));
vmcb->gs.sel = gs_sel;
vmcb->gs.attr = 0;
vmcb->gs.limit = 0;
--
2.11.0
