From: Marti Raudsepp <marti@juffo.org>
To: btrfs hackers <linux-btrfs@vger.kernel.org>,
Kernel hackers <linux-kernel@vger.kernel.org>
Subject: btrfs: compression breaks cp and cross-FS mv, FS_IOC_FIEMAP bug?
Date: Sun, 13 Feb 2011 17:49:42 +0200 [thread overview]
Message-ID: <AANLkTinZNk1j94rEMVdvRtjr2Sa-C7VpQOGHrvrY8Hph@mail.gmail.com> (raw)
Hi list!
It seems I have found a serious regression in compressed btrfs in
kernel 2.6.37. When creating a small file (less than the block size)
and then cp/mv it to *another* file system, an appropriate number of
zeroes gets written to the destination file. Case in point:
% echo foobar > foobar
% hexdump -C foobar
00000000 66 6f 6f 62 61 72 0a |foobar.|
00000007
% mv foobar /tmp
% hexdump -C /tmp/foobar
00000000 00 00 00 00 00 00 00 |.......|
00000007
% cp foobar foobar2
% hexdump -C foobar2
00000000 00 00 00 00 00 00 00 |.......|
00000007
Via strace I found that mv doesn't even attempt to read anything:
open("foobar", O_RDONLY|O_NOFOLLOW) = 3
fstat(3, {st_mode=S_IFREG|0664, st_size=7, ...}) = 0
open("/tmp/foobar", O_WRONLY|O_CREAT|O_EXCL, 0600) = 4
fstat(4, {st_mode=S_IFREG|0600, st_size=0, ...}) = 0
ioctl(3, FS_IOC_FIEMAP, 0x7fff62f6bfa0) = 0
write(4, "\0\0\0\0\0\0\0", 7) = 7
What's that, is FS_IOC_FIEMAP telling it that it's a sparse file?
Compare with ext4:
ioctl(3, FS_IOC_FIEMAP, 0x7fff2c576a90) = 0
lseek(3, 0, SEEK_SET) = 0
read(3, "foobar\n", 4096) = 7
write(4, "foobar\n", 7) = 7
I'm currently running on 2.6.37, x86_64 using Arch Linux -testing with
coreutils 8.10. Filesystem is mounted from LVM2 to /usr/src with -o
noatime,compress
This only seems to occur with compressed file systems (either zlib or
LZO). A person on IRC also reproduced the same problem in 2.6.28-rc.
I'm pretty sure this used to work correctly around 2.6.35 or 2.6.36.
This is 100% reproducible here. If anyone has trouble reproducing
this, I can dig further and provide information as needed.
Regards,
Marti
next reply other threads:[~2011-02-13 15:49 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-13 15:49 Marti Raudsepp [this message]
2011-02-13 15:57 ` btrfs: compression breaks cp and cross-FS mv, FS_IOC_FIEMAP bug? Josef Bacik
2011-02-13 16:07 ` Marti Raudsepp
2011-02-13 16:13 ` Josef Bacik
2011-02-14 15:01 ` Chris Mason
2011-02-14 17:58 ` Marti Raudsepp
2011-02-14 18:01 ` Chris Mason
2011-02-15 11:30 ` Pádraig Brady
2011-02-15 13:18 ` Josef Bacik
2011-02-13 16:31 ` Hugo Mills
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=AANLkTinZNk1j94rEMVdvRtjr2Sa-C7VpQOGHrvrY8Hph@mail.gmail.com \
--to=marti@juffo.org \
--cc=linux-btrfs@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).