Re: [PATCH] direct-IO API change

From: Linus Torvalds (torvalds@transmeta.com)
Date: Fri Oct 04 2002 - 19:11:27 EST


On Fri, 4 Oct 2002, Badari Pulavarty wrote:
>
> Only issue would be the alignment restriction on blockdev versus raw device.

Hmm.. We might want to revert the stuff that made the default block device
alignment be the maximal possible, and instead make the default be the
minimum possible.

The offending code is the "while"-loop in bd_set_size(). Just removing
that should make the default size be the minimal one (ie hardsect_size).

It _used_ to make sense to try to maximize the block-size, since it had a
noticeable impact on performance whether we did 8 512-byte requests or
just 1 4kB request. However, all the bio changes have likely made that a
non-issue, since Andrew's code ends up doing things directly one page at a
time _anyway_.

Of course, mounting a filesystem on the device will override the default
anyway, so this by no means will guarantee that direct access will always
have the minimal alignment restrictions, but once you've mounted a 4kB
blocksize filesystem on that device I think you might as well do 4kB
chunks even if you open it through the device interface, no?

                        Linus

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Mon Oct 07 2002 - 22:00:48 EST