Re: [PATCH] bhyve: fix disks address allocation

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

 



  Laine Stump wrote:

> On 08/28/2016 08:45 AM, Roman Bogorodskiy wrote:
> > As bhyve currently doesn't use controller addressing and simply
> > uses 1 implicit controller for 1 disk device, the scheme looks the
> > following:
> >
> >   pci addrees -> (implicit controller) -> disk device
> >
> > So in fact we identify disk devices by pci address of implicit
> > controller and just pass it this way to bhyve in a form:
> >
> >   -s pci_addr,ahci-(cd|hd),/path/to/disk
> >
> > Therefore, we cannot use virDeviceInfoPCIAddressWanted() because it
> > does not expect that disk devices might need PCI address assignment.
> >
> > As a result, if a disk was specified without address, it will not be
> > generated and domain will to start.
> >
> > Until proper controller addressing is not implemented in the bhyve
> 
> s/not//   :-)
> 
> > driver, force each disk to have PCI address generated if it was not
> > specified by user.
> > ---
> >   src/bhyve/bhyve_device.c | 3 ++-
> >   1 file changed, 2 insertions(+), 1 deletion(-)
> >
> 
> ACK.

Pushed with the commit message fixed, thanks! 

Roman Bogorodskiy

Attachment: signature.asc
Description: PGP signature

--
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]