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

2024-06-18 Thread Philippe Mathieu-Daudé
On 10/6/24 08:20, Philippe Mathieu-Daudé wrote: 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

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

2024-06-17 Thread Harsh Prateek Bora
On 6/10/24 11:50, Philippe Mathieu-Daudé 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

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

2024-06-13 Thread Cédric Le Goater
Thanks! As you suggested I'll wait for Nick to post the PPC pull request before respining this series (and the series depending on it). I think this patch is the only conflicting one : https://lore.kernel.org/qemu-devel/20240424093048.180966-1-...@redhat.com/ Adding it to your series

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

2024-06-13 Thread Philippe Mathieu-Daudé
On 13/6/24 14:44, Cédric Le Goater wrote: 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

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

2024-06-13 Thread Cédric Le Goater
On 6/10/24 8:20 AM, Philippe Mathieu-Daudé 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

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

2024-06-13 Thread Cédric Le Goater
On 6/13/24 11:48 AM, Philippe Mathieu-Daudé wrote: On 10/6/24 10:19, Cédric Le Goater wrote: On 6/10/24 8:20 AM, Philippe Mathieu-Daudé 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

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

2024-06-13 Thread Philippe Mathieu-Daudé
On 13/6/24 12:29, Cédric Le Goater wrote: On 6/13/24 11:48 AM, Philippe Mathieu-Daudé wrote: On 10/6/24 10:19, Cédric Le Goater wrote: On 6/10/24 8:20 AM, Philippe Mathieu-Daudé wrote: Hi, This series remove uses of Monitor in hw/ppc/, replacing by the more generic HumanReadableText. Care is

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

2024-06-13 Thread Philippe Mathieu-Daudé
On 10/6/24 10:19, Cédric Le Goater wrote: On 6/10/24 8:20 AM, Philippe Mathieu-Daudé 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.

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

2024-06-11 Thread Manos Pitsidianakis
On Mon, 10 Jun 2024 09:20, Philippe Mathieu-Daudé 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

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

2024-06-10 Thread Cédric Le Goater
On 6/10/24 8:20 AM, Philippe Mathieu-Daudé 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. Did you do any testing ? POWER[8-10] CPUs on

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

2024-06-10 Thread Cédric Le Goater
On 6/10/24 8:20 AM, Philippe Mathieu-Daudé 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

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

2024-06-10 Thread Philippe Mathieu-Daudé
On 10/6/24 08:20, Philippe Mathieu-Daudé 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:

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

2024-06-10 Thread Philippe Mathieu-Daudé
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: