From: "Duane Griffin" Subject: Re: [RFC, PATCH 4/6] jbd: refactor nested journal log recovery loop into separate functions Date: Wed, 12 Mar 2008 01:02:05 +0000 Message-ID: <20080312010205.GA14667@dastardly.plus.com> References: <8644b32ddec999bbc1da0ac55ad7b66d0b8176de.1204685366.git.duaneg@dghda.com> <20080311143550.GB6544@atrey.karlin.mff.cuni.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Duane Griffin , linux-ext4@vger.kernel.org, linux-kernel@vger.kernel.org, Theodore Tso , sct@redhat.com, akpm@linux-foundation.org, adilger@clusterfs.com To: Jan Kara Return-path: Received: from kumera.dghda.com ([80.68.90.171]:2756 "EHLO kumera.dghda.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751524AbYCLBCN (ORCPT ); Tue, 11 Mar 2008 21:02:13 -0400 Content-Disposition: inline In-Reply-To: <20080311143550.GB6544@atrey.karlin.mff.cuni.cz> Sender: linux-ext4-owner@vger.kernel.org List-ID: On Tue, Mar 11, 2008 at 03:35:50PM +0100, Jan Kara wrote: > Hmm, if I read your patch correctly, previously we aborted journal > replay when we found a block with unknown block magic but now we > continue replaying. Why have you done such change? And similarly when > some error happened when parsing revoke records block... You are right, that was an error on my part. Thanks for your careful review. Please find below an incremental patch. I'll roll this into the main patch before posting the next version. Cheers, Duane. -- "I never could learn to drink that blood and call it wine" - Bob Dylan diff --git a/fs/jbd/recovery.c b/fs/jbd/recovery.c index 453c5fe..34db55a 100644 --- a/fs/jbd/recovery.c +++ b/fs/jbd/recovery.c @@ -673,16 +673,22 @@ static int do_one_pass(journal_t *journal, case JFS_REVOKE_BLOCK: /* If we aren't in the REVOKE pass, then we can * just skip over this block. */ - if (pass == PASS_REVOKE) { - err = scan_revoke_records( - journal, bh, next_commit_ID, info); + if (pass != PASS_REVOKE) + break; + + err = scan_revoke_records(journal, bh, + next_commit_ID, info); + if (err) { + brelse(bh); + goto failed; } break; default: jbd_debug(3, "Unrecognised magic %d, end of scan.\n", blocktype); - break; + brelse(bh); + goto done; } brelse(bh); @@ -695,6 +701,7 @@ static int do_one_pass(journal_t *journal, goto failed; } +done: /* * We broke out of the log scan loop: either we came to the * known end of the log or we found an unexpected block in the