diff mbox series

tools/libxc: disable x2APIC when using nested virtualization

Message ID 20200103172935.66137-1-roger.pau@citrix.com (mailing list archive)
State New, archived
Headers show
Series tools/libxc: disable x2APIC when using nested virtualization | expand

Commit Message

Roger Pau Monné Jan. 3, 2020, 5:29 p.m. UTC
There are issues (as reported by osstest [0]) when Xen is running
nested on itself and the L1 Xen is using x2APIC. While those are being
investigated, disable announcing the x2APIC feature in CPUID when
nested HVM mode is enabled.

[0] http://logs.test-lab.xenproject.org/osstest/logs/145509/

Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
---
 tools/libxc/xc_cpuid_x86.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Andrew Cooper Jan. 3, 2020, 5:34 p.m. UTC | #1
On 03/01/2020 17:29, Roger Pau Monne wrote:
> There are issues (as reported by osstest [0]) when Xen is running
> nested on itself and the L1 Xen is using x2APIC. While those are being
> investigated, disable announcing the x2APIC feature in CPUID when
> nested HVM mode is enabled.
>
> [0] http://logs.test-lab.xenproject.org/osstest/logs/145509/
>
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
Wei Liu Jan. 3, 2020, 5:41 p.m. UTC | #2
On Fri, Jan 03, 2020 at 06:29:35PM +0100, Roger Pau Monne wrote:
> There are issues (as reported by osstest [0]) when Xen is running
> nested on itself and the L1 Xen is using x2APIC. While those are being
> investigated, disable announcing the x2APIC feature in CPUID when
> nested HVM mode is enabled.
> 
> [0] http://logs.test-lab.xenproject.org/osstest/logs/145509/

Putting a link here is probably not very useful because it will be gone
in no time.

If you have serial logs that would be great.

> 
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>

Acked-by: Wei Liu <wl@xen.org>
diff mbox series

Patch

diff --git a/tools/libxc/xc_cpuid_x86.c b/tools/libxc/xc_cpuid_x86.c
index 2540aa1e1c..ac38c1406e 100644
--- a/tools/libxc/xc_cpuid_x86.c
+++ b/tools/libxc/xc_cpuid_x86.c
@@ -653,6 +653,17 @@  int xc_cpuid_apply_policy(xc_interface *xch, uint32_t domid,
         p->extd.itsc = true;
         p->basic.vmx = true;
         p->extd.svm = true;
+
+        /*
+         * BODGE: don't announce x2APIC mode when using nested virtualization,
+         * as it doesn't work properly. This should be removed once the
+         * underlying bug(s) are fixed.
+         */
+        rc = xc_hvm_param_get(xch, domid, HVM_PARAM_NESTEDHVM, &val);
+        if ( rc )
+            goto out;
+        if ( val )
+            p->basic.x2apic = false;
     }
 
     rc = x86_cpuid_copy_to_buffer(p, leaves, &nr_leaves);