[v1] pnv/xive2: Fix TIMA special ops detection

[PATCH 0/2] pnv/xive2: Fix TIMA special ops detection

Frederic Barrat posted 2 patches 10 months, 2 weeks ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230621160306.896045-1-fbarrat@linux.ibm.com

Maintainers: "Cédric Le Goater" <clg@kaod.org>

There is a newer version of this series

hw/intc/pnv_xive.c    | 11 +++++++++
hw/intc/pnv_xive2.c   | 44 +++++++++------------------------
hw/intc/xive.c        | 57 +++++++++++++++++++++++++++++++++++++------
include/hw/ppc/xive.h |  3 +++
4 files changed, 75 insertions(+), 40 deletions(-)

Expand all Fold all

[PATCH 0/2] pnv/xive2: Fix TIMA special ops detection

Posted by Frederic Barrat 10 months, 2 weeks ago

Fix the TIMA special ops detection regression, as spotted by Coverity.

Tested by running a pseries guest on top of a powernv9 and powernv10 host.

Frederic Barrat (2):
  pnv/xive2: Add a get_config() method on the presenter class
  pnv/xive2: Check TIMA special ops against a dedicated array for P10

 hw/intc/pnv_xive.c    | 11 +++++++++
 hw/intc/pnv_xive2.c   | 44 +++++++++------------------------
 hw/intc/xive.c        | 57 +++++++++++++++++++++++++++++++++++++------
 include/hw/ppc/xive.h |  3 +++
 4 files changed, 75 insertions(+), 40 deletions(-)

-- 
2.41.0

Re: [PATCH 0/2] pnv/xive2: Fix TIMA special ops detection

Posted by Cédric Le Goater 10 months, 2 weeks ago

On 6/21/23 18:03, Frederic Barrat wrote:
> Fix the TIMA special ops detection regression, as spotted by Coverity.
> 
> Tested by running a pseries guest on top of a powernv9 and powernv10 host.

FYI,

It is possible to force Gen1 on XIVE2 also. It you set the "capabilities"
property on the command line :

   -global driver=pnv-xive2,property=capabilities,value=0x1000120076f000FC

OPAL should self adapt and QEMU will probably crash ...

C.

> 
> Frederic Barrat (2):
>    pnv/xive2: Add a get_config() method on the presenter class
>    pnv/xive2: Check TIMA special ops against a dedicated array for P10
> 
>   hw/intc/pnv_xive.c    | 11 +++++++++
>   hw/intc/pnv_xive2.c   | 44 +++++++++------------------------
>   hw/intc/xive.c        | 57 +++++++++++++++++++++++++++++++++++++------
>   include/hw/ppc/xive.h |  3 +++
>   4 files changed, 75 insertions(+), 40 deletions(-)
>

Re: [PATCH 0/2] pnv/xive2: Fix TIMA special ops detection

Posted by Frederic Barrat 10 months, 2 weeks ago


On 21/06/2023 19:23, Cédric Le Goater wrote:
> FYI,
> 
> It is possible to force Gen1 on XIVE2 also. It you set the "capabilities"
> property on the command line :
> 
>    -global driver=pnv-xive2,property=capabilities,value=0x1000120076f000FC


Thanks, I hadn't noticed. And "config" too! Good to know...

   Fred