diff options
author | Namhyung Kim <namhyung@gmail.com> | 2010-10-04 04:26:53 -0400 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2010-10-27 19:30:01 -0400 |
commit | c5639bef63a241d8d5a00e5243d7f304d7563c46 (patch) | |
tree | f0db02b256104af545b5da0eb40627b9367f12b4 /fs/jbd | |
parent | 26f78b7a423aea094c536e94cbe1f1fa4641dd7b (diff) |
jbd: Move debug message into #ifdef area
Move call to jbd_debug() into #ifdef CONFIG_JBD_DEBUG block because
'dropped' is declared there. The code could be compiled without this
change anyway, simply because jbd_debug() expands to nothing if
!CONFIG_JBD_DEBUG but IMHO it doesn't look good in general.
Signed-off-by: Namhyung Kim <namhyung@gmail.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/jbd')
-rw-r--r-- | fs/jbd/recovery.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/jbd/recovery.c b/fs/jbd/recovery.c index 81051dafebf5..5b43e96788e6 100644 --- a/fs/jbd/recovery.c +++ b/fs/jbd/recovery.c | |||
@@ -296,10 +296,10 @@ int journal_skip_recovery(journal_t *journal) | |||
296 | #ifdef CONFIG_JBD_DEBUG | 296 | #ifdef CONFIG_JBD_DEBUG |
297 | int dropped = info.end_transaction - | 297 | int dropped = info.end_transaction - |
298 | be32_to_cpu(journal->j_superblock->s_sequence); | 298 | be32_to_cpu(journal->j_superblock->s_sequence); |
299 | #endif | ||
300 | jbd_debug(1, | 299 | jbd_debug(1, |
301 | "JBD: ignoring %d transaction%s from the journal.\n", | 300 | "JBD: ignoring %d transaction%s from the journal.\n", |
302 | dropped, (dropped == 1) ? "" : "s"); | 301 | dropped, (dropped == 1) ? "" : "s"); |
302 | #endif | ||
303 | journal->j_transaction_sequence = ++info.end_transaction; | 303 | journal->j_transaction_sequence = ++info.end_transaction; |
304 | } | 304 | } |
305 | 305 | ||