On 28/01/2022 01:22, Paul Eggert wrote:
cp+install+mv started using copy_file_range in coreutils 9.0 and it
seems to be working OK, so I thought I'd change cat to use it too. While
looking into that I found some infelicities and/or unlikely integer
overflows to fix (of course there are more where these came from; these
are just the ones having to do with page-aligned buffer allocation which
is something cat does).
I installed the attached series of patches to do all that; the last
patch is the copy_file_range change.
Thanks for doing all that.
Changes look good from a quick scan.
I might update NEWS to say _why_ copy_file_range is used
(copy acceleration (copy offload + maybe reflink)).
There is a new syntax-check failure, which I fixed up in gnulib with:
https://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=commit;h=a3e1e74e
thanks,
Pádraig
