Re: [PATCH] hvf: arm: Handle ID_AA64ISAR2_EL1 reads

2022-02-08 Thread Peter Maydell
On Mon, 7 Feb 2022 at 22:52, Alexander Graf wrote: > > Recent Linux versions added support to read ID_AA64ISAR2_EL1. On M1, > those reads trap into QEMU which handles them as faults. > > However, according to the ARMv8 spec (issue D17783), reads on this > register in older ARMv8 revisions should b

Re: [PATCH] hvf: arm: Handle ID_AA64ISAR2_EL1 reads

2022-02-07 Thread Ivan Babrou
The patch addresses the current issue for me, thanks! Is it possible to make it more future proof? I can imagine a very similar situation arising in the future and it would be good to be able to handle it gracefully. If it's not possible, then maybe there's a way to output some sort of error from

Re: [PATCH] hvf: arm: Handle ID_AA64ISAR2_EL1 reads

2022-02-07 Thread Cameron Esfahani
Reviewed-by: Cameron Esfahani mailto:di...@apple.com>> Cameron > On Feb 7, 2022, at 2:52 PM, Alexander Graf wrote: > > Recent Linux versions added support to read ID_AA64ISAR2_EL1. On M1, > those reads trap into QEMU which handles them as faults. > > However, according to the ARMv8 spec (issue

[PATCH] hvf: arm: Handle ID_AA64ISAR2_EL1 reads

2022-02-07 Thread Alexander Graf
Recent Linux versions added support to read ID_AA64ISAR2_EL1. On M1, those reads trap into QEMU which handles them as faults. However, according to the ARMv8 spec (issue D17783), reads on this register in older ARMv8 revisions should be RES0. So let's treat it as such instead. Reported-by: Ivan B