diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2010-12-08 20:01:14 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2010-12-08 20:01:14 -0500 |
| commit | 87eadd7e3d6f5581d5b4cb8083212a323050e388 (patch) | |
| tree | 3f700ffc1a43eddfcecffcd3e03ce462e20f64c3 /src/include/port/linux.h | |
| parent | 799d0b4b9ede51c629149185e4058c52117cd231 (diff) | |
Force default wal_sync_method to be fdatasync on Linux.
Recent versions of the Linux system header files cause xlogdefs.h to
believe that open_datasync should be the default sync method, whereas
formerly fdatasync was the default on Linux. open_datasync is a bad
choice, first because it doesn't actually outperform fdatasync (in fact
the reverse), and second because we try to use O_DIRECT with it, causing
failures on certain filesystems (e.g., ext4 with data=journal option).
This part of the patch is largely per a proposal from Marti Raudsepp.
More extensive changes are likely to follow in HEAD, but this is as much
change as we want to back-patch.
Also clean up confusing code and incorrect documentation surrounding the
fsync_writethrough option. Those changes shouldn't result in any actual
behavioral change, but I chose to back-patch them anyway to keep the
branches looking similar in this area.
In 9.0 and HEAD, also do some copy-editing on the WAL Reliability
documentation section.
Back-patch to all supported branches, since any of them might get used
on modern Linux versions.
Diffstat (limited to 'src/include/port/linux.h')
| -rw-r--r-- | src/include/port/linux.h | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/include/port/linux.h b/src/include/port/linux.h index 0f4432a4eff..e2a4c4a114d 100644 --- a/src/include/port/linux.h +++ b/src/include/port/linux.h @@ -12,3 +12,11 @@ * to have a kernel version test here. */ #define HAVE_LINUX_EIDRM_BUG + +/* + * Set the default wal_sync_method to fdatasync. With recent Linux versions, + * xlogdefs.h's normal rules will prefer open_datasync, which (a) doesn't + * perform better and (b) causes outright failures on ext4 data=journal + * filesystems, because those don't support O_DIRECT. + */ +#define PLATFORM_DEFAULT_SYNC_METHOD SYNC_METHOD_FDATASYNC |