[libvirt] [PATCH v7 1/2] bhyve: add a basic driver

Roman Bogorodskiy bogorodskiy at gmail.com
Tue Feb 11 19:25:06 UTC 2014


  Daniel P. Berrange wrote:

> > +    virCommandAddArg(cmd, "-S");
> > +    virCommandAddArg(cmd, "31,uart");
> 
> Hmm, this is enabling a serial port right ? If so, you only want
> todo this if you have def->nserials == 1 I reckon. What is the
> string '31' saying ?

31 is pcislot here. Anyway, I've dropped these options for now.

> I'd suggest this building is done before loading the domain
> so you don't needlessly load the domain before we detect any
> possible user configuration errors.

Good idea, appreciated it while testing incorrect disk type/bus checks.
Fixed in v8.

Thanks for the review!

Roman Bogorodskiy




More information about the libvir-list mailing list