2019-08-20 03:08:40

by Dexuan Cui

[permalink] [raw]
Subject: [PATCH] Drivers: hv: vmbus: Remove the unused "tsc_page" from struct hv_context

This field is no longer used after the commit
63ed4e0c67df ("Drivers: hv: vmbus: Consolidate all Hyper-V specific clocksource code")
, because it's replaced by the global variable
"struct ms_hyperv_tsc_page *tsc_pg;" (now, the variable is in
drivers/clocksource/hyperv_timer.c).

Fixes: 63ed4e0c67df ("Drivers: hv: vmbus: Consolidate all Hyper-V specific clocksource code")
Signed-off-by: Dexuan Cui <[email protected]>
---
drivers/hv/hyperv_vmbus.h | 2 --
1 file changed, 2 deletions(-)

diff --git a/drivers/hv/hyperv_vmbus.h b/drivers/hv/hyperv_vmbus.h
index 362e70e..fb16a62 100644
--- a/drivers/hv/hyperv_vmbus.h
+++ b/drivers/hv/hyperv_vmbus.h
@@ -146,8 +146,6 @@ struct hv_context {
*/
u64 guestid;

- void *tsc_page;
-
struct hv_per_cpu_context __percpu *cpu_context;

/*
--
1.8.3.1


2019-08-20 15:30:07

by Sasha Levin

[permalink] [raw]
Subject: Re: [PATCH] Drivers: hv: vmbus: Remove the unused "tsc_page" from struct hv_context

On Tue, Aug 20, 2019 at 03:06:40AM +0000, Dexuan Cui wrote:
>This field is no longer used after the commit
>63ed4e0c67df ("Drivers: hv: vmbus: Consolidate all Hyper-V specific clocksource code")
>, because it's replaced by the global variable
>"struct ms_hyperv_tsc_page *tsc_pg;" (now, the variable is in
>drivers/clocksource/hyperv_timer.c).
>
>Fixes: 63ed4e0c67df ("Drivers: hv: vmbus: Consolidate all Hyper-V specific clocksource code")
>Signed-off-by: Dexuan Cui <[email protected]>

Queued up for hyperv-fixes, thank you.

--
Thanks,
Sasha