linux-ext4.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: bugzilla-daemon@bugzilla.kernel.org
To: linux-ext4@vger.kernel.org
Subject: [Bug 36172] "kernel BUG at fs/ext4/super.c" triggered after tune2fs/remount
Date: Fri, 24 Aug 2012 14:01:37 +0000 (UTC)	[thread overview]
Message-ID: <20120824140137.E253B11FC6C@bugzilla.kernel.org> (raw)
In-Reply-To: <bug-36172-13602@https.bugzilla.kernel.org/>

https://bugzilla.kernel.org/show_bug.cgi?id=36172


Theodore Tso <tytso@mit.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |CLOSED
                 CC|                            |tytso@mit.edu
         Resolution|                            |INVALID




--- Comment #5 from Theodore Tso <tytso@mit.edu>  2012-08-24 14:01:37 ---
We should just fix this in tune2fs, and not allow the journal to be removed
when the file system is mounted, even if it is only mounted read/only.    I
don't know what I was thinking when I allowed tune2fs to do this, but it just
doesn't work.  Even with this patch, the problem is that the journal (and its
blocks) are released, and so when the file system gets remounted read/write,
the kernel will try to use journaling after the remount, and this will cause
all sorts of problems.

We could add code to ext4 to detect this case and release the journalling
structure, but it's going to be problematic for ext3, so we should just remove
this capability from tune2fs.   Fortunately it's not going to hit that many
people, so it's probably not worth it to add support in ext4 just for the older
versions of e2fsprogs; it's pretty rare that someone will want to disable the
journal while the fs is mounted.

So I'm going to close this as a kernel bug, and treat it as an e2fsprogs bug.

-- 
Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.

  parent reply	other threads:[~2012-08-24 14:01 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-29 19:23 [Bug 36172] New: "kernel BUG at fs/ext4/super.c" triggered after tune2fs/remount bugzilla-daemon
2011-05-29 21:50 ` [Bug 36172] " bugzilla-daemon
2011-05-31 19:07 ` bugzilla-daemon
2011-05-31 19:29 ` bugzilla-daemon
2012-08-24 12:48 ` bugzilla-daemon
2012-08-24 14:01 ` bugzilla-daemon [this message]
2012-08-24 15:36 ` bugzilla-daemon

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=20120824140137.E253B11FC6C@bugzilla.kernel.org \
    --to=bugzilla-daemon@bugzilla.kernel.org \
    --cc=linux-ext4@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).