From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Sandeen Subject: Re: [PATCH] jbd2: Make barrier messages less scary Date: Tue, 27 Jul 2010 09:21:37 -0500 Message-ID: <4C4EEB71.4080709@redhat.com> References: <4C4DE515.5020307@redhat.com> <20100727135823.GG18966@thunk.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: ext4 development To: "Ted Ts'o" Return-path: Received: from mx1.redhat.com ([209.132.183.28]:39862 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754476Ab0G0OVm (ORCPT ); Tue, 27 Jul 2010 10:21:42 -0400 In-Reply-To: <20100727135823.GG18966@thunk.org> Sender: linux-ext4-owner@vger.kernel.org List-ID: Ted Ts'o wrote: > On Mon, Jul 26, 2010 at 02:42:13PM -0500, Eric Sandeen wrote: >> 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 >> --- >> >> diff --git a/fs/jbd2/commit.c b/fs/jbd2/commit.c >> index 75716d3..3d15b28 100644 >> --- a/fs/jbd2/commit.c >> +++ b/fs/jbd2/commit.c >> @@ -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: barriers no longer supported on %s - " >> + "disabling barriers\n", __func__, journal->j_devname); > > Any reason this message has "no longer supported" as opposed to the > "not supported by device" printed above? Hm. In the non-async case, we'd have already tested it in journal_submit_commit_record, right, and if we failed here, they got lost somehow. But in the async case, this is the first test we'd get... so maybe it should just use the same message as the first. -Eric > - Ted