Commit 2982635a authored by Anirudh Rayabharam's avatar Anirudh Rayabharam Committed by Wei Liu

x86/hyperv: fix invalid writes to MSRs during root partition kexec

hyperv_cleanup resets the hypercall page by setting the MSR to 0. However,
the root partition is not allowed to write to the GPA bits of the MSR.
Instead, it uses the hypercall page provided by the MSR. Similar is the
case with the reference TSC MSR.

Clear only the enable bit instead of zeroing the entire MSR to make
the code valid for root partition too.
Signed-off-by: default avatarAnirudh Rayabharam <anrayabh@linux.microsoft.com>
Reviewed-by: default avatarMichael Kelley <mikelley@microsoft.com>
Link: https://lore.kernel.org/r/20221027095729.1676394-3-anrayabh@linux.microsoft.comSigned-off-by: default avatarWei Liu <wei.liu@kernel.org>
parent 4ad1aa57
...@@ -535,6 +535,7 @@ void __init hyperv_init(void) ...@@ -535,6 +535,7 @@ void __init hyperv_init(void)
void hyperv_cleanup(void) void hyperv_cleanup(void)
{ {
union hv_x64_msr_hypercall_contents hypercall_msr; union hv_x64_msr_hypercall_contents hypercall_msr;
union hv_reference_tsc_msr tsc_msr;
unregister_syscore_ops(&hv_syscore_ops); unregister_syscore_ops(&hv_syscore_ops);
...@@ -550,12 +551,14 @@ void hyperv_cleanup(void) ...@@ -550,12 +551,14 @@ void hyperv_cleanup(void)
hv_hypercall_pg = NULL; hv_hypercall_pg = NULL;
/* Reset the hypercall page */ /* Reset the hypercall page */
hypercall_msr.as_uint64 = 0; hypercall_msr.as_uint64 = hv_get_register(HV_X64_MSR_HYPERCALL);
wrmsrl(HV_X64_MSR_HYPERCALL, hypercall_msr.as_uint64); hypercall_msr.enable = 0;
hv_set_register(HV_X64_MSR_HYPERCALL, hypercall_msr.as_uint64);
/* Reset the TSC page */ /* Reset the TSC page */
hypercall_msr.as_uint64 = 0; tsc_msr.as_uint64 = hv_get_register(HV_X64_MSR_REFERENCE_TSC);
wrmsrl(HV_X64_MSR_REFERENCE_TSC, hypercall_msr.as_uint64); tsc_msr.enable = 0;
hv_set_register(HV_X64_MSR_REFERENCE_TSC, tsc_msr.as_uint64);
} }
void hyperv_report_panic(struct pt_regs *regs, long err, bool in_die) void hyperv_report_panic(struct pt_regs *regs, long err, bool in_die)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment