Page MenuHomeFreeBSD

copy_file_range: Call vn_rdwr() even for zero-length files
ClosedPublic

Authored by markj on Jan 20 2024, 5:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 25, 3:07 PM
Unknown Object (File)
Sat, May 25, 3:07 PM
Unknown Object (File)
Thu, May 23, 9:33 PM
Unknown Object (File)
May 14 2024, 6:18 AM
Unknown Object (File)
May 12 2024, 11:29 AM
Unknown Object (File)
May 10 2024, 8:14 AM
Unknown Object (File)
May 5 2024, 11:09 PM
Unknown Object (File)
Apr 26 2024, 4:14 PM
Subscribers

Details

Summary

This ensures that we invoke VOP_READ on the input file even if it's
empty, which in turn helps ensure that filesystems update the atime of
the file.

PR: 274615

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj requested review of this revision.Jan 20 2024, 5:29 PM
This revision is now accepted and ready to land.Jan 20 2024, 6:45 PM

Needs rebasing but passes the additional test from D44609.

sys/kern/vfs_vnops.c
3592

Instead of adding a flag, you could simply switch from while to do-while.

sys/kern/vfs_vnops.c
3592

It's not that simple - we could have interrupted != 0 before the first iteration.

This revision was automatically updated to reflect the committed changes.