hv: initialize and save/restore IA32_TSC_AUX MSR for guest
Commit cbf3825
"hv: Pass-through IA32_TSC_AUX MSR to L1 guest"
lets guest own the physical MSR IA32_TSC_AUX and does not handle this MSR
in the hypervisor.
If multiple vCPUs share the same pCPU, when one vCPU reads MSR IA32_TSC_AUX,
it may get the value set by other vCPUs.
To fix this issue, this patch does:
- initialize the MSR content to 0 for the given vCPU, which is consistent with
the value specified in SDM Vol3 "Table 9-1. IA-32 and Intel 64 Processor
States Following Power-up, Reset, or INIT"
- save/restore the MSR content for the given vCPU during context switch
v1 -> v2:
* According to Table 9-1, the content of IA32_TSC_AUX MSR is unchanged
following INIT, v2 updates the initialization logic so that the content for
vCPU is consistent with SDM.
Tracked-On: #6799
Signed-off-by: Shiqing Gao <shiqing.gao@intel.com>
Reviewed-by: Zide Chen <zide.chen@intel.com>
Acked-by: Eddie Dong <eddie.dong@intel.com>
This commit is contained in:
parent
6023388e79
commit
7bbd17ce80
|
@ -149,6 +149,7 @@ static void save_world_ctx(struct acrn_vcpu *vcpu, struct ext_context *ext_ctx)
|
|||
ext_ctx->ia32_lstar = msr_read(MSR_IA32_LSTAR);
|
||||
ext_ctx->ia32_fmask = msr_read(MSR_IA32_FMASK);
|
||||
ext_ctx->ia32_kernel_gs_base = msr_read(MSR_IA32_KERNEL_GS_BASE);
|
||||
ext_ctx->tsc_aux = msr_read(MSR_IA32_TSC_AUX);
|
||||
|
||||
/* XSAVE area */
|
||||
save_xsave_area(vcpu, ext_ctx);
|
||||
|
@ -201,6 +202,7 @@ static void load_world_ctx(struct acrn_vcpu *vcpu, const struct ext_context *ext
|
|||
msr_write(MSR_IA32_LSTAR, ext_ctx->ia32_lstar);
|
||||
msr_write(MSR_IA32_FMASK, ext_ctx->ia32_fmask);
|
||||
msr_write(MSR_IA32_KERNEL_GS_BASE, ext_ctx->ia32_kernel_gs_base);
|
||||
msr_write(MSR_IA32_TSC_AUX, ext_ctx->tsc_aux);
|
||||
|
||||
/* XSAVE area */
|
||||
rstore_xsave_area(vcpu, ext_ctx);
|
||||
|
|
|
@ -271,7 +271,7 @@ static void vcpu_reset_internal(struct acrn_vcpu *vcpu, enum reset_mode mode)
|
|||
vlapic = vcpu_vlapic(vcpu);
|
||||
vlapic_reset(vlapic, apicv_ops, mode);
|
||||
|
||||
reset_vcpu_regs(vcpu);
|
||||
reset_vcpu_regs(vcpu, mode);
|
||||
|
||||
for (i = 0; i < VCPU_EVENT_NUM; i++) {
|
||||
reset_event(&vcpu->events[i]);
|
||||
|
@ -467,9 +467,26 @@ bool sanitize_cr0_cr4_pattern(void)
|
|||
return ret;
|
||||
}
|
||||
|
||||
void reset_vcpu_regs(struct acrn_vcpu *vcpu)
|
||||
void reset_vcpu_regs(struct acrn_vcpu *vcpu, enum reset_mode mode)
|
||||
{
|
||||
set_vcpu_regs(vcpu, &realmode_init_vregs);
|
||||
|
||||
/*
|
||||
* According to SDM Vol3 "Table 9-1. IA-32 and Intel 64 Processor States Following Power-up, Reset, or INIT",
|
||||
* for some registers, the state following INIT is different from the state following Power-up, Reset.
|
||||
* (For all registers mentioned in Table 9-1, the state following Power-up and following Reset are same.)
|
||||
*
|
||||
* To distinguish this kind of case for vCPU:
|
||||
* - If the state following INIT is same as the state following Power-up/Reset, handle it in
|
||||
* set_vcpu_regs above.
|
||||
* - Otherwise, handle it below.
|
||||
*/
|
||||
if (mode != INIT_RESET) {
|
||||
struct ext_context *ectx = &(vcpu->arch.contexts[vcpu->arch.cur_context].ext_ctx);
|
||||
|
||||
/* IA32_TSC_AUX: 0 following Power-up/Reset, unchanged following INIT */
|
||||
ectx->tsc_aux = 0UL;
|
||||
}
|
||||
}
|
||||
|
||||
void init_vcpu_protect_mode_regs(struct acrn_vcpu *vcpu, uint64_t vgdt_base_gpa)
|
||||
|
@ -902,6 +919,7 @@ static void context_switch_out(struct thread_object *prev)
|
|||
ectx->ia32_lstar = msr_read(MSR_IA32_LSTAR);
|
||||
ectx->ia32_fmask = msr_read(MSR_IA32_FMASK);
|
||||
ectx->ia32_kernel_gs_base = msr_read(MSR_IA32_KERNEL_GS_BASE);
|
||||
ectx->tsc_aux = msr_read(MSR_IA32_TSC_AUX);
|
||||
|
||||
save_xsave_area(vcpu, ectx);
|
||||
}
|
||||
|
@ -919,6 +937,7 @@ static void context_switch_in(struct thread_object *next)
|
|||
msr_write(MSR_IA32_LSTAR, ectx->ia32_lstar);
|
||||
msr_write(MSR_IA32_FMASK, ectx->ia32_fmask);
|
||||
msr_write(MSR_IA32_KERNEL_GS_BASE, ectx->ia32_kernel_gs_base);
|
||||
msr_write(MSR_IA32_TSC_AUX, ectx->tsc_aux);
|
||||
|
||||
if (pcpu_has_cap(X86_FEATURE_WAITPKG)) {
|
||||
vmsr_val = vcpu_get_guest_msr(vcpu, MSR_IA32_UMWAIT_CONTROL);
|
||||
|
|
|
@ -411,6 +411,7 @@ struct ext_context {
|
|||
|
||||
uint64_t dr7;
|
||||
uint64_t tsc_offset;
|
||||
uint64_t tsc_aux;
|
||||
|
||||
struct xsave_area xs_area;
|
||||
uint64_t xcr0;
|
||||
|
|
|
@ -581,10 +581,11 @@ void set_vcpu_regs(struct acrn_vcpu *vcpu, struct acrn_regs *vcpu_regs);
|
|||
* Reset target vCPU's all registers in run_context to initial values.
|
||||
*
|
||||
* @param[inout] vcpu pointer to vcpu data structure
|
||||
* @param[in] mode the reset mode
|
||||
*
|
||||
* @return None
|
||||
*/
|
||||
void reset_vcpu_regs(struct acrn_vcpu *vcpu);
|
||||
void reset_vcpu_regs(struct acrn_vcpu *vcpu, enum reset_mode mode);
|
||||
|
||||
bool sanitize_cr0_cr4_pattern(void);
|
||||
|
||||
|
|
Loading…
Reference in New Issue