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

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

Frederic Barrat posted 2 patches 10 months, 2 weeks ago

Diff against v1
Download series mbox

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

Maintainers: "Cédric Le Goater" <clg@kaod.org>, Daniel Henrique Barboza <danielhb413@gmail.com>, David Gibson <david@gibson.dropbear.id.au>, Greg Kurz <groug@kaod.org>, Harsh Prateek Bora <harshpb@linux.ibm.com>

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

Expand all Fold all

[PATCH v2 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.

Changelog:
v2: define get_config for spapr


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/spapr_xive.c  | 16 ++++++++++++
 hw/intc/xive.c        | 57 +++++++++++++++++++++++++++++++++++++------
 include/hw/ppc/xive.h |  3 +++
 5 files changed, 91 insertions(+), 40 deletions(-)

-- 
2.41.0

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

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

On 6/22/23 18:25, 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.
> 
> Changelog:
> v2: define get_config for spapr
> 
> 
> 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/spapr_xive.c  | 16 ++++++++++++
>   hw/intc/xive.c        | 57 +++++++++++++++++++++++++++++++++++++------
>   include/hw/ppc/xive.h |  3 +++
>   5 files changed, 91 insertions(+), 40 deletions(-)
> 

Applied to ppc-next.

Thanks,

C.