diff options
| author | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2017-11-21 20:32:30 +1100 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2017-11-28 20:39:07 -0600 |
| commit | 7db8a9a31649d970922e4d78f9c65d9922ba8700 (patch) | |
| tree | 88ece27dcac759843b0bc1f30c8e727a54847071 /hw/xive.c | |
| parent | 7d385d1b942c6080d05548d2ab319e7af640f735 (diff) | |
| download | skiboot-7db8a9a31649d970922e4d78f9c65d9922ba8700.zip skiboot-7db8a9a31649d970922e4d78f9c65d9922ba8700.tar.gz skiboot-7db8a9a31649d970922e4d78f9c65d9922ba8700.tar.bz2 | |
xive: Don't bother cleaning up disabled EQs in reset
Additionally, warn if we find an enabled one that isn't one
of the firmware built-in queues.
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hw/xive.c')
| -rw-r--r-- | hw/xive.c | 13 |
1 files changed, 9 insertions, 4 deletions
@@ -4494,10 +4494,15 @@ static void xive_reset_one(struct xive *x) * we will incorrectly free the EQs that are reserved * for the physical CPUs */ - eq0 = *eq; - xive_cleanup_eq(&eq0); - xive_eqc_cache_update(x, x->block_id, - idx, 0, 4, &eq0, false, true); + if (eq->w0 & EQ_W0_VALID) { + if (!(eq->w0 & EQ_W0_FIRMWARE)) + xive_dbg(x, "EQ 0x%x:0x%x is valid at reset: %08x %08x\n", + x->block_id, idx, eq->w0, eq->w1); + eq0 = *eq; + xive_cleanup_eq(&eq0); + xive_eqc_cache_update(x, x->block_id, + idx, 0, 4, &eq0, false, true); + } if (eq->w0 & EQ_W0_FIRMWARE) eq_firmware = true; } |