[PATCHv2 0/4] target/arm: fix missing exception class

Jamie Iles posted 4 patches 2 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/next-importer-push tags/patchew/20210526121847.1935454-1-jamie@nuviainc.com
Maintainers: Peter Maydell <peter.maydell@linaro.org>
target/arm/m_helper.c   |  5 +----
target/arm/mte_helper.c | 11 ++---------
target/arm/op_helper.c  | 28 +++++++---------------------
3 files changed, 10 insertions(+), 34 deletions(-)
[PATCHv2 0/4] target/arm: fix missing exception class
Posted by Jamie Iles 2 years, 10 months ago
Thanks Peter for the suggestions, I also added a patch to switch a 
couple of cpu_restore_state+raise_exception pairs in stack limit 
exception handling for both v7m and v8m.

v2:
 - fix raise_exception_ra to restore state before raising exception
 - remove redundant do_raise_exception
 - remove now redundant open coded raise_exception_ra from MTE and stack 
   limit exception handling

Jamie Iles (4):
  target/arm: fix missing exception class
  target/arm: fold do_raise_exception into raise_exception
  target/arm: use raise_exception_ra for MTE check failure
  target/arm: use raise_exception_ra for stack limit exception

 target/arm/m_helper.c   |  5 +----
 target/arm/mte_helper.c | 11 ++---------
 target/arm/op_helper.c  | 28 +++++++---------------------
 3 files changed, 10 insertions(+), 34 deletions(-)

-- 
2.30.2


Re: [PATCHv2 0/4] target/arm: fix missing exception class
Posted by Peter Maydell 2 years, 10 months ago
On Wed, 26 May 2021 at 13:24, Jamie Iles <jamie@nuviainc.com> wrote:
>
> Thanks Peter for the suggestions, I also added a patch to switch a
> couple of cpu_restore_state+raise_exception pairs in stack limit
> exception handling for both v7m and v8m.
>
> v2:
>  - fix raise_exception_ra to restore state before raising exception
>  - remove redundant do_raise_exception
>  - remove now redundant open coded raise_exception_ra from MTE and stack
>    limit exception handling

Applied to target-arm.next, thanks. (I tweaked a couple of comments
and commit messages.)

-- PMM