linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Austin S. Hemmelgarn" <ahferroin7@gmail.com>
To: Adam Borowski <kilobyte@angband.pl>, Marat Khalili <mkh@rqc.ru>
Cc: Duncan <1i5t5.duncan@cox.net>, linux-btrfs <linux-btrfs@vger.kernel.org>
Subject: Re: qemu-kvm VM died during partial raid1 problems of btrfs
Date: Tue, 12 Sep 2017 13:36:48 -0400	[thread overview]
Message-ID: <d1f6450c-fc09-171c-82c8-32caa2cbd230@gmail.com> (raw)
In-Reply-To: <20170912172125.rb6gtqdxqneb36js@angband.pl>

On 2017-09-12 13:21, Adam Borowski wrote:
> On Tue, Sep 12, 2017 at 02:26:39PM +0300, Marat Khalili wrote:
>> On 12/09/17 14:12, Adam Borowski wrote:
>>> Why would you need support in the hypervisor if cp --reflink=always is
>>> enough?
>> +1 :)
>>
>> But I've already found one problem: I use rsync snapshots for backups, and
>> although rsync does have --sparse argument, apparently it conflicts with
>> --inplace. You cannot have all nice things :(
(Replying here to the above, as I can't seem to find the original in my 
e-mail client to reply to)

--inplace and --sparse are inherently at odds with each other.  The only 
way that they could work together is if rsync was taught about the 
FALLOCATE_PUNCH_HOLES ioctl, and that isn't likely to ever happen 
because it's Linux specific (at least, it's functionally Linux 
specific).  Without that ioctl, the only way to create a sparse file is 
to seek over areas that are supposed to be empty when writing the file 
out initially, but you can't do that with an existing file because you 
then have old data where you're supposed to have zeroes.
> 
> There's fallocate -d, but that for some reason touches mtime which makes
> rsync go again.  This can be handled manually but is still not nice.It touches mtime because it updates the block allocations, which in turn 
touch ctime, which on most (possibly all, not sure though) POSIX systems 
implies an mtime update.  It's essentially the same as truncate updating 
the mtime when you extend the file, the only difference is that the 
FALLOCATE_PUNCH_HOLES ioctl doesn't change the file size.

  reply	other threads:[~2017-09-12 17:36 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-12  8:02 qemu-kvm VM died during partial raid1 problems of btrfs Marat Khalili
2017-09-12  8:25 ` Timofey Titovets
2017-09-12  8:42   ` Marat Khalili
2017-09-12  9:21     ` Timofey Titovets
2017-09-12  9:29       ` Marat Khalili
2017-09-12  9:35         ` Timofey Titovets
2017-09-12 10:01     ` Duncan
2017-09-12 10:32       ` Adam Borowski
2017-09-12 10:39         ` Marat Khalili
2017-09-12 11:01           ` Timofey Titovets
2017-09-12 11:12             ` Adam Borowski
2017-09-12 11:17               ` Timofey Titovets
2017-09-12 11:26               ` Marat Khalili
2017-09-12 17:21                 ` Adam Borowski
2017-09-12 17:36                   ` Austin S. Hemmelgarn [this message]
2017-09-12 18:43                     ` Adam Borowski
2017-09-12 18:47                       ` Christoph Hellwig
2017-09-12 19:12                         ` Austin S. Hemmelgarn
2017-09-12 19:11                       ` Austin S. Hemmelgarn
2017-09-12 20:00                         ` Adam Borowski
2017-09-12 20:12                           ` Austin S. Hemmelgarn
2017-09-12 21:13                             ` Adam Borowski
2017-09-13  0:52                               ` Timofey Titovets
2017-09-13 12:55                                 ` Austin S. Hemmelgarn
2017-09-13 12:21                               ` Austin S. Hemmelgarn
2017-09-18 11:53                                 ` Adam Borowski
2017-09-13 14:47                               ` Martin Raiber
2017-09-13 15:25                                 ` Austin S. Hemmelgarn
2017-09-12 11:09         ` Roman Mamedov
2017-09-13 13:23 ` Chris Murphy
2017-09-13 14:15   ` Marat Khalili
2017-09-13 17:52     ` Goffredo Baroncelli

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=d1f6450c-fc09-171c-82c8-32caa2cbd230@gmail.com \
    --to=ahferroin7@gmail.com \
    --cc=1i5t5.duncan@cox.net \
    --cc=kilobyte@angband.pl \
    --cc=linux-btrfs@vger.kernel.org \
    --cc=mkh@rqc.ru \
    /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).