[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 00/18] target/riscv: Various fixes to gdbstub and CSR access

From: Daniel Henrique Barboza
Subject: Re: [PATCH 00/18] target/riscv: Various fixes to gdbstub and CSR access
Date: Mon, 13 Feb 2023 16:19:45 -0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.1


I received only patches 1-11. I don't see the remaining patches in patchwork:


or in the qemu-devel archives:


Can you please verify? Thanks,


On 2/13/23 15:01, Bin Meng wrote:
At present gdbstub reports an incorrect / incomplete CSR list in the
target description XML, for example:

- menvcfg is reported in 'sifive_u' machine
- fcsr is missing in a F/D enabled processor

The issue is caused by:
- priv spec version check is missing when reporting CSRs
- CSR predicate() routine is called without turning on the debugger flag

This series aims to generate a correct and complete CSR list for gdbstub.

Bin Meng (18):
   target/riscv: gdbstub: Check priv spec version before reporting CSR
   target/riscv: Correct the priority policy of riscv_csrrw_check()
   target/riscv: gdbstub: Minor change for better readability
   target/riscv: gdbstub: Do not generate CSR XML if Zicsr is disabled
   target/riscv: Coding style fixes in csr.c
   target/riscv: Use 'bool' type for read_only
   target/riscv: Simplify {read,write}_pmpcfg() a little bit
   target/riscv: Simplify getting RISCVCPU pointer from env
   target/riscv: Avoid reporting odd-numbered pmpcfgX in the CSR XML for
   target/riscv: gdbstub: Turn on debugger mode before calling CSR
   target/riscv: gdbstub: Drop the vector CSRs in riscv-vector.xml
   target/riscv: Allow debugger to access user timer and counter CSRs
   target/riscv: Allow debugger to access seed CSR
   target/riscv: Allow debugger to access {h,s}stateen CSRs
   target/riscv: Allow debugger to access sstc CSRs
   target/riscv: Drop priv level check in mseccfg predicate()
   target/riscv: Group all predicate() routines together
   target/riscv: Move configuration check to envcfg CSRs predicate()

  target/riscv/csr.c     | 360 +++++++++++++++++++++++------------------
  target/riscv/gdbstub.c | 100 +++---------
  2 files changed, 221 insertions(+), 239 deletions(-)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]