Page MenuHomeFreeBSD

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

Authored by kib on Aug 16 2016, 8:32 AM.

Diff Detail

rS FreeBSD src repository - subversion
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

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.
6 ↗(On Diff #19325)


78–81 ↗(On Diff #19325)

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.
78 ↗(On Diff #19329)

Passive -> active:

If the file metadata has already been committed, using
81 ↗(On Diff #19329)

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

can be more efficient than
.Fn fsync .
95 ↗(On Diff #19329)

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:"
"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.

42 ↗(On Diff #19350)

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

42 ↗(On Diff #19350)

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.
37 ↗(On Diff #19350)


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.