diff options
author | James Hogan <james.hogan@imgtec.com> | 2014-05-29 05:16:43 -0400 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2014-05-30 07:05:37 -0400 |
commit | 6e95bfd2675ea5332b10e9414de8886243ca53ae (patch) | |
tree | 42555013c3a3909fa225dee9d7be4ddb11ef1d37 /arch/mips/kvm | |
parent | d5c704d525aa5e2e7b2bdfbe3e99e4621e29d6bf (diff) |
MIPS: KVM: Quieten kvm_info() logging
The logging from MIPS KVM is fairly noisy with kvm_info() in places
where it shouldn't be, such as on VM creation and migration to a
different CPU. Replace these kvm_info() calls with kvm_debug().
Signed-off-by: James Hogan <james.hogan@imgtec.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Gleb Natapov <gleb@kernel.org>
Cc: kvm@vger.kernel.org
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: linux-mips@linux-mips.org
Cc: Sanjay Lal <sanjayl@kymasys.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'arch/mips/kvm')
-rw-r--r-- | arch/mips/kvm/kvm_mips.c | 27 | ||||
-rw-r--r-- | arch/mips/kvm/kvm_tlb.c | 16 |
2 files changed, 21 insertions, 22 deletions
diff --git a/arch/mips/kvm/kvm_mips.c b/arch/mips/kvm/kvm_mips.c index 97e9fe1626ce..dd7b115c26e8 100644 --- a/arch/mips/kvm/kvm_mips.c +++ b/arch/mips/kvm/kvm_mips.c | |||
@@ -125,8 +125,8 @@ static void kvm_mips_init_vm_percpu(void *arg) | |||
125 | int kvm_arch_init_vm(struct kvm *kvm, unsigned long type) | 125 | int kvm_arch_init_vm(struct kvm *kvm, unsigned long type) |
126 | { | 126 | { |
127 | if (atomic_inc_return(&kvm_mips_instance) == 1) { | 127 | if (atomic_inc_return(&kvm_mips_instance) == 1) { |
128 | kvm_info("%s: 1st KVM instance, setup host TLB parameters\n", | 128 | kvm_debug("%s: 1st KVM instance, setup host TLB parameters\n", |
129 | __func__); | 129 | __func__); |
130 | on_each_cpu(kvm_mips_init_vm_percpu, kvm, 1); | 130 | on_each_cpu(kvm_mips_init_vm_percpu, kvm, 1); |
131 | } | 131 | } |
132 | 132 | ||
@@ -181,8 +181,8 @@ void kvm_arch_destroy_vm(struct kvm *kvm) | |||
181 | 181 | ||
182 | /* If this is the last instance, restore wired count */ | 182 | /* If this is the last instance, restore wired count */ |
183 | if (atomic_dec_return(&kvm_mips_instance) == 0) { | 183 | if (atomic_dec_return(&kvm_mips_instance) == 0) { |
184 | kvm_info("%s: last KVM instance, restoring TLB parameters\n", | 184 | kvm_debug("%s: last KVM instance, restoring TLB parameters\n", |
185 | __func__); | 185 | __func__); |
186 | on_each_cpu(kvm_mips_uninit_tlbs, NULL, 1); | 186 | on_each_cpu(kvm_mips_uninit_tlbs, NULL, 1); |
187 | } | 187 | } |
188 | } | 188 | } |
@@ -244,9 +244,8 @@ void kvm_arch_commit_memory_region(struct kvm *kvm, | |||
244 | goto out; | 244 | goto out; |
245 | } | 245 | } |
246 | 246 | ||
247 | kvm_info | 247 | kvm_debug("Allocated space for Guest PMAP Table (%ld pages) @ %p\n", |
248 | ("Allocated space for Guest PMAP Table (%ld pages) @ %p\n", | 248 | npages, kvm->arch.guest_pmap); |
249 | npages, kvm->arch.guest_pmap); | ||
250 | 249 | ||
251 | /* Now setup the page table */ | 250 | /* Now setup the page table */ |
252 | for (i = 0; i < npages; i++) { | 251 | for (i = 0; i < npages; i++) { |
@@ -291,7 +290,7 @@ struct kvm_vcpu *kvm_arch_vcpu_create(struct kvm *kvm, unsigned int id) | |||
291 | if (err) | 290 | if (err) |
292 | goto out_free_cpu; | 291 | goto out_free_cpu; |
293 | 292 | ||
294 | kvm_info("kvm @ %p: create cpu %d at %p\n", kvm, id, vcpu); | 293 | kvm_debug("kvm @ %p: create cpu %d at %p\n", kvm, id, vcpu); |
295 | 294 | ||
296 | /* Allocate space for host mode exception handlers that handle | 295 | /* Allocate space for host mode exception handlers that handle |
297 | * guest mode exits | 296 | * guest mode exits |
@@ -311,8 +310,8 @@ struct kvm_vcpu *kvm_arch_vcpu_create(struct kvm *kvm, unsigned int id) | |||
311 | err = -ENOMEM; | 310 | err = -ENOMEM; |
312 | goto out_free_cpu; | 311 | goto out_free_cpu; |
313 | } | 312 | } |
314 | kvm_info("Allocated %d bytes for KVM Exception Handlers @ %p\n", | 313 | kvm_debug("Allocated %d bytes for KVM Exception Handlers @ %p\n", |
315 | ALIGN(size, PAGE_SIZE), gebase); | 314 | ALIGN(size, PAGE_SIZE), gebase); |
316 | 315 | ||
317 | /* Save new ebase */ | 316 | /* Save new ebase */ |
318 | vcpu->arch.guest_ebase = gebase; | 317 | vcpu->arch.guest_ebase = gebase; |
@@ -337,9 +336,9 @@ struct kvm_vcpu *kvm_arch_vcpu_create(struct kvm *kvm, unsigned int id) | |||
337 | 336 | ||
338 | /* General handler, relocate to unmapped space for sanity's sake */ | 337 | /* General handler, relocate to unmapped space for sanity's sake */ |
339 | offset = 0x2000; | 338 | offset = 0x2000; |
340 | kvm_info("Installing KVM Exception handlers @ %p, %#x bytes\n", | 339 | kvm_debug("Installing KVM Exception handlers @ %p, %#x bytes\n", |
341 | gebase + offset, | 340 | gebase + offset, |
342 | mips32_GuestExceptionEnd - mips32_GuestException); | 341 | mips32_GuestExceptionEnd - mips32_GuestException); |
343 | 342 | ||
344 | memcpy(gebase + offset, mips32_GuestException, | 343 | memcpy(gebase + offset, mips32_GuestException, |
345 | mips32_GuestExceptionEnd - mips32_GuestException); | 344 | mips32_GuestExceptionEnd - mips32_GuestException); |
@@ -356,7 +355,7 @@ struct kvm_vcpu *kvm_arch_vcpu_create(struct kvm *kvm, unsigned int id) | |||
356 | goto out_free_gebase; | 355 | goto out_free_gebase; |
357 | } | 356 | } |
358 | 357 | ||
359 | kvm_info("Allocated COMM page @ %p\n", vcpu->arch.kseg0_commpage); | 358 | kvm_debug("Allocated COMM page @ %p\n", vcpu->arch.kseg0_commpage); |
360 | kvm_mips_commpage_init(vcpu); | 359 | kvm_mips_commpage_init(vcpu); |
361 | 360 | ||
362 | /* Init */ | 361 | /* Init */ |
diff --git a/arch/mips/kvm/kvm_tlb.c b/arch/mips/kvm/kvm_tlb.c index 15ad06d717fd..8a5a700ad8de 100644 --- a/arch/mips/kvm/kvm_tlb.c +++ b/arch/mips/kvm/kvm_tlb.c | |||
@@ -679,17 +679,17 @@ void kvm_arch_vcpu_load(struct kvm_vcpu *vcpu, int cpu) | |||
679 | vcpu->arch.guest_user_mm.context.asid[cpu]; | 679 | vcpu->arch.guest_user_mm.context.asid[cpu]; |
680 | newasid++; | 680 | newasid++; |
681 | 681 | ||
682 | kvm_info("[%d]: cpu_context: %#lx\n", cpu, | 682 | kvm_debug("[%d]: cpu_context: %#lx\n", cpu, |
683 | cpu_context(cpu, current->mm)); | 683 | cpu_context(cpu, current->mm)); |
684 | kvm_info("[%d]: Allocated new ASID for Guest Kernel: %#x\n", | 684 | kvm_debug("[%d]: Allocated new ASID for Guest Kernel: %#x\n", |
685 | cpu, vcpu->arch.guest_kernel_asid[cpu]); | 685 | cpu, vcpu->arch.guest_kernel_asid[cpu]); |
686 | kvm_info("[%d]: Allocated new ASID for Guest User: %#x\n", cpu, | 686 | kvm_debug("[%d]: Allocated new ASID for Guest User: %#x\n", cpu, |
687 | vcpu->arch.guest_user_asid[cpu]); | 687 | vcpu->arch.guest_user_asid[cpu]); |
688 | } | 688 | } |
689 | 689 | ||
690 | if (vcpu->arch.last_sched_cpu != cpu) { | 690 | if (vcpu->arch.last_sched_cpu != cpu) { |
691 | kvm_info("[%d->%d]KVM VCPU[%d] switch\n", | 691 | kvm_debug("[%d->%d]KVM VCPU[%d] switch\n", |
692 | vcpu->arch.last_sched_cpu, cpu, vcpu->vcpu_id); | 692 | vcpu->arch.last_sched_cpu, cpu, vcpu->vcpu_id); |
693 | /* | 693 | /* |
694 | * Migrate the timer interrupt to the current CPU so that it | 694 | * Migrate the timer interrupt to the current CPU so that it |
695 | * always interrupts the guest and synchronously triggers a | 695 | * always interrupts the guest and synchronously triggers a |