Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1763617AbYCGBbk (ORCPT ); Thu, 6 Mar 2008 20:31:40 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757619AbYCGBb3 (ORCPT ); Thu, 6 Mar 2008 20:31:29 -0500 Received: from kumera.dghda.com ([80.68.90.171]:1981 "EHLO kumera.dghda.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756860AbYCGBb2 (ORCPT ); Thu, 6 Mar 2008 20:31:28 -0500 From: "Duane Griffin" To: linux-ext4@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Theodore Tso , sct@redhat.com, akpm@linux-foundation.org, adilger@clusterfs.com, Duane Griffin Subject: [PATCH 1/3] jbd2: eliminate duplicated code in revocation table init/destroy functions Date: Fri, 7 Mar 2008 01:31:22 +0000 Message-Id: <1204853484-25968-1-git-send-email-duaneg@dghda.com> X-Mailer: git-send-email 1.5.3.7 Message-Id: <5e28cd633c71f6354a203a43000cbe5fef045589.1204844851.git.duaneg@dghda.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5830 Lines: 186 The revocation table initialisation/destruction code is repeated for each of the two revocation tables stored in the journal. Refactoring the duplicated code into functions is tidier, simplifies the logic in initialisation in particular, and slightly reduces the code size. There should not be any functional change. Signed-off-by: Duane Griffin --- fs/jbd2/revoke.c | 125 +++++++++++++++++++++++------------------------------- 1 files changed, 53 insertions(+), 72 deletions(-) diff --git a/fs/jbd2/revoke.c b/fs/jbd2/revoke.c index df36f42..1bf4c1f 100644 --- a/fs/jbd2/revoke.c +++ b/fs/jbd2/revoke.c @@ -196,108 +196,89 @@ void jbd2_journal_destroy_revoke_caches(void) jbd2_revoke_table_cache = NULL; } -/* Initialise the revoke table for a given journal to a given size. */ - -int jbd2_journal_init_revoke(journal_t *journal, int hash_size) +static int jbd2_journal_init_revoke_table(struct jbd2_revoke_table_s *table, + int size) { - int shift, tmp; + int shift = 0; + int tmp = size; - J_ASSERT (journal->j_revoke_table[0] == NULL); - - shift = 0; - tmp = hash_size; while((tmp >>= 1UL) != 0UL) shift++; - journal->j_revoke_table[0] = kmem_cache_alloc(jbd2_revoke_table_cache, GFP_KERNEL); - if (!journal->j_revoke_table[0]) - return -ENOMEM; - journal->j_revoke = journal->j_revoke_table[0]; - - /* Check that the hash_size is a power of two */ - J_ASSERT(is_power_of_2(hash_size)); - - journal->j_revoke->hash_size = hash_size; - - journal->j_revoke->hash_shift = shift; - - journal->j_revoke->hash_table = - kmalloc(hash_size * sizeof(struct list_head), GFP_KERNEL); - if (!journal->j_revoke->hash_table) { - kmem_cache_free(jbd2_revoke_table_cache, journal->j_revoke_table[0]); - journal->j_revoke = NULL; + table->hash_size = size; + table->hash_shift = shift; + table->hash_table = kmalloc( + size * sizeof(struct list_head), GFP_KERNEL); + if (!table->hash_table) return -ENOMEM; - } - for (tmp = 0; tmp < hash_size; tmp++) - INIT_LIST_HEAD(&journal->j_revoke->hash_table[tmp]); + for (tmp = 0; tmp < size; tmp++) + INIT_LIST_HEAD(&table->hash_table[tmp]); - journal->j_revoke_table[1] = kmem_cache_alloc(jbd2_revoke_table_cache, GFP_KERNEL); - if (!journal->j_revoke_table[1]) { - kfree(journal->j_revoke_table[0]->hash_table); - kmem_cache_free(jbd2_revoke_table_cache, journal->j_revoke_table[0]); - return -ENOMEM; - } - - journal->j_revoke = journal->j_revoke_table[1]; + return 0; +} - /* Check that the hash_size is a power of two */ +/* Initialise the revoke table for a given journal to a given size. */ +int jbd2_journal_init_revoke(journal_t *journal, int hash_size) +{ + J_ASSERT(journal->j_revoke_table[0] == NULL); J_ASSERT(is_power_of_2(hash_size)); - journal->j_revoke->hash_size = hash_size; + journal->j_revoke_table[0] = kmem_cache_alloc( + jbd2_revoke_table_cache, GFP_KERNEL); + if (!journal->j_revoke_table[0]) + goto failed_alloc1; + if (jbd2_journal_init_revoke_table(journal->j_revoke_table[0], hash_size)) + goto failed_init1; - journal->j_revoke->hash_shift = shift; + journal->j_revoke_table[1] = kmem_cache_alloc( + jbd2_revoke_table_cache, GFP_KERNEL); + if (!journal->j_revoke_table[1]) + goto failed_alloc2; + if (jbd2_journal_init_revoke_table(journal->j_revoke_table[1], hash_size)) + goto failed_init2; - journal->j_revoke->hash_table = - kmalloc(hash_size * sizeof(struct list_head), GFP_KERNEL); - if (!journal->j_revoke->hash_table) { - kfree(journal->j_revoke_table[0]->hash_table); - kmem_cache_free(jbd2_revoke_table_cache, journal->j_revoke_table[0]); - kmem_cache_free(jbd2_revoke_table_cache, journal->j_revoke_table[1]); - journal->j_revoke = NULL; - return -ENOMEM; - } - - for (tmp = 0; tmp < hash_size; tmp++) - INIT_LIST_HEAD(&journal->j_revoke->hash_table[tmp]); + journal->j_revoke = journal->j_revoke_table[1]; spin_lock_init(&journal->j_revoke_lock); return 0; -} -/* Destoy a journal's revoke table. The table must already be empty! */ +failed_init2: + kmem_cache_free(jbd2_revoke_table_cache, journal->j_revoke_table[1]); +failed_alloc2: + kfree(journal->j_revoke_table[0]->hash_table); +failed_init1: + kmem_cache_free(jbd2_revoke_table_cache, journal->j_revoke_table[0]); +failed_alloc1: + return -ENOMEM; +} -void jbd2_journal_destroy_revoke(journal_t *journal) +static void jbd2_journal_destroy_revoke_table(struct jbd2_revoke_table_s *table) { - struct jbd2_revoke_table_s *table; - struct list_head *hash_list; int i; + struct list_head *hash_list; - table = journal->j_revoke_table[0]; - if (!table) - return; - - for (i=0; ihash_size; i++) { + for (i = 0; i < table->hash_size; i++) { hash_list = &table->hash_table[i]; - J_ASSERT (list_empty(hash_list)); + J_ASSERT(list_empty(hash_list)); } kfree(table->hash_table); kmem_cache_free(jbd2_revoke_table_cache, table); - journal->j_revoke = NULL; +} - table = journal->j_revoke_table[1]; - if (!table) +/* Destroy a journal's revoke table. The table must already be empty! */ +void jbd2_journal_destroy_revoke(journal_t *journal) +{ + if (!journal->j_revoke_table[0]) return; + jbd2_journal_destroy_revoke_table(journal->j_revoke_table[0]); + journal->j_revoke = NULL; - for (i=0; ihash_size; i++) { - hash_list = &table->hash_table[i]; - J_ASSERT (list_empty(hash_list)); - } - - kfree(table->hash_table); - kmem_cache_free(jbd2_revoke_table_cache, table); + if (!journal->j_revoke_table[1]) + return; + jbd2_journal_destroy_revoke_table(journal->j_revoke_table[1]); journal->j_revoke = NULL; } -- 1.5.3.7 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/