On Fri, Nov 15, 2019 at 05:24:20PM +0100, Cédric Le Goater wrote: > A context should be 'valid' when pulled from the thread interrupt > context registers. > > Signed-off-by: Cédric Le Goater <c...@kaod.org>
Applied, thanks.
> ---
> hw/intc/xive.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/hw/intc/xive.c b/hw/intc/xive.c
> index 42e9a11ef731..511e1a936347 100644
> --- a/hw/intc/xive.c
> +++ b/hw/intc/xive.c
> @@ -377,6 +377,11 @@ static uint64_t xive_tm_pull_os_ctx(XiveTCTX *tctx,
> hwaddr offset,
>
> qw1w2 = xive_tctx_get_os_cam(tctx, &nvt_blk, &nvt_idx, &vo);
>
> + if (!vo) {
> + qemu_log_mask(LOG_GUEST_ERROR, "XIVE: pulling invalid NVT %x/%x
> !?\n",
> + nvt_blk, nvt_idx);
> + }
> +
> /* Invalidate CAM line */
> qw1w2_new = xive_set_field32(TM_QW1W2_VO, qw1w2, 0);
> xive_tctx_set_os_cam(tctx, qw1w2_new);
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
