hw/ppc: Avoid using Monitor in spapr_xive_pic_print_info()

Replace Monitor API by HumanReadableText one.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Reviewed-by: Harsh Prateek Bora <harshpb@linux.ibm.com>
Message-Id: <20240610062105.49848-9-philmd@linaro.org>
This commit is contained in:
Philippe Mathieu-Daudé 2024-06-07 14:48:53 +02:00
parent 950f1273ab
commit 4d62448c1c

View File

@ -157,7 +157,7 @@ static void spapr_xive_end_pic_print_info(SpaprXive *xive, XiveEND *end,
#define spapr_xive_in_kernel(xive) \ #define spapr_xive_in_kernel(xive) \
(kvm_irqchip_in_kernel() && (xive)->fd != -1) (kvm_irqchip_in_kernel() && (xive)->fd != -1)
static void spapr_xive_pic_print_info(SpaprXive *xive, Monitor *mon) static void spapr_xive_pic_print_info(SpaprXive *xive, GString *buf)
{ {
XiveSource *xsrc = &xive->source; XiveSource *xsrc = &xive->source;
int i; int i;
@ -172,7 +172,7 @@ static void spapr_xive_pic_print_info(SpaprXive *xive, Monitor *mon)
} }
} }
monitor_printf(mon, " LISN PQ EISN CPU/PRIO EQ\n"); g_string_append_printf(buf, " LISN PQ EISN CPU/PRIO EQ\n");
for (i = 0; i < xive->nr_irqs; i++) { for (i = 0; i < xive->nr_irqs; i++) {
uint8_t pq = xive_source_esb_get(xsrc, i); uint8_t pq = xive_source_esb_get(xsrc, i);
@ -182,19 +182,17 @@ static void spapr_xive_pic_print_info(SpaprXive *xive, Monitor *mon)
continue; continue;
} }
monitor_printf(mon, " %08x %s %c%c%c %s %08x ", i, g_string_append_printf(buf, " %08x %s %c%c%c %s %08x ", i,
xive_source_irq_is_lsi(xsrc, i) ? "LSI" : "MSI", xive_source_irq_is_lsi(xsrc, i) ? "LSI" : "MSI",
pq & XIVE_ESB_VAL_P ? 'P' : '-', pq & XIVE_ESB_VAL_P ? 'P' : '-',
pq & XIVE_ESB_VAL_Q ? 'Q' : '-', pq & XIVE_ESB_VAL_Q ? 'Q' : '-',
xive_source_is_asserted(xsrc, i) ? 'A' : ' ', xive_source_is_asserted(xsrc, i) ? 'A' : ' ',
xive_eas_is_masked(eas) ? "M" : " ", xive_eas_is_masked(eas) ? "M" : " ",
(int) xive_get_field64(EAS_END_DATA, eas->w)); (int) xive_get_field64(EAS_END_DATA, eas->w));
if (!xive_eas_is_masked(eas)) { if (!xive_eas_is_masked(eas)) {
uint32_t end_idx = xive_get_field64(EAS_END_INDEX, eas->w); uint32_t end_idx = xive_get_field64(EAS_END_INDEX, eas->w);
XiveEND *end; XiveEND *end;
g_autoptr(GString) buf = g_string_new("");
g_autoptr(HumanReadableText) info = NULL;
assert(end_idx < xive->nr_ends); assert(end_idx < xive->nr_ends);
end = &xive->endt[end_idx]; end = &xive->endt[end_idx];
@ -203,10 +201,8 @@ static void spapr_xive_pic_print_info(SpaprXive *xive, Monitor *mon)
spapr_xive_end_pic_print_info(xive, end, buf); spapr_xive_end_pic_print_info(xive, end, buf);
} }
info = human_readable_text_from_str(buf);
monitor_puts(mon, info->human_readable_text);
} }
monitor_printf(mon, "\n"); g_string_append_c(buf, '\n');
} }
} }
@ -717,10 +713,10 @@ static void spapr_xive_print_info(SpaprInterruptController *intc, Monitor *mon)
xive_tctx_pic_print_info(spapr_cpu_state(cpu)->tctx, buf); xive_tctx_pic_print_info(spapr_cpu_state(cpu)->tctx, buf);
} }
spapr_xive_pic_print_info(xive, buf);
info = human_readable_text_from_str(buf); info = human_readable_text_from_str(buf);
monitor_puts(mon, info->human_readable_text); monitor_puts(mon, info->human_readable_text);
spapr_xive_pic_print_info(xive, mon);
} }
static void spapr_xive_dt(SpaprInterruptController *intc, uint32_t nr_servers, static void spapr_xive_dt(SpaprInterruptController *intc, uint32_t nr_servers,