qemu/timer: Clarify QEMUTimer new/free API

[PATCH 0/2] qemu/timer: Clarify QEMUTimer new/free API

Posted by Philippe Mathieu-Daudé 2 months, 1 week ago

Update few QEMUTimer docstring and add a
sanity check during timer initialization.

Noticed trying to understand leaks in QDev
Realize -> Unrealize -> Realize transition.

Philippe Mathieu-Daudé (2):
  qemu/timer: Clarify timer_new*() must be freed with timer_free()
  qemu/timer: Sanity check timer_list in timer_init_full()

 include/qemu/timer.h | 12 +++++++++++-
 util/qemu-timer.c    |  1 +
 2 files changed, 12 insertions(+), 1 deletion(-)

-- 
2.47.1

Re: [PATCH 0/2] qemu/timer: Clarify QEMUTimer new/free API

Posted by Philippe Mathieu-Daudé 1 month, 4 weeks ago

ping?

On 25/1/25 19:24, Philippe Mathieu-Daudé wrote:
> Update few QEMUTimer docstring and add a
> sanity check during timer initialization.
> 
> Noticed trying to understand leaks in QDev
> Realize -> Unrealize -> Realize transition.
> 
> Philippe Mathieu-Daudé (2):
>    qemu/timer: Clarify timer_new*() must be freed with timer_free()
>    qemu/timer: Sanity check timer_list in timer_init_full()
> 
>   include/qemu/timer.h | 12 +++++++++++-
>   util/qemu-timer.c    |  1 +
>   2 files changed, 12 insertions(+), 1 deletion(-)
>

Re: [PATCH 0/2] qemu/timer: Clarify QEMUTimer new/free API

Posted by Philippe Mathieu-Daudé 1 month, 3 weeks ago

Cc'ing more developers.

On 4/2/25 22:44, Philippe Mathieu-Daudé wrote:
> ping?
> 
> On 25/1/25 19:24, Philippe Mathieu-Daudé wrote:
>> Update few QEMUTimer docstring and add a
>> sanity check during timer initialization.
>>
>> Noticed trying to understand leaks in QDev
>> Realize -> Unrealize -> Realize transition.
>>
>> Philippe Mathieu-Daudé (2):
>>    qemu/timer: Clarify timer_new*() must be freed with timer_free()
>>    qemu/timer: Sanity check timer_list in timer_init_full()
>>
>>   include/qemu/timer.h | 12 +++++++++++-
>>   util/qemu-timer.c    |  1 +
>>   2 files changed, 12 insertions(+), 1 deletion(-)
>>
>