From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: [PATCH v5 3/5] libxl: add support for vscsi Date: Fri, 15 May 2015 08:29:52 +0200 Message-ID: <20150515062952.GA4587@aepfle.de> References: <1430918892-20928-1-git-send-email-olaf@aepfle.de> <1430918892-20928-4-git-send-email-olaf@aepfle.de> <1431527008.8263.295.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1431527008.8263.295.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: Wei Liu , Stefano Stabellini , Ian Jackson , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Wed, May 13, Ian Campbell wrote: > On Wed, 2015-05-06 at 13:28 +0000, Olaf Hering wrote: > > +++ b/docs/man/xl.pod.1 > > @@ -1328,6 +1328,24 @@ List virtual trusted platform modules for a domain. > > > > =back > > > > +=head2 PVSCSI DEVICES > > + > > +=over 4 > > + > > +=item B I I I,I<[feature-host]> > > Unlike in the xl.cfg disk spec the pdev and vdev are separated with > space rather than ",", is that deliberate? (I don't mind, just want to > check it's intended). Yes, pdev and vdev are space separated. I have to double check how xm handled the additional feature-host option, it was most likely comma separated. > > +=item B I I<[domain-id] ...> > > + > > +List vscsi devices for the domain specified by I. > > Does/could omitting the domid list them all? No, at least one domid is required. Not sure how desirable (and racy) it is to walk every domain and look for vscsi devices. Olaf