[0/2] target/riscv: add traces for exceptions

Message

Daniel Henrique Barboza Dec. 19, 2024, 5:46 p.m. UTC

Hi,

Let's add trace capabilities in riscv_raise_exception() to allow users
of qemu-riscv(32/64) to have a little more information when a SIGILL
occurs. This is done in patch 2.

Patch 1 is a "look and feel" patch that I figured was worth doing.

Daniel Henrique Barboza (2):
  target/riscv: use RISCVException enum in exception helpers
  target/riscv: add trace in riscv_raise_exception()

 target/riscv/cpu.h        | 3 ++-
 target/riscv/op_helper.c  | 9 ++++++++-
 target/riscv/trace-events | 3 +++
 target/riscv/translate.c  | 2 +-
 4 files changed, 14 insertions(+), 3 deletions(-)

Comments

Alistair Francis Jan. 6, 2025, 3:36 a.m. UTC | #1

On Fri, Dec 20, 2024 at 3:47 AM Daniel Henrique Barboza
<[e-mail]> wrote:
>
> Hi,
>
> Let's add trace capabilities in riscv_raise_exception() to allow users
> of qemu-riscv(32/64) to have a little more information when a SIGILL
> occurs. This is done in patch 2.
>
> Patch 1 is a "look and feel" patch that I figured was worth doing.
>
> Daniel Henrique Barboza (2):
>   target/riscv: use RISCVException enum in exception helpers
>   target/riscv: add trace in riscv_raise_exception()

Do you mind rebasing on
https://github.com/alistair23/qemu/tree/riscv-to-apply.next ?

Alistair

>
>  target/riscv/cpu.h        | 3 ++-
>  target/riscv/op_helper.c  | 9 ++++++++-
>  target/riscv/trace-events | 3 +++
>  target/riscv/translate.c  | 2 +-
>  4 files changed, 14 insertions(+), 3 deletions(-)
>
> --
> 2.47.1
>
>