[PATCH 1/5] KVM: arm64: Add accessor for per-CPU state

[Marc Zyngier]

On Mon, 11 Mar 2024 04:50:23 +0000,
Dongli Zhang <dongli.zhang at oracle.com> wrote:
> 
> 
> 
> On 3/2/24 03:19, Marc Zyngier wrote:
> > In order to facilitate the introduction of new per-CPU state,
> > add a new host_data_ptr() helped that hides some of the per-CPU
> > verbosity, and make it easier to move that state around in the
> > future.
> > 
> > Signed-off-by: Marc Zyngier <maz at kernel.org>
> > ---
> >  arch/arm64/include/asm/kvm_host.h         | 13 +++++++++++++
> >  arch/arm64/kvm/arm.c                      |  2 +-
> >  arch/arm64/kvm/hyp/include/hyp/debug-sr.h |  4 ++--
> >  arch/arm64/kvm/hyp/include/hyp/switch.h   | 11 +++++------
> >  arch/arm64/kvm/hyp/nvhe/psci-relay.c      |  2 +-
> >  arch/arm64/kvm/hyp/nvhe/setup.c           |  3 +--
> >  arch/arm64/kvm/hyp/nvhe/switch.c          |  4 ++--
> >  arch/arm64/kvm/hyp/vhe/switch.c           |  4 ++--
> >  arch/arm64/kvm/hyp/vhe/sysreg-sr.c        |  4 ++--
> >  arch/arm64/kvm/pmu.c                      |  2 +-
> >  10 files changed, 30 insertions(+), 19 deletions(-)
> > 
> > diff --git a/arch/arm64/include/asm/kvm_host.h b/arch/arm64/include/asm/kvm_host.h
> > index 21c57b812569..3ca2a9444f21 100644
> > --- a/arch/arm64/include/asm/kvm_host.h
> > +++ b/arch/arm64/include/asm/kvm_host.h
> > @@ -492,6 +492,17 @@ struct kvm_cpu_context {
> >  	u64 *vncr_array;
> >  };
> >  
> > +/*
> > + * This structure is instanciated on a per-CPU basis, and contains
> 
> instantiated?

Yup, thanks.

> 
> 
> May this patchset (at least the first, second, third and fifth) be qualified as
> "non functional change" in the commit message?

I disagree. No change is without any functional change unless it is
only changing comments, and this one definitely introduces a bug. So I
will not make any such statement.

> That provides some hints when backporting this patchset to some old kernel in
> the future. Thank you very much!

two things:

- you should not be backporting these patches. Full stop. They don't
  fix anything, they do not enable anything. They are just preliminary
  work for future things.

- you really should perform a full review of what you are backporting
  in the light of the *target* kernel. I am never going to do this job
  (I only care about the current state), and that makes your earlier
  request even less realistic.

Thanks,

	M.

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