jbd2: Make barrier messages less scary
authorEric Sandeen <sandeen@redhat.com>
Tue, 27 Jul 2010 15:56:08 +0000 (11:56 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Tue, 27 Jul 2010 15:56:08 +0000 (11:56 -0400)
Saying things like "sync failed" when a device does
not support barriers makes users slightly more worried than
they need to be; rather than talking about sync failures,
let's just state the barrier-based facts.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
fs/jbd2/commit.c

index 75716d3..af05681 100644 (file)
@@ -150,8 +150,8 @@ static int journal_submit_commit_record(journal_t *journal,
         */
        if (ret == -EOPNOTSUPP && barrier_done) {
                printk(KERN_WARNING
-                      "JBD: barrier-based sync failed on %s - "
-                      "disabling barriers\n", journal->j_devname);
+                      "JBD2: Disabling barriers on %s, "
+                      "not supported by device\n", journal->j_devname);
                spin_lock(&journal->j_state_lock);
                journal->j_flags &= ~JBD2_BARRIER;
                spin_unlock(&journal->j_state_lock);
@@ -180,8 +180,8 @@ retry:
        wait_on_buffer(bh);
        if (buffer_eopnotsupp(bh) && (journal->j_flags & JBD2_BARRIER)) {
                printk(KERN_WARNING
-                      "JBD2: wait_on_commit_record: sync failed on %s - "
-                      "disabling barriers\n", journal->j_devname);
+                      "JBD2: %s: disabling barries on %s - not supported "
+                      "by device\n", __func__, journal->j_devname);
                spin_lock(&journal->j_state_lock);
                journal->j_flags &= ~JBD2_BARRIER;
                spin_unlock(&journal->j_state_lock);