qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] build-sys: fix win32 compilation with --target-list=''


From: Paolo Bonzini
Subject: Re: [PATCH] build-sys: fix win32 compilation with --target-list=''
Date: Thu, 17 Dec 2020 12:46:57 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.5.0

On 17/12/20 12:32, Claudio Fontana wrote:
Is the root cause elsewhere though?

I don't like stubs very much, because often they are introduced as the easy way 
out of a problem instead of doing the necessary refactoring,
and they end up confusing the hell out of someone trying to understand what is 
actually used where, never mind trying to debug the linker errors.

There is already an bunch of #ifndef _WIN32, #else , ... in util/main-loop.c 
(quite a bunch of them really),
is that what actually needs reworking, and putting the pieces together in the 
build system in a way that makes sense?

qemu_fd_register is almost not needed at all, since we have

        WSAEventSelect(node->pfd.fd, event, bitmask);

in aio_set_fd_handler. I think we can remove the call to qemu_fd_register from qemu_try_set_nonblock, and that should fix the issue as well.

Paolo




reply via email to

[Prev in Thread] Current Thread [Next in Thread]