From 1d8338d6b71447a5bf18750ca44724345aae5b60 Mon Sep 17 00:00:00 2001 From: fanquake Date: Tue, 28 Jul 2020 19:55:00 +0800 Subject: util: use HAVE_FDATASYNC to determine fdatasync() use Rather than just using it on Linux and NetBSD, use `fdatasync()` based on whether it's available. i.e it is available in newer versions of FreeBSD (11.1 and later). This also aligns our code more closely with what is being done in leveldb. Was pointed out by Luke in #19430. --- src/util/system.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/util') diff --git a/src/util/system.cpp b/src/util/system.cpp index 8164e884b1..e63f2e5c70 100644 --- a/src/util/system.cpp +++ b/src/util/system.cpp @@ -1021,7 +1021,7 @@ bool FileCommit(FILE *file) return false; } #else - #if defined(__linux__) || defined(__NetBSD__) + #if defined(HAVE_FDATASYNC) if (fdatasync(fileno(file)) != 0 && errno != EINVAL) { // Ignore EINVAL for filesystems that don't support sync LogPrintf("%s: fdatasync failed: %d\n", __func__, errno); return false; -- cgit v1.2.3