linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Willem Riede <osst@riede.org>
Cc: James Bottomley <James.Bottomley@hansenpartnership.com>,
	Christoph Hellwig <hch@infradead.org>,
	FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
	linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: [PATCH] remove ide-scsi
Date: Sat, 6 Dec 2008 16:59:01 +0100	[thread overview]
Message-ID: <200812061659.02200.bzolnier@gmail.com> (raw)
In-Reply-To: <150283e90812060725x482d8428kfc7e41f8bc509e12@mail.gmail.com>

On Saturday 06 December 2008, Willem Riede wrote:
> On Sat, Dec 6, 2008 at 7:51 AM, Bartlomiej Zolnierkiewicz
> <bzolnier@gmail.com> wrote:
> > On Wednesday 03 December 2008, James Bottomley wrote:
> >> On Wed, 2008-12-03 at 05:06 -0500, Christoph Hellwig wrote:
> >> > On Wed, Dec 03, 2008 at 10:38:54AM +0900, FUJITA Tomonori wrote:
> >> > > This is for 2.6.29 (not 2.6.28) as feature-removal-schedule.txt says.
> >> > >
> >> > > It's against linux-next (which seems to has some changes to ide-scsi
> >> > > for 2.6.29 from the ide tree).
> >> >
> >> > Isn't ide-scsi the only way to use ATAPI OnStream tapes supported by
> >> > the osst driver?
> >>
> >> Depends.  If you're still using drivers/ide then yes, it is.  With
> >> libata (which is what most modern distros use), osst just works as an
> >> ATAPI transport.
> >>
> >> git log tells me ide-scsi has been updated quite a bit recently, but it
> >> mostly looks to be fallout around the drivers/ide churn.  Can we get ide
> >> maintainer's buy in for this (I think they've been maintaining ide-tape
> >> and ide-cd in preference to ide-scsi)?
> >
> > Certainly, native ide-{cd,gd,tape} drivers are far superior over ide-scsi
> > and are actively maintained.
> >
> > ide-scsi has been practically a dead code for a long time now (IIRC it has
> > even been broken -- by some general kernel changes -- for few releases and
> > nobody noticed, till Boaz discovered it while doing unrelated SCSI fixes)
> > but I kept it on the live support anyway.  However there needs to be some
> > limit to it, especially given that driver has been officially orphaned for
> > over a year now and nobody stepped in...
> >
> > I applied the scheduled removal patch to pata-2.6 tree, thanks Tomo!
> >
> > Thanks,
> > Bart
> >
> > PS If somebody wants to work on OSST support for IDE we can provide an
> > assistance into porting osst.c over generic ATAPI code -- it would still
> > be much less hassle than trying to figure out remaining ide-scsi issues
> > (lifetime rules for IDE / SCSI / IDE-SCSI objects, error handling etc.).
> >
> Don't know exactly what you mean with "porting osst.c over generic ATAPI code",
> but osst is SCSI by design - a SCSI version of the drive exists.
> Making osst ATAPI

I mean porting it over generic IDE ATAPI code (ide-atapi.c) in case somebody
wants to use it with drivers/ide instead of libata.

> is therefore not appropriate. If osst works with libata as earlier suggested,
> then that would be the best solution.

It should work fine I just doubt that anybody has ever tested it because of
lack of osst users in general.

> Unfortunately, my IDE test drive died, OnStream went out of business some time
> ago, so I can't get a new one, hence I (osst maintainer) can't test this :-(

:-( indeed.

Thanks,
Bart

  reply	other threads:[~2008-12-06 16:31 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-03  1:38 [PATCH] remove ide-scsi FUJITA Tomonori
2008-12-03 10:06 ` Christoph Hellwig
2008-12-03 13:31   ` Willem Riede
2008-12-03 13:55     ` Matthew Wilcox
2008-12-03 14:02       ` Alan Cox
2008-12-03 15:09   ` James Bottomley
2008-12-06  6:12     ` Pete Zaitcev
2008-12-06 14:06       ` Bartlomiej Zolnierkiewicz
2008-12-06 14:51     ` Bartlomiej Zolnierkiewicz
2008-12-06 15:06       ` Alan Cox
2008-12-06 16:29         ` Bartlomiej Zolnierkiewicz
2008-12-06 15:25       ` Willem Riede
2008-12-06 15:59         ` Bartlomiej Zolnierkiewicz [this message]
2008-12-06 17:00       ` Dan Noé
2008-12-06 21:41         ` Bartlomiej Zolnierkiewicz
2008-12-06 22:24           ` Alan Cox
2008-12-06 22:52             ` Sergei Shtylyov
2008-12-06 23:02               ` Alan Cox
2008-12-06 23:19                 ` Sergei Shtylyov
2008-12-06 23:32                   ` Alan Cox
2008-12-07  0:08                     ` Sergei Shtylyov
2008-12-07 11:40                       ` Alan Cox
2008-12-07 14:46                         ` Sergei Shtylyov
2008-12-07 15:04                   ` James Bottomley
2008-12-07 15:21                     ` Sergei Shtylyov
2008-12-09 22:21                     ` libata / scsi separation Matthew Wilcox
2008-12-09 22:38                       ` James Bottomley
2008-12-10  3:37                         ` Matthew Wilcox
2008-12-10  1:54                       ` Tejun Heo
2008-12-10  2:29                         ` Grant Grundler
2008-12-10  2:47                           ` Tejun Heo
2008-12-10  3:23                             ` Grant Grundler
2008-12-10  3:44                               ` Tejun Heo
2008-12-10 15:24                                 ` Matthew Wilcox
2008-12-10 15:33                                   ` Tejun Heo
2008-12-10 16:01                                     ` Matthew Wilcox
2008-12-10 17:11                                     ` Grant Grundler
2008-12-10 17:21                                   ` Grant Grundler
2008-12-07  0:19                 ` [PATCH] remove ide-scsi Sergei Shtylyov
2008-12-07  9:59                   ` Sergei Shtylyov
2008-12-07 10:41                 ` Sergei Shtylyov
2008-12-09 21:41                 ` Matthew Wilcox
2008-12-10 17:46                   ` Sergei Shtylyov
2008-12-06 23:28               ` Jeff Garzik
2008-12-06 23:42                 ` Sergei Shtylyov
2008-12-06 23:48                   ` Jeff Garzik
2008-12-07  3:36                     ` Yinghai Lu
2008-12-07  4:17                       ` Jeff Garzik
2008-12-07  5:07                         ` Yinghai Lu
2008-12-07 11:00                           ` Sergei Shtylyov
2008-12-09 19:59                         ` Mark Lord
2008-12-09 20:07                           ` Jeff Garzik
2008-12-09 21:04                             ` James Bottomley
2008-12-06 23:45                 ` Bartlomiej Zolnierkiewicz
2008-12-06 23:50                   ` Jeff Garzik
2008-12-06 23:40             ` Bartlomiej Zolnierkiewicz
2008-12-06 23:51               ` Alan Cox
2008-12-07  0:56                 ` Bartlomiej Zolnierkiewicz
2008-12-07  1:14                   ` Alan Cox
2008-12-07 10:32                     ` Sergei Shtylyov
2008-12-06 23:51               ` Jeff Garzik
2008-12-06 22:33           ` Al Viro
2008-12-06 23:13             ` Bartlomiej Zolnierkiewicz
2008-12-06 23:17             ` Willem Riede
2008-12-07  0:09               ` Al Viro

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200812061659.02200.bzolnier@gmail.com \
    --to=bzolnier@gmail.com \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=hch@infradead.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=osst@riede.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).