Re: [Qemu-devel] [PATCH 0/4] qemu-timer: introduce usable pointer-free API

[Fam Zheng <famz@redhat.com>]

On Thu, 01/08 11:03, Paolo Bonzini wrote:
> The current pointer free API for timers is very low level.  Introduce
> a new API that matches timer_new_ns/us/ms and also a new API timer_deinit
> that can be used instead of timer_free.
> 
> Finally, mechanically change timer macro names in vmstate, to make it
> clear that they accept a pointer.
> 
> This prepares for conversion of timers from QEMUTimer * to QEMUTimer.

Novice question: what's the advantage, please?

Fam

> 
> Paolo
> 
> Paolo Bonzini (4):
>   qemu-timer: rename timer_init to timer_init_tl
>   qemu-timer: add timer_init and timer_init_ns/us/ms
>   qemu-timer: introduce timer_deinit
>   vmstate: accept QEMUTimer in VMSTATE_TIMER*, add VMSTATE_TIMER_PTR*
> 
>  hw/acpi/ich9.c              |  2 +-
>  hw/acpi/piix4.c             |  2 +-
>  hw/arm/stellaris.c          |  2 +-
>  hw/block/fdc.c              |  2 +-
>  hw/char/cadence_uart.c      |  2 +-
>  hw/char/serial.c            |  4 +-
>  hw/core/ptimer.c            |  2 +-
>  hw/dma/pl330.c              |  2 +-
>  hw/input/lm832x.c           |  2 +-
>  hw/intc/armv7m_nvic.c       |  2 +-
>  hw/isa/vt82c686.c           |  2 +-
>  hw/misc/macio/cuda.c        |  2 +-
>  hw/net/pcnet.c              |  2 +-
>  hw/sd/sdhci.c               |  4 +-
>  hw/timer/a9gtimer.c         |  2 +-
>  hw/timer/arm_mptimer.c      |  2 +-
>  hw/timer/hpet.c             |  2 +-
>  hw/timer/mc146818rtc.c      |  4 +-
>  hw/usb/hcd-ehci.c           |  2 +-
>  hw/usb/hcd-ohci.c           |  2 +-
>  hw/usb/hcd-uhci.c           |  2 +-
>  hw/usb/hcd-xhci.c           |  2 +-
>  hw/usb/redirect.c           |  2 +-
>  hw/watchdog/wdt_i6300esb.c  |  2 +-
>  hw/watchdog/wdt_ib700.c     |  2 +-
>  include/block/aio.h         |  2 +-
>  include/migration/vmstate.h | 18 +++++++--
>  include/qemu/timer.h        | 94 ++++++++++++++++++++++++++++++++++++++++++---
>  qemu-timer.c                | 20 +++++++---
>  target-arm/machine.c        |  4 +-
>  30 files changed, 149 insertions(+), 45 deletions(-)
> 
> -- 
> 2.1.0
> 
>