[PATCH v2 00/11] clean up after "slot sharing" patches

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

 



(this is just a rebase of the v1 patches to make review easier)

Some of the patches that enabled sharing PCI slots by multiple
pcie-root-ports made the idea of reserving an entire slot obsolete. To
reduce confusion and misunderstandings, this patch series gets rid of
the name "Slot" in all of the functions that reserve and release PCI
addresses.

None of these patches affect any functional change.

This is a lot of patches, but each patch is trivial, I promise - most
are simply renaming a single function, or changing one argument to
some calls to a function.

Laine Stump (11):
  conf: fix fromConfig argument to virDomainPCIAddressReserveAddr()
  conf: fix fromConfig argument to virDomainPCIAddressValidate()
  conf: rename virDomainPCIAddressGetNextSlot() to ...GetNextAddr()
  conf: eliminate virDomainPCIAddressReserveNextSlot()
  qemu: replace virDomainPCIAddressReserveAddr with
    virDomainPCIAddressReserveSlot
  conf: make virDomainPCIAddressReserveAddr() a static function
  conf: rename virDomainPCIAddressReserveAddr() to ...Internal()
  conf: rename virDomainPCIAddressReserveSlot() to ...Addr()
  qemu: remove qemuDomainPCIAddressReserveNextAddr()
  qemu: rename qemuDomainPCIAddressReserveNextSlot() to ...Addr()
  conf: eliminate virDomainPCIAddressReleaseSlot() in favor of ...Addr()

 src/bhyve/bhyve_device.c       |  26 ++++++----
 src/conf/domain_addr.c         |  60 +++++------------------
 src/conf/domain_addr.h         |  15 ------
 src/libvirt_private.syms       |   4 +-
 src/qemu/qemu_domain_address.c | 105 ++++++++++++++++++++---------------------
 5 files changed, 80 insertions(+), 130 deletions(-)

-- 
2.7.4

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