| Message ID | 20240909211038.27440-10-kowal@linux.ibm.com |
|---|---|
| State | New |
| Headers | show |
| Series | XIVE2 changes for TIMA operations | expand |
On 9/9/24 23:10, Michael Kowal wrote: > Some the functions that have been created are specific to a ring or context. Some > of these same functions are being changed to operate on any ring/context. This will > simplify the next patch sets that are adding additional ring/context operations. > > Signed-off-by: Michael Kowal <kowal@linux.ibm.com> > --- > include/hw/ppc/xive.h | 2 +- > hw/intc/xive.c | 6 ++-- > hw/intc/xive2.c | 77 ++++++++++++++++++++++++++++--------------- > 3 files changed, 55 insertions(+), 30 deletions(-) > > diff --git a/include/hw/ppc/xive.h b/include/hw/ppc/xive.h > index 28c181faa2..31242f0406 100644 > --- a/include/hw/ppc/xive.h > +++ b/include/hw/ppc/xive.h > @@ -533,7 +533,7 @@ Object *xive_tctx_create(Object *cpu, XivePresenter *xptr, Error **errp); > void xive_tctx_reset(XiveTCTX *tctx); > void xive_tctx_destroy(XiveTCTX *tctx); > void xive_tctx_ipb_update(XiveTCTX *tctx, uint8_t ring, uint8_t ipb); > -void xive_tctx_reset_os_signal(XiveTCTX *tctx); > +void xive_tctx_reset_signal(XiveTCTX *tctx, uint8_t ring); > > /* > * KVM XIVE device helpers > diff --git a/hw/intc/xive.c b/hw/intc/xive.c > index 5b66a3aec5..f1d007d9a6 100644 > --- a/hw/intc/xive.c > +++ b/hw/intc/xive.c > @@ -114,7 +114,7 @@ static void xive_tctx_notify(XiveTCTX *tctx, uint8_t ring) > } > } > > -void xive_tctx_reset_os_signal(XiveTCTX *tctx) > +void xive_tctx_reset_signal(XiveTCTX *tctx, uint8_t ring) > { > /* > * Lower the External interrupt. Used when pulling an OS > @@ -122,7 +122,7 @@ void xive_tctx_reset_os_signal(XiveTCTX *tctx) > * context. It should be raised again when re-pushing the OS > * context. > */ > - qemu_irq_lower(xive_tctx_output(tctx, TM_QW1_OS)); > + qemu_irq_lower(xive_tctx_output(tctx, ring)); > } > > static void xive_tctx_set_cppr(XiveTCTX *tctx, uint8_t ring, uint8_t cppr) > @@ -424,7 +424,7 @@ static uint64_t xive_tm_pull_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, > qw1w2_new = xive_set_field32(TM_QW1W2_VO, qw1w2, 0); > xive_tctx_set_os_cam(tctx, qw1w2_new); > > - xive_tctx_reset_os_signal(tctx); > + xive_tctx_reset_signal(tctx, TM_QW1_OS); > return qw1w2; > } > > diff --git a/hw/intc/xive2.c b/hw/intc/xive2.c > index 8d3d69a0db..38a4b0ff68 100644 > --- a/hw/intc/xive2.c > +++ b/hw/intc/xive2.c > @@ -270,13 +270,14 @@ static void xive2_end_enqueue(Xive2End *end, uint32_t data) > * the NVP by changing the H bit while the context is enabled > */ > > -static void xive2_tctx_save_os_ctx(Xive2Router *xrtr, XiveTCTX *tctx, > - uint8_t nvp_blk, uint32_t nvp_idx) > +static void xive2_tctx_save_ctx(Xive2Router *xrtr, XiveTCTX *tctx, > + uint8_t nvp_blk, uint32_t nvp_idx, > + uint8_t ring) > { > CPUPPCState *env = &POWERPC_CPU(tctx->cs)->env; > uint32_t pir = env->spr_cb[SPR_PIR].default_value; > Xive2Nvp nvp; > - uint8_t *regs = &tctx->regs[TM_QW1_OS]; > + uint8_t *regs = &tctx->regs[ring]; > > if (xive2_router_get_nvp(xrtr, nvp_blk, nvp_idx, &nvp)) { > qemu_log_mask(LOG_GUEST_ERROR, "XIVE: No NVP %x/%x\n", > @@ -321,13 +322,13 @@ static void xive2_tctx_save_os_ctx(Xive2Router *xrtr, XiveTCTX *tctx, > xive2_router_write_nvp(xrtr, nvp_blk, nvp_idx, &nvp, 1); > } > > -static void xive2_os_cam_decode(uint32_t cam, uint8_t *nvp_blk, > - uint32_t *nvp_idx, bool *vo, bool *ho) > +static void xive2_cam_decode(uint32_t cam, uint8_t *nvp_blk, > + uint32_t *nvp_idx, bool *valid, bool *hw) > { > *nvp_blk = xive2_nvp_blk(cam); > *nvp_idx = xive2_nvp_idx(cam); > - *vo = !!(cam & TM2_QW1W2_VO); > - *ho = !!(cam & TM2_QW1W2_HO); > + *valid = !!(cam & TM2_QW1W2_VO); > + *hw = !!(cam & TM2_QW1W2_HO); > } > > > @@ -351,37 +352,54 @@ static uint32_t xive2_tctx_hw_cam_line(XivePresenter *xptr, XiveTCTX *tctx) > return xive2_nvp_cam_line(blk, 1 << tid_shift | (pir & tid_mask)); > } > > -uint64_t xive2_tm_pull_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, > - hwaddr offset, unsigned size) > +static uint64_t xive2_tm_pull_ctx(XivePresenter *xptr, XiveTCTX *tctx, > + hwaddr offset, unsigned size, uint8_t ring) The changes below are not strictly adding a ring parameter. Some belong to another patch. Thanks, C. > { > Xive2Router *xrtr = XIVE2_ROUTER(xptr); > - uint32_t qw1w2 = xive_tctx_word2(&tctx->regs[TM_QW1_OS]); > - uint32_t qw1w2_new; > - uint32_t cam = be32_to_cpu(qw1w2); > + uint32_t target_ringw2 = xive_tctx_word2(&tctx->regs[ring]); > + uint32_t cam = be32_to_cpu(target_ringw2); > uint8_t nvp_blk; > uint32_t nvp_idx; > - bool vo; > + uint8_t cur_ring; > + bool valid; > bool do_save; > > - xive2_os_cam_decode(cam, &nvp_blk, &nvp_idx, &vo, &do_save); > + xive2_cam_decode(cam, &nvp_blk, &nvp_idx, &valid, &do_save); > > - if (!vo) { > + if (!valid) { > qemu_log_mask(LOG_GUEST_ERROR, "XIVE: pulling invalid NVP %x/%x !?\n", > nvp_blk, nvp_idx); > } > > - /* Invalidate CAM line */ > - qw1w2_new = xive_set_field32(TM2_QW1W2_VO, qw1w2, 0); > - memcpy(&tctx->regs[TM_QW1_OS + TM_WORD2], &qw1w2_new, 4); > + /* Invalidate CAM line of requested ring and all lower rings */ > + for (cur_ring = TM_QW0_USER; cur_ring <= ring; > + cur_ring += XIVE_TM_RING_SIZE) { > + uint32_t ringw2 = xive_tctx_word2(&tctx->regs[cur_ring]); > + uint32_t ringw2_new = xive_set_field32(TM2_QW1W2_VO, ringw2, 0); > + memcpy(&tctx->regs[cur_ring + TM_WORD2], &ringw2_new, 4); > + } > > if (xive2_router_get_config(xrtr) & XIVE2_VP_SAVE_RESTORE && do_save) { > - xive2_tctx_save_os_ctx(xrtr, tctx, nvp_blk, nvp_idx); > + xive2_tctx_save_ctx(xrtr, tctx, nvp_blk, nvp_idx, ring); > } > > - xive_tctx_reset_os_signal(tctx); > - return qw1w2; > + /* > + * Lower external interrupt line of requested ring and below except for > + * USER, which doesn't exist. > + */ > + for (cur_ring = TM_QW1_OS; cur_ring <= ring; > + cur_ring += XIVE_TM_RING_SIZE) { > + xive_tctx_reset_signal(tctx, cur_ring); > + } > + return target_ringw2; > } > > +uint64_t xive2_tm_pull_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, > + hwaddr offset, unsigned size) > +{ > + return xive2_tm_pull_ctx(xptr, tctx, offset, size, TM_QW1_OS); > + } > + > #define REPORT_LINE_GEN1_SIZE 16 > > static void xive2_tm_report_line_gen1(XiveTCTX *tctx, uint8_t *data, > @@ -424,8 +442,9 @@ static void xive2_tm_report_line_gen1(XiveTCTX *tctx, uint8_t *data, > } > } > > -void xive2_tm_pull_os_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, > - hwaddr offset, uint64_t value, unsigned size) > +static void xive2_tm_pull_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, > + hwaddr offset, uint64_t value, > + unsigned size, uint8_t ring) > { > Xive2Router *xrtr = XIVE2_ROUTER(xptr); > uint32_t hw_cam, nvp_idx, xive2_cfg, reserved; > @@ -473,8 +492,14 @@ void xive2_tm_pull_os_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, > assert(result == MEMTX_OK); > } > > - /* the rest is similar to pull OS context to registers */ > - xive2_tm_pull_os_ctx(xptr, tctx, offset, size); > + /* the rest is similar to pull context to registers */ > + xive2_tm_pull_ctx(xptr, tctx, offset, size, ring); > +} > + > +void xive2_tm_pull_os_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, > + hwaddr offset, uint64_t value, unsigned size) > +{ > + xive2_tm_pull_ctx_ol(xptr, tctx, offset, value, size, TM_QW1_OS); > } > > static uint8_t xive2_tctx_restore_os_ctx(Xive2Router *xrtr, XiveTCTX *tctx, > @@ -573,7 +598,7 @@ void xive2_tm_push_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, > bool vo; > bool do_restore; > > - xive2_os_cam_decode(cam, &nvp_blk, &nvp_idx, &vo, &do_restore); > + xive2_cam_decode(cam, &nvp_blk, &nvp_idx, &vo, &do_restore); > > /* First update the thead context */ > memcpy(&tctx->regs[TM_QW1_OS + TM_WORD2], &qw1w2, 4);
On 9/12/2024 1:27 AM, Cédric Le Goater wrote: > On 9/9/24 23:10, Michael Kowal wrote: >> Some the functions that have been created are specific to a ring or >> context. Some >> of these same functions are being changed to operate on any >> ring/context. This will >> simplify the next patch sets that are adding additional ring/context >> operations. >> >> Signed-off-by: Michael Kowal <kowal@linux.ibm.com> >> --- >> include/hw/ppc/xive.h | 2 +- >> hw/intc/xive.c | 6 ++-- >> hw/intc/xive2.c | 77 ++++++++++++++++++++++++++++--------------- >> 3 files changed, 55 insertions(+), 30 deletions(-) >> >> diff --git a/include/hw/ppc/xive.h b/include/hw/ppc/xive.h >> index 28c181faa2..31242f0406 100644 >> --- a/include/hw/ppc/xive.h >> +++ b/include/hw/ppc/xive.h >> @@ -533,7 +533,7 @@ Object *xive_tctx_create(Object *cpu, >> XivePresenter *xptr, Error **errp); >> void xive_tctx_reset(XiveTCTX *tctx); >> void xive_tctx_destroy(XiveTCTX *tctx); >> void xive_tctx_ipb_update(XiveTCTX *tctx, uint8_t ring, uint8_t ipb); >> -void xive_tctx_reset_os_signal(XiveTCTX *tctx); >> +void xive_tctx_reset_signal(XiveTCTX *tctx, uint8_t ring); >> /* >> * KVM XIVE device helpers >> diff --git a/hw/intc/xive.c b/hw/intc/xive.c >> index 5b66a3aec5..f1d007d9a6 100644 >> --- a/hw/intc/xive.c >> +++ b/hw/intc/xive.c >> @@ -114,7 +114,7 @@ static void xive_tctx_notify(XiveTCTX *tctx, >> uint8_t ring) >> } >> } >> -void xive_tctx_reset_os_signal(XiveTCTX *tctx) >> +void xive_tctx_reset_signal(XiveTCTX *tctx, uint8_t ring) >> { >> /* >> * Lower the External interrupt. Used when pulling an OS >> @@ -122,7 +122,7 @@ void xive_tctx_reset_os_signal(XiveTCTX *tctx) >> * context. It should be raised again when re-pushing the OS >> * context. >> */ >> - qemu_irq_lower(xive_tctx_output(tctx, TM_QW1_OS)); >> + qemu_irq_lower(xive_tctx_output(tctx, ring)); >> } >> static void xive_tctx_set_cppr(XiveTCTX *tctx, uint8_t ring, >> uint8_t cppr) >> @@ -424,7 +424,7 @@ static uint64_t xive_tm_pull_os_ctx(XivePresenter >> *xptr, XiveTCTX *tctx, >> qw1w2_new = xive_set_field32(TM_QW1W2_VO, qw1w2, 0); >> xive_tctx_set_os_cam(tctx, qw1w2_new); >> - xive_tctx_reset_os_signal(tctx); >> + xive_tctx_reset_signal(tctx, TM_QW1_OS); >> return qw1w2; >> } >> diff --git a/hw/intc/xive2.c b/hw/intc/xive2.c >> index 8d3d69a0db..38a4b0ff68 100644 >> --- a/hw/intc/xive2.c >> +++ b/hw/intc/xive2.c >> @@ -270,13 +270,14 @@ static void xive2_end_enqueue(Xive2End *end, >> uint32_t data) >> * the NVP by changing the H bit while the context is enabled >> */ >> -static void xive2_tctx_save_os_ctx(Xive2Router *xrtr, XiveTCTX *tctx, >> - uint8_t nvp_blk, uint32_t nvp_idx) >> +static void xive2_tctx_save_ctx(Xive2Router *xrtr, XiveTCTX *tctx, >> + uint8_t nvp_blk, uint32_t nvp_idx, >> + uint8_t ring) >> { >> CPUPPCState *env = &POWERPC_CPU(tctx->cs)->env; >> uint32_t pir = env->spr_cb[SPR_PIR].default_value; >> Xive2Nvp nvp; >> - uint8_t *regs = &tctx->regs[TM_QW1_OS]; >> + uint8_t *regs = &tctx->regs[ring]; >> if (xive2_router_get_nvp(xrtr, nvp_blk, nvp_idx, &nvp)) { >> qemu_log_mask(LOG_GUEST_ERROR, "XIVE: No NVP %x/%x\n", >> @@ -321,13 +322,13 @@ static void xive2_tctx_save_os_ctx(Xive2Router >> *xrtr, XiveTCTX *tctx, >> xive2_router_write_nvp(xrtr, nvp_blk, nvp_idx, &nvp, 1); >> } >> -static void xive2_os_cam_decode(uint32_t cam, uint8_t *nvp_blk, >> - uint32_t *nvp_idx, bool *vo, bool *ho) >> +static void xive2_cam_decode(uint32_t cam, uint8_t *nvp_blk, >> + uint32_t *nvp_idx, bool *valid, bool *hw) >> { >> *nvp_blk = xive2_nvp_blk(cam); >> *nvp_idx = xive2_nvp_idx(cam); >> - *vo = !!(cam & TM2_QW1W2_VO); >> - *ho = !!(cam & TM2_QW1W2_HO); >> + *valid = !!(cam & TM2_QW1W2_VO); >> + *hw = !!(cam & TM2_QW1W2_HO); >> } >> @@ -351,37 +352,54 @@ static uint32_t >> xive2_tctx_hw_cam_line(XivePresenter *xptr, XiveTCTX *tctx) >> return xive2_nvp_cam_line(blk, 1 << tid_shift | (pir & tid_mask)); >> } >> -uint64_t xive2_tm_pull_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, >> - hwaddr offset, unsigned size) >> +static uint64_t xive2_tm_pull_ctx(XivePresenter *xptr, XiveTCTX *tctx, >> + hwaddr offset, unsigned size, >> uint8_t ring) > > The changes below are not strictly adding a ring parameter. Some belong > to another patch. > > > Thanks, > > C. > The two patch sets go together and the functions added the ring parm so I included those changes here. I can move some of the changes from here to patch set 10. MAK > >> { >> Xive2Router *xrtr = XIVE2_ROUTER(xptr); >> - uint32_t qw1w2 = xive_tctx_word2(&tctx->regs[TM_QW1_OS]); >> - uint32_t qw1w2_new; >> - uint32_t cam = be32_to_cpu(qw1w2); >> + uint32_t target_ringw2 = xive_tctx_word2(&tctx->regs[ring]); >> + uint32_t cam = be32_to_cpu(target_ringw2); >> uint8_t nvp_blk; >> uint32_t nvp_idx; >> - bool vo; >> + uint8_t cur_ring; >> + bool valid; >> bool do_save; >> - xive2_os_cam_decode(cam, &nvp_blk, &nvp_idx, &vo, &do_save); >> + xive2_cam_decode(cam, &nvp_blk, &nvp_idx, &valid, &do_save); >> - if (!vo) { >> + if (!valid) { >> qemu_log_mask(LOG_GUEST_ERROR, "XIVE: pulling invalid NVP >> %x/%x !?\n", >> nvp_blk, nvp_idx); >> } >> - /* Invalidate CAM line */ >> - qw1w2_new = xive_set_field32(TM2_QW1W2_VO, qw1w2, 0); >> - memcpy(&tctx->regs[TM_QW1_OS + TM_WORD2], &qw1w2_new, 4); >> + /* Invalidate CAM line of requested ring and all lower rings */ >> + for (cur_ring = TM_QW0_USER; cur_ring <= ring; >> + cur_ring += XIVE_TM_RING_SIZE) { >> + uint32_t ringw2 = xive_tctx_word2(&tctx->regs[cur_ring]); >> + uint32_t ringw2_new = xive_set_field32(TM2_QW1W2_VO, ringw2, >> 0); >> + memcpy(&tctx->regs[cur_ring + TM_WORD2], &ringw2_new, 4); >> + } >> if (xive2_router_get_config(xrtr) & XIVE2_VP_SAVE_RESTORE && >> do_save) { >> - xive2_tctx_save_os_ctx(xrtr, tctx, nvp_blk, nvp_idx); >> + xive2_tctx_save_ctx(xrtr, tctx, nvp_blk, nvp_idx, ring); >> } >> - xive_tctx_reset_os_signal(tctx); >> - return qw1w2; >> + /* >> + * Lower external interrupt line of requested ring and below >> except for >> + * USER, which doesn't exist. >> + */ >> + for (cur_ring = TM_QW1_OS; cur_ring <= ring; >> + cur_ring += XIVE_TM_RING_SIZE) { >> + xive_tctx_reset_signal(tctx, cur_ring); >> + } >> + return target_ringw2; >> } >> +uint64_t xive2_tm_pull_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, >> + hwaddr offset, unsigned size) >> +{ >> + return xive2_tm_pull_ctx(xptr, tctx, offset, size, TM_QW1_OS); >> + } >> + >> #define REPORT_LINE_GEN1_SIZE 16 >> static void xive2_tm_report_line_gen1(XiveTCTX *tctx, uint8_t *data, >> @@ -424,8 +442,9 @@ static void xive2_tm_report_line_gen1(XiveTCTX >> *tctx, uint8_t *data, >> } >> } >> -void xive2_tm_pull_os_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, >> - hwaddr offset, uint64_t value, unsigned >> size) >> +static void xive2_tm_pull_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, >> + hwaddr offset, uint64_t value, >> + unsigned size, uint8_t ring) >> { >> Xive2Router *xrtr = XIVE2_ROUTER(xptr); >> uint32_t hw_cam, nvp_idx, xive2_cfg, reserved; >> @@ -473,8 +492,14 @@ void xive2_tm_pull_os_ctx_ol(XivePresenter >> *xptr, XiveTCTX *tctx, >> assert(result == MEMTX_OK); >> } >> - /* the rest is similar to pull OS context to registers */ >> - xive2_tm_pull_os_ctx(xptr, tctx, offset, size); >> + /* the rest is similar to pull context to registers */ >> + xive2_tm_pull_ctx(xptr, tctx, offset, size, ring); >> +} >> + >> +void xive2_tm_pull_os_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, >> + hwaddr offset, uint64_t value, unsigned >> size) >> +{ >> + xive2_tm_pull_ctx_ol(xptr, tctx, offset, value, size, TM_QW1_OS); >> } >> static uint8_t xive2_tctx_restore_os_ctx(Xive2Router *xrtr, >> XiveTCTX *tctx, >> @@ -573,7 +598,7 @@ void xive2_tm_push_os_ctx(XivePresenter *xptr, >> XiveTCTX *tctx, >> bool vo; >> bool do_restore; >> - xive2_os_cam_decode(cam, &nvp_blk, &nvp_idx, &vo, &do_restore); >> + xive2_cam_decode(cam, &nvp_blk, &nvp_idx, &vo, &do_restore); >> /* First update the thead context */ >> memcpy(&tctx->regs[TM_QW1_OS + TM_WORD2], &qw1w2, 4); >
diff --git a/include/hw/ppc/xive.h b/include/hw/ppc/xive.h index 28c181faa2..31242f0406 100644 --- a/include/hw/ppc/xive.h +++ b/include/hw/ppc/xive.h @@ -533,7 +533,7 @@ Object *xive_tctx_create(Object *cpu, XivePresenter *xptr, Error **errp); void xive_tctx_reset(XiveTCTX *tctx); void xive_tctx_destroy(XiveTCTX *tctx); void xive_tctx_ipb_update(XiveTCTX *tctx, uint8_t ring, uint8_t ipb); -void xive_tctx_reset_os_signal(XiveTCTX *tctx); +void xive_tctx_reset_signal(XiveTCTX *tctx, uint8_t ring); /* * KVM XIVE device helpers diff --git a/hw/intc/xive.c b/hw/intc/xive.c index 5b66a3aec5..f1d007d9a6 100644 --- a/hw/intc/xive.c +++ b/hw/intc/xive.c @@ -114,7 +114,7 @@ static void xive_tctx_notify(XiveTCTX *tctx, uint8_t ring) } } -void xive_tctx_reset_os_signal(XiveTCTX *tctx) +void xive_tctx_reset_signal(XiveTCTX *tctx, uint8_t ring) { /* * Lower the External interrupt. Used when pulling an OS @@ -122,7 +122,7 @@ void xive_tctx_reset_os_signal(XiveTCTX *tctx) * context. It should be raised again when re-pushing the OS * context. */ - qemu_irq_lower(xive_tctx_output(tctx, TM_QW1_OS)); + qemu_irq_lower(xive_tctx_output(tctx, ring)); } static void xive_tctx_set_cppr(XiveTCTX *tctx, uint8_t ring, uint8_t cppr) @@ -424,7 +424,7 @@ static uint64_t xive_tm_pull_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, qw1w2_new = xive_set_field32(TM_QW1W2_VO, qw1w2, 0); xive_tctx_set_os_cam(tctx, qw1w2_new); - xive_tctx_reset_os_signal(tctx); + xive_tctx_reset_signal(tctx, TM_QW1_OS); return qw1w2; } diff --git a/hw/intc/xive2.c b/hw/intc/xive2.c index 8d3d69a0db..38a4b0ff68 100644 --- a/hw/intc/xive2.c +++ b/hw/intc/xive2.c @@ -270,13 +270,14 @@ static void xive2_end_enqueue(Xive2End *end, uint32_t data) * the NVP by changing the H bit while the context is enabled */ -static void xive2_tctx_save_os_ctx(Xive2Router *xrtr, XiveTCTX *tctx, - uint8_t nvp_blk, uint32_t nvp_idx) +static void xive2_tctx_save_ctx(Xive2Router *xrtr, XiveTCTX *tctx, + uint8_t nvp_blk, uint32_t nvp_idx, + uint8_t ring) { CPUPPCState *env = &POWERPC_CPU(tctx->cs)->env; uint32_t pir = env->spr_cb[SPR_PIR].default_value; Xive2Nvp nvp; - uint8_t *regs = &tctx->regs[TM_QW1_OS]; + uint8_t *regs = &tctx->regs[ring]; if (xive2_router_get_nvp(xrtr, nvp_blk, nvp_idx, &nvp)) { qemu_log_mask(LOG_GUEST_ERROR, "XIVE: No NVP %x/%x\n", @@ -321,13 +322,13 @@ static void xive2_tctx_save_os_ctx(Xive2Router *xrtr, XiveTCTX *tctx, xive2_router_write_nvp(xrtr, nvp_blk, nvp_idx, &nvp, 1); } -static void xive2_os_cam_decode(uint32_t cam, uint8_t *nvp_blk, - uint32_t *nvp_idx, bool *vo, bool *ho) +static void xive2_cam_decode(uint32_t cam, uint8_t *nvp_blk, + uint32_t *nvp_idx, bool *valid, bool *hw) { *nvp_blk = xive2_nvp_blk(cam); *nvp_idx = xive2_nvp_idx(cam); - *vo = !!(cam & TM2_QW1W2_VO); - *ho = !!(cam & TM2_QW1W2_HO); + *valid = !!(cam & TM2_QW1W2_VO); + *hw = !!(cam & TM2_QW1W2_HO); } @@ -351,37 +352,54 @@ static uint32_t xive2_tctx_hw_cam_line(XivePresenter *xptr, XiveTCTX *tctx) return xive2_nvp_cam_line(blk, 1 << tid_shift | (pir & tid_mask)); } -uint64_t xive2_tm_pull_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, - hwaddr offset, unsigned size) +static uint64_t xive2_tm_pull_ctx(XivePresenter *xptr, XiveTCTX *tctx, + hwaddr offset, unsigned size, uint8_t ring) { Xive2Router *xrtr = XIVE2_ROUTER(xptr); - uint32_t qw1w2 = xive_tctx_word2(&tctx->regs[TM_QW1_OS]); - uint32_t qw1w2_new; - uint32_t cam = be32_to_cpu(qw1w2); + uint32_t target_ringw2 = xive_tctx_word2(&tctx->regs[ring]); + uint32_t cam = be32_to_cpu(target_ringw2); uint8_t nvp_blk; uint32_t nvp_idx; - bool vo; + uint8_t cur_ring; + bool valid; bool do_save; - xive2_os_cam_decode(cam, &nvp_blk, &nvp_idx, &vo, &do_save); + xive2_cam_decode(cam, &nvp_blk, &nvp_idx, &valid, &do_save); - if (!vo) { + if (!valid) { qemu_log_mask(LOG_GUEST_ERROR, "XIVE: pulling invalid NVP %x/%x !?\n", nvp_blk, nvp_idx); } - /* Invalidate CAM line */ - qw1w2_new = xive_set_field32(TM2_QW1W2_VO, qw1w2, 0); - memcpy(&tctx->regs[TM_QW1_OS + TM_WORD2], &qw1w2_new, 4); + /* Invalidate CAM line of requested ring and all lower rings */ + for (cur_ring = TM_QW0_USER; cur_ring <= ring; + cur_ring += XIVE_TM_RING_SIZE) { + uint32_t ringw2 = xive_tctx_word2(&tctx->regs[cur_ring]); + uint32_t ringw2_new = xive_set_field32(TM2_QW1W2_VO, ringw2, 0); + memcpy(&tctx->regs[cur_ring + TM_WORD2], &ringw2_new, 4); + } if (xive2_router_get_config(xrtr) & XIVE2_VP_SAVE_RESTORE && do_save) { - xive2_tctx_save_os_ctx(xrtr, tctx, nvp_blk, nvp_idx); + xive2_tctx_save_ctx(xrtr, tctx, nvp_blk, nvp_idx, ring); } - xive_tctx_reset_os_signal(tctx); - return qw1w2; + /* + * Lower external interrupt line of requested ring and below except for + * USER, which doesn't exist. + */ + for (cur_ring = TM_QW1_OS; cur_ring <= ring; + cur_ring += XIVE_TM_RING_SIZE) { + xive_tctx_reset_signal(tctx, cur_ring); + } + return target_ringw2; } +uint64_t xive2_tm_pull_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, + hwaddr offset, unsigned size) +{ + return xive2_tm_pull_ctx(xptr, tctx, offset, size, TM_QW1_OS); + } + #define REPORT_LINE_GEN1_SIZE 16 static void xive2_tm_report_line_gen1(XiveTCTX *tctx, uint8_t *data, @@ -424,8 +442,9 @@ static void xive2_tm_report_line_gen1(XiveTCTX *tctx, uint8_t *data, } } -void xive2_tm_pull_os_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, - hwaddr offset, uint64_t value, unsigned size) +static void xive2_tm_pull_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, + hwaddr offset, uint64_t value, + unsigned size, uint8_t ring) { Xive2Router *xrtr = XIVE2_ROUTER(xptr); uint32_t hw_cam, nvp_idx, xive2_cfg, reserved; @@ -473,8 +492,14 @@ void xive2_tm_pull_os_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, assert(result == MEMTX_OK); } - /* the rest is similar to pull OS context to registers */ - xive2_tm_pull_os_ctx(xptr, tctx, offset, size); + /* the rest is similar to pull context to registers */ + xive2_tm_pull_ctx(xptr, tctx, offset, size, ring); +} + +void xive2_tm_pull_os_ctx_ol(XivePresenter *xptr, XiveTCTX *tctx, + hwaddr offset, uint64_t value, unsigned size) +{ + xive2_tm_pull_ctx_ol(xptr, tctx, offset, value, size, TM_QW1_OS); } static uint8_t xive2_tctx_restore_os_ctx(Xive2Router *xrtr, XiveTCTX *tctx, @@ -573,7 +598,7 @@ void xive2_tm_push_os_ctx(XivePresenter *xptr, XiveTCTX *tctx, bool vo; bool do_restore; - xive2_os_cam_decode(cam, &nvp_blk, &nvp_idx, &vo, &do_restore); + xive2_cam_decode(cam, &nvp_blk, &nvp_idx, &vo, &do_restore); /* First update the thead context */ memcpy(&tctx->regs[TM_QW1_OS + TM_WORD2], &qw1w2, 4);
Some the functions that have been created are specific to a ring or context. Some of these same functions are being changed to operate on any ring/context. This will simplify the next patch sets that are adding additional ring/context operations. Signed-off-by: Michael Kowal <kowal@linux.ibm.com> --- include/hw/ppc/xive.h | 2 +- hw/intc/xive.c | 6 ++-- hw/intc/xive2.c | 77 ++++++++++++++++++++++++++++--------------- 3 files changed, 55 insertions(+), 30 deletions(-)