[libvirt] [PATCH 3/5] interface: always build all available backends

[Doug Goldstein]

Always build all available backends to avoid bit-rot. At run time we
select the correct backend and load it by attempting netcf first and
then udev.
---
 src/Makefile.am                         |    6 ++--
 src/interface/interface_backend_netcf.c |    2 +-
 src/interface/interface_backend_udev.c  |    6 +++-
 src/interface/interface_driver.c        |   37 +++++++++++++++++++++++++++++++
 src/interface/interface_driver.h        |    3 ++
 5 files changed, 48 insertions(+), 6 deletions(-)
 create mode 100644 src/interface/interface_driver.c

diff --git a/src/Makefile.am b/src/Makefile.am
index 98448c1..34bc75c 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -557,18 +557,18 @@ INTERFACE_DRIVER_SOURCES =
 
 if WITH_INTERFACE
 INTERFACE_DRIVER_SOURCES +=					\
-		interface/interface_driver.h
+		interface/interface_driver.h        \
+		interface/interface_driver.c
 
 if WITH_NETCF
 INTERFACE_DRIVER_SOURCES +=					\
 		interface/interface_backend_netcf.c
-else
+endif
 if HAVE_UDEV
 INTERFACE_DRIVER_SOURCES +=					\
 		interface/interface_backend_udev.c
 endif
 endif
-endif
 
 SECRET_DRIVER_SOURCES =						\
 		secret/secret_driver.h secret/secret_driver.c
diff --git a/src/interface/interface_backend_netcf.c b/src/interface/interface_backend_netcf.c
index b857ac7..140fa9f 100644
--- a/src/interface/interface_backend_netcf.c
+++ b/src/interface/interface_backend_netcf.c
@@ -805,7 +805,7 @@ static virInterfaceDriver interfaceDriver = {
 #endif /* HAVE_NETCF_TRANSACTIONS */
 };
 
-int interfaceRegister(void) {
+int netcfIfaceRegister(void) {
     virRegisterInterfaceDriver(&interfaceDriver);
     return 0;
 }
diff --git a/src/interface/interface_backend_udev.c b/src/interface/interface_backend_udev.c
index bb161ab..b001e6e 100644
--- a/src/interface/interface_backend_udev.c
+++ b/src/interface/interface_backend_udev.c
@@ -503,9 +503,11 @@ static virInterfaceDriver udevIfaceDriver = {
 };
 
 int
-interfaceRegister(void) {
-    if (virRegisterInterfaceDriver(&udevIfaceDriver) < 0)
+udevIfaceRegister(void) {
+    if (virRegisterInterfaceDriver(&udevIfaceDriver) < 0) {
         virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                        _("failed to register udev interface driver"));
+        return -1;
+    }
     return 0;
 }
diff --git a/src/interface/interface_driver.c b/src/interface/interface_driver.c
new file mode 100644
index 0000000..fbf861e
--- /dev/null
+++ b/src/interface/interface_driver.c
@@ -0,0 +1,37 @@
+/*
+ * interface_driver.c: loads the appropriate backend
+ *
+ * Copyright (C) 2012 Doug Goldstein <cardoe at cardoe.com>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library;  If not, see
+ * <http://www.gnu.org/licenses/>.
+ */
+#include <config.h>
+
+#include "interface_driver.h"
+
+int
+interfaceRegister(void) {
+#ifdef WITH_NETCF
+    /* Attempt to load the netcf based backend first */
+    if (netcfIfaceRegister() == 0)
+        return 0;
+#endif /* WITH_NETCF */
+#if HAVE_UDEV
+    /* If there's no netcf or it failed to load, register the udev backend */
+    if (udevIfaceRegister() == 0)
+        return 0;
+#endif /* HAVE_UDEV */
+    return -1;
+}
diff --git a/src/interface/interface_driver.h b/src/interface/interface_driver.h
index 67b6218..80ada5c 100644
--- a/src/interface/interface_driver.h
+++ b/src/interface/interface_driver.h
@@ -26,4 +26,7 @@
 
 int interfaceRegister(void);
 
+int netcfIfaceRegister(void);
+int udevIfaceRegister(void);
+
 #endif /* __VIR_INTERFACE__DRIVER_H */
-- 
1.7.8.6