Daniel P. Berrange wrote:

> > +    virCommandAddArg(cmd, "-s");
> > +    virCommandAddArgFormat(cmd, "2:0,ahci-hd,%s", disk->src);
> 
> What is the '2:0' bit here ?  Is that disk controller/unit
> number ?

Quoting the manpage [1]:

pcislot[:function]

                             The pcislot value is 0 to 31 and the optional
                             function value is 0 to 7.  If not specified, the
                             function value defaults to 0.

> I'd suggest  s/INFO/DEBUG/ here and earlier.  If you want user visible
> log messages about key operations, then we should talk about what is
> needed to make  viraudit.{c,h} work on FreeBSD, and use domain_audit.c
> for this.

Will do s/INFO/DEBUG/. I'll take a look at viraudit later too.

1: 
http://www.freebsd.org/cgi/man.cgi?query=bhyve&apropos=0&sektion=0&manpath=FreeBSD+10.0-RELEASE&arch=default&format=html

Thanks for the review!

Roman Bogorodskiy

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to