[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] coroutine: Drop co_sleep_ns
From: |
Benoît Canet |
Subject: |
Re: [Qemu-devel] [PATCH] coroutine: Drop co_sleep_ns |
Date: |
Tue, 26 Aug 2014 17:05:31 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
The Tuesday 26 Aug 2014 à 15:15:43 (+0800), Fam Zheng wrote :
> block_job_sleep_ns is the only user. Since we are moving towards
> AioContext aware code, it's better to use the explict version and drop
s/explict/explicit/g ?G
> the old one.
>
> Signed-off-by: Fam Zheng <address@hidden>
> ---
> blockjob.c | 2 +-
> include/block/coroutine.h | 8 --------
> qemu-coroutine-sleep.c | 12 ------------
> 3 files changed, 1 insertion(+), 21 deletions(-)
>
> diff --git a/blockjob.c b/blockjob.c
> index ca0b4e2..0689fdd 100644
> --- a/blockjob.c
> +++ b/blockjob.c
> @@ -205,7 +205,7 @@ void block_job_sleep_ns(BlockJob *job, QEMUClockType
> type, int64_t ns)
> if (block_job_is_paused(job)) {
> qemu_coroutine_yield();
> } else {
> - co_sleep_ns(type, ns);
> + co_aio_sleep_ns(bdrv_get_aio_context(job->bs), type, ns);
> }
> job->busy = true;
> }
> diff --git a/include/block/coroutine.h b/include/block/coroutine.h
> index b9b7f48..793df0e 100644
> --- a/include/block/coroutine.h
> +++ b/include/block/coroutine.h
> @@ -203,14 +203,6 @@ void qemu_co_rwlock_unlock(CoRwlock *lock);
> /**
> * Yield the coroutine for a given duration
> *
> - * Note this function uses timers and hence only works when a main loop is in
> - * use. See main-loop.h and do not use from qemu-tool programs.
> - */
> -void coroutine_fn co_sleep_ns(QEMUClockType type, int64_t ns);
> -
> -/**
> - * Yield the coroutine for a given duration
> - *
> * Behaves similarly to co_sleep_ns(), but the sleeping coroutine will be
> * resumed when using aio_poll().
> */
> diff --git a/qemu-coroutine-sleep.c b/qemu-coroutine-sleep.c
> index ad78fba..9abb7fd 100644
> --- a/qemu-coroutine-sleep.c
> +++ b/qemu-coroutine-sleep.c
> @@ -27,18 +27,6 @@ static void co_sleep_cb(void *opaque)
> qemu_coroutine_enter(sleep_cb->co, NULL);
> }
>
> -void coroutine_fn co_sleep_ns(QEMUClockType type, int64_t ns)
> -{
> - CoSleepCB sleep_cb = {
> - .co = qemu_coroutine_self(),
> - };
> - sleep_cb.ts = timer_new(type, SCALE_NS, co_sleep_cb, &sleep_cb);
> - timer_mod(sleep_cb.ts, qemu_clock_get_ns(type) + ns);
> - qemu_coroutine_yield();
> - timer_del(sleep_cb.ts);
> - timer_free(sleep_cb.ts);
> -}
> -
> void coroutine_fn co_aio_sleep_ns(AioContext *ctx, QEMUClockType type,
> int64_t ns)
> {
> --
> 2.1.0
>
>
Reviewed-by: Benoît Canet <address@hidden>