arch-arm: Fix Data Abort ISS when caused by Atomic operation
authorGiacomo Travaglini <giacomo.travaglini@arm.com>
Wed, 31 Jul 2019 08:41:48 +0000 (09:41 +0100)
committerGiacomo Travaglini <giacomo.travaglini@arm.com>
Wed, 18 Sep 2019 13:30:01 +0000 (13:30 +0000)
commita4bf7c3c20756f01f39e644d65f97fd9813502f0
treef9488099d2ccdddce14473881d82d59be82b0605
parentdc70987e470d66f584e0ddf606e9f07da994ba75
arch-arm: Fix Data Abort ISS when caused by Atomic operation

Data Aborts caused by an atomic instruction have a special rule for
their syndrome:
From a ISS point of view they count as read if a read to that address
would generate a fault; they count as writes otherwise (ISS.WnR bit)
This patch is implementing this in the TLB. For permission faults we
need to explicitly check if a read would trigger a fault
(e.g. checking for the AP bits) since permissions can allow read-only
accesses.
For other MMU exceptions (like translation faults) we are confident the
nature of the access doesn't affect the genration of a fault.
This means that if the access is atomic, we treat it as a read from an
ISS.WnR point of view.

Change-Id: Ia524aa6ae07f81513cdc26c516b5fd9b01a931c3
Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/20981
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/arch/arm/table_walker.cc
src/arch/arm/tlb.cc