On Fri, 22 Feb 2019 08:49:42 +0000
Julien Thierry <[email protected]> wrote:
> Hi Shaokun,
>
> On 22/02/2019 06:34, Shaokun Zhang wrote:
> > The 'timer' local variable became unused after commit bee038a67487
> > ("KVM: arm/arm64: Rework the timer code to use a timer_map").
> > Remove it to avoid [-Wunused-but-set-variable] warning.
> >
> > Cc: Christoffer Dall <[email protected]>
> > Cc: Marc Zyngier <[email protected]>
> > Cc: James Morse <[email protected]>
> > Cc: Julien Thierry <[email protected]>
> > Cc: Suzuki K Pouloze <[email protected]>
> > Signed-off-by: Shaokun Zhang <[email protected]>
> > ---
> > virt/kvm/arm/arch_timer.c | 2 --
> > 1 file changed, 2 deletions(-)
> >
> > diff --git a/virt/kvm/arm/arch_timer.c b/virt/kvm/arm/arch_timer.c
> > index af8f2f1d01cc..3417f2dbc366 100644
> > --- a/virt/kvm/arm/arch_timer.c
> > +++ b/virt/kvm/arm/arch_timer.c
> > @@ -236,14 +236,12 @@ static enum hrtimer_restart kvm_hrtimer_expire(struct
> > hrtimer *hrt)
> >
> > static bool kvm_timer_should_fire(struct arch_timer_context *timer_ctx)
> > {
> > - struct arch_timer_cpu *timer;
> > enum kvm_arch_timers index;
> > u64 cval, now;
> >
> > if (!timer_ctx)
> > return false;
> >
> > - timer = vcpu_timer(timer_ctx->vcpu);
> > index = arch_timer_ctx_index(timer_ctx);
> >
> > if (timer_ctx->loaded) {
>
> This seems to be a consequence of moving the loaded state from
> arch_timer_cpu to arch_timer_context. Thanks for fixing it.
>
> Reviewed-by: Julien Thierry <[email protected]>
Applied to -next.
Thanks,
M.
--
Without deviation from the norm, progress is not possible.
_______________________________________________
kvmarm mailing list
[email protected]
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
