RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-03-11 02:39:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

os/newstore: use fdatasync instead of fsync

Browse Source 
On XFS at least, fdatasync is sufficient to make data readable.

Signed-off-by: Sage Weil <sage@redhat.com>
This commit is contained in:
Sage Weil 2015-04-15 15:35:16 -07:00
parent 1321b880cc
commit 04f55d8d18
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/os/newstore/NewStore.cc
View File

@ -2352,7 +2352,7 @@ int NewStore::_do_wal_transaction(wal_transaction_t& wt)
for (vector<int>::iterator p = sync_fds.begin();
p != sync_fds.end();
++p) {
int r = ::fsync(*p);
int r = ::fdatasync(*p);
assert(r == 0);
VOID_TEMP_FAILURE_RETRY(::close(*p));
}
@ -2446,7 +2446,7 @@ int NewStore::queue_transactions(
for (list<fsync_item>::iterator p = txc->fds.begin();
p != txc->fds.end(); ++p) {
dout(30) << __func__ << " fsync " << p->fd << dendl;
int r = ::fsync(p->fd);
int r = ::fdatasync(p->fd);
if (r < 0) {
r = -errno;
derr << __func__ << " fsync: " << cpp_strerror(r) << dendl;