KVM: Cleanup the kvm_print functions and introduce pr_XX wrappers

Introduces a couple of print functions, which are essentially wrappers
around standard printk functions, with a KVM: prefix.

Functions introduced or modified are:
 - kvm_err(fmt, ...)
 - kvm_info(fmt, ...)
 - kvm_debug(fmt, ...)
 - kvm_pr_unimpl(fmt, ...)
 - pr_unimpl(vcpu, fmt, ...) -> vcpu_unimpl(vcpu, fmt, ...)

Signed-off-by: Christoffer Dall <c.dall@virtualopensystems.com>
Signed-off-by: Avi Kivity <avi@redhat.com>

1 files changed, 27 insertions, 27 deletions

diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index f12a52408cda..a01a4241bc6b 100644
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -1437,8 +1437,8 @@ static int set_msr_hyperv_pw(struct kvm_vcpu *vcpu, u32 msr, u64 data)
                 break;
         }
         default:
-                pr_unimpl(vcpu, "HYPER-V unimplemented wrmsr: 0x%x "
-                          "data 0x%llx\n", msr, data);
+                vcpu_unimpl(vcpu, "HYPER-V unimplemented wrmsr: 0x%x "
+                            "data 0x%llx\n", msr, data);
                 return 1;
         }
         return 0;
@@ -1470,8 +1470,8 @@ static int set_msr_hyperv(struct kvm_vcpu *vcpu, u32 msr, u64 data)
         case HV_X64_MSR_TPR:
                 return kvm_hv_vapic_msr_write(vcpu, APIC_TASKPRI, data);
         default:
-                pr_unimpl(vcpu, "HYPER-V unimplemented wrmsr: 0x%x "
-                          "data 0x%llx\n", msr, data);
+                vcpu_unimpl(vcpu, "HYPER-V unimplemented wrmsr: 0x%x "
+                            "data 0x%llx\n", msr, data);
                 return 1;
         }
 
@@ -1551,15 +1551,15 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 data)
                 data &= ~(u64)0x100;    /* ignore ignne emulation enable */
                 data &= ~(u64)0x8;      /* ignore TLB cache disable */
                 if (data != 0) {
-                        pr_unimpl(vcpu, "unimplemented HWCR wrmsr: 0x%llx\n",
-                                data);
+                        vcpu_unimpl(vcpu, "unimplemented HWCR wrmsr: 0x%llx\n",
+                                    data);
                         return 1;
                 }
                 break;
         case MSR_FAM10H_MMIO_CONF_BASE:
                 if (data != 0) {
-                        pr_unimpl(vcpu, "unimplemented MMIO_CONF_BASE wrmsr: "
-                                "0x%llx\n", data);
+                        vcpu_unimpl(vcpu, "unimplemented MMIO_CONF_BASE wrmsr: "
+                                    "0x%llx\n", data);
                         return 1;
                 }
                 break;
@@ -1574,8 +1574,8 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 data)
                            thus reserved and should throw a #GP */
                         return 1;
                 }
-                pr_unimpl(vcpu, "%s: MSR_IA32_DEBUGCTLMSR 0x%llx, nop\n",
-                        __func__, data);
+                vcpu_unimpl(vcpu, "%s: MSR_IA32_DEBUGCTLMSR 0x%llx, nop\n",
+                            __func__, data);
                 break;
         case MSR_IA32_UCODE_REV:
         case MSR_IA32_UCODE_WRITE:
@@ -1671,8 +1671,8 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 data)
         case MSR_K7_EVNTSEL2:
         case MSR_K7_EVNTSEL3:
                 if (data != 0)
-                        pr_unimpl(vcpu, "unimplemented perfctr wrmsr: "
-                                "0x%x data 0x%llx\n", msr, data);
+                        vcpu_unimpl(vcpu, "unimplemented perfctr wrmsr: "
+                                    "0x%x data 0x%llx\n", msr, data);
                 break;
         /* at least RHEL 4 unconditionally writes to the perfctr registers,
          * so we ignore writes to make it happy.
@@ -1681,8 +1681,8 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 data)
         case MSR_K7_PERFCTR1:
         case MSR_K7_PERFCTR2:
         case MSR_K7_PERFCTR3:
-                pr_unimpl(vcpu, "unimplemented perfctr wrmsr: "
-                        "0x%x data 0x%llx\n", msr, data);
+                vcpu_unimpl(vcpu, "unimplemented perfctr wrmsr: "
+                            "0x%x data 0x%llx\n", msr, data);
                 break;
         case MSR_P6_PERFCTR0:
         case MSR_P6_PERFCTR1:
@@ -1693,8 +1693,8 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 data)
                         return kvm_pmu_set_msr(vcpu, msr, data);
 
                 if (pr || data != 0)
-                        pr_unimpl(vcpu, "disabled perfctr wrmsr: "
-                                "0x%x data 0x%llx\n", msr, data);
+                        vcpu_unimpl(vcpu, "disabled perfctr wrmsr: "
+                                    "0x%x data 0x%llx\n", msr, data);
                 break;
         case MSR_K7_CLK_CTL:
                 /*
@@ -1720,7 +1720,7 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 data)
                 /* Drop writes to this legacy MSR -- see rdmsr
                  * counterpart for further detail.
                  */
-                pr_unimpl(vcpu, "ignored wrmsr: 0x%x data %llx\n", msr, data);
+                vcpu_unimpl(vcpu, "ignored wrmsr: 0x%x data %llx\n", msr, data);
                 break;
         case MSR_AMD64_OSVW_ID_LENGTH:
                 if (!guest_cpuid_has_osvw(vcpu))
@@ -1738,12 +1738,12 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 data)
                 if (kvm_pmu_msr(vcpu, msr))
                         return kvm_pmu_set_msr(vcpu, msr, data);
                 if (!ignore_msrs) {
-                        pr_unimpl(vcpu, "unhandled wrmsr: 0x%x data %llx\n",
-                                msr, data);
+                        vcpu_unimpl(vcpu, "unhandled wrmsr: 0x%x data %llx\n",
+                                    msr, data);
                         return 1;
                 } else {
-                        pr_unimpl(vcpu, "ignored wrmsr: 0x%x data %llx\n",
-                                msr, data);
+                        vcpu_unimpl(vcpu, "ignored wrmsr: 0x%x data %llx\n",
+                                    msr, data);
                         break;
                 }
         }
@@ -1846,7 +1846,7 @@ static int get_msr_hyperv_pw(struct kvm_vcpu *vcpu, u32 msr, u64 *pdata)
                 data = kvm->arch.hv_hypercall;
                 break;
         default:
-                pr_unimpl(vcpu, "Hyper-V unhandled rdmsr: 0x%x\n", msr);
+                vcpu_unimpl(vcpu, "Hyper-V unhandled rdmsr: 0x%x\n", msr);
                 return 1;
         }
 
@@ -1877,7 +1877,7 @@ static int get_msr_hyperv(struct kvm_vcpu *vcpu, u32 msr, u64 *pdata)
                 data = vcpu->arch.hv_vapic;
                 break;
         default:
-                pr_unimpl(vcpu, "Hyper-V unhandled rdmsr: 0x%x\n", msr);
+                vcpu_unimpl(vcpu, "Hyper-V unhandled rdmsr: 0x%x\n", msr);
                 return 1;
         }
         *pdata = data;
@@ -2030,10 +2030,10 @@ int kvm_get_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 *pdata)
                 if (kvm_pmu_msr(vcpu, msr))
                         return kvm_pmu_get_msr(vcpu, msr, pdata);
                 if (!ignore_msrs) {
-                        pr_unimpl(vcpu, "unhandled rdmsr: 0x%x\n", msr);
+                        vcpu_unimpl(vcpu, "unhandled rdmsr: 0x%x\n", msr);
                         return 1;
                 } else {
-                        pr_unimpl(vcpu, "ignored rdmsr: 0x%x\n", msr);
+                        vcpu_unimpl(vcpu, "ignored rdmsr: 0x%x\n", msr);
                         data = 0;
                 }
                 break;
@@ -4116,7 +4116,7 @@ static unsigned long emulator_get_cr(struct x86_emulate_ctxt *ctxt, int cr)
                 value = kvm_get_cr8(vcpu);
                 break;
         default:
-                vcpu_printf(vcpu, "%s: unexpected cr %u\n", __func__, cr);
+                kvm_err("%s: unexpected cr %u\n", __func__, cr);
                 return 0;
         }
 
@@ -4145,7 +4145,7 @@ static int emulator_set_cr(struct x86_emulate_ctxt *ctxt, int cr, ulong val)
                 res = kvm_set_cr8(vcpu, val);
                 break;
         default:
-                vcpu_printf(vcpu, "%s: unexpected cr %u\n", __func__, cr);
+                kvm_err("%s: unexpected cr %u\n", __func__, cr);
                 res = -1;
         }