From: Frederic Barrat <fbarrat@linux.ibm.com>
Moving xive2_nvp_pic_print_info() align with the other "pic_print_info"
functions and allows us to call functions internal to xive2.c.
In XIVE Gen 2 there were some minor changes to the TIMA header that were
updated when printed.
Additional END state 'info pic' information as added. The 'ignore',
'crowd' and 'precluded escalation control' bits of an Event Notification
Descriptor are all used when delivering an interrupt targeting a VP-group
or crowd.
Signed-off-by: Frederic Barrat <fbarrat@linux.ibm.com>
Signed-off-by: Michael Kowal <kowal@linux.vnet.ibm.com>
---
include/hw/ppc/xive2_regs.h | 9 +++++++++
hw/intc/pnv_xive2.c | 27 ---------------------------
hw/intc/xive.c | 12 +++++++++---
hw/intc/xive2.c | 33 +++++++++++++++++++++++++++++++--
4 files changed, 49 insertions(+), 32 deletions(-)
diff --git a/include/hw/ppc/xive2_regs.h b/include/hw/ppc/xive2_regs.h
index 4e5e17cd89..4349d009d0 100644
--- a/include/hw/ppc/xive2_regs.h
+++ b/include/hw/ppc/xive2_regs.h
@@ -97,6 +97,7 @@ typedef struct Xive2End {
uint32_t w6;
#define END2_W6_FORMAT_BIT PPC_BIT32(0)
#define END2_W6_IGNORE PPC_BIT32(1)
+#define END2_W6_CROWD PPC_BIT32(2)
#define END2_W6_VP_BLOCK PPC_BITMASK32(4, 7)
#define END2_W6_VP_OFFSET PPC_BITMASK32(8, 31)
#define END2_W6_VP_OFFSET_GEN1 PPC_BITMASK32(13, 31)
@@ -111,6 +112,8 @@ typedef struct Xive2End {
#define xive2_end_is_notify(end) \
(be32_to_cpu((end)->w0) & END2_W0_UCOND_NOTIFY)
#define xive2_end_is_backlog(end) (be32_to_cpu((end)->w0) & END2_W0_BACKLOG)
+#define xive2_end_is_precluded_escalation(end) \
+ (be32_to_cpu((end)->w0) & END2_W0_PRECL_ESC_CTL)
#define xive2_end_is_escalate(end) \
(be32_to_cpu((end)->w0) & END2_W0_ESCALATE_CTL)
#define xive2_end_is_uncond_escalation(end) \
@@ -123,6 +126,10 @@ typedef struct Xive2End {
(be32_to_cpu((end)->w0) & END2_W0_FIRMWARE1)
#define xive2_end_is_firmware2(end) \
(be32_to_cpu((end)->w0) & END2_W0_FIRMWARE2)
+#define xive2_end_is_ignore(end) \
+ (be32_to_cpu((end)->w6) & END2_W6_IGNORE)
+#define xive2_end_is_crowd(end) \
+ (be32_to_cpu((end)->w6) & END2_W6_CROWD)
static inline uint64_t xive2_end_qaddr(Xive2End *end)
{
@@ -194,6 +201,8 @@ static inline uint32_t xive2_nvp_blk(uint32_t cam_line)
return (cam_line >> XIVE2_NVP_SHIFT) & 0xf;
}
+void xive2_nvp_pic_print_info(Xive2Nvp *nvp, uint32_t nvp_idx, GString *buf);
+
/*
* Notification Virtual Group or Crowd (NVG/NVC)
*/
diff --git a/hw/intc/pnv_xive2.c b/hw/intc/pnv_xive2.c
index ad0d6f45e4..7846f2ef33 100644
--- a/hw/intc/pnv_xive2.c
+++ b/hw/intc/pnv_xive2.c
@@ -2433,33 +2433,6 @@ static void pnv_xive2_register_types(void)
type_init(pnv_xive2_register_types)
-static void xive2_nvp_pic_print_info(Xive2Nvp *nvp, uint32_t nvp_idx,
- GString *buf)
-{
- uint8_t eq_blk = xive_get_field32(NVP2_W5_VP_END_BLOCK, nvp->w5);
- uint32_t eq_idx = xive_get_field32(NVP2_W5_VP_END_INDEX, nvp->w5);
-
- if (!xive2_nvp_is_valid(nvp)) {
- return;
- }
-
- g_string_append_printf(buf, " %08x end:%02x/%04x IPB:%02x",
- nvp_idx, eq_blk, eq_idx,
- xive_get_field32(NVP2_W2_IPB, nvp->w2));
- /*
- * When the NVP is HW controlled, more fields are updated
- */
- if (xive2_nvp_is_hw(nvp)) {
- g_string_append_printf(buf, " CPPR:%02x",
- xive_get_field32(NVP2_W2_CPPR, nvp->w2));
- if (xive2_nvp_is_co(nvp)) {
- g_string_append_printf(buf, " CO:%04x",
- xive_get_field32(NVP2_W1_CO_THRID, nvp->w1));
- }
- }
- g_string_append_c(buf, '\n');
-}
-
/*
* If the table is direct, we can compute the number of PQ entries
* provisioned by FW.
diff --git a/hw/intc/xive.c b/hw/intc/xive.c
index 70f11f993b..5a02dd8e02 100644
--- a/hw/intc/xive.c
+++ b/hw/intc/xive.c
@@ -692,9 +692,15 @@ void xive_tctx_pic_print_info(XiveTCTX *tctx, GString *buf)
}
}
- g_string_append_printf(buf, "CPU[%04x]: "
- "QW NSR CPPR IPB LSMFB ACK# INC AGE PIPR W2\n",
- cpu_index);
+ if (xive_presenter_get_config(tctx->xptr) & XIVE_PRESENTER_GEN1_TIMA_OS) {
+ g_string_append_printf(buf, "CPU[%04x]: "
+ "QW NSR CPPR IPB LSMFB ACK# INC AGE PIPR"
+ " W2\n", cpu_index);
+ } else {
+ g_string_append_printf(buf, "CPU[%04x]: "
+ "QW NSR CPPR IPB LSMFB - LGS T PIPR"
+ " W2\n", cpu_index);
+ }
for (i = 0; i < XIVE_TM_RING_COUNT; i++) {
char *s = xive_tctx_ring_print(&tctx->regs[i * XIVE_TM_RING_SIZE]);
diff --git a/hw/intc/xive2.c b/hw/intc/xive2.c
index 3e7238c663..1f150685bf 100644
--- a/hw/intc/xive2.c
+++ b/hw/intc/xive2.c
@@ -89,7 +89,7 @@ void xive2_end_pic_print_info(Xive2End *end, uint32_t end_idx, GString *buf)
pq = xive_get_field32(END2_W1_ESn, end->w1);
g_string_append_printf(buf,
- " %08x %c%c %c%c%c%c%c%c%c%c%c%c "
+ " %08x %c%c %c%c%c%c%c%c%c%c%c%c%c %c%c "
"prio:%d nvp:%02x/%04x",
end_idx,
pq & XIVE_ESB_VAL_P ? 'P' : '-',
@@ -98,12 +98,15 @@ void xive2_end_pic_print_info(Xive2End *end, uint32_t end_idx, GString *buf)
xive2_end_is_enqueue(end) ? 'q' : '-',
xive2_end_is_notify(end) ? 'n' : '-',
xive2_end_is_backlog(end) ? 'b' : '-',
+ xive2_end_is_precluded_escalation(end) ? 'p' : '-',
xive2_end_is_escalate(end) ? 'e' : '-',
xive2_end_is_escalate_end(end) ? 'N' : '-',
xive2_end_is_uncond_escalation(end) ? 'u' : '-',
xive2_end_is_silent_escalation(end) ? 's' : '-',
xive2_end_is_firmware1(end) ? 'f' : '-',
xive2_end_is_firmware2(end) ? 'F' : '-',
+ xive2_end_is_ignore(end) ? 'i' : '-',
+ xive2_end_is_crowd(end) ? 'c' : '-',
priority, nvp_blk, nvp_idx);
if (qaddr_base) {
@@ -137,6 +140,32 @@ void xive2_end_eas_pic_print_info(Xive2End *end, uint32_t end_idx,
(uint32_t) xive_get_field64(EAS2_END_DATA, eas->w));
}
+void xive2_nvp_pic_print_info(Xive2Nvp *nvp, uint32_t nvp_idx, GString *buf)
+{
+ uint8_t eq_blk = xive_get_field32(NVP2_W5_VP_END_BLOCK, nvp->w5);
+ uint32_t eq_idx = xive_get_field32(NVP2_W5_VP_END_INDEX, nvp->w5);
+
+ if (!xive2_nvp_is_valid(nvp)) {
+ return;
+ }
+
+ g_string_append_printf(buf, " %08x end:%02x/%04x IPB:%02x",
+ nvp_idx, eq_blk, eq_idx,
+ xive_get_field32(NVP2_W2_IPB, nvp->w2));
+ /*
+ * When the NVP is HW controlled, more fields are updated
+ */
+ if (xive2_nvp_is_hw(nvp)) {
+ g_string_append_printf(buf, " CPPR:%02x",
+ xive_get_field32(NVP2_W2_CPPR, nvp->w2));
+ if (xive2_nvp_is_co(nvp)) {
+ g_string_append_printf(buf, " CO:%04x",
+ xive_get_field32(NVP2_W1_CO_THRID, nvp->w1));
+ }
+ }
+ g_string_append_c(buf, '\n');
+}
+
static void xive2_end_enqueue(Xive2End *end, uint32_t data)
{
uint64_t qaddr_base = xive2_end_qaddr(end);
@@ -650,7 +679,7 @@ static void xive2_router_end_notify(Xive2Router *xrtr, uint8_t end_blk,
}
found = xive_presenter_notify(xrtr->xfb, format, nvp_blk, nvp_idx,
- xive_get_field32(END2_W6_IGNORE, end.w7),
+ xive2_end_is_ignore(&end),
priority,
xive_get_field32(END2_W7_F1_LOG_SERVER_ID, end.w7));
--
2.43.0On 7/15/24 20:33, Michael Kowal wrote:
> From: Frederic Barrat <fbarrat@linux.ibm.com>
>
> Moving xive2_nvp_pic_print_info() align with the other "pic_print_info"
> functions and allows us to call functions internal to xive2.c.
>
> In XIVE Gen 2 there were some minor changes to the TIMA header that were
> updated when printed.
>
> Additional END state 'info pic' information as added. The 'ignore',
> 'crowd' and 'precluded escalation control' bits of an Event Notification
> Descriptor are all used when delivering an interrupt targeting a VP-group
> or crowd.
>
> Signed-off-by: Frederic Barrat <fbarrat@linux.ibm.com>
> Signed-off-by: Michael Kowal <kowal@linux.vnet.ibm.com>
There are multiple unrelated changes in this patch.
> ---
> include/hw/ppc/xive2_regs.h | 9 +++++++++
> hw/intc/pnv_xive2.c | 27 ---------------------------
> hw/intc/xive.c | 12 +++++++++---
> hw/intc/xive2.c | 33 +++++++++++++++++++++++++++++++--
> 4 files changed, 49 insertions(+), 32 deletions(-)
>
> diff --git a/include/hw/ppc/xive2_regs.h b/include/hw/ppc/xive2_regs.h
> index 4e5e17cd89..4349d009d0 100644
> --- a/include/hw/ppc/xive2_regs.h
> +++ b/include/hw/ppc/xive2_regs.h
> @@ -97,6 +97,7 @@ typedef struct Xive2End {
> uint32_t w6;
> #define END2_W6_FORMAT_BIT PPC_BIT32(0)
> #define END2_W6_IGNORE PPC_BIT32(1)
> +#define END2_W6_CROWD PPC_BIT32(2)
> #define END2_W6_VP_BLOCK PPC_BITMASK32(4, 7)
> #define END2_W6_VP_OFFSET PPC_BITMASK32(8, 31)
> #define END2_W6_VP_OFFSET_GEN1 PPC_BITMASK32(13, 31)
> @@ -111,6 +112,8 @@ typedef struct Xive2End {
> #define xive2_end_is_notify(end) \
> (be32_to_cpu((end)->w0) & END2_W0_UCOND_NOTIFY)
> #define xive2_end_is_backlog(end) (be32_to_cpu((end)->w0) & END2_W0_BACKLOG)
> +#define xive2_end_is_precluded_escalation(end) \
> + (be32_to_cpu((end)->w0) & END2_W0_PRECL_ESC_CTL)
> #define xive2_end_is_escalate(end) \
> (be32_to_cpu((end)->w0) & END2_W0_ESCALATE_CTL)
> #define xive2_end_is_uncond_escalation(end) \
> @@ -123,6 +126,10 @@ typedef struct Xive2End {
> (be32_to_cpu((end)->w0) & END2_W0_FIRMWARE1)
> #define xive2_end_is_firmware2(end) \
> (be32_to_cpu((end)->w0) & END2_W0_FIRMWARE2)
> +#define xive2_end_is_ignore(end) \
> + (be32_to_cpu((end)->w6) & END2_W6_IGNORE)
> +#define xive2_end_is_crowd(end) \
> + (be32_to_cpu((end)->w6) & END2_W6_CROWD)
This is a first patch, plus the xive2_end_pic_print_info and
xive2_router_end_notify changes
> static inline uint64_t xive2_end_qaddr(Xive2End *end)
> {
> @@ -194,6 +201,8 @@ static inline uint32_t xive2_nvp_blk(uint32_t cam_line)
> return (cam_line >> XIVE2_NVP_SHIFT) & 0xf;
> }
>
> +void xive2_nvp_pic_print_info(Xive2Nvp *nvp, uint32_t nvp_idx, GString *buf);
> +
> /*
> * Notification Virtual Group or Crowd (NVG/NVC)
> */
> diff --git a/hw/intc/pnv_xive2.c b/hw/intc/pnv_xive2.c
> index ad0d6f45e4..7846f2ef33 100644
> --- a/hw/intc/pnv_xive2.c
> +++ b/hw/intc/pnv_xive2.c
> @@ -2433,33 +2433,6 @@ static void pnv_xive2_register_types(void)
>
> type_init(pnv_xive2_register_types)
>
> -static void xive2_nvp_pic_print_info(Xive2Nvp *nvp, uint32_t nvp_idx,
> - GString *buf)
> -{
> - uint8_t eq_blk = xive_get_field32(NVP2_W5_VP_END_BLOCK, nvp->w5);
> - uint32_t eq_idx = xive_get_field32(NVP2_W5_VP_END_INDEX, nvp->w5);
> -
> - if (!xive2_nvp_is_valid(nvp)) {
> - return;
> - }
> -
> - g_string_append_printf(buf, " %08x end:%02x/%04x IPB:%02x",
> - nvp_idx, eq_blk, eq_idx,
> - xive_get_field32(NVP2_W2_IPB, nvp->w2));
> - /*
> - * When the NVP is HW controlled, more fields are updated
> - */
> - if (xive2_nvp_is_hw(nvp)) {
> - g_string_append_printf(buf, " CPPR:%02x",
> - xive_get_field32(NVP2_W2_CPPR, nvp->w2));
> - if (xive2_nvp_is_co(nvp)) {
> - g_string_append_printf(buf, " CO:%04x",
> - xive_get_field32(NVP2_W1_CO_THRID, nvp->w1));
> - }
> - }
> - g_string_append_c(buf, '\n');
> -}
This is a second patch but I don't see where else xive2_nvp_pic_print_info()
is used ?
> /*
> * If the table is direct, we can compute the number of PQ entries
> * provisioned by FW.
> diff --git a/hw/intc/xive.c b/hw/intc/xive.c
> index 70f11f993b..5a02dd8e02 100644
> --- a/hw/intc/xive.c
> +++ b/hw/intc/xive.c
> @@ -692,9 +692,15 @@ void xive_tctx_pic_print_info(XiveTCTX *tctx, GString *buf)
> }
> }
>
> - g_string_append_printf(buf, "CPU[%04x]: "
> - "QW NSR CPPR IPB LSMFB ACK# INC AGE PIPR W2\n",
> - cpu_index);
> + if (xive_presenter_get_config(tctx->xptr) & XIVE_PRESENTER_GEN1_TIMA_OS) {
> + g_string_append_printf(buf, "CPU[%04x]: "
> + "QW NSR CPPR IPB LSMFB ACK# INC AGE PIPR"
> + " W2\n", cpu_index);
> + } else {
> + g_string_append_printf(buf, "CPU[%04x]: "
> + "QW NSR CPPR IPB LSMFB - LGS T PIPR"
> + " W2\n", cpu_index);
> + }
and this is patch 3.
Thanks,
C.
> for (i = 0; i < XIVE_TM_RING_COUNT; i++) {
> char *s = xive_tctx_ring_print(&tctx->regs[i * XIVE_TM_RING_SIZE]);
> diff --git a/hw/intc/xive2.c b/hw/intc/xive2.c
> index 3e7238c663..1f150685bf 100644
> --- a/hw/intc/xive2.c
> +++ b/hw/intc/xive2.c
> @@ -89,7 +89,7 @@ void xive2_end_pic_print_info(Xive2End *end, uint32_t end_idx, GString *buf)
> pq = xive_get_field32(END2_W1_ESn, end->w1);
>
> g_string_append_printf(buf,
> - " %08x %c%c %c%c%c%c%c%c%c%c%c%c "
> + " %08x %c%c %c%c%c%c%c%c%c%c%c%c%c %c%c "
> "prio:%d nvp:%02x/%04x",
> end_idx,
> pq & XIVE_ESB_VAL_P ? 'P' : '-',
> @@ -98,12 +98,15 @@ void xive2_end_pic_print_info(Xive2End *end, uint32_t end_idx, GString *buf)
> xive2_end_is_enqueue(end) ? 'q' : '-',
> xive2_end_is_notify(end) ? 'n' : '-',
> xive2_end_is_backlog(end) ? 'b' : '-',
> + xive2_end_is_precluded_escalation(end) ? 'p' : '-',
> xive2_end_is_escalate(end) ? 'e' : '-',
> xive2_end_is_escalate_end(end) ? 'N' : '-',
> xive2_end_is_uncond_escalation(end) ? 'u' : '-',
> xive2_end_is_silent_escalation(end) ? 's' : '-',
> xive2_end_is_firmware1(end) ? 'f' : '-',
> xive2_end_is_firmware2(end) ? 'F' : '-',
> + xive2_end_is_ignore(end) ? 'i' : '-',
> + xive2_end_is_crowd(end) ? 'c' : '-',
> priority, nvp_blk, nvp_idx);
>
> if (qaddr_base) {
> @@ -137,6 +140,32 @@ void xive2_end_eas_pic_print_info(Xive2End *end, uint32_t end_idx,
> (uint32_t) xive_get_field64(EAS2_END_DATA, eas->w));
> }
>
> +void xive2_nvp_pic_print_info(Xive2Nvp *nvp, uint32_t nvp_idx, GString *buf)
> +{
> + uint8_t eq_blk = xive_get_field32(NVP2_W5_VP_END_BLOCK, nvp->w5);
> + uint32_t eq_idx = xive_get_field32(NVP2_W5_VP_END_INDEX, nvp->w5);
> +
> + if (!xive2_nvp_is_valid(nvp)) {
> + return;
> + }
> +
> + g_string_append_printf(buf, " %08x end:%02x/%04x IPB:%02x",
> + nvp_idx, eq_blk, eq_idx,
> + xive_get_field32(NVP2_W2_IPB, nvp->w2));
> + /*
> + * When the NVP is HW controlled, more fields are updated
> + */
> + if (xive2_nvp_is_hw(nvp)) {
> + g_string_append_printf(buf, " CPPR:%02x",
> + xive_get_field32(NVP2_W2_CPPR, nvp->w2));
> + if (xive2_nvp_is_co(nvp)) {
> + g_string_append_printf(buf, " CO:%04x",
> + xive_get_field32(NVP2_W1_CO_THRID, nvp->w1));
> + }
> + }
> + g_string_append_c(buf, '\n');
> +}
> +
> static void xive2_end_enqueue(Xive2End *end, uint32_t data)
> {
> uint64_t qaddr_base = xive2_end_qaddr(end);
> @@ -650,7 +679,7 @@ static void xive2_router_end_notify(Xive2Router *xrtr, uint8_t end_blk,
> }
>
> found = xive_presenter_notify(xrtr->xfb, format, nvp_blk, nvp_idx,
> - xive_get_field32(END2_W6_IGNORE, end.w7),
> + xive2_end_is_ignore(&end),
> priority,
> xive_get_field32(END2_W7_F1_LOG_SERVER_ID, end.w7));
>
© 2016 - 2024 Red Hat, Inc.