Page MenuHomeFreeBSD
Differential D19811

vn_vmap_seekhole(): align running offset to the block boundary.
ClosedPublic
Actions

Authored by kib on Apr 3 2019, 2:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 22, 11:29 PM
Unknown Object (File)
Thu, Mar 19, 6:02 PM
Unknown Object (File)
Wed, Mar 18, 9:51 PM
Unknown Object (File)
Sat, Mar 14, 9:27 PM
Unknown Object (File)
Sat, Mar 14, 6:30 AM
Unknown Object (File)
Sat, Mar 14, 6:25 AM
Unknown Object (File)
Sat, Mar 14, 1:47 AM
Unknown Object (File)
Fri, Mar 13, 2:32 AM
View All Files
Subscribers
emaste
imp

Details

Reviewers
markj
Commits
rS345955: vn_vmap_seekhole(): align running offset to the block boundary.
Summary

Otherwise we might miss the last iteration where EOF appears below unaligned noff.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib created this revision.Apr 3 2019, 2:08 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 3 2019, 2:08 PM
Harbormaster completed remote builds in B23490: Diff 55776.Apr 3 2019, 2:08 PM
kib mentioned this in D19599: Add a UFS-specific implementation for lseek(SEEK_DATA)..Apr 3 2019, 2:08 PM
emaste added a subscriber: emaste.Apr 3 2019, 2:48 PM
markj added inline comments.Apr 3 2019, 11:02 PM
sys/kern/vfs_vnops.c
2171 ↗(On Diff #55776)

Shouldn't it be noff += noff % bsize == 0 ? bsize : bsize - noff % bsize? Or even noff += bsize - noff % bsize.

kib updated this revision to Diff 55802.Apr 4 2019, 1:11 PM

Fix invalid calculation of realignment.

Harbormaster completed remote builds in B23511: Diff 55802.Apr 4 2019, 1:11 PM
markj accepted this revision.Apr 4 2019, 11:06 PM
This revision is now accepted and ready to land.Apr 4 2019, 11:06 PM
Closed by commit rS345955: vn_vmap_seekhole(): align running offset to the block boundary. (authored by kib). · Explain WhyApr 5 2019, 4:14 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
kern/
3 lines

Diff 55840

View Options

head/sys/kern/vfs_vnops.c

Loading...