[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC] [PATCHv7 01/22] aio / timers: Add qemu_clock_free
From: |
liu ping fan |
Subject: |
Re: [Qemu-devel] [RFC] [PATCHv7 01/22] aio / timers: Add qemu_clock_free and expose qemu_clock_new and clock types |
Date: |
Thu, 8 Aug 2013 14:21:34 +0800 |
On Wed, Aug 7, 2013 at 7:48 AM, Alex Bligh <address@hidden> wrote:
> Add qemu_clock_free and expose qemu_new_clock as qemu_clock_new.
>
> Expose clock types.
>
> Signed-off-by: Alex Bligh <address@hidden>
> ---
> include/qemu/timer.h | 22 ++++++++++++++++++++++
> qemu-timer.c | 11 ++++++-----
> 2 files changed, 28 insertions(+), 5 deletions(-)
>
> diff --git a/include/qemu/timer.h b/include/qemu/timer.h
> index 9dd206c..3dd20bf 100644
> --- a/include/qemu/timer.h
> +++ b/include/qemu/timer.h
> @@ -11,6 +11,10 @@
> #define SCALE_US 1000
> #define SCALE_NS 1
>
> +#define QEMU_CLOCK_REALTIME 0
> +#define QEMU_CLOCK_VIRTUAL 1
> +#define QEMU_CLOCK_HOST 2
> +
> typedef struct QEMUClock QEMUClock;
> typedef void QEMUTimerCB(void *opaque);
>
> @@ -32,6 +36,24 @@ extern QEMUClock *vm_clock;
> the virtual clock. */
> extern QEMUClock *host_clock;
>
> +/**
> + * qemu_clock_new:
> + * @type: the type of clock to be created
> + *
> + * Create a new clock of type @type
> + *
> + * Returns: A pointer to the clock object
> + */
> +QEMUClock *qemu_clock_new(int type);
> +
> +/**
> + * qemu_clock_free:
> + * @clock: the clock to free
> + *
> + * Free clock object @clock
> + */
> +void qemu_clock_free(QEMUClock *clock);
Go through this series, I can not find the caller of it. I think as
clock source, the QEMUClock can not be destroyed, except exit(2), so
could we let exit to reclaim the resource? Or at least, this interface
should not be public.
> +
> int64_t qemu_get_clock_ns(QEMUClock *clock);
> int64_t qemu_clock_has_timers(QEMUClock *clock);
> int64_t qemu_clock_expired(QEMUClock *clock);
> diff --git a/qemu-timer.c b/qemu-timer.c
> index b2d95e2..531aa05 100644
> --- a/qemu-timer.c
> +++ b/qemu-timer.c
> @@ -40,10 +40,6 @@
> /***********************************************************/
> /* timers */
>
> -#define QEMU_CLOCK_REALTIME 0
> -#define QEMU_CLOCK_VIRTUAL 1
> -#define QEMU_CLOCK_HOST 2
> -
> struct QEMUClock {
> QEMUTimer *active_timers;
>
> @@ -231,7 +227,7 @@ QEMUClock *rt_clock;
> QEMUClock *vm_clock;
> QEMUClock *host_clock;
>
> -static QEMUClock *qemu_new_clock(int type)
> +QEMUClock *qemu_clock_new(int type)
> {
> QEMUClock *clock;
>
> @@ -243,6 +239,11 @@ static QEMUClock *qemu_new_clock(int type)
> return clock;
> }
>
> +void qemu_clock_free(QEMUClock *clock)
> +{
> + g_free(clock);
> +}
> +
> void qemu_clock_enable(QEMUClock *clock, bool enabled)
> {
> bool old = clock->enabled;
> --
> 1.7.9.5
>
- [Qemu-devel] [RFC] [PATCHv7 00/22] aio / timers: Add AioContext timers and use ppoll, Alex Bligh, 2013/08/06
- [Qemu-devel] [RFC] [PATCHv7 02/22] aio / timers: add qemu-timer.c utility functions, Alex Bligh, 2013/08/06
- [Qemu-devel] [RFC] [PATCHv7 03/22] aio / timers: Consistent treatment of disabled clocks for deadlines, Alex Bligh, 2013/08/06
- [Qemu-devel] [RFC] [PATCHv7 04/22] aio / timers: add ppoll support with qemu_poll_ns, Alex Bligh, 2013/08/06
- [Qemu-devel] [RFC] [PATCHv7 05/22] aio / timers: Add prctl(PR_SET_TIMERSLACK, 1, ...) to reduce timer slack, Alex Bligh, 2013/08/06
- [Qemu-devel] [RFC] [PATCHv7 06/22] aio / timers: Make qemu_run_timers and qemu_run_all_timers return progress, Alex Bligh, 2013/08/06
- [Qemu-devel] [RFC] [PATCHv7 09/22] aio / timers: Add QEMUTimerListGroup and helper functions, Alex Bligh, 2013/08/06
- [Qemu-devel] [RFC] [PATCHv7 10/22] aio / timers: Add QEMUTimerListGroup to AioContext, Alex Bligh, 2013/08/06
- [Qemu-devel] [RFC] [PATCHv7 08/22] aio / timers: Untangle include files, Alex Bligh, 2013/08/06
- [Qemu-devel] [RFC] [PATCHv7 12/22] aio / timers: aio_ctx_prepare sets timeout from AioContext timers, Alex Bligh, 2013/08/06