[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 01/15] coroutine: add co_sleep_ns() coroutine
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v5 01/15] coroutine: add co_sleep_ns() coroutine sleep function |
Date: |
Tue, 17 Jan 2012 14:38:48 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111115 Thunderbird/8.0 |
Am 17.01.2012 14:31, schrieb Stefan Hajnoczi:
> On Tue, Jan 17, 2012 at 12:54 PM, Kevin Wolf <address@hidden> wrote:
>> Am 13.01.2012 14:14, schrieb Stefan Hajnoczi:
>>> Signed-off-by: Stefan Hajnoczi <address@hidden>
>>> ---
>>> Makefile.objs | 1 +
>>> qemu-coroutine-sleep.c | 38 ++++++++++++++++++++++++++++++++++++++
>>> qemu-coroutine.h | 9 +++++++++
>>> 3 files changed, 48 insertions(+), 0 deletions(-)
>>> create mode 100644 qemu-coroutine-sleep.c
>>>
>>> diff --git a/Makefile.objs b/Makefile.objs
>>> index 4f6d26c..f4f52e0 100644
>>> --- a/Makefile.objs
>>> +++ b/Makefile.objs
>>> @@ -13,6 +13,7 @@ oslib-obj-$(CONFIG_POSIX) += oslib-posix.o
>>> qemu-thread-posix.o
>>> #######################################################################
>>> # coroutines
>>> coroutine-obj-y = qemu-coroutine.o qemu-coroutine-lock.o
>>> qemu-coroutine-io.o
>>> +coroutine-obj-y += qemu-coroutine-sleep.o
>>> ifeq ($(CONFIG_UCONTEXT_COROUTINE),y)
>>> coroutine-obj-$(CONFIG_POSIX) += coroutine-ucontext.o
>>> else
>>> diff --git a/qemu-coroutine-sleep.c b/qemu-coroutine-sleep.c
>>> new file mode 100644
>>> index 0000000..fd65274
>>> --- /dev/null
>>> +++ b/qemu-coroutine-sleep.c
>>> @@ -0,0 +1,38 @@
>>> +/*
>>> + * QEMU coroutine sleep
>>> + *
>>> + * Copyright IBM, Corp. 2011
>>> + *
>>> + * Authors:
>>> + * Stefan Hajnoczi <address@hidden>
>>> + *
>>> + * This work is licensed under the terms of the GNU LGPL, version 2 or
>>> later.
>>> + * See the COPYING.LIB file in the top-level directory.
>>> + *
>>> + */
>>> +
>>> +#include "qemu-coroutine.h"
>>> +#include "qemu-timer.h"
>>> +
>>> +typedef struct CoSleepCB {
>>> + QEMUTimer *ts;
>>> + Coroutine *co;
>>> +} CoSleepCB;
>>> +
>>> +static void co_sleep_cb(void *opaque)
>>> +{
>>> + CoSleepCB *sleep_cb = opaque;
>>> +
>>> + qemu_free_timer(sleep_cb->ts);
>>
>> I think you need to call qemu_del_timer() first.
>
> I think we're okay with just qemu_free_timer(). qemu_run_timers()
> removes the timer from the active_timers list before invoking its
> callback. qemu_del_timer() is not needed because it just searches the
> active_timers list and removes the timer, if found. We're no longer
> on the active_timers list at the point when co_sleep_cb() is called,
> so there's no need.
Yes, seems you're right. qemu_del_timer() isn't a good name...
Kevin
- [Qemu-devel] [PATCH v5 00/15] block: generic image streaming, Stefan Hajnoczi, 2012/01/13
- [Qemu-devel] [PATCH v5 02/15] block: check bdrv_in_use() before blockdev operations, Stefan Hajnoczi, 2012/01/13
- [Qemu-devel] [PATCH v5 05/15] block: rate-limit streaming operations, Stefan Hajnoczi, 2012/01/13
- [Qemu-devel] [PATCH v5 04/15] block: add image streaming block job, Stefan Hajnoczi, 2012/01/13
- [Qemu-devel] [PATCH v5 03/15] block: add BlockJob interface for long-running operations, Stefan Hajnoczi, 2012/01/13
[Qemu-devel] [PATCH v5 07/15] qmp: add block_job_set_speed command, Stefan Hajnoczi, 2012/01/13
[Qemu-devel] [PATCH v5 14/15] docs: describe live block operations, Stefan Hajnoczi, 2012/01/13
[Qemu-devel] [PATCH v5 11/15] block: add bdrv_find_backing_image, Stefan Hajnoczi, 2012/01/13