Re: [PATCH v9] bhyve: add a basic driver

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

 



  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@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]