Commit 440523b9 authored by Marc Zyngier's avatar Marc Zyngier
Browse files

KVM: arm64: vgic: Demote userspace-triggered console prints to kvm_debug() 



Running the KVM selftests results in these messages being dumped
in the kernel console:

[  188.051073] kvm [469]: VGIC redist and dist frames overlap
[  188.056820] kvm [469]: VGIC redist and dist frames overlap
[  188.076199] kvm [469]: VGIC redist and dist frames overlap

Being amle to trigger this from userspace is definitely not on,
so demote these warnings to kvm_debug().

Signed-off-by: default avatarMarc Zyngier <maz@kernel.org>
Link: https://lore.kernel.org/r/20211216104507.1482017-1-maz@kernel.org
parent c95b1d7c

arch/arm64/kvm/vgic/vgic-v2.c

+2 −2
Original line number Diff line number Diff line
@@ -293,12 +293,12 @@ int vgic_v2_map_resources(struct kvm *kvm) 


	if (IS_VGIC_ADDR_UNDEF(dist->vgic_dist_base) ||

	    IS_VGIC_ADDR_UNDEF(dist->vgic_cpu_base)) {

		kvm_err("Need to set vgic cpu and dist addresses first\n");

		kvm_debug("Need to set vgic cpu and dist addresses first\n");

		return -ENXIO;

	}



	if (!vgic_v2_check_base(dist->vgic_dist_base, dist->vgic_cpu_base)) {

		kvm_err("VGIC CPU and dist frames overlap\n");

		kvm_debug("VGIC CPU and dist frames overlap\n");

		return -EINVAL;

	}

arch/arm64/kvm/vgic/vgic-v3.c

+2 −2
Original line number Diff line number Diff line
@@ -554,12 +554,12 @@ int vgic_v3_map_resources(struct kvm *kvm) 
	}



	if (IS_VGIC_ADDR_UNDEF(dist->vgic_dist_base)) {

		kvm_err("Need to set vgic distributor addresses first\n");

		kvm_debug("Need to set vgic distributor addresses first\n");

		return -ENXIO;

	}



	if (!vgic_v3_check_base(kvm)) {

		kvm_err("VGIC redist and dist frames overlap\n");

		kvm_debug("VGIC redist and dist frames overlap\n");

		return -EINVAL;

	}