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