[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v13 15/21] bottom halves: introduce bh call
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [RFC PATCH v13 15/21] bottom halves: introduce bh call function |
Date: |
Wed, 06 May 2015 16:39:36 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 |
On 06/05/2015 16:04, Pavel Dovgalyuk wrote:
> This patch introduces aio_bh_call function. It is used to execute
> bottom halves as callbacks without adding them to the queue.
>
> Signed-off-by: Pavel Dovgalyuk <address@hidden>
> ---
> async.c | 8 +++++++-
> include/block/aio.h | 5 +++++
> 2 files changed, 12 insertions(+), 1 deletions(-)
>
> diff --git a/async.c b/async.c
> index 46d9e63..734c76c 100644
> --- a/async.c
> +++ b/async.c
> @@ -59,6 +59,12 @@ QEMUBH *aio_bh_new(AioContext *ctx, QEMUBHFunc *cb, void
> *opaque)
> return bh;
> }
>
> +void aio_bh_call(void *opaque)
> +{
> + QEMUBH *bh = (QEMUBH *)opaque;
This can have a QEMUBH * argument. I can fix it when committing.
> + bh->cb(bh->opaque);
> +}
> +
> /* Multiple occurrences of aio_bh_poll cannot be called concurrently */
> int aio_bh_poll(AioContext *ctx)
> {
> @@ -82,7 +88,7 @@ int aio_bh_poll(AioContext *ctx)
> if (!bh->idle)
> ret = 1;
> bh->idle = 0;
> - bh->cb(bh->opaque);
> + aio_bh_call(bh);
> }
> }
>
> diff --git a/include/block/aio.h b/include/block/aio.h
> index d2bb423..b498704 100644
> --- a/include/block/aio.h
> +++ b/include/block/aio.h
> @@ -157,6 +157,11 @@ QEMUBH *aio_bh_new(AioContext *ctx, QEMUBHFunc *cb, void
> *opaque);
> void aio_notify(AioContext *ctx);
>
> /**
> + * aio_bh_call: Executes callback function of the specified BH.
> + */
> +void aio_bh_call(void *opaque);
> +
> +/**
> * aio_bh_poll: Poll bottom halves for an AioContext.
> *
> * These are internal functions used by the QEMU main loop.
>
Works for me! I am not sure why patch 16 works though. :) I'll ponder
on it, in the meanwhile some extra explanation from you would be welcome...
Paolo
- [Qemu-devel] [RFC PATCH v13 07/21] cpu-exec: allow temporary disabling icount, (continued)
- [Qemu-devel] [RFC PATCH v13 07/21] cpu-exec: allow temporary disabling icount, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 08/21] cpu: replay instructions sequence, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 09/21] i386: interrupt poll processing, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 10/21] replay: interrupts and exceptions, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 11/21] replay: asynchronous events infrastructure, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 12/21] replay: recording and replaying clock ticks, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 13/21] replay: shutdown event, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 14/21] replay: checkpoints, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 16/21] replay: ptimer, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 15/21] bottom halves: introduce bh call function, Pavel Dovgalyuk, 2015/05/06
- Re: [Qemu-devel] [RFC PATCH v13 15/21] bottom halves: introduce bh call function,
Paolo Bonzini <=
- [Qemu-devel] [RFC PATCH v13 17/21] typedef: add typedef for QemuOpts, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 18/21] replay: initialization and deinitialization, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 19/21] replay: replay blockers for devices, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 20/21] replay: command line options, Pavel Dovgalyuk, 2015/05/06
- [Qemu-devel] [RFC PATCH v13 21/21] replay: recording of the user input, Pavel Dovgalyuk, 2015/05/06
- Re: [Qemu-devel] [RFC PATCH v13 00/21] Deterministic replay core, Pavel Dovgaluk, 2015/05/19