On Mon, May 05, 2014 at 05:00:34PM +0300, Laine Stump wrote:
> On 05/05/2014 03:07 PM, Ján Tomko wrote:
> > On 05/03/2014 06:31 PM, Roman Bogorodskiy wrote:
> >> Extract PCI handling related structs that could be shared
> >> with other drivers.
> >>
> >> List of structs moved to virpci.h and new names:
> >>
> >> qemuDomainPCIAddressBus -> virDomainPCIAddressBus
> >> qemuDomainPCIAddressBusPtr -> virDomainPCIAddressBusPtr
> >> _qemuDomainPCIAddressSet -> virDomainPCIAddressSet
> >> qemuDomainPCIAddressSetPtr -> virDomainPCIAddressSetPtr
> >> qemuDomainPCIConnectFlags -> virDomainPCIConnectFlags
> > I would drop the 'Domain', to make the prefix match the file.
>
> I was thinking about that and came to a different opinion. The functions
> that are currently in virpci.c are dealing with manipulating and
> reporting about PCI devices on the *host* (reading and writing sysfs
> files to attach and detach drivers, determining the list of virtual
> functions for an SRIOV physical function, etc), while these functions
> that Roman is moving are only concerned with managing the allocation of
> PCI addresses to devices in a domain.
>
> Because of that, I think it's reasonable (a good idea really) to keep
> "Domain" in the function names.
>
> Beyond that, I was going to say that I think these functions belong in
> their own file, *not* virpci.c (and maybe we even want to rename
> virpci.c to virhostpci.c or something). I think it's *essential* that
> the two sets of functions are separated from each other, since what is
> in virpci.c is Linux-specific, but the virDomainPCI... functions should
> be host-agnostic.
Agreed, I think this should go into a src/conf/domain_addr.{c,h}
file since it has a dep on other things in src/conf, which is
not allowed for files in src/util.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list