On 2016-12-16 23:04:13 -0500, Robert Haas wrote: > >> BTW, I suggest this rewritten comment: > >> > >> /*------ > >> * FD_READ events are edge-triggered on Windows per > >> * > >> https://msdn.microsoft.com/en-us/library/windows/desktop/ms741576(v=vs.85).aspx > >> > > > > Isn't the statement in above doc. "For FD_READ, FD_OOB, and FD_ACCEPT > > network events, network event recording and event object signaling are > > level-triggered." says that FD_READ events are level-triggered which > > seems to be contradictory with above comment? > > Argh. I see your point. Maybe we'd better rephrase that. The > document does say that, but the behavior they described is actually a > weird hybrid of level-triggered and edge-triggered. We should > probably just avoid that terminology altogether and explain that > read-ready won't necessarily be re-signalled unless there is an > intervening recv().
(just looked at some latch code, reminding me of this) FWIW, I wonder if the issue here isn't whether WaitForMultipleObjects / WSAEventSelect is level triggered, but that WSAEnumNetworkEvents() pretty much explicitly is not. And it seems to have effects over multiple handles for the same socket. The relevant line from the docs is: "The WSAEnumNetworkEvents function discovers occurrences of network events for the indicated socket, clear internal network event records, and reset event objects (optional)." Note that it clears the state from the socket, *not* just the handle. That behaviour of WSAEnumNetworkEvents() also seems to explains why http://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=f7819baa618c528f60e266874051563ecfe08207 was necessary, and why all the weird workaround in win32/socket.c exist. Greetings, Andres Freund -- Sent via pgsql-hackers mailing list (firstname.lastname@example.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers