[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.12] iothread: fix breakage on windows
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH for-2.12] iothread: fix breakage on windows |
Date: |
Thu, 22 Mar 2018 10:13:20 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 22/03/2018 09:56, Peter Xu wrote:
> OOB can enable iothread for parsing even on Windows. We need some tunes
> to enable that on Windows otherwise it'll break Windows users. This
> patch fixes the breakage on Windows with qemu-system-ppc.exe.
>
> Reported-by: Howard Spoelstra <address@hidden>
> Tested-by: Howard Spoelstra <address@hidden>
> Suggested-by: Paolo Bonzini <address@hidden>
> Signed-off-by: Peter Xu <address@hidden>
> ---
> iothread.c | 4 ++++
> util/aio-win32.c | 4 +++-
> 2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/iothread.c b/iothread.c
> index 1b3463cb00..9f52270ae1 100644
> --- a/iothread.c
> +++ b/iothread.c
> @@ -31,11 +31,15 @@ typedef ObjectClass IOThreadClass;
> #define IOTHREAD_CLASS(klass) \
> OBJECT_CLASS_CHECK(IOThreadClass, klass, TYPE_IOTHREAD)
>
> +#ifdef CONFIG_LINUX
> /* Benchmark results from 2016 on NVMe SSD drives show max polling times
> around
> * 16-32 microseconds yield IOPS improvements for both iodepth=1 and
> iodepth=32
> * workloads.
> */
> #define IOTHREAD_POLL_MAX_NS_DEFAULT 32768ULL
> +#else
> +#define IOTHREAD_POLL_MAX_NS_DEFAULT 0ULL
> +#endif
>
> static __thread IOThread *my_iothread;
>
> diff --git a/util/aio-win32.c b/util/aio-win32.c
> index d6d5e02f00..a67b00c6ad 100644
> --- a/util/aio-win32.c
> +++ b/util/aio-win32.c
> @@ -410,5 +410,7 @@ void aio_context_setup(AioContext *ctx)
> void aio_context_set_poll_params(AioContext *ctx, int64_t max_ns,
> int64_t grow, int64_t shrink, Error **errp)
> {
> - error_setg(errp, "AioContext polling is not implemented on Windows");
> + if (max_ns) {
> + error_setg(errp, "AioContext polling is not implemented on Windows");
> + }
> }
>
Queued, thanks.
Paolo