Drivers: hv: vmbus: Move the check for hypercall page setup
authorK. Y. Srinivasan <kys@microsoft.com>
Thu, 19 Jan 2017 18:51:49 +0000 (11:51 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 20 Jan 2017 13:48:03 +0000 (14:48 +0100)
As part of the effort to separate out architecture specific code, move the
check for detecting if the hypercall page is setup.

Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
arch/x86/hyperv/hv_init.c
arch/x86/include/asm/mshyperv.h
drivers/hv/hv.c

index ce5fc7394814f9051e177bb3a41e8737ea69df44..d289bc29d2822eb9453efe78ea5d3b459ed1e98f 100644 (file)
@@ -235,3 +235,18 @@ void hyperv_report_panic(struct pt_regs *regs)
        wrmsrl(HV_X64_MSR_CRASH_CTL, HV_CRASH_CTL_CRASH_NOTIFY);
 }
 EXPORT_SYMBOL_GPL(hyperv_report_panic);
+
+bool hv_is_hypercall_page_setup(void)
+{
+       union hv_x64_msr_hypercall_contents hypercall_msr;
+
+       /* Check if the hypercall page is setup */
+       hypercall_msr.as_uint64 = 0;
+       rdmsrl(HV_X64_MSR_HYPERCALL, hypercall_msr.as_uint64);
+
+       if (!hypercall_msr.enable)
+               return false;
+
+       return true;
+}
+EXPORT_SYMBOL_GPL(hv_is_hypercall_page_setup);
index 64e682d8868465cc69e2e3c5729ae5da2dbc3ba1..c843ef64defea9cd9bbd6cf97f173085dfadae7d 100644 (file)
@@ -115,5 +115,6 @@ void hv_remove_crash_handler(void);
 #if IS_ENABLED(CONFIG_HYPERV)
 void hyperv_init(void);
 void hyperv_report_panic(struct pt_regs *regs);
+bool hv_is_hypercall_page_setup(void);
 #endif
 #endif
index 9985a347ed03c2ccb33877d912c7fb5944405d65..d28a8731baa016f331e5f339f6e3fde1da909130 100644 (file)
@@ -49,7 +49,6 @@ struct hv_context hv_context = {
  */
 int hv_init(void)
 {
-       union hv_x64_msr_hypercall_contents hypercall_msr;
 
        memset(hv_context.synic_event_page, 0, sizeof(void *) * NR_CPUS);
        memset(hv_context.synic_message_page, 0,
@@ -65,11 +64,7 @@ int hv_init(void)
        memset(hv_context.clk_evt, 0,
               sizeof(void *) * NR_CPUS);
 
-       /* See if the hypercall page is already set */
-       hypercall_msr.as_uint64 = 0;
-       rdmsrl(HV_X64_MSR_HYPERCALL, hypercall_msr.as_uint64);
-
-       if (!hypercall_msr.enable)
+       if (!hv_is_hypercall_page_setup())
                return -ENOTSUPP;
 
        return 0;