Re: [PATCH 5/5] server: get rid of sock_try_event

[Alexandre Julliard]

Mike Kaplinskiy <mike.kaplinskiy(a)gmail.com> writes:
> We *might* need to ignore POLLERR messages for UDP sockets, as the
> condition is actually temporary and sock_try_event was hiding this.

It doesn't work here:

../../../tools/runtest -q -P wine -M ws2_32.dll -T ../../.. -p ws2_32_test.exe.so sock.c && touch sock.ok
sock.c:941: Test failed: some threads have not completed: 102
sock.c:3567: Tests skipped: WSAIoctl failed to get AcceptEx with ret -1 + errno 10045
make: *** [sock.ok] Error 1

-- 
Alexandre Julliard
julliard(a)winehq.org

[PATCH 5/5] server: get rid of sock_try_event

[Mike Kaplinskiy]

On Thu, May 13, 2010 at 6:28 AM, Alexandre Julliard <julliard(a)winehq.org> wrote:
> Mike Kaplinskiy <mike.kaplinskiy(a)gmail.com> writes:
> > We *might* need to ignore POLLERR messages for UDP sockets, as the
> > condition is actually temporary and sock_try_event was hiding this.
>
> It doesn't work here:
>
> ../../../tools/runtest -q -P wine -M ws2_32.dll -T ../../.. -p ws2_32_test.exe.so sock.c && touch sock.ok
> sock.c:941: Test failed: some threads have not completed: 102
> sock.c:3567: Tests skipped: WSAIoctl failed to get AcceptEx with ret -1 + errno 10045
> make: *** [sock.ok] Error 1
>
> --
> Alexandre Julliard
> julliard(a)winehq.org

Hmm, this turns out to be somewhat intermittent. Looks like I forgot
one of the patches in the series (we tend to ignore POLLHUP on
SOCK_SHUTDOWN_EOF). Sorry about that, I'll resend for tomorrow. The
patches before this one should be ok.

Mike.