Re: [PATCH v2] target/riscv: Allocate itrigger timers only once

2023-08-31 Thread Alistair Francis
On Fri, Aug 18, 2023 at 1:42 PM Akihiko Odaki wrote: > > riscv_trigger_init() had been called on reset events that can happen > several times for a CPU and it allocated timers for itrigger. If old > timers were present, they were simply overwritten by the new timers, > resulting in a memory leak.

[PATCH v2] target/riscv: Allocate itrigger timers only once

2023-08-17 Thread Akihiko Odaki
riscv_trigger_init() had been called on reset events that can happen several times for a CPU and it allocated timers for itrigger. If old timers were present, they were simply overwritten by the new timers, resulting in a memory leak. Divide riscv_trigger_init() into two functions, namely riscv_tr