From: Andrea Arcangeli <andrea@suse.de>
To: Joel Becker <jlbec@evilplan.org>,
Marcelo Tosatti <marcelo@conectiva.com.br>,
lkml <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] O_DIRECT with hardware blocksize alignment
Date: Sat, 12 Jan 2002 13:31:22 +0100 [thread overview]
Message-ID: <20020112133122.I1482@inspiron.school.suse.de> (raw)
In-Reply-To: <20020109195606.A16884@parcelfarce.linux.theplanet.co.uk>
In-Reply-To: <20020109195606.A16884@parcelfarce.linux.theplanet.co.uk>; from jlbec@evilplan.org on Wed, Jan 09, 2002 at 07:56:07PM +0000
On Wed, Jan 09, 2002 at 07:56:07PM +0000, Joel Becker wrote:
> Folks,
> Some major users of O_DIRECT (Oracle, for instance) align and
> size I/O based on the 512byte hardware blocksize common to most hard
> disk drives. The current O_DIRECT code enforces that the alignment and
> size of the I/O match the software blocksize (inot->i_sb->s_blocksize).
> This patch relaxes that restriction to a minimum of the hardware
> blocksize. In the interest of efficiency,
> min(I/O alignment, s_blocksize) is used as the effective
> blocksize. eg:
>
> I/O alignment s_blocksize final blocksize
> 8192 4096 4096
> 4096 4096 4096
> 512 4096 512
this falls in the same risky category of the vary-I/O patch from Badari
(check the discussion on l-k) for rawio, so to make it safe it also will
need to check for the same new per-blkdev bitflag before you can trigger
the scaling (if the bitflag says 'no', it will have to enforce
softblocksize b_size etc). (and then you will get the same optimization
in brw_kiovec as well as rawio, also while not doing softblocksize
aligned I/O, but still large I/O) So I suggest you to check Badari's
stuff and the thread on l-k and to make a new patch incremental with his
code (note: I still had some problem with his latest patch but it's not
far from a final version)
Andrea
next prev parent reply other threads:[~2002-01-12 12:34 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-01-09 19:56 Joel Becker
2002-01-12 12:31 ` Andrea Arcangeli [this message]
2002-01-15 3:21 ` Joel Becker
2002-01-15 12:20 ` Andrea Arcangeli
2002-01-15 13:08 ` Jens Axboe
2002-01-15 13:55 ` Jens Axboe
2002-01-15 21:23 ` Badari Pulavarty
2002-01-15 21:44 ` Alan Cox
2002-01-24 0:44 ` [PATCH] small bugfix for ll_rw_bio() for 2.5.3-pre3 Badari Pulavarty
2002-01-24 21:52 ` O_DIRECT broken in 2.5.3-preX ? Badari Pulavarty
2002-01-28 2:06 ` Andrea Arcangeli
2002-01-16 0:08 ` [PATCH] O_DIRECT with hardware blocksize alignment Joel Becker
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=20020112133122.I1482@inspiron.school.suse.de \
--to=andrea@suse.de \
--cc=jlbec@evilplan.org \
--cc=linux-kernel@vger.kernel.org \
--cc=marcelo@conectiva.com.br \
/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
all inboxes | Powered by JetHome®