mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Alok Kataria <alokkataria1@gmail.com>
To: James Bottomley <James.Bottomley@suse.de>
Cc: akataria@vmware.com, Robert Love <robert.w.love@intel.com>,
	Randy Dunlap <randy.dunlap@oracle.com>,
	Mike Christie <michaelc@cs.wisc.edu>,
	"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
	LKML <linux-kernel@vger.kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Dmitry Torokhov <dtor@vmware.com>,
	Rolf Eike Beer <eike-kernel@sf-tec.de>,
	Maxime Austruy <maustruy@vmware.com>,
	Bhavesh Davda <bhavesh@vmware.com>
Subject: Re: [PATCH] SCSI driver for VMware's virtual HBA.
Date: Tue, 1 Sep 2009 09:08:48 -0700	[thread overview]
Message-ID: <35f686220909010908k24f0164nc09b1158b6841ff3@mail.gmail.com> (raw)
In-Reply-To: <1251815012.3864.55.camel@mulgrave.site>

Hi James,

On Tue, Sep 1, 2009 at 7:23 AM, James Bottomley<James.Bottomley@suse.de> wrote:
> On Mon, 2009-08-31 at 14:53 -0700, Alok Kataria wrote:
>> Hi James,
>>
>> Thanks for your comments.
>>
>> On Mon, 2009-08-31 at 11:00 -0700, James Bottomley wrote:
>> > On Mon, 2009-08-31 at 10:28 -0700, Alok Kataria wrote:
>> > > VMware PVSCSI driver - v2.
>> >
>> > OK, so the first thing that springs to mind is that we already have one
>> > of these things: the ibmvscsi ... is there no way we can share code
>> > between this and the other PV drivers?
>>
>> I took a quick look at the ibmvscsi driver, and there are lot of
>> differences between the two, mainly the ABI that is shared between the
>> hypervisor and driver differ.
>
> Well, that's pretty abstractable, surely?  However, there is an
> interesting question of what the best hypervisor interface actually is.
>
>>  Also the ibmvscsi driver seems to offer a
>> lot of other features as well, like the SRP.
>
> SRP is the protocol transfer abstraction.  It's just a way of packaging
> up SCSI commands for transfer over a DMA protocol (OK, so it was
> envisaged that the DMA protocol would be RDMA, but a hypervisor
> interface is also a DMA protocol).
>
>> The pvscsi driver is a simple SCSI adapter driver and is basically no
>> different than any other SCSI driver written for a particular HBA.
>
> Well, it is really ... hopefully all the hypervisor interfaces won't
> decide to be completely incompatible, so there's a good chance of code
> sharing between them.

Well, going by that theory every SCSI driver can share some code with
any other SCSI driver, In short we can come up with an low level SCSI
driver interface which every SCSI driver can make use of, to reduce
code duplicity.  We don't do that, and the reason being, that its a
device and we shouldn't expect every device to share common features
or design decisions.

We already virtualize storage at the lowest (read device) level to
avoid duplicating stuff in a guest operating system.
If this was say a block level virtaulizing scheme sharing would be
more eminent, but given that we are a device its better to keep things
separate and IMHO, it might be an overkill to design a new pv-driver
or some such layer for every device that you virtualize.

Thanks,
Alok



>
> James
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>

  reply	other threads:[~2009-09-01 16:08 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-27 23:17 Alok Kataria
2009-08-28  6:03 ` Rolf Eike Beer
2009-08-31 17:26   ` Alok Kataria
2009-08-31 18:51     ` Rolf Eike Beer
2009-08-31 21:54       ` Alok Kataria
2009-08-28 21:18 ` Chetan.Loke
2009-08-28 22:30   ` Alok Kataria
2009-08-29 12:04     ` Chetan.Loke
2009-08-31 22:35       ` Alok Kataria
2009-08-31 17:28 ` Alok Kataria
2009-08-31 18:00   ` James Bottomley
2009-08-31 21:53     ` Alok Kataria
2009-09-01 14:23       ` James Bottomley
2009-09-01 16:08         ` Alok Kataria [this message]
2009-09-01 16:13           ` Matthew Wilcox
2009-09-01 16:20             ` Boaz Harrosh
2009-09-01 16:47               ` Alok Kataria
2009-09-01 14:26       ` James Bottomley
2009-09-01 11:12     ` Bart Van Assche
2009-09-01 14:17       ` James Bottomley
2009-09-01 16:12       ` Roland Dreier
2009-09-01 16:16         ` Matthew Wilcox
2009-09-01 16:33           ` Dmitry Torokhov
2009-09-01 16:52             ` James Bottomley
2009-09-01 16:59               ` Alok Kataria
2009-09-01 17:25                 ` James Bottomley
2009-09-01 17:41                   ` Alok Kataria
2009-09-01 18:15                     ` James Bottomley
2009-09-02  2:55                       ` Alok Kataria
2009-09-02 15:06                         ` James Bottomley
2009-09-02 17:16                           ` Alok Kataria
2009-09-03 20:03                             ` James Bottomley
2009-09-03 20:31                               ` Dmitry Torokhov
2009-09-03 21:21                                 ` Ric Wheeler
2009-09-03 21:41                                   ` Dmitry Torokhov
2009-09-04  3:28                               ` Alok Kataria
2009-09-01 17:25               ` Roland Dreier
2009-09-01 17:40                 ` James Bottomley
2009-09-01 17:54                   ` Alok Kataria
2009-09-01 18:38                     ` Christoph Hellwig
2009-09-02  9:50                       ` Bart Van Assche
2009-09-01 16:34         ` Bart Van Assche

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=35f686220909010908k24f0164nc09b1158b6841ff3@mail.gmail.com \
    --to=alokkataria1@gmail.com \
    --cc=James.Bottomley@suse.de \
    --cc=akataria@vmware.com \
    --cc=akpm@linux-foundation.org \
    --cc=bhavesh@vmware.com \
    --cc=dtor@vmware.com \
    --cc=eike-kernel@sf-tec.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=maustruy@vmware.com \
    --cc=michaelc@cs.wisc.edu \
    --cc=randy.dunlap@oracle.com \
    --cc=robert.w.love@intel.com \
    /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

all inboxes | Powered by JetHome®