On 17/05/26, Philippe Mathieu-Daudé wrote: > On 15/5/26 20:04, Anton Johansson wrote: > > Fix to 64 bits to hold all relevant values. > > > > Signed-off-by: Anton Johansson <[email protected]> > > Reviewed-by: Pierrick Bouvier <[email protected]> > > Reviewed-by: Philippe Mathieu-Daudé <[email protected]> > > Acked-by: Alistair Francis <[email protected]> > > --- > > target/riscv/cpu.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h > > index e9d3bed6f9..269bc07e2d 100644 > > --- a/target/riscv/cpu.h > > +++ b/target/riscv/cpu.h > > @@ -224,7 +224,7 @@ typedef struct PMUCTRState { > > /* Snapshot value of a counter */ > > uint64_t mhpmcounter_prev; > > /* Value beyond UINT32_MAX/UINT64_MAX before overflow interrupt > > trigger */ > > Should we update the comment?
Right, I'll do that: "Value beyond INT64_MAX ..." seems correct to me. -- Anton Johansson rev.ng Labs Srl.
