Adjust pg_fsync_writethrough so that it will set errno when failing

on a platform that doesn't support this operation.  The former coding
would allow an unrelated errno to be reported, which would be quite
misleading.  Not sure if this has anything to do with the current
buildfarm failures, but it's certainly bogus as-is.
REL9_0_ALPHA5_BRANCH
Tom Lane 16 years ago
parent 3229db2d42
commit e9a383303c
  1. 3
      src/backend/storage/file/fd.c

@ -7,7 +7,7 @@
* Portions Copyright (c) 1994, Regents of the University of California
*
* IDENTIFICATION
* $PostgreSQL: pgsql/src/backend/storage/file/fd.c,v 1.154 2010/02/15 00:50:57 stark Exp $
* $PostgreSQL: pgsql/src/backend/storage/file/fd.c,v 1.155 2010/02/22 15:26:14 tgl Exp $
*
* NOTES:
*
@ -292,6 +292,7 @@ pg_fsync_writethrough(int fd)
#elif defined(F_FULLFSYNC)
return (fcntl(fd, F_FULLFSYNC, 0) == -1) ? -1 : 0;
#else
errno = ENOSYS;
return -1;
#endif
}

Loading…
Cancel
Save