public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: viro@parcelfarce.linux.theplanet.co.uk
To: Rene Herman <rene.herman@keyaccess.nl>
Cc: Linus Torvalds <torvalds@osdl.org>, linux-kernel@vger.kernel.org
Subject: Re: [RFC] removal of legacy cdrom drivers (Re: [PATCH] mcdx.c insanity removal)
Date: Mon, 3 May 2004 20:45:58 +0100	[thread overview]
Message-ID: <20040503194558.GF17014@parcelfarce.linux.theplanet.co.uk> (raw)
In-Reply-To: <40968A9F.6070608@keyaccess.nl>

On Mon, May 03, 2004 at 08:08:31PM +0200, Rene Herman wrote:
> viro@parcelfarce.linux.theplanet.co.uk wrote:
> 
> >OK...  So we have
> >	* potentially faulty mcdx (2.4, apparently either driver corrupts
> >memory in some conditions or isofs does the same for some IO failures -
> >need to take a look at that report more carefully).
> >	* cdu31a (FUBAR driver, nasty to fix, "most of the time" works on
> >2.6)
> >	* sbpcd (at least two, both untested with 2.6)
> 
> 3y25:~$ mount | grep cdrom
> /dev/sbpcd on /mnt/cdrom type iso9660 (ro,noexec,nosuid,nodev)
> 3y25:~$ ls /mnt/cdrom/
> cd.id*        install.exe*  lecdemos/     readme.doc*   resource/ 
> support/
> 
> However, any "cp" from cd-rom oopses the box.

oopses in driver, oopses by triggering BUG() or oopses in fs/*?  The last
two would be more interesting - isofs _MUST_ be able to survive any IO
errors, simply because CDs get scratched, etc. and that shouldn't crash
the box.

> I was actually planning to get around to that at some point. Somewhat 
> fond of this drive. As you say, driver is a disaster area; a few trivial 
> fixes are not what it wants and at this point, fixing it properly would 
> not be a trivial undertaking for me. Am also currently very busy 
> elsewhere. Could it be kept around a bit?

Free advice: if you want to handle that one, fork off the bits you really
need.  Keeping the drivers for all variants together simply doesn't pay,
and even the excuse of keeping the development in single codebase doesn't
apply anymore for what, 7 years?

  reply	other threads:[~2004-05-03 19:46 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20040502024637.GV17014@parcelfarce.linux.theplanet.co.uk>
     [not found] ` <Pine.LNX.4.58.0405011953140.18014@ppc970.osdl.org>
2004-05-03  1:16   ` [RFC] removal of legacy cdrom drivers (Re: [PATCH] mcdx.c insanity removal) viro
2004-05-03  3:21     ` Rene Herman
2004-05-03  5:59       ` viro
2004-05-03 18:08         ` Rene Herman
2004-05-03 19:45           ` viro [this message]
2004-05-03 21:22             ` Rene Herman
2004-05-03 21:30               ` Linus Torvalds
2004-05-03 21:34               ` viro
2004-05-05 23:41     ` Adrian Bunk

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=20040503194558.GF17014@parcelfarce.linux.theplanet.co.uk \
    --to=viro@parcelfarce.linux.theplanet.co.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rene.herman@keyaccess.nl \
    --cc=torvalds@osdl.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