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)
Mon, Jan 27, 4:58 PM
Unknown Object (File)
Sat, Jan 25, 9:26 AM
Unknown Object (File)
Fri, Jan 24, 5:59 PM
Unknown Object (File)
Mon, Jan 20, 11:04 AM
Unknown Object (File)
Wed, Jan 15, 10:07 AM
Unknown Object (File)
Fri, Jan 10, 10:01 AM
Unknown Object (File)
Fri, Jan 10, 4:54 AM
Unknown Object (File)
Nov 23 2024, 2:14 PM
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