2008-07-25 23:06:30

by Andrew Morton

[permalink] [raw]
Subject: - jbd-replace-potentially-false-assertion-with-if-block.patch removed from -mm tree

The patch titled
jbd: replace potentially false assertion with if block
has been removed from the -mm tree. Its filename was

This patch was dropped because it was merged into mainline or a subsystem tree

The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/

Subject: jbd: replace potentially false assertion with if block
From: "Duane Griffin" <[email protected]>

If an error occurs during jbd cache initialisation it is possible for the
journal_head_cache to be NULL when journal_destroy_journal_head_cache is
called. Replace the J_ASSERT with an if block to handle the situation

Note that even with this fix things will break badly if jbd is statically
compiled in and cache initialisation fails.

Signed-off-by: Duane Griffin <[email protected]
Cc: <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>

fs/jbd/journal.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)

diff -puN fs/jbd/journal.c~jbd-replace-potentially-false-assertion-with-if-block fs/jbd/journal.c
--- a/fs/jbd/journal.c~jbd-replace-potentially-false-assertion-with-if-block
+++ a/fs/jbd/journal.c
@@ -1636,9 +1636,10 @@ static int journal_init_journal_head_cac

static void journal_destroy_journal_head_cache(void)
- J_ASSERT(journal_head_cache != NULL);
- kmem_cache_destroy(journal_head_cache);
- journal_head_cache = NULL;
+ if (journal_head_cache) {
+ kmem_cache_destroy(journal_head_cache);
+ journal_head_cache = NULL;
+ }


Patches currently in -mm which might be from [email protected] are