[PATCH 1/2] spapr: Add SPAPR_CAP_AIL_MODE_3 for AIL mode 3 support for H_SET_MODE hcall

2022-02-14 Thread Nicholas Piggin
The behaviour of the Address Translation Mode on Interrupt resource is not consistently supported by all CPU versions or all KVM versions. In particular KVM HV only supports mode 0 on POWER7 processors and some early POWER9 processors, and does not support mode 2 anywhere. KVM PR only supports mode

Re: [PATCH 1/2] spapr: Add SPAPR_CAP_AIL_MODE_3 for AIL mode 3 support for H_SET_MODE hcall

2022-02-14 Thread David Gibson
On Mon, Feb 14, 2022 at 09:17:48PM +1000, Nicholas Piggin wrote: > The behaviour of the Address Translation Mode on Interrupt resource is > not consistently supported by all CPU versions or all KVM versions. In > particular KVM HV only supports mode 0 on POWER7 processors and some > early POWER9 pr

Re: [PATCH 1/2] spapr: Add SPAPR_CAP_AIL_MODE_3 for AIL mode 3 support for H_SET_MODE hcall

2022-02-15 Thread Nicholas Piggin
Excerpts from David Gibson's message of February 15, 2022 11:10 am: > On Mon, Feb 14, 2022 at 09:17:48PM +1000, Nicholas Piggin wrote: >> diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c >> index 222c1b6bbd..5dec056796 100644 >> --- a/hw/ppc/spapr_hcall.c >> +++ b/hw/ppc/spapr_hcall.c >> @@

Re: [PATCH 1/2] spapr: Add SPAPR_CAP_AIL_MODE_3 for AIL mode 3 support for H_SET_MODE hcall

2022-02-15 Thread David Gibson
On Wed, Feb 16, 2022 at 11:50:34AM +1000, Nicholas Piggin wrote: > Excerpts from David Gibson's message of February 15, 2022 11:10 am: > > On Mon, Feb 14, 2022 at 09:17:48PM +1000, Nicholas Piggin wrote: > >> diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c > >> index 222c1b6bbd..5dec056796

Re: [PATCH 1/2] spapr: Add SPAPR_CAP_AIL_MODE_3 for AIL mode 3 support for H_SET_MODE hcall

2022-02-16 Thread Cédric Le Goater
If we do, we should probably change the default value for this cap based on cpu model in default_caps_with_cpu(). We allegedly still support POWER7 KVM in Linux. I've never tested it and I don't know how much it's used at all. Probably should keep it working here if possible. I'll look into defa