[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20070619055629.GE18542@lst.de>
Date: Tue, 19 Jun 2007 07:56:29 +0200
From: Christoph Hellwig <hch@....de>
To: David Miller <davem@...emloft.net>
Cc: James.Bottomley@...elEye.com, axboe@...nel.dk,
linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
linuxppc-dev@...abs.org, paulus@...ba.org,
Geert.Uytterhoeven@...ycom.com, dwmw2@...radead.org,
alan@...rguk.ukuu.org.uk
Subject: Re: [patch 4/6] ps3: Disk Storage Driver
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 change the hypervisor nor the server
> speaking on the other end. And when I do write a server I do want
> it to be able to speak to all of the existing clients.
>
> There's SCSI command pass through as well, as I keep mentioning as
> it's an important reason I don't want to go with any of the non-SCSI
> solutions (other than perhaps ATA) being suggested.
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 decoding
command blocks and translating them to deep magic. The ps3 storage
drivers this thread discussed are a good example for that. We now
have a very nice and simple disk, scsi and flash chardev driver each
that don't include abstractions layers and cruft. Combine that with
their initial scsi layer driver that was full of internal dispatches
because each of these device types speaks a completely different command
set.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists