ext3 aborts journal

ext3 aborts journal

[Norbert Preining]

Hi all,

(please Cc)

just a few minutes ago my laptop told me:

[ 4759.267418] __log_wait_for_space: no transactions
[ 4759.267428] Aborting journal on device sda3.
[ 4759.267469] ext3_abort called.
[ 4759.267474] EXT3-fs error (device sda3): ext3_journal_start_sb: Detected aborted journal
[ 4759.267481] Remounting filesystem read-only
[ 4759.267727] journal commit I/O error
[ 4784.453791] Monitor-Mwait will be used to enter C-2 state

That made me a bit nervous. It is with kernel 2.6.28-rc3, now I rebooted
in to -rc4.

Any comments or explanations? Do I have to take any action? Reformat,
...?

Best wishes

Norbert

-------------------------------------------------------------------------------
Dr. Norbert Preining <preining@logic.at>        Vienna University of Technology
Debian Developer <preining@debian.org>                         Debian TeX Group
gpg DSA: 0x09C5B094      fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
-------------------------------------------------------------------------------
QUALL (vb.)
To speak with the voice of one who requires another to do something
for them.
			--- Douglas Adams, The Meaning of Liff

Re: ext3 aborts journal

[Eric Sandeen]

Norbert Preining wrote:
> Hi all,
> 
> (please Cc)
> 
> just a few minutes ago my laptop told me:
> 
> [ 4759.267418] __log_wait_for_space: no transactions
> [ 4759.267428] Aborting journal on device sda3.
> [ 4759.267469] ext3_abort called.
> [ 4759.267474] EXT3-fs error (device sda3): ext3_journal_start_sb: Detected aborted journal
> [ 4759.267481] Remounting filesystem read-only
> [ 4759.267727] journal commit I/O error
> [ 4784.453791] Monitor-Mwait will be used to enter C-2 state
> 
> That made me a bit nervous. It is with kernel 2.6.28-rc3, now I rebooted
> in to -rc4.
> 
> Any comments or explanations? Do I have to take any action? Reformat,
> ...?
> 
> Best wishes
> 
> Norbert

I believe this was fixed by:

From: Theodore Ts'o <tytso@mit.edu>
Date: Fri, 7 Nov 2008 03:37:59 +0000 (-0500)
Subject: jbd: don't give up looking for space so easily in
__log_wait_for_space
X-Git-Url:
http://git.kernel.org/?p=linux%2Fkernel%2Fgit%2Ftorvalds%2Flinux-2.6.git;a=commitdiff_plain;h=e219cca082f52e7dfea41f3be264b7b5eb204227

jbd: don't give up looking for space so easily in __log_wait_for_space

Commit be07c4ed introducd a regression because it assumed that if
there were no transactions ready to be checkpointed, that no progress
could be made on making space available in the journal, and so the
journal should be aborted.  This assumption is false; it could be the
case that simply calling cleanup_journal_tail() will recover the
necessary space, or, for small journals, the currently committing
transaction could be responsible for chewing up the required space in
the log, so we need to wait for the currently committing transaction
to finish before trying to force a checkpoint operation.

This patch fixes the bug reported by Meelis Roos at:
http://bugzilla.kernel.org/show_bug.cgi?id=11937

Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Cc: Duane Griffin <duaneg@dghda.com>
Cc: Toshiyuki Okajima <toshi.okajima@jp.fujitsu.com>
---

-Eric

> -------------------------------------------------------------------------------
> Dr. Norbert Preining <preining@logic.at>        Vienna University of Technology
> Debian Developer <preining@debian.org>                         Debian TeX Group
> gpg DSA: 0x09C5B094      fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
> -------------------------------------------------------------------------------
> QUALL (vb.)
> To speak with the voice of one who requires another to do something
> for them.
> 			--- Douglas Adams, The Meaning of Liff
> --
> To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: ext3 aborts journal

[Norbert Preining]

Dear Eric,

On Mo, 10 Nov 2008, Eric Sandeen wrote:
> I believe this was fixed by:

thanks, that seems to be in rc4, at least patch did not apply, but the
code in checkpoint.c looks suspiciously like a variant of this patch has
already been applied.

Thanks a lot again

Norbert

-------------------------------------------------------------------------------
Dr. Norbert Preining <preining@logic.at>        Vienna University of Technology
Debian Developer <preining@debian.org>                         Debian TeX Group
gpg DSA: 0x09C5B094      fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
-------------------------------------------------------------------------------
TINCLETON (n.)
A man who amuses himself in your lavatory by pulling the chain in
mid-pee and then seeing if he can finish before the flush does.
			--- Douglas Adams, The Meaning of Liff

Re: ext3 aborts journal

[Theodore Tso]

On Tue, Nov 11, 2008 at 10:43:46AM +0100, Norbert Preining wrote:
> Dear Eric,
> 
> On Mo, 10 Nov 2008, Eric Sandeen wrote:
> > I believe this was fixed by:
> 
> thanks, that seems to be in rc4, at least patch did not apply, but the
> code in checkpoint.c looks suspiciously like a variant of this patch has
> already been applied.

Correct, the fix is in -rc4.

Regards,

						- Ted