coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] copy: adjust fiemap handling of sparse files


From: Pádraig Brady
Subject: Re: [PATCH] copy: adjust fiemap handling of sparse files
Date: Thu, 10 Feb 2011 11:32:09 +0000
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.8) Gecko/20100227 Thunderbird/3.0.3

On 10/02/11 10:53, Jim Meyering wrote:
> While using --sparse=always is a lot less common,
> it looks like there's room for improvement there, too.
> I.e., we should be able to eliminate all of these
> lseek calls on the output descriptor there, too:
> 
>     $ strace -e lseek cp --sparse=always /tmp/t.f /tmp/t2 2>&1|wc -l
>     16384

True. So merge the "lseeks" in the sparse detection in sparse_copy.
I'll have a look at adding that this evening.
Note we'd only get 1 lseek for the fallocated file above
if the mentioned FIEMAP_EXTENT_UNWRITTEN idea works.

cheers,
Pádraig.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]