Messages in this thread | | | From | KY Srinivasan <> | Subject | RE: [PATCH char-misc 1/2] Drivers: hv: vmbus: Remove x86 MSR refs in arch independent code | Date | Sat, 12 May 2018 08:37:47 +0000 |
| |
> -----Original Message----- > From: mhkelley58@gmail.com <mhkelley58@gmail.com> > Sent: Tuesday, May 8, 2018 8:38 AM > To: gregkh@linuxfoundation.org; linux-kernel@vger.kernel.org; > devel@linuxdriverproject.org; olaf@aepfle.de; apw@canonical.com; > vkuznets@redhat.com; jasowang@redhat.com; > leann.ogasawara@canonical.com; marcelo.cerri@canonical.com; Stephen > Hemminger <sthemmin@microsoft.com>; KY Srinivasan > <kys@microsoft.com> > Subject: [PATCH char-misc 1/2] Drivers: hv: vmbus: Remove x86 MSR refs in > arch independent code > > From: Michael Kelley <mikelley@microsoft.com> > > In architecture independent code for manipulating Hyper-V synthetic timers > and synthetic interrupts, pass in an ordinal number identifying the timer > or interrupt, rather than an actual MSR register address. Then in > x86/x64 specific code, map the ordinal number to the appropriate MSR. > This change facilitates the introduction of an ARM64 version of Hyper-V, > which uses the same synthetic timers and interrupts, but a different > mechanism for accessing them. > > Signed-off-by: Michael Kelley <mikelley@microsoft.com> > --- > arch/x86/include/asm/mshyperv.h | 12 ++++++++---- > drivers/hv/hv.c | 20 ++++++++------------ > 2 files changed, 16 insertions(+), 16 deletions(-) > > diff --git a/arch/x86/include/asm/mshyperv.h > b/arch/x86/include/asm/mshyperv.h > index b90e796..caf9035 100644 > --- a/arch/x86/include/asm/mshyperv.h > +++ b/arch/x86/include/asm/mshyperv.h > @@ -75,8 +75,10 @@ static inline void vmbus_signal_eom(struct > hv_message *msg, u32 old_msg_type) > } > } > > -#define hv_init_timer(timer, tick) wrmsrl(timer, tick) > -#define hv_init_timer_config(config, val) wrmsrl(config, val) > +#define hv_init_timer(timer, tick) \ > + wrmsrl(HV_X64_MSR_STIMER0_COUNT + (2*timer), tick) > +#define hv_init_timer_config(timer, val) \ > + wrmsrl(HV_X64_MSR_STIMER0_CONFIG + (2*timer), val) Why are we stepping in units of 2?
K. Y
| |