From: Eric Sandeen <sandeen@redhat.com>
To: Christoph Anton Mitterer <calestyo@scientia.net>,
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: Mon, 7 Dec 2015 17:06:34 -0600 [thread overview]
Message-ID: <566610FA.1080408@redhat.com> (raw)
In-Reply-To: <1449521684.6881.13.camel@scientia.net>
On 12/7/15 2:54 PM, Christoph Anton Mitterer wrote:
...
> 2) a section that describes "ro" in btrfs-mount(5) which describes that
> normal "ro" alone may cause changes on the device and which then refers
> to hard-ro and/or the list of options (currently nologreplay) which are
> required right now to make it truly ro.
>
>
> I think this is important as an end-user probably expects "ro" to be
> truly ro,
Yeah, I don't know that this is true. It hasn't been true for over a
decade (2?), with the most widely-used filesystem in linux history, i.e.
ext3. So if btrfs wants to go on this re-education crusade, more power
to you, but I don't know that it's really a fight worth fighting. ;)
> so if he looks it up in the documentation (2) he should find
> enough information that this isn't the case and what to do instead.
> Further, as one might expect that in the future, other places (than
> just the log) may cause changes to a device, even though mounted ro...
> I think it's better for the end user to also have a real "hard-ro" like
> option, than a possibly growing list of "noXYZ" where the end-user may
> have no clue that something else is now also required to get the truly
> read-only behaviour.
# blockdev --setro ...
-Eric
next prev parent reply other threads:[~2015-12-07 23:06 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 [this message]
2015-12-08 0:00 ` Christoph Anton Mitterer
2015-12-08 12:15 ` Austin S Hemmelgarn
2015-12-08 19:20 ` Christoph Anton Mitterer
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=566610FA.1080408@redhat.com \
--to=sandeen@redhat.com \
--cc=calestyo@scientia.net \
--cc=chandan@linux.vnet.ibm.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=quwenruo@cn.fujitsu.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.