lkml.org 
[lkml]   [2002]   [Apr]   [16]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateTue, 16 Apr 2002 08:43:35 -0700 (PDT)
FromLinus Torvalds <>
SubjectRe: [PATCH] 2.5.8 IDE 36
On Tue, 16 Apr 2002, Richard Gooch wrote:
> 
> This gratuitous removal of features in the guise of "cleanups" is why
> you got flamed earlier this year. I thought you'd learned :-/

Richard, have you looked at the IDE mess? That "feature" is a bug, the way 
it was implemented - and considering that it's implementable at a 
different level much more cleanly for the (few) people who actually need 
it...
Also note that performance is likely to _increase_ by removing that stupid
feature - using DMA to do the actual IO and them byteswapping in some
higher level than the driver is likely to be a _lot_ faster than doing PIO
(and byteswap in-place, resulting in random mmap corruption).

Do you realize that because the current bswap writeback reverses the bytes 
in place, you can actually seriously corrupt your filesystem by just being 
unlucky in timing (ie a bswap on a dirty metadata block at the same time 
another process accesses it)?

It's a BUG guys, not a feature.

		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/

\
 
 \ /
  Last update: 2005-03-22 12:22    [from the cache]
©2003-2008