[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v8 3/5] libxl: add support for vscsi



Olaf Hering writes ("Re: [PATCH v8 3/5] libxl: add support for vscsi"):
> On Mon, Feb 15, Ian Jackson wrote:
> > One reason you might define a virtual controller with no devices yet
> > is so that you have a stable and pre-expected device path for any
> > actual targets you choose to hotplug later.
> 
> Would it be acceptable to reuse the devid as the "group index"?
> The various vdev in vscsi=['pdev,vdev'] will be assigned to the same
> vscsictrl if the host part in host:chn:target:lun matches. Right now an
> empty vscsictrl has no property to store the "host" part. This could be
> handled by either reusing devid, or by introducing a new xenstore
> property such as "libxl_vscsictrl_index". The value itself has no
> meaning other than being an index or label.

I haven't been following this design in detail, but: why is the
vscictrl `host' number not part of the xenstore path for the
controller, which in turn contains the devices ?

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.