public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jens Axboe <axboe@suse.de>
To: Andrew Morton <akpm@osdl.org>
Cc: Chris Mason <mason@suse.com>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ext3 barrier bits
Date: Sat, 22 May 2004 10:42:15 +0200	[thread overview]
Message-ID: <20040522084214.GS1952@suse.de> (raw)
In-Reply-To: <20040522011139.01a7da10.akpm@osdl.org>

On Sat, May 22 2004, Andrew Morton wrote:
> 
> May as well cc lkml on this.  It's to do with the disk write barrier
> implementation.
> 
> 
> - How do I know that the barrier code is actually doing stuff?  It doesn't
>   seem to affect benchmarks much, if at all.

Usually not a lot of barriers would be generated. If you use reiser and
heavy fsync load, it should be pretty apparent of barriers are issued or
not (ie compare without barrier + wb cache, with barrier + wb cache, and
with/without - wb cache).

> - Does reiserfs support `mount -o remount,barrier=flush'? and "=none"?

Chris?

> - How do I test the "oh, barriers aren't working" fallback code in ext3?

A quick hack would be to replace 'err' with '1' in this line in
drivers/ide/ide-io.c:void ide_end_drive_cmd():

        if (blk_barrier_preflush(rq) || blk_barrier_postflush(rq))
                ide_complete_barrier(drive, rq, err);

that'll fail the first barrier, should trigger the fall back. Or just
try -o barrier=flush on a SCSI drive, should fail the barrier too.

> - Does the kernel tell you if your disk doesn't supoprt barriers?  ie:
>   how does the user know if it's working or not?

IDE will not tell you, the file system should flag the disabling of
barriers (like reiser currently does). Try SCSI case again.

-- 
Jens Axboe


  parent reply	other threads:[~2004-05-22  8:42 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20040521093207.GA1952@suse.de>
     [not found] ` <20040521023807.0de63c7a.akpm@osdl.org>
     [not found]   ` <20040521100234.GK1952@suse.de>
     [not found]     ` <20040521235044.6160cccb.akpm@osdl.org>
     [not found]       ` <20040522073540.GO1952@suse.de>
2004-05-22  8:11         ` [PATCH] ext3 barrier bits Andrew Morton
2004-05-22  8:24           ` Arjan van de Ven
2004-05-22  8:33             ` Andrew Morton
2004-05-22 19:15               ` Chris Mason
2004-05-22  8:43             ` Jens Axboe
2004-05-22  8:42           ` Jens Axboe [this message]
2004-05-22 19:31           ` Chris Mason

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20040522084214.GS1952@suse.de \
    --to=axboe@suse.de \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mason@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox