2008-11-10 08:22:42

by Norbert Preining

[permalink] [raw]
Subject: ext3 aborts journal

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 <[email protected]> Vienna University of Technology
Debian Developer <[email protected]> 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


2008-11-10 16:01:53

by Eric Sandeen

[permalink] [raw]
Subject: Re: ext3 aborts journal

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 <[email protected]>
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" <[email protected]>
Cc: Duane Griffin <[email protected]>
Cc: Toshiyuki Okajima <[email protected]>
---

-Eric

> -------------------------------------------------------------------------------
> Dr. Norbert Preining <[email protected]> Vienna University of Technology
> Debian Developer <[email protected]> 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 [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html


2008-11-11 09:43:48

by Norbert Preining

[permalink] [raw]
Subject: Re: ext3 aborts journal

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 <[email protected]> Vienna University of Technology
Debian Developer <[email protected]> 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

2008-11-11 14:53:09

by Theodore Ts'o

[permalink] [raw]
Subject: Re: ext3 aborts journal

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