Thread (2 messages) 2 messages, 2 authors, 2003-01-25

Re: 2.5.59-mm5

From: Jens Axboe <hidden>
Date: 2003-01-25 12:19:54
Also in: lkml

On Fri, Jan 24 2003, Valdis.Kletnieks@vt.edu wrote:
On Fri, 24 Jan 2003 21:04:34 +0100, Jens Axboe said:
quoted
Nicks comment refers to the block layer situation, we obviously cannot
merge reads and writes there. You would basically have to rewrite the
entire request submission structure and break all drivers. And for zero
benefit. Face it, it would be stupid to even attempt such a manuever.
As I *said* - "hairy beyond benefit", not "cant".
Hairy is ok as long as it provides substantial benefit in some way, and
this does definitely not qualify.
quoted
Since you bring it up, you must know if a device which can take a single
command that says "read blocks a to b, and write blocks x to z"? Even
such thing existed,
They do exist.

IBM mainframe disks (the 3330/50/80 series) are able to do much more
than that in one CCW chain  So it was *quite* possible to even express
things like "Go to this cylinder/track, search for each record that
has value XYZ in the 'key' field, and if found, write value ABC in the
data field". (In fact, the DASD I/O
opcodes for CCW chains are Turing-complete).
Well as interesting as that is, it is still an obscurity that will not
be generally supported. As I said, if you wanted to do such a thing you
can do it in the driver. Complicating the block layer in this way is
totally unacceptable, and is just bound to be an endless source of data
corrupting driver bugs.
quoted
So I quite agree with the "obviously".
My complaint was the confusion of "obviously cant" with "we have decided we
don't want to".
Ok fair enough, make that a strong "obviously wont" instead then.

-- 
Jens Axboe

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help