Drivers: hv: vmbus: don't manipulate with clocksources on crash
clocksource_change_rating() involves mutex usage and can't be called in interrupt context. It also makes sense to avoid doing redundant work on crash. Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com> Signed-off-by: K. Y. Srinivasan <kys@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
415719160d
commit
3ccb4fd8f4
|
@ -295,8 +295,14 @@ void hv_cleanup(void)
|
||||||
* Cleanup the TSC page based CS.
|
* Cleanup the TSC page based CS.
|
||||||
*/
|
*/
|
||||||
if (ms_hyperv.features & HV_X64_MSR_REFERENCE_TSC_AVAILABLE) {
|
if (ms_hyperv.features & HV_X64_MSR_REFERENCE_TSC_AVAILABLE) {
|
||||||
clocksource_change_rating(&hyperv_cs_tsc, 10);
|
/*
|
||||||
clocksource_unregister(&hyperv_cs_tsc);
|
* Crash can happen in an interrupt context and unregistering
|
||||||
|
* a clocksource is impossible and redundant in this case.
|
||||||
|
*/
|
||||||
|
if (!oops_in_progress) {
|
||||||
|
clocksource_change_rating(&hyperv_cs_tsc, 10);
|
||||||
|
clocksource_unregister(&hyperv_cs_tsc);
|
||||||
|
}
|
||||||
|
|
||||||
hypercall_msr.as_uint64 = 0;
|
hypercall_msr.as_uint64 = 0;
|
||||||
wrmsrl(HV_X64_MSR_REFERENCE_TSC, hypercall_msr.as_uint64);
|
wrmsrl(HV_X64_MSR_REFERENCE_TSC, hypercall_msr.as_uint64);
|
||||||
|
|
Loading…
Reference in New Issue