Use virFileFindResource to locate driver plugins

Replace virDriverModuleInitialize with virFileFindResource
usage.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>

diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c
index c5915bd30803bea51f151f89da247ee6d47484ef..e549783d40773b8c7607a0e16309dd33bec47846 100644 (file)
--- a/daemon/libvirtd.c
+++ b/daemon/libvirtd.c
@@ -1167,25 +1167,14 @@ int main(int argc, char **argv) {
             exit(EXIT_FAILURE);
         }
         *tmp = '\0';
-        char *driverdir;
-        if (virAsprintfQuiet(&driverdir, "%s/../../src/.libs", argv[0]) < 0 ||
-            virAsprintfQuiet(&cpumap, "%s/../../src/cpu/cpu_map.xml",
+        if (virAsprintfQuiet(&cpumap, "%s/../../src/cpu/cpu_map.xml",
                              argv[0]) < 0) {
             fprintf(stderr, _("%s: initialization failed\n"), argv[0]);
             exit(EXIT_FAILURE);
         }
-        if (access(driverdir, R_OK) < 0) {
-            fprintf(stderr, _("%s: expected driver directory '%s' is missing\n"),
-                    argv[0], driverdir);
-            exit(EXIT_FAILURE);
-        }
-#ifdef WITH_DRIVER_MODULES
-        virDriverModuleInitialize(driverdir);
-#endif
         cpuMapOverride(cpumap);
         VIR_FREE(cpumap);
         *tmp = '/';
-        /* Must not free 'driverdir' - it is still used */
     }
 
     while (1) {

diff --git a/src/driver.c b/src/driver.c
index 6b79b5e3857f51e4bea7c54637cabe6e1cda61ce..9e3a2eb1dc0114541531ac8f8386d2aa9efe6f65 100644 (file)
--- a/src/driver.c
+++ b/src/driver.c
@@ -26,6 +26,7 @@
 
 #include "driver.h"
 #include "viralloc.h"
+#include "virfile.h"
 #include "virlog.h"
 #include "virutil.h"
 #include "configmake.h"
@@ -41,21 +42,6 @@ VIR_LOG_INIT("driver");
 
 # include <dlfcn.h>
 
-static const char *moddir = NULL;
-
-void
-virDriverModuleInitialize(const char *defmoddir)
-{
-    const char *custommoddir = virGetEnvBlockSUID("LIBVIRT_DRIVER_DIR");
-    if (custommoddir)
-        moddir = custommoddir;
-    else if (defmoddir)
-        moddir = defmoddir;
-    else
-        moddir = DEFAULT_DRIVER_DIR;
-    VIR_DEBUG("Module dir %s", moddir);
-}
-
 void *
 virDriverLoadModule(const char *name)
 {
@@ -63,12 +49,14 @@ virDriverLoadModule(const char *name)
     void *handle = NULL;
     int (*regsym)(void);
 
-    if (moddir == NULL)
-        virDriverModuleInitialize(NULL);
-
     VIR_DEBUG("Module load %s", name);
 
-    if (virAsprintfQuiet(&modfile, "%s/libvirt_driver_%s.so", moddir, name) < 0)
+    if (!(modfile = virFileFindResourceFull(name,
+                                            "libvirt_driver_",
+                                            ".so",
+                                            "src/.libs",
+                                            LIBDIR "/libvirt/connection-driver",
+                                            "LIBVIRT_DRIVER_DIR")))
         return NULL;
 
     if (access(modfile, R_OK) < 0) {

diff --git a/src/driver.h b/src/driver.h
index e66fc7ae5f8ea4b0a2256e468d4b07bf7719cff3..729e7438c81927ac09db7b5f463a685a49de7046 100644 (file)
--- a/src/driver.h
+++ b/src/driver.h
@@ -2164,7 +2164,6 @@ int virRegisterNWFilterDriver(virNWFilterDriverPtr) ATTRIBUTE_RETURN_CHECK;
 # ifdef WITH_LIBVIRTD
 int virRegisterStateDriver(virStateDriverPtr) ATTRIBUTE_RETURN_CHECK;
 # endif
-void virDriverModuleInitialize(const char *defmoddir);
 void *virDriverLoadModule(const char *name);
 
 #endif /* __VIR_DRIVER_H__ */

diff --git a/src/libvirt_driver_modules.syms b/src/libvirt_driver_modules.syms
index 35ac1de44ea000f229dbf8b4cc0b9e156d089b68..f9d0ee9b971407ca4c429c38e25c68558aeffa09 100644 (file)
--- a/src/libvirt_driver_modules.syms
+++ b/src/libvirt_driver_modules.syms
@@ -4,7 +4,6 @@
 
 # driver.h
 virDriverLoadModule;
-virDriverModuleInitialize;
 
 # Let emacs know we want case-insensitive sorting
 # Local Variables:

diff --git a/tests/virdrivermoduletest.c b/tests/virdrivermoduletest.c
index 4203f5bab6782e49d7c04f7b49bcbccebc63e977..840fc280f6b5e06a97c9d62cec16f7f4de6dc2cf 100644 (file)
--- a/tests/virdrivermoduletest.c
+++ b/tests/virdrivermoduletest.c
@@ -65,8 +65,6 @@ mymain(void)
             ret = -1;                                                   \
     } while (0)
 
-    virDriverModuleInitialize(abs_builddir "/../src/.libs");
-
 #ifdef WITH_NETWORK
 # define USE_NETWORK "network"
     TEST("network", NULL);