Commit 0f7b791b authored Nov 06, 2018 by Peter Maydell

target/arm: Set S and PTW in 64-bit PAR format



In do_ats_write() we construct a PAR value based on the result
of the translation.  A comment says "S2WLK and FSTAGE are always
zero, because we don't implement virtualization".
Since we do in fact now implement virtualization, add the missing
code that sets these bits based on the reported ARMMMUFaultInfo.

(These bits are named PTW and S in ARMv8, so we follow that
convention in the new comments in this patch.)

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-id: 20181016093703.10637-2-peter.maydell@linaro.org

parent 76a82ced

target/arm/helper.c

+6 −4

Original line number	Diff line number	Diff line
		@@ -2347,10 +2347,12 @@ static uint64_t do_ats_write(CPUARMState *env, uint64_t value,

		par64 \|= 1; /* F */
		par64 \|= (fsr & 0x3f) << 1; /* FS */
		/* Note that S2WLK and FSTAGE are always zero, because we don't
		* implement virtualization and therefore there can't be a stage 2
		* fault.
		*/
		if (fi.stage2) {
		par64 \|= (1 << 9); /* S */
		}
		if (fi.s1ptw) {
		par64 \|= (1 << 8); /* PTW */
		}
		}
		} else {
		/* fsr is a DFSR/IFSR value for the short descriptor