public inbox for linux-btrfs@vger.kernel.org
 help / color / mirror / Atom feed
* Re: A file cloned with "cp --reflink" different from the original one?
       [not found]   ` <119341710911230614h10d7071dq3700cdf39e0c28fb@mail.gmail.com>
@ 2009-11-23 15:47     ` Pádraig Brady
  2009-11-23 15:50       ` Jian Lin
  0 siblings, 1 reply; 3+ messages in thread
From: Pádraig Brady @ 2009-11-23 15:47 UTC (permalink / raw)
  To: Jian Lin; +Cc: bug-coreutils, linux-btrfs

Jian Lin wrote:
> 2009/11/23 Pádraig Brady <P@draigbrady.com>:
>> Jian Lin wrote:
>>> I installed BtrFS 0.19 and GNU coreutils 8.1 on my Ubuntu 9.10.
>>> I tried to clone some files with "cp --reflink" to make them "copy-on-write".
>>> However, I found some of the files cloned have different MD5s to the
>>> original one.
>>>
>>> Is BtrFS (or cp with reflink) buggy?
>>> Or it is indeed a feature that I used incorrectly?
>>>
>>> Thanks!
>>>
>>> root@lj-laptop:/mnt# uname -a
>>> Linux lj-laptop 2.6.31-15-generic #50-Ubuntu SMP Tue Nov 10 14:54:29
>>> UTC 2009 i686 GNU/Linux
>>>
>>> root@lj-laptop:/mnt# mount | grep btrfs
>>> /dev/sda4 on /mnt type btrfs (rw)
>>>
>>> root@lj-laptop:/mnt# cp --version
>>> cp (GNU coreutils) 8.1
>>>
>>> root@lj-laptop:/mnt# cd WinXP_CHS/
>>> root@lj-laptop:/mnt/WinXP_CHS# cp WinXP_CHS.vmdk WinXP_CHS-ref.vmdk --reflink
>>>
>>> root@lj-laptop:/mnt/WinXP_CHS# ls -la WinXP_CHS.vmdk WinXP_CHS-ref.vmdk
>>> -rwxr-xr-x 1 root root 1713766400 Nov 23 15:11 WinXP_CHS-ref.vmdk
>>> -rwxr-xr-x 1 root root 1713766400 Nov 23 13:32 WinXP_CHS.vmdk
>>>
>>> root@lj-laptop:/mnt/WinXP_CHS# md5sum WinXP_CHS.vmdk WinXP_CHS-ref.vmdk
>>> f6da592d056167a44b089d70fa46f863  WinXP_CHS.vmdk
>>> e3ee0c2c17771811c80eed088c20987d  WinXP_CHS-ref.vmdk
>> Yikes. Are you 100% sure nothing is changing those vm files?
> 
> I think so. I run md5sum immediately after cloning the file.
> 
>> Note I wouldn't 100% trust the mtime, so an md5sum before
>> and after would be good to confirm.
>>
>> You could try this simple prog to do the clones to
>> implicate coreutils or otherwise:
>>
>> #include <sys/ioctl.h>
>> #include <sys/types.h>
>> #include <sys/stat.h>
>> #include <fcntl.h>
>> #include <stdio.h>
>> #define BTRFS_IOC_CLONE 1074041865
>>
>> int main(int argc, const char **argv)
>> {
>>  int in = open(argv[1], O_RDONLY);
>>  int out = open(argv[2], O_CREAT|O_TRUNC|O_WRONLY, 0644);
>>  if (ioctl(out, BTRFS_IOC_CLONE, in)==-1) {
>>      fprintf(stderr, "ioctl error = %m\n");
>>  }
>>  close(in);
>>  close(out);
>>  return 0;
>> }
> 
> I compiled and tried this program.
> ./clone WinXP_CHS.vmdk WinXP_CHS-clone.vmdk
> then I got md5 of WinXP_CHS-clone.vmdk, which is the same as that of
> WinXP_CHS-ref.vmdk (made by cp --reflink), and different from the md5
> of original WinXP_CHS.vmdk.
> 
> The cloned vm cannot run correctly, which said "hal.dll lost". The
> cloned Linux vm I mentioned before also reported "/bin/bash lost".
> Another tiny linux vm I cloned with the same md5 to its original vmdk
> can run correctly.

OK then it sounds like an issue with BTRFS (CC'd)
(which could already be fixed since it's in active development).

cheers,
Pádraig.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: A file cloned with "cp --reflink" different from the original one?
  2009-11-23 15:47     ` A file cloned with "cp --reflink" different from the original one? Pádraig Brady
@ 2009-11-23 15:50       ` Jian Lin
  2009-11-23 15:52         ` Chris Mason
  0 siblings, 1 reply; 3+ messages in thread
From: Jian Lin @ 2009-11-23 15:50 UTC (permalink / raw)
  To: Pádraig Brady; +Cc: bug-coreutils, linux-btrfs

Thanks. I will try the new version soon.

2009/11/23 P=A8=A2draig Brady <P@draigbrady.com>:
> Jian Lin wrote:
>> 2009/11/23 P=A8=A2draig Brady <P@draigbrady.com>:
>>> Jian Lin wrote:
>>>> I installed BtrFS 0.19 and GNU coreutils 8.1 on my Ubuntu 9.10.
>>>> I tried to clone some files with "cp --reflink" to make them "copy=
-on-write".
>>>> However, I found some of the files cloned have different MD5s to t=
he
>>>> original one.
>>>>
>>>> Is BtrFS (or cp with reflink) buggy?
>>>> Or it is indeed a feature that I used incorrectly?
>>>>
>>>> Thanks!
>>>>
>>>> root@lj-laptop:/mnt# uname -a
>>>> Linux lj-laptop 2.6.31-15-generic #50-Ubuntu SMP Tue Nov 10 14:54:=
29
>>>> UTC 2009 i686 GNU/Linux
>>>>
>>>> root@lj-laptop:/mnt# mount | grep btrfs
>>>> /dev/sda4 on /mnt type btrfs (rw)
>>>>
>>>> root@lj-laptop:/mnt# cp --version
>>>> cp (GNU coreutils) 8.1
>>>>
>>>> root@lj-laptop:/mnt# cd WinXP_CHS/
>>>> root@lj-laptop:/mnt/WinXP_CHS# cp WinXP_CHS.vmdk WinXP_CHS-ref.vmd=
k --reflink
>>>>
>>>> root@lj-laptop:/mnt/WinXP_CHS# ls -la WinXP_CHS.vmdk WinXP_CHS-ref=
=2Evmdk
>>>> -rwxr-xr-x 1 root root 1713766400 Nov 23 15:11 WinXP_CHS-ref.vmdk
>>>> -rwxr-xr-x 1 root root 1713766400 Nov 23 13:32 WinXP_CHS.vmdk
>>>>
>>>> root@lj-laptop:/mnt/WinXP_CHS# md5sum WinXP_CHS.vmdk WinXP_CHS-ref=
=2Evmdk
>>>> f6da592d056167a44b089d70fa46f863  WinXP_CHS.vmdk
>>>> e3ee0c2c17771811c80eed088c20987d  WinXP_CHS-ref.vmdk
>>> Yikes. Are you 100% sure nothing is changing those vm files?
>>
>> I think so. I run md5sum immediately after cloning the file.
>>
>>> Note I wouldn't 100% trust the mtime, so an md5sum before
>>> and after would be good to confirm.
>>>
>>> You could try this simple prog to do the clones to
>>> implicate coreutils or otherwise:
>>>
>>> #include <sys/ioctl.h>
>>> #include <sys/types.h>
>>> #include <sys/stat.h>
>>> #include <fcntl.h>
>>> #include <stdio.h>
>>> #define BTRFS_IOC_CLONE 1074041865
>>>
>>> int main(int argc, const char **argv)
>>> {
>>>  int in =3D open(argv[1], O_RDONLY);
>>>  int out =3D open(argv[2], O_CREAT|O_TRUNC|O_WRONLY, 0644);
>>>  if (ioctl(out, BTRFS_IOC_CLONE, in)=3D=3D-1) {
>>>      fprintf(stderr, "ioctl error =3D %m\n");
>>>  }
>>>  close(in);
>>>  close(out);
>>>  return 0;
>>> }
>>
>> I compiled and tried this program.
>> ./clone WinXP_CHS.vmdk WinXP_CHS-clone.vmdk
>> then I got md5 of WinXP_CHS-clone.vmdk, which is the same as that of
>> WinXP_CHS-ref.vmdk (made by cp --reflink), and different from the md=
5
>> of original WinXP_CHS.vmdk.
>>
>> The cloned vm cannot run correctly, which said "hal.dll lost". The
>> cloned Linux vm I mentioned before also reported "/bin/bash lost".
>> Another tiny linux vm I cloned with the same md5 to its original vmd=
k
>> can run correctly.
>
> OK then it sounds like an issue with BTRFS (CC'd)
> (which could already be fixed since it's in active development).
>
> cheers,
> P=A8=A2draig.
>



--=20

=C1=D6=BD=A1
=D6=D0=B9=FA=BF=C6=D1=A7=D4=BA=BC=C6=CB=E3=BC=BC=CA=F5=D1=D0=BE=BF=CB=F9=
=CD=F8=C2=E7=D6=D8=B5=E3=CA=B5=D1=E9=CA=D2
=B1=B1=BE=A9=CA=D0=BA=A3=B5=ED=C7=F8=BF=C6=D1=A7=D4=BA=C4=CF=C2=B76=BA=C5=
 =D3=CA=B1=E0=A3=BA100190

Jian LIN
Key Laboratory of Network Science and Technology,
Institute of Computing Technology,
Chinese Academy of Sciences,
P.O.Box 2704, Beijing 100190, China.

Website:    http://www.linjian.org
Blog:       http://blog.linjian.org
E-mail/IM:  lj@linjian.org
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" =
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: A file cloned with "cp --reflink" different from the original one?
  2009-11-23 15:50       ` Jian Lin
@ 2009-11-23 15:52         ` Chris Mason
  0 siblings, 0 replies; 3+ messages in thread
From: Chris Mason @ 2009-11-23 15:52 UTC (permalink / raw)
  To: Jian Lin; +Cc: Pádraig Brady, bug-coreutils, linux-btrfs

On Mon, Nov 23, 2009 at 11:50:47PM +0800, Jian Lin wrote:
> Thanks. I will try the new version soon.

Note that you'll have to reclone the files, the new version won't fix
the old clone I'm afraid.

-chris

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-11-23 15:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <119341710911230208j5e60734cvf09cfcacd86ba9f6@mail.gmail.com>
     [not found] ` <4B0A8FC1.4050208@draigBrady.com>
     [not found]   ` <119341710911230614h10d7071dq3700cdf39e0c28fb@mail.gmail.com>
2009-11-23 15:47     ` A file cloned with "cp --reflink" different from the original one? Pádraig Brady
2009-11-23 15:50       ` Jian Lin
2009-11-23 15:52         ` Chris Mason

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox