[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] tools: Use real async.c instead of stubs
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] Re: [PATCH] tools: Use real async.c instead of stubs |
Date: |
Wed, 23 Feb 2011 12:47:40 +0000 |
On Wed, Feb 23, 2011 at 11:54 AM, Kevin Wolf <address@hidden> wrote:
> It's wrong to call BHs directly, even in tools. The only operations that
> schedule BHs are called in a loop that (indirectly) contains a call to
> qemu_bh_poll anyway, so we're not losing the scheduled BHs: Tools either use
> synchronous functions, which are guaranteed to have completed (including any
> BHs) when they return; or if they use asynchronous functions, they need to
> call
> qemu_aio_wait() or similar functions already today.
>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> Makefile.objs | 4 ++--
> qemu-tool.c | 47 ++++-------------------------------------------
> 2 files changed, 6 insertions(+), 45 deletions(-)
We discussed this on IRC and I think it makes sense. Review from
others would be appreciated.
Reviewed-by: Stefan Hajnoczi <address@hidden>