Page MenuHomeFreeBSD

install: Don't skip syncing in the common case.
ClosedPublic

Authored by des on Apr 12 2024, 2:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 14, 8:18 AM
Unknown Object (File)
Wed, Nov 5, 4:27 PM
Unknown Object (File)
Wed, Nov 5, 4:27 PM
Unknown Object (File)
Wed, Nov 5, 3:50 PM
Unknown Object (File)
Tue, Nov 4, 11:24 PM
Unknown Object (File)
Oct 15 2025, 3:02 AM
Unknown Object (File)
Oct 12 2025, 9:30 AM
Unknown Object (File)
Oct 12 2025, 9:30 AM
Subscribers

Details

Summary

In copy(), if no digest was requested (which is the common case), we
use copy_file_range() to avoid needlessly copying the contents of the
file into user space and back. When copy_file_range() returns
successfully (which, again, is the common case), we simply return, and
therefore never get to the point where we call fsync() if the -S
option was specified. Fix this.

MFC after: 1 week
Sponsored by: Klara, Inc.

Diff Detail

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