On 08/21/2018 03:35 PM, Alexander Graf wrote:
On 10/23/2017 06:11 PM, Marc Zyngier wrote:
The only case where we actually need to perform a dcache maintenance
is when we map the page for the first time, and subsequent permission
faults do not require cache maintenance. Let's make it conditional
on not being a permission fault (and thus a translation fault).
Reviewed-by: Christoffer Dall <christoffer.dall@xxxxxxxxxx>
Signed-off-by: Marc Zyngier <marc.zyngier@xxxxxxx>
This patch unfortunately breaks something on Hi1616 SoCs when running
32bit guests. With this patch applied (and thus with 4.18) I get
random illegal instruction warnings from 32bit code inside VMs. I do
not know at this point whether this affects other CPUs as well.
If anyone is interested in a reproducer, I have something handy. But
for now I believe we should just revert this patch.
Ok, I'm slightly confused. The patch in question is already upstream
since 4.16, but the regression reportedly came with the switch from 4.17
to 4.18. I'll try to bisect it down a bit further ...
Alex
