Three new functions useful in other files:
virDomainHostdevInsert:
Add a new hostdev at the end of the array. This would more sensibly be
called virDomainHostdevAppend, but the existing functions for other
types of devices are called Insert.
virDomainHostdevRemove:
Eliminates one entry from the hostdevs array, but doesn't free it;
patterned after the code at the end of the two
qemuDomainDetachHostXXXDevice functions (and also other pre-existing
virDomainXXXRemove functions for other device types).
virDomainHostdevFind:
This function is patterned from the search loops at the top of
qemuDomainDetachHostPciDevice and qemuDomainDetachHostUsbDevice, and
will be used to re-factor those (and other detach-related) functions.
---
New patch for V2.
src/conf/domain_conf.c | 94 ++
src/conf/domain_conf.h |5 ++
src/libvirt_private.syms |3 +
3 files changed, 102 insertions(+), 0 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 93fd8d7..94ee634 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -6769,6 +6769,100 @@ virDomainChrTargetTypeToString(int deviceType,
}
int
+virDomainHostdevInsert(virDomainDefPtr def, virDomainHostdevDefPtr hostdev)
+{
+if (VIR_REALLOC_N(def->hostdevs, def->nhostdevs + 1) < 0)
+return -1;
+def->hostdevs[def->nhostdevs++] = hostdev;
+return 0;
+}
+
+void
+virDomainHostdevRemove(virDomainDefPtr def, size_t i)
+{
+if (def->nhostdevs > 1) {
+memmove(def->hostdevs + i,
+def->hostdevs + i + 1,
+sizeof(*def->hostdevs) *
+(def->nhostdevs - (i + 1)));
+def->nhostdevs--;
+if (VIR_REALLOC_N(def->hostdevs, def->nhostdevs) < 0) {
+/* ignore, harmless */
+}
+} else {
+VIR_FREE(def->hostdevs);
+def->nhostdevs = 0;
+}
+}
+
+/* Find an entry in hostdevs that matches the source spec in
+ * @match. return pointer to the entry in @found (if found is
+ * non-NULL). Returns index (within hostdevs) of matched entry, or -1
+ * if no match was found.
+ */
+int
+virDomainHostdevFind(virDomainDefPtr def,
+ virDomainHostdevDefPtr match,
+ virDomainHostdevDefPtr *found)
+{
+virDomainHostdevDefPtr local_found;
+virDomainHostdevSubsysPtr m_subsys = &match->source.subsys;
+int i;
+
+if (!found)
+found = &local_found;
+*found = NULL;
+
+/* There is no code that uses _MODE_CAPABILITIES, and nothing to
+ * compare if it did, so don't allow it.
+ */
+if (match->mode != VIR_DOMAIN_HOSTDEV_MODE_SUBSYS)
+return -1;
+
+for (i = 0 ; i < def->nhostdevs ; i++) {
+virDomainHostdevDefPtr compare = def->hostdevs[i];
+virDomainHostdevSubsysPtr c_subsys = &compare->source.subsys;
+
+if ((compare->mode != VIR_DOMAIN_HOSTDEV_MODE_SUBSYS) ||
+(c_subsys->type != m_subsys->type)) {
+continue;
+}
+
+switch (m_subsys->type)
+{
+case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI:
+if ((c_subsys->u.pci.domain == m_subsys->u.pci.domain) &&
+(c_subsys->u.pci.bus == m_subsys->u.pci.bus) &&
+(c_subsys->u.pci.slot == m_subsys->u.pci.slot) &&
+(c_subsys->u.pci.function == m_subsys->u.pci.function)) {
+*found = compare;
+}
+break;
+case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_USB:
+if (m_subsys->u.usb.bus && m_subsys->u.usb.device) {
+/* specified by bus location on host */
+if ((c_subsys->u.usb.bus== m_subsys->u.usb.bus) &&
+(c_subsys->u.usb.device == m_subsys->u.usb.device)) {
+*found = compare;
+}
+} else {
+/* specified by product & vendor id */
+if ((c_subsys->u.usb.product == m_subsys->u.usb.product) &&
+(c_subsys->u.usb.vendor == m_subsys->u.usb.vendor)) {
+*found = compare;
+}
+}
+break;
+default:
+break;
+}
+if (*found)
+break;
+}
+return *found ? i : -1;
+}
+
+int
virDomainDiskIndexByName(virDomainDefPtr def, const char *name,
bool allow_ambiguous)
{
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index 1a29fdb..343e48d 100644
--- a/src/conf/domain_conf.h
+++ b/src/conf/domain_conf.h
@@ -1895,6 +1895,11 @@ int virDomainNetIndexByMac(virDomainDefPtr def, const
unsigned char *mac);
int virDomainNetInsert(virDomainDefPtr def, virDomainNetDefPtr net);
int virDomainNetRemoveByMac(virDomainDefPtr def, const unsigned char *mac);
+int virDomainHostdevInsert(virDomainDefPtr def, virDomainHostdevDefPtr
hostdev);
+void virDomainHostdevRemove(virDomainDefPtr def, size_t i);
+int virDomainHostdevFind(vir