lkml.org 
[lkml]   [2003]   [Aug]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: ide-tape broken (was Re: [PATCH] use ide-identify.h, fix endian bug)

On Fri, 8 Aug 2003, Mikael Pettersson wrote:

> On Thu, 7 Aug 2003 22:51:31 +0200 (MET DST), Bartlomiej Zolnierkiewicz wrote:
> >On Thu, 7 Aug 2003 Andries.Brouwer@cwi.nl wrote:
> >
> >> Given ide-identify.h, we can simplify ide-floppy.c and ide-tape.c a lot.
> >> In fact ide-tape.c was broken on big-endian machines.
> >> (Unfortunately much more is broken that was fixed here,
> >> ide-tape.c is not in a good shape today.)
> >
> >ide-tape is broken because nobody cares, so I don't care too
> >(was broken even before). It needs rewrite and testing.
> >
> >So once again if anybody cares and has hardware to test,
> >please contact me and I will try fix it.
> >
> >Until then I don't touch it et all and consider it obsoleted.
>
> I used to use the ide-tape driver for my Seagate STT8000A,
> but its prolonged brokenness in 2.5 (caused initially by the
> bio changes) made me switch to ide-scsi + st instead since
> that at least _works_.
>
> I can test updates as they hit Linus 2.6-releases, but frankly

Great.

> I'd rather use ide-scsi+st or a new clean ATAPI tape driver
> than ide-tape.c. (I've studied ide-tape.c. It reeks of poor

"new clean ATAPI tape drive" == the one yet to be written?

> coding style, kludges for Onstream, and an over-engineered
> buffering scheme. And it's known to have problems with DMA.)

So you are familiar with the code, cool. ;-)

I will try to simplify ide-tape.c as much as possible.

--bartlomiej

-
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 13:47    [W:0.050 / U:1.176 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site