[patch 4/6] ps3: Disk Storage Driver

2007-07-04 Thread Geert Uytterhoeven
From: Geert Uytterhoeven <[EMAIL PROTECTED]> Add a Disk Storage Driver for the PS3: - Implemented as a block device driver with a dynamic major - Disk names (and partitions) are of the format ps3d%c(%u) - Uses software scatter-gather with a 64 KiB bounce buffer as the hypervisor doesn't

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-19 Thread Christoph Hellwig
On Tue, Jun 19, 2007 at 02:51:25PM +0200, Geert Uytterhoeven wrote: > On Tue, 19 Jun 2007, Christoph Hellwig wrote: > > On Fri, Jun 15, 2007 at 01:39:23PM +0200, Geert Uytterhoeven wrote: > > > From: Geert Uytterhoeven <[EMAIL PROTECTED]> > > > > > > Add a Disk Storage Driver for the PS3: > > >

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-19 Thread Geert Uytterhoeven
On Tue, 19 Jun 2007, Christoph Hellwig wrote: > On Fri, Jun 15, 2007 at 01:39:23PM +0200, Geert Uytterhoeven wrote: > > From: Geert Uytterhoeven <[EMAIL PROTECTED]> > > > > Add a Disk Storage Driver for the PS3: > > - Implemented as a block device driver with a dynamic major > > - Disk names (

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-19 Thread Christoph Hellwig
On Mon, Jun 18, 2007 at 11:07:31PM -0700, David Miller wrote: > The main disk I/O block read and write is done using descriptors > sent to the disk server. SCSI pass-through is provided (optionally) > so that disk analysis tools can do things like MODE_SENSE on the > disk. SG_IO can easily be imp

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-19 Thread Geert Uytterhoeven
On Mon, 18 Jun 2007, David Miller wrote: > What I really care about is what will work transparently for existing > userspace. In particular, distribution installers and existing tools > like fdisk. > > When a slice it being exported, it's not being exported like that so > that the client can just

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-18 Thread David Miller
From: Christoph Hellwig <[EMAIL PROTECTED]> Date: Tue, 19 Jun 2007 07:56:29 +0200 > A SCSI pass through is of course perfectly fine. If you have a separate > block passthrough that has additional magic a separate block driver is > the way to go because it actually is simpler than a scsi driver de

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-18 Thread David Miller
From: Christoph Hellwig <[EMAIL PROTECTED]> Date: Tue, 19 Jun 2007 07:53:58 +0200 > On Fri, Jun 15, 2007 at 02:19:17PM -0700, David Miller wrote: > > Another quirk I have to deal with is that under LDOMs you > > can export full disks and also just slices. So I'll have > > to get down into the par

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-18 Thread Christoph Hellwig
On Fri, Jun 15, 2007 at 04:08:58PM -0700, David Miller wrote: > That's not gonna work, it's a totally different model. > > I have a predefined protocol over hypervisor provided "channels" and > page flipping also done by the hypervisor for the bulk data transfer. > For the client side I cannot cha

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-18 Thread Christoph Hellwig
On Fri, Jun 15, 2007 at 02:19:17PM -0700, David Miller wrote: > Another quirk I have to deal with is that under LDOMs you > can export full disks and also just slices. So I'll have > to get down into the partition machinery to support that > somehow. What's the problem with that? Partition machi

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-18 Thread Christoph Hellwig
On Fri, Jun 15, 2007 at 01:39:23PM +0200, Geert Uytterhoeven wrote: > From: Geert Uytterhoeven <[EMAIL PROTECTED]> > > Add a Disk Storage Driver for the PS3: > - Implemented as a block device driver with a dynamic major > - Disk names (and partitions) are of the format ps3d%c(%u) > - Uses so

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread Geert Uytterhoeven
On Fri, 15 Jun 2007, David Miller wrote: > From: Alan Cox <[EMAIL PROTECTED]> > Date: Fri, 15 Jun 2007 17:15:45 +0100 > > > > > Any particular reason why this is done as a separate block device driver > > > > rather than as SCSI? > > > > > > Because no new fake SCSI drivers are accepted anymore.

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread David Miller
From: James Bottomley <[EMAIL PROTECTED]> Date: Fri, 15 Jun 2007 16:28:03 -0700 > On Fri, 2007-06-15 at 16:08 -0700, David Miller wrote: > > From: James Bottomley <[EMAIL PROTECTED]> > > Date: Fri, 15 Jun 2007 15:40:42 -0700 > > > > > On Fri, 2007-06-15 at 14:19 -0700, David Miller wrote: > > > >

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread James Bottomley
On Fri, 2007-06-15 at 16:08 -0700, David Miller wrote: > From: James Bottomley <[EMAIL PROTECTED]> > Date: Fri, 15 Jun 2007 15:40:42 -0700 > > > On Fri, 2007-06-15 at 14:19 -0700, David Miller wrote: > > > Another quirk I have to deal with is that under LDOMs you > > > can export full disks and al

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread David Miller
From: James Bottomley <[EMAIL PROTECTED]> Date: Fri, 15 Jun 2007 15:40:42 -0700 > On Fri, 2007-06-15 at 14:19 -0700, David Miller wrote: > > Another quirk I have to deal with is that under LDOMs you > > can export full disks and also just slices. So I'll have > > to get down into the partition ma

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread Alan Cox
> That sounds like a good idea for my virtual I/O case on > Niagara too actually :-) It was actually meant semi-seriously in that drivers/ata sits on top of the abstract parts of drivers/scsi which James keeps asking for someone to get split properly off and which would sort all these out. > Anot

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread James Bottomley
On Fri, 2007-06-15 at 14:19 -0700, David Miller wrote: > From: Alan Cox <[EMAIL PROTECTED]> > Date: Fri, 15 Jun 2007 17:15:45 +0100 > > > > > Any particular reason why this is done as a separate block device driver > > > > rather than as SCSI? > > > > > > Because no new fake SCSI drivers are acce

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread David Miller
From: Jeff Garzik <[EMAIL PROTECTED]> Date: Fri, 15 Jun 2007 17:50:50 -0400 > FWIW that's why libata followed the path it did: faking SCSI required > the least amount of code for the maximal amount of existing storage > driver and platform installer support. > > Well, that and the fact that AT

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread Jeff Garzik
David Miller wrote: It's senseless to make people build new major/minor numbers for all these new quirky storage drivers. People have to add support for the new major number to installers and all kinds of other tools. FWIW that's why libata followed the path it did: faking SCSI required the

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread David Miller
From: Jeff Garzik <[EMAIL PROTECTED]> Date: Fri, 15 Jun 2007 17:28:31 -0400 > David Miller wrote: > > If the SCSI guys were smart, there would be a totally generic helper > > layer that allows anyone to hook into the SCSI layer as a virtual SCSI > > disk provider in like 10 lines of code. :-) > >

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread Jeff Garzik
David Miller wrote: From: Geert Uytterhoeven <[EMAIL PROTECTED]> Because no new fake SCSI drivers are accepted anymore. If you can access the firmware/hypervisor code, have that provide SCSI. Then the Linux driver piece is easy, obvious and flexible SCSI, and is not "fake." If the SCS

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread David Miller
From: Alan Cox <[EMAIL PROTECTED]> Date: Fri, 15 Jun 2007 17:15:45 +0100 > > > Any particular reason why this is done as a separate block device driver > > > rather than as SCSI? > > > > Because no new fake SCSI drivers are accepted anymore. > > Where did drivers/ata come from ;) > > How about

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread David Miller
From: Geert Uytterhoeven <[EMAIL PROTECTED]> Date: Fri, 15 Jun 2007 16:43:05 +0200 (CEST) > On Fri, 15 Jun 2007, David Woodhouse wrote: > > On Fri, 2007-06-15 at 13:39 +0200, Geert Uytterhoeven wrote: > > > Add a Disk Storage Driver for the PS3: > > > - Implemented as a block device driver with

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread Geert Uytterhoeven
On Fri, 15 Jun 2007, Alan Cox wrote: > > > Any particular reason why this is done as a separate block device driver > > > rather than as SCSI? > > > > Because no new fake SCSI drivers are accepted anymore. > > Where did drivers/ata come from ;) I was told that one got in just before the moratori

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread Alan Cox
> > Any particular reason why this is done as a separate block device driver > > rather than as SCSI? > > Because no new fake SCSI drivers are accepted anymore. Where did drivers/ata come from ;) How about making it a fake ata driver if James is being fussy 8) - To unsubscribe from this list: se

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread Geert Uytterhoeven
On Fri, 15 Jun 2007, David Woodhouse wrote: > On Fri, 2007-06-15 at 13:39 +0200, Geert Uytterhoeven wrote: > > Add a Disk Storage Driver for the PS3: > > - Implemented as a block device driver with a dynamic major > > - Disk names (and partitions) are of the format ps3d%c(%u) > > - Uses softw

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread Arnd Bergmann
On Friday 15 June 2007, David Woodhouse wrote: > > On Fri, 2007-06-15 at 13:39 +0200, Geert Uytterhoeven wrote: > > Add a Disk Storage Driver for the PS3: > >   - Implemented as a block device driver with a dynamic major > >   - Disk names (and partitions) are of the format ps3d%c(%u) > >   - Uses

Re: [patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread David Woodhouse
On Fri, 2007-06-15 at 13:39 +0200, Geert Uytterhoeven wrote: > Add a Disk Storage Driver for the PS3: > - Implemented as a block device driver with a dynamic major > - Disk names (and partitions) are of the format ps3d%c(%u) > - Uses software scatter-gather with a 64 KiB bounce buffer as the

[patch 4/6] ps3: Disk Storage Driver

2007-06-15 Thread Geert Uytterhoeven
From: Geert Uytterhoeven <[EMAIL PROTECTED]> Add a Disk Storage Driver for the PS3: - Implemented as a block device driver with a dynamic major - Disk names (and partitions) are of the format ps3d%c(%u) - Uses software scatter-gather with a 64 KiB bounce buffer as the hypervisor doesn't