[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL v3 7/7] i386: Disable Intel PT if packets IP payloads
From: |
Eduardo Habkost |
Subject: |
[Qemu-devel] [PULL v3 7/7] i386: Disable Intel PT if packets IP payloads have LIP values |
Date: |
Mon, 19 Mar 2018 17:12:55 -0300 |
From: Luwei Kang <address@hidden>
Intel processor trace should be disabled when
CPUID.(EAX=14H,ECX=0H).ECX.[bit31] is set.
Generated packets which contain IP payloads will have LIP
values when this bit is set, or IP payloads will have RIP
values.
Currently, The information of CPUID 14H is constant to make
live migration safty and this bit is always 0 in guest even
if host support LIP values.
Guest sees the bit is 0 will expect IP payloads with RIP
values, but the host CPU will generate IP payloads with
LIP values if this bit is set in HW.
To make sure the value of IP payloads correctly, Intel PT
should be disabled when bit[31] is set.
Signed-off-by: Luwei Kang <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
---
target/i386/cpu.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/target/i386/cpu.c b/target/i386/cpu.c
index 30b4af897b..4f62fe776d 100644
--- a/target/i386/cpu.c
+++ b/target/i386/cpu.c
@@ -195,6 +195,8 @@
* bit[02]: Support Single-Range Output scheme;
*/
#define INTEL_PT_MINIMAL_ECX 0x7
+/* generated packets which contain IP payloads have LIP values */
+#define INTEL_PT_IP_LIP (1 << 31)
#define INTEL_PT_ADDR_RANGES_NUM 0x2 /* Number of configurable address ranges
*/
#define INTEL_PT_ADDR_RANGES_NUM_MASK 0x3
#define INTEL_PT_MTC_BITMAP (0x0249 << 16) /* Support ART(0,3,6,9) */
@@ -4167,7 +4169,8 @@ static int x86_cpu_filter_features(X86CPU *cpu)
((eax_1 & INTEL_PT_ADDR_RANGES_NUM_MASK) <
INTEL_PT_ADDR_RANGES_NUM) ||
((ebx_1 & (INTEL_PT_PSB_BITMAP | INTEL_PT_CYCLE_BITMAP)) !=
- (INTEL_PT_PSB_BITMAP | INTEL_PT_CYCLE_BITMAP))) {
+ (INTEL_PT_PSB_BITMAP | INTEL_PT_CYCLE_BITMAP)) ||
+ (ecx_0 & INTEL_PT_IP_LIP)) {
/*
* Processor Trace capabilities aren't configurable, so if the
* host can't emulate the capabilities we report on
--
2.14.3
- [Qemu-devel] [PULL v3 0/7] Machine and x86 queue, 2018-03-19, Eduardo Habkost, 2018/03/19
- [Qemu-devel] [PULL v3 1/7] nios2: 10m50_devboard: replace cpu_model with cpu_type, Eduardo Habkost, 2018/03/19
- [Qemu-devel] [PULL v3 2/7] tests: add machine 'none' with -cpu test, Eduardo Habkost, 2018/03/19
- [Qemu-devel] [PULL v3 3/7] cpu: add CPU_RESOLVING_TYPE macro, Eduardo Habkost, 2018/03/19
- [Qemu-devel] [PULL v3 4/7] Use cpu_create(type) instead of cpu_init(cpu_model), Eduardo Habkost, 2018/03/19
- [Qemu-devel] [PULL v3 5/7] cpu: get rid of unused cpu_init() defines, Eduardo Habkost, 2018/03/19
- [Qemu-devel] [PULL v3 6/7] cpu: drop unnecessary NULL check and cpu_common_class_by_name(), Eduardo Habkost, 2018/03/19
- [Qemu-devel] [PULL v3 7/7] i386: Disable Intel PT if packets IP payloads have LIP values,
Eduardo Habkost <=
- Re: [Qemu-devel] [PULL v3 0/7] Machine and x86 queue, 2018-03-19, Peter Maydell, 2018/03/20