On Mon, Mar 22, 2010 at 09:55:49PM +0800, jeff.liu wrote:
> I found cp(1) has an '--reflink' option, at first, I think it is means the 
> OCFS2 reflink
> operation(please refer to http://lwn.net/Articles/331576/).
> 
> But by checking through the source code, looks it does another thing for 
> Btrfs only.

        Yeah, this was the cp folks conflating my reflink patch and
chris's btrfs operation.

> btw, is it acceptable to add the OCFS2 reflink into the cp(1) program, it 
> could be implemented very
> similar to the current "clone_file()" function(also call ioctl(2), due to 
> reflink(2) does not merged
> into mainline for now), and share the same command line trigger(i.e. 
> --reflink=[WHEN]).
> If so, I'd like to write the patch.

        If you can make the cp --reflink option do the btrfs thing for
btrfs and the ocfs2 thing for ocfs2, go for it!

Joel

-- 

"I have never let my schooling interfere with my education."
        - Mark Twain

Joel Becker
Principal Software Developer
Oracle
E-mail: [email protected]
Phone: (650) 506-8127


Reply via email to