Re: [PATCH 00/26] hw/ppc: Prefer HumanReadableText over Monitor

[Manos Pitsidianakis <manos.pitsidianakis@linaro.org>]

On Mon, 10 Jun 2024 09:20, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>Hi,
>
>This series remove uses of Monitor in hw/ppc/,
>replacing by the more generic HumanReadableText.
>Care is taken to keep the commit bisectables by
>updating functions one by one, also easing review.
>
>For rationale see previous series from Daniel:
>https://lore.kernel.org/qemu-devel/20211028155457.967291-1-berrange@redhat.com/
>
>Regards,
>
>Phil.
>
>Philippe Mathieu-Daudé (26):
>  hw/ppc: Avoid using Monitor in pnv_phb3_msi_pic_print_info()
>  hw/ppc: Avoid using Monitor in icp_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive_tctx_pic_print_info()
>  hw/ppc: Avoid using Monitor in ics_pic_print_info()
>  hw/ppc: Avoid using Monitor in PnvChipClass::intc_print_info()
>  hw/ppc: Avoid using Monitor in xive_end_queue_pic_print_info()
>  hw/ppc: Avoid using Monitor in spapr_xive_end_pic_print_info()
>  hw/ppc: Avoid using Monitor in spapr_xive_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive_source_pic_print_info()
>  hw/ppc: Avoid using Monitor in pnv_phb4_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive_eas_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive_end_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive_end_eas_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive_nvt_pic_print_info()
>  hw/ppc: Avoid using Monitor in pnv_xive_pic_print_info()
>  hw/ppc: Avoid using Monitor in pnv_psi_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive2_eas_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive2_end_eas_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive2_end_queue_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive2_end_pic_print_info()
>  hw/ppc: Avoid using Monitor in xive2_nvp_pic_print_info()
>  hw/ppc: Avoid using Monitor in pnv_xive2_pic_print_info()
>  hw/ppc: Avoid using Monitor in
>    SpaprInterruptControllerClass::print_info()
>  hw/ppc: Avoid using Monitor in spapr_irq_print_info()
>  hw/ppc: Avoid using Monitor in pnv_chip_power9_pic_print_info_child()
>  hw/ppc: Avoid using Monitor in pic_print_info()
>
> include/hw/pci-host/pnv_phb3.h |   2 +-
> include/hw/pci-host/pnv_phb4.h |   2 +-
> include/hw/ppc/pnv_chip.h      |   4 +-
> include/hw/ppc/pnv_psi.h       |   2 +-
> include/hw/ppc/pnv_xive.h      |   4 +-
> include/hw/ppc/spapr_irq.h     |   4 +-
> include/hw/ppc/xics.h          |   4 +-
> include/hw/ppc/xive.h          |   4 +-
> include/hw/ppc/xive2_regs.h    |   8 +--
> include/hw/ppc/xive_regs.h     |   8 +--
> hw/intc/pnv_xive.c             |  38 ++++++------
> hw/intc/pnv_xive2.c            |  48 +++++++--------
> hw/intc/spapr_xive.c           |  41 ++++++-------
> hw/intc/xics.c                 |  25 ++++----
> hw/intc/xics_spapr.c           |   7 +--
> hw/intc/xive.c                 | 108 ++++++++++++++++-----------------
> hw/intc/xive2.c                |  87 +++++++++++++-------------
> hw/pci-host/pnv_phb3_msi.c     |  21 +++----
> hw/pci-host/pnv_phb4.c         |  17 +++---
> hw/ppc/pnv.c                   |  52 ++++++++--------
> hw/ppc/pnv_psi.c               |   9 ++-
> hw/ppc/spapr.c                 |  11 +++-
> hw/ppc/spapr_irq.c             |   4 +-
> 23 files changed, 256 insertions(+), 254 deletions(-)
>
>-- 
>2.41.0

For the series:

Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>