linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Jeff Garzik <jeff@garzik.org>
Cc: "Mark Lord" <liml@rtr.ca>, "Yinghai Lu" <yinghai@kernel.org>,
	"Sergei Shtylyov" <sshtylyov@ru.mvista.com>,
	"Alan Cox" <alan@lxorguk.ukuu.org.uk>,
	"Bartlomiej Zolnierkiewicz" <bzolnier@gmail.com>,
	"Dan Noé" <dpn@isomerica.net>,
	"Christoph Hellwig" <hch@infradead.org>,
	"FUJITA Tomonori" <fujita.tomonori@lab.ntt.co.jp>,
	linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org,
	osst@riede.org
Subject: Re: [PATCH] remove ide-scsi
Date: Tue, 09 Dec 2008 15:04:20 -0600	[thread overview]
Message-ID: <1228856661.3263.37.camel@localhost.localdomain> (raw)
In-Reply-To: <493ED006.7020201@garzik.org>

On Tue, 2008-12-09 at 15:07 -0500, Jeff Garzik wrote:
> Mark Lord wrote:
> > Jeff Garzik wrote:
> >> Yinghai Lu wrote:
> >>> On Sat, Dec 6, 2008 at 3:48 PM, Jeff Garzik <jeff@garzik.org> wrote:
> > ..
> >>> interesting, so will get /dev/ada instead of /dev/sda?
> >>
> >> That is an interesting question.  The easiest thing is to allocate a 
> >> new 32-bit block major.  But the more compatible (and more 
> >> controversial) solution is to allocate from the SCSI disk blkmajor space.
> > ..
> > 
> > Perhaps "sdx" and "scdx" should simply be redefined as "System disk"
> > and "System CD", and moved under block layer ownership from SCSI ?
> 
> Yes, precisely.  SCSI and libata (and others) would allocate disk/MMC 
> majors from the block layer.

Actually, The mmc major will still be sr (11).  The split out disk major
will only really be used for the current block devices served by sd.

> The biggest problem with this plan is that it introduces a mismatch in 
> ioctl capabilities, a behavior change from the current situation where 
> SCSI exports SCSI_IOCTL_SEND_COMMAND

That's deprecated and has been for several years.

>  in addition to the more universal 
> SG_IO.  Installers and existing clients may rely on this portion of the 
> Linux application ABI.

Hopefully not.  It warns noisily on use of deprecated ioctls.  The door
lock ones still aren't, so libata will have to grow it's own removable
media handling for them, but I think all the rest are (except the
bus/host probes which return largely made up numbers).

James



  reply	other threads:[~2008-12-09 21:04 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
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 [this message]
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=1228856661.3263.37.camel@localhost.localdomain \
    --to=james.bottomley@hansenpartnership.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=bzolnier@gmail.com \
    --cc=dpn@isomerica.net \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=hch@infradead.org \
    --cc=jeff@garzik.org \
    --cc=liml@rtr.ca \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=osst@riede.org \
    --cc=sshtylyov@ru.mvista.com \
    --cc=yinghai@kernel.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).