From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <ian.campbell@citrix.com>
Subject: Re: [PATCH 4/4] libxl: add support for vscsi
Date: Mon, 2 Mar 2015 14:31:06 +0000
Message-ID: <1425306666.21151.58.camel@citrix.com>
References: <1425291362-30228-1-git-send-email-olaf@aepfle.de>
	<1425291362-30228-5-git-send-email-olaf@aepfle.de>
	<20150302142040.GA17851@aepfle.de>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <20150302142040.GA17851@aepfle.de>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Olaf Hering <olaf@aepfle.de>
Cc: Wei Liu <wei.liu2@citrix.com>, Stefano Stabellini <stefano.stabellini@eu.citrix.com>, Ian Jackson <ian.jackson@eu.citrix.com>, xen-devel@lists.xen.org
List-Id: xen-devel@lists.xenproject.org

On Mon, 2015-03-02 at 15:20 +0100, Olaf Hering wrote:
> On Mon, Mar 02, Olaf Hering wrote:
> 
> > +++ b/tools/libxl/libxl_types.idl
> 
> > +libxl_vscsi_hctl = Struct("vscsi_hctl", [
> > +    ("hst", uint32),
> > +    ("chn", uint32),
> > +    ("tgt", uint32),
> > +    ("lun", uint32),
> > +    ])
> > +
> > +libxl_vscsi_dev = Struct("vscsi_dev", [
> > +    ("vscsi_dev_id",     libxl_devid),
> > +    ("remove",           bool),
> > +    ("p_devname",        string),
> > +    ("pdev",             libxl_vscsi_hctl),
> > +    ("vdev",             libxl_vscsi_hctl),
> > +    ])
> > +
> > +libxl_device_vscsi = Struct("device_vscsi", [
> > +    ("backend_domid",    libxl_domid),
> > +    ("devid",            libxl_devid),
> > +    ("v_hst",            uint32),
> > +    ("vscsi_devs",       Array(libxl_vscsi_dev, "num_vscsi_devs")),
> > +    ("feature_host",     bool),
> > +    ])
> 
> IanC, you made a comment about "vscsi_dev_id" in the pastebin.com
> comment about libxl_vscsi_hctl:
> '- devid is just "devid" in all other places, not "TYPE_dev_id".'

> Its used as <domid>/<vhost>/vscsi-devs/dev-<vscsi_dev_id>/.  What name
> should be used to reference the devices? Also just "devid" even if it
> refers to devices "private" to the vscsi host?

Not sure. PCI uses vdevfn, which by extension would be vdevid here
(assuming I've understood what you are suggesting correctly and
therefore the analogy is valid).

Or could it be the case that there is no need to expose this value to
the user in the first place and libxl could just make up the index
numbers internally? Why would they want to configure it or retrieve it?

Is it actually the same as the offset into the vscsi_devs array?

How is it intended to be different from the vdev field?

Ian.