Commit ba82e06c authored by Marc Zyngier's avatar Marc Zyngier Committed by Oliver Upton
Browse files

KVM: arm64: timers: Don't BUG() on unhandled timer trap 



Although not handling a trap is a pretty bad situation to be in,
panicing the kernel isn't useful and provides no valuable
information to help debugging the situation.

Instead, dump the encoding of the unhandled sysreg, and inject
an UNDEF in the guest. At least, this gives a user an opportunity
to report the issue with some information to help debugging it.

Signed-off-by: default avatarMarc Zyngier <maz@kernel.org>
Link: https://lore.kernel.org/r/20230112123829.458912-4-maz@kernel.org


Signed-off-by: default avatarOliver Upton <oliver.upton@linux.dev>
parent fc6ee952

arch/arm64/kvm/sys_regs.c

+3 −1
Original line number Diff line number Diff line
@@ -1049,7 +1049,9 @@ static bool access_arch_timer(struct kvm_vcpu *vcpu, 
		treg = TIMER_REG_CVAL;

		break;

	default:

		BUG();

		print_sys_reg_msg(p, "%s", "Unhandled trapped timer register");

		kvm_inject_undefined(vcpu);

		return false;

	}



	if (p->is_write)