Paths

Table of Contentst

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)
	Sat, Sep 27, 2:32 AM

	Unknown Object (File)
	Sun, Sep 21, 12:21 AM

	Unknown Object (File)
	Thu, Sep 18, 10:21 PM

	Unknown Object (File)
	Sep 14 2025, 8:47 AM

	Unknown Object (File)
	Sep 13 2025, 10:26 PM

	Unknown Object (File)
	Aug 23 2025, 5:30 AM

	Unknown Object (File)
	Aug 16 2025, 4:26 AM

	Unknown Object (File)
	Aug 3 2025, 8:18 AM

Subscribers

Details

Reviewers

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`.

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

Path

Size

head/

sys/

kern/

3 lines

Diff 55840

head/sys/kern/vfs_vnops.c

Loading...