Re: [PATCH v3 1/2] target/riscv/debug.c: use wp size = 4 for 32-bit CPUs

2025-01-28 Thread Alistair Francis
On Wed, Jan 22, 2025 at 3:07 AM Daniel Henrique Barboza wrote: > > The mcontrol select bit (19) is always zero, meaning our triggers will > always match virtual addresses. In this condition, if the user does not > specify a size for the trigger, the access size defaults to XLEN. > > At this moment

Re: [PATCH v3 1/2] target/riscv/debug.c: use wp size = 4 for 32-bit CPUs

2025-01-21 Thread Philippe Mathieu-Daudé
On 21/1/25 18:06, Daniel Henrique Barboza wrote: The mcontrol select bit (19) is always zero, meaning our triggers will always match virtual addresses. In this condition, if the user does not specify a size for the trigger, the access size defaults to XLEN. At this moment we're using def_size =

[PATCH v3 1/2] target/riscv/debug.c: use wp size = 4 for 32-bit CPUs

2025-01-21 Thread Daniel Henrique Barboza
The mcontrol select bit (19) is always zero, meaning our triggers will always match virtual addresses. In this condition, if the user does not specify a size for the trigger, the access size defaults to XLEN. At this moment we're using def_size = 8 regardless of CPU XLEN. Use def_size = 4 in case