Page MenuHomeFreeBSD

freebsd32 shims for copy_file_range(2)
ClosedPublic

Authored by kib on Jul 27 2019, 8:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jan 21, 8:48 AM
Unknown Object (File)
Tue, Jan 14, 4:59 PM
Unknown Object (File)
Sat, Jan 11, 5:18 PM
Unknown Object (File)
Thu, Jan 9, 5:48 AM
Unknown Object (File)
Dec 7 2024, 2:42 PM
Unknown Object (File)
Nov 17 2024, 6:39 PM
Unknown Object (File)
Oct 10 2024, 11:57 PM
Unknown Object (File)
Oct 2 2024, 11:04 PM
Subscribers

Details

Test Plan

Rick, could you, please, provide me with the test program you used for the syscall check ?

Diff Detail

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

Event Timeline

I don't really know anything about this, but it looks fine to me.
The only change is uint32_t replaces size_t, which makes sense to 32bits.
(I'm guessing this is only used for amd64, where "int" and "unsigned int"
remain 32bits.)

Hopefully brooks can review this, but if not, I'll click reviewed in a few days.

Why is a shim required? All arguments are extended to 64-bits as part of the combat layer. There would be an issue if len were an off_t, but it's a size_t. The only pointers point to off_t's which are always 64-bit and thus require no special handling.

Why is a shim required? All arguments are extended to 64-bits as part of the combat layer. There would be an issue if len were an off_t, but it's a size_t. The only pointers point to off_t's which are always 64-bit and thus require no special handling.

Yes, it started with len having off_t, see some discussion on fs@. I am on fence there, I can drop the shim and change STD to NOPROTO, of course.

This revision is now accepted and ready to land.Jul 30 2019, 10:16 PM
This revision was automatically updated to reflect the committed changes.