Page MenuHomeFreeBSD

Add fdatasync(2) man page, combined with fsync(2).
ClosedPublic

Authored by kib on Aug 16 2016, 8:32 AM.
Tags
None
Referenced Files
F81677882: D7522.id19403.diff
Fri, Apr 19, 7:34 PM
Unknown Object (File)
Thu, Apr 18, 12:51 PM
Unknown Object (File)
Jan 11 2024, 8:05 AM
Unknown Object (File)
Dec 20 2023, 5:49 AM
Unknown Object (File)
Dec 14 2023, 6:27 PM
Unknown Object (File)
Nov 19 2023, 5:12 AM
Unknown Object (File)
Nov 15 2023, 9:56 AM
Unknown Object (File)
Nov 9 2023, 6:27 PM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kib retitled this revision from to Add fdatasync(2) man page, combined with fsync(2)..
kib updated this object.
kib edited the test plan for this revision. (Show Details)
kib added a reviewer: emaste.
kib set the repository for this revision to rS FreeBSD src repository - subversion.
lib/libc/sys/fsync.2
6

s/was/were/

78–81

Can we be more specific here, or at least avoid "appear"?

kib marked 2 inline comments as done.
kib edited edge metadata.

Notes by Ed.

wblock added inline comments.
lib/libc/sys/fsync.2
78

Passive -> active:

If the file metadata has already been committed, using
81

"for the same purpose" is a little unclear. How about

can be more efficient than
.Fn fsync .
95

Either remove the "The" at the beginning of this sentence, or add a word before "fail" to say what they are:

"fsync and fdatasync fail if:"
or
"The fsync and fdatasync calls fail if:"

kib marked 3 inline comments as done.
kib edited edge metadata.

wblock changes

Other than the optional change I suggested for the '.Nd' line, this looks good to me.

lib/libc/sys/fsync.2
42

"synchronize changes to a file's data, without or with it's metadata"

lib/libc/sys/fsync.2
42

Wouldn't it clutter the tag line ? Some changes are synced, which are is described in the main text.

fdatasync(2) cannot be a default choice, it requires careful consideration if used.

emaste edited edge metadata.
emaste added inline comments.
lib/libc/sys/fsync.2
37

Update

This revision is now accepted and ready to land.Aug 17 2016, 10:08 AM
This revision was automatically updated to reflect the committed changes.