Re: [PATCH] target/arm: fix missing exception class

2021-05-24 Thread Jamie Iles
Hi Peter, On Mon, May 24, 2021 at 10:41:58AM +0100, Peter Maydell wrote: > On Mon, 24 May 2021 at 09:42, Jamie Iles wrote: > > > > The DAIF and PAC checks used raise_exception_ra to raise an exception > > and unwind CPU state but raise_exception_ra is currently designed for > > handling data abor

[PATCH] target/arm: fix missing exception class

2021-05-24 Thread Jamie Iles
The DAIF and PAC checks used raise_exception_ra to raise an exception and unwind CPU state but raise_exception_ra is currently designed for handling data aborts as the syndrome is partially precomputed and encoded in the TB and then merged in merge_syn_data_abort when handling the data abort. Usin

Re: [PATCH] target/arm: fix missing exception class

2021-05-24 Thread Peter Maydell
On Mon, 24 May 2021 at 13:36, Jamie Iles wrote: > On Mon, May 24, 2021 at 10:41:58AM +0100, Peter Maydell wrote: > > raise_exception() and raise_exception_ra() are supposed to have > > the same semantics apart from one of them being passed a return > > address. So perhaps we should look at trying

Re: [PATCH] target/arm: fix missing exception class

2021-05-24 Thread Peter Maydell
On Mon, 24 May 2021 at 09:42, Jamie Iles wrote: > > The DAIF and PAC checks used raise_exception_ra to raise an exception > and unwind CPU state but raise_exception_ra is currently designed for > handling data aborts as the syndrome is partially precomputed and > encoded in the TB and then merged