diff options
author | Dave Kleikamp <shaggy@austin.ibm.com> | 2005-06-03 15:09:54 -0400 |
---|---|---|
committer | Dave Kleikamp <shaggy@austin.ibm.com> | 2005-06-03 15:09:54 -0400 |
commit | 72e3148a6e987974e3e949c5668e5ca812d7c818 (patch) | |
tree | abdf33dbbe33029ab81e7a209f11f29d2bc6b801 | |
parent | c2731509cfb538b9b38feaf657fab2334ea45253 (diff) |
JFS: Fix compiler warning in jfs_logmgr.c
fs/jfs/jfs_logmgr.c: In function `jfs_flush_journal':
fs/jfs/jfs_logmgr.c:1632: warning: unused variable `mp'
Some debug code in jfs_flush_journal does nothing when CONFIG_JFS_DEBUG
is not defined. Place the whole code segment within an ifdef to avoid
unnecessary code to be compiled and the warning to be issued.
Signed-off-by: Dave Kleikamp <shaggy@austin.ibm.com>
-rw-r--r-- | fs/jfs/jfs_logmgr.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/jfs/jfs_logmgr.c b/fs/jfs/jfs_logmgr.c index c5cc03bcae69..7c8387ed4192 100644 --- a/fs/jfs/jfs_logmgr.c +++ b/fs/jfs/jfs_logmgr.c | |||
@@ -1617,6 +1617,8 @@ void jfs_flush_journal(struct jfs_log *log, int wait) | |||
1617 | } | 1617 | } |
1618 | } | 1618 | } |
1619 | assert(list_empty(&log->cqueue)); | 1619 | assert(list_empty(&log->cqueue)); |
1620 | |||
1621 | #ifdef CONFIG_JFS_DEBUG | ||
1620 | if (!list_empty(&log->synclist)) { | 1622 | if (!list_empty(&log->synclist)) { |
1621 | struct logsyncblk *lp; | 1623 | struct logsyncblk *lp; |
1622 | 1624 | ||
@@ -1631,9 +1633,8 @@ void jfs_flush_journal(struct jfs_log *log, int wait) | |||
1631 | dump_mem("orphan tblock", lp, | 1633 | dump_mem("orphan tblock", lp, |
1632 | sizeof(struct tblock)); | 1634 | sizeof(struct tblock)); |
1633 | } | 1635 | } |
1634 | // current->state = TASK_INTERRUPTIBLE; | ||
1635 | // schedule(); | ||
1636 | } | 1636 | } |
1637 | #endif | ||
1637 | //assert(list_empty(&log->synclist)); | 1638 | //assert(list_empty(&log->synclist)); |
1638 | clear_bit(log_FLUSH, &log->flag); | 1639 | clear_bit(log_FLUSH, &log->flag); |
1639 | } | 1640 | } |