diff options
| author | Thomas Munro <tmunro@postgresql.org> | 2021-02-15 15:43:39 +1300 |
|---|---|---|
| committer | Thomas Munro <tmunro@postgresql.org> | 2021-02-15 16:04:59 +1300 |
| commit | f900a79ecdc1864a6ead72c97c34a41012227eaf (patch) | |
| tree | 79dfa5d680b37149cc299be5fb52ef78aca962f8 /src | |
| parent | 2c8b42b50df6cc5ba6987c400a857d6bbfdf0300 (diff) | |
| download | postgresql-f900a79ecdc1864a6ead72c97c34a41012227eaf.tar.gz postgresql-f900a79ecdc1864a6ead72c97c34a41012227eaf.zip | |
Default to wal_sync_method=fdatasync on FreeBSD.
FreeBSD 13 gained O_DSYNC, which would normally cause wal_sync_method to
choose open_datasync as its default value. That may not be a good
choice for all systems, and performs worse than fdatasync in some
scenarios. Let's preserve the existing default behavior for now.
Like commit 576477e73c4, which did the same for Linux, back-patch to all
supported releases.
Discussion: https://postgr.es/m/CA%2BhUKGLsAMXBQrCxCXoW-JsUYmdOL8ALYvaX%3DCrHqWxm-nWbGA%40mail.gmail.com
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/utils/misc/postgresql.conf.sample | 2 | ||||
| -rw-r--r-- | src/include/port/freebsd.h | 9 |
2 files changed, 10 insertions, 1 deletions
diff --git a/src/backend/utils/misc/postgresql.conf.sample b/src/backend/utils/misc/postgresql.conf.sample index bd57e917e16..db6db376eb0 100644 --- a/src/backend/utils/misc/postgresql.conf.sample +++ b/src/backend/utils/misc/postgresql.conf.sample @@ -206,7 +206,7 @@ #wal_sync_method = fsync # the default is the first option # supported by the operating system: # open_datasync - # fdatasync (default on Linux) + # fdatasync (default on Linux and FreeBSD) # fsync # fsync_writethrough # open_sync diff --git a/src/include/port/freebsd.h b/src/include/port/freebsd.h index 2e36d3da4f4..2e2e749a6b6 100644 --- a/src/include/port/freebsd.h +++ b/src/include/port/freebsd.h @@ -1 +1,10 @@ /* src/include/port/freebsd.h */ + +/* + * Set the default wal_sync_method to fdatasync. xlogdefs.h's normal rules + * would prefer open_datasync on FreeBSD 13+, but that is not a good choice on + * many systems. + */ +#ifdef HAVE_FDATASYNC +#define PLATFORM_DEFAULT_SYNC_METHOD SYNC_METHOD_FDATASYNC +#endif |