Fix failure-to-read-man-page in commit 899bd785c.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 26 Sep 2017 17:42:53 +0000 (13:42 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 26 Sep 2017 17:43:00 +0000 (13:43 -0400)
commitd29f30d8c3b2a9d8c57324355df8a8d9da1d9c12
tree5007ac8ae6b74a043c9c1ece2da5ef11038cdb38
parent01c5de88ff242b379a033e46e4da6476f2213029
Fix failure-to-read-man-page in commit 899bd785c.

posix_fallocate() is not quite a drop-in replacement for fallocate(),
because it is defined to return the error code as its function result,
not in "errno".  I (tgl) missed this because RHEL6's version seems
to set errno as well.  That is not the case on more modern Linuxen,
though, as per buildfarm results.

Aside from fixing the return-convention confusion, remove the test
for ENOSYS; we expect that glibc will mask that for posix_fallocate,
though it does not for fallocate.  Keep the test for EINTR, because
POSIX specifies that as a possible result, and buildfarm results
suggest that it can happen in practice.

Back-patch to 9.4, like the previous commit.

Thomas Munro

Discussion: https://postgr.es/m/1002664500.12301802.1471008223422.JavaMail.yahoo@mail.yahoo.com
src/backend/storage/ipc/dsm_impl.c