Re: [PATCH rcu v2 17/20] kvm: Remove "select SRCU"

From: Paul E. McKenney
Date: Tue Jan 17 2023 - 22:56:54 EST


On Wed, Jan 18, 2023 at 01:27:36AM +0000, Sean Christopherson wrote:
> On Thu, Jan 12, 2023, Paul E. McKenney wrote:
> > Now that the SRCU Kconfig option is unconditionally selected, there is
> > no longer any point in selecting it. Therefore, remove the "select SRCU"
> > Kconfig statements from the various KVM Kconfig files.
> >
> > Signed-off-by: Paul E. McKenney <paulmck@xxxxxxxxxx>
> > Cc: Huacai Chen <chenhuacai@xxxxxxxxxx>
> > Cc: Aleksandar Markovic <aleksandar.qemu.devel@xxxxxxxxx>
> > Cc: Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx>
> > Cc: Michael Ellerman <mpe@xxxxxxxxxxxxxx>
> > Cc: Christophe Leroy <christophe.leroy@xxxxxxxxxx>
> > Cc: Sean Christopherson <seanjc@xxxxxxxxxx>
> > Cc: Paolo Bonzini <pbonzini@xxxxxxxxxx>
> > Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> > Cc: Ingo Molnar <mingo@xxxxxxxxxx>
> > Cc: Borislav Petkov <bp@xxxxxxxxx>
> > Cc: Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx>
> > Cc: "H. Peter Anvin" <hpa@xxxxxxxxx>
> > Cc: <kvm@xxxxxxxxxxxxxxx>
> > Acked-by: Marc Zyngier <maz@xxxxxxxxxx> (arm64)
> > Acked-by: Michael Ellerman <mpe@xxxxxxxxxxxxxx> (powerpc)
> > Acked-by: Anup Patel <anup@xxxxxxxxxxxxxx> (riscv)
> > Acked-by: Heiko Carstens <hca@xxxxxxxxxxxxx> (s390)
> > Reviewed-by: John Ogness <john.ogness@xxxxxxxxxxxxx>
> > ---
>
> Reviewed-by: Sean Christopherson <seanjc@xxxxxxxxxx>
>
>
> Ugh, my apologies Paul. I didn't realize 0cd7e350abc4 ("rcu: Make SRCU mandatory")
> is already in Linus' tree, i.e. bundling this in a single patch was unnecessary work
> for you.
>
> Anyways, since there isn't an external dependency, this can go through the KVM
> tree. Unless you prefer to take it directly, I'll make sure Paolo sends it along
> at some point before v6.2 final.

I am very happy to have you guys take it. Just let me know when you
have it in place, and I will drop my copy on my next rebase after that.

Thanx, Paul