Re: [Xen-devel] [PATCH v3 4/4] libxl: add support for vscsi

[Olaf Hering <olaf@xxxxxxxxx>]

On Thu, Mar 26, Ian Campbell wrote:

> On Thu, 2015-03-26 at 13:55 +0100, Olaf Hering wrote:
> > On Fri, Mar 13, Ian Campbell wrote:
> > > Also, should this be a defbool or not?
> > This is a good idea. I remmeber xend had code to make sure that all
> > devices within a vhost had the same value for "feature-host". I think
> > the intention was that whole physical hosts are passed to a domU. A
> > defbool makes it easier to preserve the initial value based on the first
> > config item.
> 
> In which case it sounds to me like this flag should be at a per-vhost
> level and not at a per device level.

Its per vhost already, "libxl_device_vscsi" refers to the virtual scsi
host, and "libxl_vscsi_dev" refers to a device connected to that host.
Perhaps the latter needs a better name to make the difference more
obvious.

Olaf

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