public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@steeleye.com>
To: "Randy.Dunlap" <rddunlap@osdl.org>
Cc: James Bottomley <James.Bottomley@SteelEye.com>,
	Marcelo Tosatti <marcelo@conectiva.com.br>,
	linux-kernel@vger.kernel.org,
	Erik Andersen <andersen@codepoet.org>
Subject: Re: [PATCH] cdrom sane fallback vs 2.4.20-pre1
Date: Tue, 13 Aug 2002 11:21:38 -0500	[thread overview]
Message-ID: <200208131621.g7DGLc202919@localhost.localdomain> (raw)
In-Reply-To: Message from "Randy.Dunlap" <rddunlap@osdl.org>  of "Tue, 13 Aug 2002 08:48:28 PDT." <Pine.LNX.4.33L2.0208130847380.5175-100000@dragon.pdx.osdl.net>

rddunlap@osdl.org said:
> and that's precisely the wrong attitude IMO.

I wasn't expressing an opinion, just stating what could and could not be done 
in 2.4.

> I was glad to see that Marcelo asked about the hardcoded values. They
> hurt. 

Well, this is a rather big and particularly rancid can of worms.  If you look 
a little further, you'll see that cdrom.h has its own definition of the 
(effectively SCSI) struct request_sense that sr.c uses, yet the sense key is 
defined in scsi/scsi.h.  Then you notice that cdrom.h also duplicates all of 
the scsi commands with a GPCMD_ prefix.

If you'd like to take this particular can of worms off somewhere, clean it out 
and return it neatly labelled, I'd be more than grateful...just don't take the 
lid off too close to me.

James



  parent reply	other threads:[~2002-08-13 16:17 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-08-13 14:13 [PATCH] cdrom sane fallback vs 2.4.20-pre1 James Bottomley
2002-08-13 15:48 ` Randy.Dunlap
2002-08-13 16:14   ` Jens Axboe
2002-08-13 16:21   ` James Bottomley [this message]
2002-08-13 16:25     ` Randy.Dunlap
2002-08-13 16:37       ` Jens Axboe
2002-08-13 17:10         ` James Bottomley
  -- strict thread matches above, loose matches on Subject: below --
2002-08-11 21:59 Erik Andersen
2002-08-13  2:58 ` Marcelo Tosatti
2002-08-13  4:12   ` Erik Andersen
2002-08-16  3:12     ` Oliver Xymoron
2002-08-13  8:42   ` Jens Axboe

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=200208131621.g7DGLc202919@localhost.localdomain \
    --to=james.bottomley@steeleye.com \
    --cc=andersen@codepoet.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcelo@conectiva.com.br \
    --cc=rddunlap@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