Date: Fri, 8 Aug 2003 02:10:19 +0200 (MET DST)
From: Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>
To: Mikael Pettersson <mikpe@csd.uu.se>
cc: <linux-kernel@vger.kernel.org>
Subject: Re: ide-tape broken (was Re: [PATCH] use ide-identify.h, fix endian
 bug)
In-Reply-To: <200308072341.h77NfeF4025095@harpo.it.uu.se>
Message-ID: <Pine.SOL.4.30.0308080205490.24371-100000@mion.elka.pw.edu.pl>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1691
Lines: 47


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/