|
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
[Prev in Thread] | Current Thread | [Next in Thread] |