diff options
author | Jan Kara <jack@suse.cz> | 2013-11-25 17:30:26 -0500 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2013-12-04 06:27:46 -0500 |
commit | 301d4c9a286bc7dc4fb3cda21131be91a582fa79 (patch) | |
tree | 0376fdc743ce2d403db16fb07dbcdc39e3310adb /fs/jbd/journal.c | |
parent | df4e7ac0bb70abc97fbfd9ef09671fc084b3f9db (diff) |
jbd: Revise KERN_EMERG error messages
Some of KERN_EMERG printk messages do not really deserve this log level
and the one in log_wait_commit() is even rather useless (the journal has
been previously aborted and *that* is where we should have been
complaining). So make some messages just KERN_ERR and remove the useless
message.
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/jbd/journal.c')
-rw-r--r-- | fs/jbd/journal.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/fs/jbd/journal.c b/fs/jbd/journal.c index 2d04f9afafd7..06fe11e0abfa 100644 --- a/fs/jbd/journal.c +++ b/fs/jbd/journal.c | |||
@@ -573,7 +573,7 @@ int log_wait_commit(journal_t *journal, tid_t tid) | |||
573 | #ifdef CONFIG_JBD_DEBUG | 573 | #ifdef CONFIG_JBD_DEBUG |
574 | spin_lock(&journal->j_state_lock); | 574 | spin_lock(&journal->j_state_lock); |
575 | if (!tid_geq(journal->j_commit_request, tid)) { | 575 | if (!tid_geq(journal->j_commit_request, tid)) { |
576 | printk(KERN_EMERG | 576 | printk(KERN_ERR |
577 | "%s: error: j_commit_request=%d, tid=%d\n", | 577 | "%s: error: j_commit_request=%d, tid=%d\n", |
578 | __func__, journal->j_commit_request, tid); | 578 | __func__, journal->j_commit_request, tid); |
579 | } | 579 | } |
@@ -604,10 +604,8 @@ int log_wait_commit(journal_t *journal, tid_t tid) | |||
604 | out_unlock: | 604 | out_unlock: |
605 | spin_unlock(&journal->j_state_lock); | 605 | spin_unlock(&journal->j_state_lock); |
606 | 606 | ||
607 | if (unlikely(is_journal_aborted(journal))) { | 607 | if (unlikely(is_journal_aborted(journal))) |
608 | printk(KERN_EMERG "journal commit I/O error\n"); | ||
609 | err = -EIO; | 608 | err = -EIO; |
610 | } | ||
611 | return err; | 609 | return err; |
612 | } | 610 | } |
613 | 611 | ||
@@ -2136,7 +2134,7 @@ static void __exit journal_exit(void) | |||
2136 | #ifdef CONFIG_JBD_DEBUG | 2134 | #ifdef CONFIG_JBD_DEBUG |
2137 | int n = atomic_read(&nr_journal_heads); | 2135 | int n = atomic_read(&nr_journal_heads); |
2138 | if (n) | 2136 | if (n) |
2139 | printk(KERN_EMERG "JBD: leaked %d journal_heads!\n", n); | 2137 | printk(KERN_ERR "JBD: leaked %d journal_heads!\n", n); |
2140 | #endif | 2138 | #endif |
2141 | jbd_remove_debugfs_entry(); | 2139 | jbd_remove_debugfs_entry(); |
2142 | journal_destroy_caches(); | 2140 | journal_destroy_caches(); |