Page MenuHomeFreeBSD
Differential D25160

ufs: add O_DSYNC support
ClosedPublic
Actions

Authored by tmunro on Jun 5 2020, 10:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 26, 1:44 PM
Unknown Object (File)
Wed, Dec 25, 9:26 AM
Unknown Object (File)
Wed, Dec 11, 9:25 PM
Unknown Object (File)
Dec 3 2024, 4:17 AM
Unknown Object (File)
Oct 22 2024, 6:02 PM
Unknown Object (File)
Oct 9 2024, 4:22 AM
Unknown Object (File)
Oct 3 2024, 12:59 PM
Unknown Object (File)
Oct 3 2024, 10:08 AM
View All 61 Files
Subscribers
None

Details

Reviewers
kib
mjg
mckusick
Summary

Building on top of D25090, here is a patch to teach UFS to understand O_DSYNC.

Test Plan

Using a test program at https://github.com/macdice/some-io-tests/blob/master/sync.c and watching IO with dwatch -X io, we can verify that O_DSYNC updates the inode only when fdatasync would (eg for size change, but not for overwrite).

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

tmunro requested review of this revision.Jun 5 2020, 10:15 PM
tmunro created this revision.
kib added a comment.Jun 23 2020, 4:14 AM

I am not sure what you see in dtrace, but I see a bug in ffs_write() regardless of IO_DATASYNC. D25404

tmunro updated this revision to Diff 81741.Jan 6 2021, 11:47 AM
tmunro edited the summary of this revision. (Show Details)
tmunro edited the test plan for this revision. (Show Details)

Rebased. Previous confusion turned out to be explained by problems fixed by D27353 and problems in my test methodology.

tmunro mentioned this in D25071: aio_fsync(O_DSYNC).Jan 6 2021, 11:49 AM
kib added a reviewer: mckusick.Jan 6 2021, 1:36 PM
kib accepted this revision.Jan 6 2021, 1:44 PM
This revision is now accepted and ready to land.Jan 6 2021, 1:44 PM
tmunro mentioned this in rGe7347be9e34d: ffs: Support O_DSYNC..Jan 8 2021, 12:17 AM
tmunro closed this revision.Jan 9 2021, 7:34 PM

Revision Contents
Changeset List

PathSize
sys/
ufs/
ffs/
4 lines

Diff 72748

View Options

sys/ufs/ffs/ffs_vnops.c

Loading...