Differential D7522

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

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

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 6, 2:38 PM

	Unknown Object (File)
	Sun, Dec 1, 1:16 AM

	Unknown Object (File)
	Nov 14 2024, 4:26 PM

	Unknown Object (File)
	Nov 11 2024, 2:00 AM

	Unknown Object (File)
	Oct 31 2024, 12:03 PM

	Unknown Object (File)
	Oct 7 2024, 5:13 AM

	Unknown Object (File)
	Oct 2 2024, 8:04 PM

	Unknown Object (File)
	Sep 20 2024, 9:21 AM

View All 71 Files

Subscribers

Details

Reviewers

Group Reviewers

Commits

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

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib updated this revision to Diff 19325.Aug 16 2016, 8:32 AM

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.

Herald added a reviewer: manpages. · View Herald TranscriptAug 16 2016, 8:32 AM

Herald added a subscriber: imp. · View Herald Transcript

emaste added inline comments.Aug 16 2016, 8:50 AM

lib/libc/sys/fsync.2
6 ↗	(On Diff #19325)	s/was/were/
78–81 ↗	(On Diff #19325)	Can we be more specific here, or at least avoid "appear"?

Notes by Ed.

Herald edited edge metadata. · View Herald TranscriptAug 16 2016, 9:23 AM

wblock added a subscriber: wblock.Aug 16 2016, 11:48 AM

wblock added inline comments.

lib/libc/sys/fsync.2
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:" or "The fsync and fdatasync calls fail if:"

wblock changes

Herald edited edge metadata. · View Herald TranscriptAug 16 2016, 2:01 PM

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

lib/libc/sys/fsync.2
42 ↗	(On Diff #19350)	"synchronize changes to a file's data, without or with it's metadata"

kib added inline comments.Aug 16 2016, 2:47 PM

lib/libc/sys/fsync.2
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 accepted this revision.Aug 17 2016, 10:08 AM

emaste edited edge metadata.

emaste added inline comments.

lib/libc/sys/fsync.2
37 ↗	(On Diff #19350)	Update

This revision is now accepted and ready to land.Aug 17 2016, 10:08 AM

Closed by commit rS304287: Add fdatasync(2) man page, combined with fsync(2). (authored by kib). · Explain WhyAug 17 2016, 10:17 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

lib/

libc/

sys/

1 line

51 lines

Diff 19403

head/lib/libc/sys/Makefile.inc

Loading...

head/lib/libc/sys/fsync.2

Loading...