linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Austin S. Hemmelgarn" <ahferroin7@gmail.com>
To: linux-btrfs@vger.kernel.org
Subject: Re: Problem with file system
Date: Fri, 3 Nov 2017 07:33:25 -0400	[thread overview]
Message-ID: <4d26d20d-5c07-ccb3-b26c-0c4876b8fe3a@gmail.com> (raw)
In-Reply-To: <20171103084222.05a4e226@jupiter.sol.kaishome.de>

On 2017-11-03 03:42, Kai Krakow wrote:
> Am Tue, 31 Oct 2017 07:28:58 -0400
> schrieb "Austin S. Hemmelgarn" <ahferroin7@gmail.com>:
> 
>> On 2017-10-31 01:57, Marat Khalili wrote:
>>> On 31/10/17 00:37, Chris Murphy wrote:
>>>> But off hand it sounds like hardware was sabotaging the expected
>>>> write ordering. How to test a given hardware setup for that, I
>>>> think, is really overdue. It affects literally every file system,
>>>> and Linux storage technology.
>>>>
>>>> It kinda sounds like to me something other than supers is being
>>>> overwritten too soon, and that's why it's possible for none of the
>>>> backup roots to find a valid root tree, because all four possible
>>>> root trees either haven't actually been written yet (still) or
>>>> they've been overwritten, even though the super is updated. But
>>>> again, it's speculation, we don't actually know why your system
>>>> was no longer mountable.
>>> Just a detached view: I know hardware should respect
>>> ordering/barriers and such, but how hard is it really to avoid
>>> overwriting at least one complete metadata tree for half an hour
>>> (even better, yet another one for a day)? Just metadata, not data
>>> extents.
>> If you're running on an SSD (or thinly provisioned storage, or
>> something else which supports discards) and have the 'discard' mount
>> option enabled, then there is no backup metadata tree (this issue was
>> mentioned on the list a while ago, but nobody ever replied), because
>> it's already been discarded.  This is ideally something which should
>> be addressed (we need some sort of discard queue for handling in-line
>> discards), but it's not easy to address.
>>
>> Otherwise, it becomes a question of space usage on the filesystem,
>> and this is just another reason to keep some extra slack space on the
>> FS (though that doesn't help _much_, it does help).  This, in theory,
>> could be addressed, but it probably can't be applied across mounts of
>> a filesystem without an on-disk format change.
> 
> Well, maybe inline discard is working at the wrong level. It should
> kick in when the reference through any of the backup roots is dropped,
> not when the current instance is dropped.
Indeed.
> 
> Without knowledge of the internals, I guess discards could be added to
> a queue within a new tree in btrfs, and only added to that queue when
> dropped from the last backup root referencing it. But this will
> probably add some bad performance spikes.
Inline discards can already cause bad performance spikes.
> 
> I wonder how a regular fstrim run through cron applies to this problem?
You functionally lose any old (freed) trees, they just get kept around 
until you call fstrim.


  reply	other threads:[~2017-11-03 11:33 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-24 15:27 Problem with file system Fred Van Andel
2017-04-24 17:02 ` Chris Murphy
2017-04-25  4:05   ` Duncan
2017-04-25  0:26 ` Qu Wenruo
2017-04-25  5:33   ` Marat Khalili
2017-04-25  6:13     ` Qu Wenruo
2017-04-26 16:43       ` Fred Van Andel
2017-10-30  3:31         ` Dave
2017-10-30 21:37           ` Chris Murphy
2017-10-31  5:57             ` Marat Khalili
2017-10-31 11:28               ` Austin S. Hemmelgarn
2017-11-03  7:42                 ` Kai Krakow
2017-11-03 11:33                   ` Austin S. Hemmelgarn [this message]
2017-11-03 22:03                 ` Chris Murphy
2017-11-04  4:46                   ` Adam Borowski
2017-11-04 12:00                     ` Marat Khalili
2017-11-04 17:14                     ` Chris Murphy
2017-11-06 13:29                       ` Austin S. Hemmelgarn
2017-11-06 18:45                         ` Chris Murphy
2017-11-06 19:12                           ` Austin S. Hemmelgarn
2017-11-04  7:26             ` Dave
2017-11-04 17:25               ` Chris Murphy
2017-11-07  7:01                 ` Dave
2017-11-07 13:02                   ` Austin S. Hemmelgarn
2017-11-08  4:50                     ` Chris Murphy
2017-11-08 12:13                       ` Austin S. Hemmelgarn
2017-11-08 17:17                         ` Chris Murphy
2017-11-08 17:22                           ` Hugo Mills
2017-11-08 17:54                             ` Chris Murphy
2017-11-08 18:10                               ` Austin S. Hemmelgarn
2017-11-08 18:31                                 ` Chris Murphy
2017-11-08 19:29                                   ` Austin S. Hemmelgarn
2017-10-31  1:58           ` Duncan

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=4d26d20d-5c07-ccb3-b26c-0c4876b8fe3a@gmail.com \
    --to=ahferroin7@gmail.com \
    --cc=linux-btrfs@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).