[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 19/26] hw/intc/arm_gicv3_its: Don't use data if reading comma
From: |
Alex Bennée |
Subject: |
Re: [PATCH 19/26] hw/intc/arm_gicv3_its: Don't use data if reading command failed |
Date: |
Mon, 13 Dec 2021 14:49:40 +0000 |
User-agent: |
mu4e 1.7.5; emacs 28.0.90 |
Peter Maydell <peter.maydell@linaro.org> writes:
> In process_cmdq(), we read 64 bits of the command packet, which
> contain the command identifier, which we then switch() on to dispatch
> to an appropriate sub-function. However, if address_space_ldq_le()
> reports a memory transaction failure, we still read the command
> identifier out of the data and switch() on it. Restructure the code
> so that we stop immediately (stalling the command queue) in this
> case.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
--
Alex Bennée
- Re: [PATCH 26/26] hw/intc/arm_gicv3_its: Factor out "find address of table entry" code, (continued)