From: Christoph Anton Mitterer <calestyo@scientia.net>
To: Austin S Hemmelgarn <ahferroin7@gmail.com>,
Eric Sandeen <sandeen@redhat.com>,
Chandan Rajendra <chandan@linux.vnet.ibm.com>
Cc: Qu Wenruo <quwenruo@cn.fujitsu.com>, linux-btrfs@vger.kernel.org
Subject: Re: [PATCH] btrfs: Introduce new mount option to disable tree log replay
Date: Tue, 08 Dec 2015 20:20:02 +0100 [thread overview]
Message-ID: <1449602402.6649.6.camel@scientia.net> (raw)
In-Reply-To: <5666C9F7.2030509@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1108 bytes --]
On Tue, 2015-12-08 at 07:15 -0500, Austin S Hemmelgarn wrote:
> Despite this, it really isn't a widely known or well documented
> behavior
> outside of developers, forensic specialists, and people who have had
> to
> deal with the implications it has on data recovery. There really
> isn't
> any way that the user would know about it without being explicitly
> told,
> and it's something that can have a serious impact on being able to
> recover a broken filesystem. TBH, I really feel that _every_
> filesystem's documentation should have something about how to make it
> mount truly read-only, even if it's just a reference to how to mark
> the
> block device read-only.
Exactly what I've meant.
And the developers here, should definitely consider that every normal
end-user, may easily assume the role of e.g. a forensics specialist
(especially with btrfs ;-) ), when recovery in case of corruptions is
tried.
I don't think that "it has always been improperly documented" (i.e. the
"ro" option) is a good excuse to continue doing it that way =)
Cheers,
Chris.
[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 5313 bytes --]
next prev parent reply other threads:[~2015-12-08 19:20 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-07 6:06 [PATCH] btrfs: Introduce new mount option to disable tree log replay Qu Wenruo
2015-12-07 15:38 ` Chandan Rajendra
2015-12-07 23:51 ` Qu Wenruo
2015-12-07 16:27 ` Eric Sandeen
2015-12-07 16:52 ` Chandan Rajendra
2015-12-07 17:29 ` Eric Sandeen
2015-12-07 20:54 ` Christoph Anton Mitterer
2015-12-07 23:06 ` Eric Sandeen
2015-12-08 0:00 ` Christoph Anton Mitterer
2015-12-08 12:15 ` Austin S Hemmelgarn
2015-12-08 19:20 ` Christoph Anton Mitterer [this message]
2015-12-08 20:29 ` Austin S Hemmelgarn
2015-12-07 16:36 ` Austin S Hemmelgarn
2015-12-08 6:08 ` Qu Wenruo
2015-12-08 12:16 ` Austin S Hemmelgarn
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=1449602402.6649.6.camel@scientia.net \
--to=calestyo@scientia.net \
--cc=ahferroin7@gmail.com \
--cc=chandan@linux.vnet.ibm.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=quwenruo@cn.fujitsu.com \
--cc=sandeen@redhat.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.