target/arm: Fix aarch64 illegal opcode exceptions

[Qemu-devel] [PATCH 0/2] target/arm: Fix aarch64 illegal opcode exceptions

Richard Henderson posted 2 patches 6 years, 2 months ago

Download series mbox

Test docker-clang@ubuntu passed

Test FreeBSD passed

Test checkpatch passed

Test docker-mingw@fedora passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190826151536.6771-1-richard.henderson@linaro.org

Maintainers: Peter Maydell <peter.maydell@linaro.org>

target/arm/translate-a64.h | 2 ++
target/arm/translate.h     | 2 --
target/arm/translate-a64.c | 7 +++++++
target/arm/translate.c     | 2 +-
4 files changed, 10 insertions(+), 3 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/2] target/arm: Fix aarch64 illegal opcode exceptions

Posted by Richard Henderson 6 years, 2 months ago

As Laurent pointed out, gen_exception_insn is very different
between aarch64 and aarch32, so we shouldn't accidentally
redirect one to the other.


r~


Richard Henderson (2):
  Revert "target/arm: Use unallocated_encoding for aarch32"
  target/arm: Factor out unallocated_encoding for aarch32

 target/arm/translate-a64.h | 2 ++
 target/arm/translate.h     | 2 --
 target/arm/translate-a64.c | 7 +++++++
 target/arm/translate.c     | 2 +-
 4 files changed, 10 insertions(+), 3 deletions(-)

-- 
2.17.1

Re: [Qemu-devel] [PATCH 0/2] target/arm: Fix aarch64 illegal opcode exceptions

Posted by Peter Maydell 6 years, 2 months ago

On Mon, 26 Aug 2019 at 16:15, Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> As Laurent pointed out, gen_exception_insn is very different
> between aarch64 and aarch32, so we shouldn't accidentally
> redirect one to the other.
>
>
> r~
>
>
> Richard Henderson (2):
>   Revert "target/arm: Use unallocated_encoding for aarch32"
>   target/arm: Factor out unallocated_encoding for aarch32



Applied to target-arm.next, thanks.

-- PMM