[PATCH 13/16] KVM: arm64: nv: timers: Support hyp timer emulation

Marc Zyngier maz at kernel.org
Sat Feb 25 02:34:05 PST 2023


On Fri, 24 Feb 2023 20:08:15 +0000,
Colton Lewis <coltonlewis at google.com> wrote:
> 
> I hope someone else looks at this one because I don't understand NV.

Don't worry, you're not alone.

> Marc Zyngier <maz at kernel.org> writes:
> 
> > +	struct arch_timer_context *ctx;
> > +
> > +	ctx = (vcpu_has_nv(vcpu) && is_hyp_ctxt(vcpu)) ? vcpu_hvtimer(vcpu)
> > +						       : vcpu_vtimer(vcpu);
> 
> I don't think that ternary is the easiest to read and would prefer an if
> statement.

As the saying goes, there is no accounting for taste. Do you see a
technical issue with this statement?

	M.

-- 
Without deviation from the norm, progress is not possible.



More information about the linux-arm-kernel mailing list