]> granicus.if.org Git - postgresql/commit
Allow latches to wait for socket writability without waiting for readability.
authorAndres Freund <andres@anarazel.de>
Tue, 13 Jan 2015 11:58:43 +0000 (12:58 +0100)
committerAndres Freund <andres@anarazel.de>
Tue, 13 Jan 2015 11:58:43 +0000 (12:58 +0100)
commit4bad60e3fd9a5fc6070fd4d1bd820a280e174654
treed9ce09bcc3c9290e1c3e1eaf11d7c5eb58910515
parent3dfce37627b76e4da9e1d6090beedb608cefafcb
Allow latches to wait for socket writability without waiting for readability.

So far WaitLatchOrSocket() required to pass in WL_SOCKET_READABLE as
that solely was used to indicate error conditions, like EOF. Waiting
for WL_SOCKET_WRITEABLE would have meant to busy wait upon socket
errors.

Adjust the API to signal errors by returning the socket as readable,
writable or both, depending on WL_SOCKET_READABLE/WL_SOCKET_WRITEABLE
being specified.  It would arguably be nicer to return WL_SOCKET_ERROR
but that's not possible on platforms and would probably also result in
more complex callsites.

This previously had explicitly been forbidden in e42a21b9e6c9, as
there was no strong use case at that point. We now are looking into
making FE/BE communication use latches, so changing this makes sense.

There also are some portability concerns because there cases of older
platforms where select(2) is known to, in violation of POSIX, not
return a socket as writable after the peer has closed it.  So far the
platforms where that's the case provide a working poll(2). If we find
one where that's not the case, we'll need to add a workaround for that
platform.

Discussion: 20140927191243.GD5423@alap3.anarazel.de
Reviewed-By: Heikki Linnakangas, Noah Misch
src/backend/port/unix_latch.c
src/backend/port/win32_latch.c