Make the Python plugin provider a libtool module
[babeltrace.git] / src / lib / plugin / plugin.c
index 36059c4ad3b13f6b04eb17ca7ad1dcd075948469..8f2f6da347cb38792b108ed1c8ee3d6cb8c75dbf 100644 (file)
@@ -33,6 +33,7 @@
 #include "common/common.h"
 #include <babeltrace2/plugin/plugin-const.h>
 #include <babeltrace2/graph/component-class-const.h>
+#include <babeltrace2/current-thread.h>
 #include "lib/graph/component-class.h"
 #include <babeltrace2/types.h>
 #include <glib.h>
@@ -48,7 +49,8 @@
 #include "plugin-so.h"
 #include "lib/func-status.h"
 
-#define PYTHON_PLUGIN_PROVIDER_FILENAME        "libbabeltrace2-python-plugin-provider." G_MODULE_SUFFIX
+#define PYTHON_PLUGIN_PROVIDER_FILENAME        "babeltrace2-python-plugin-provider." G_MODULE_SUFFIX
+#define PYTHON_PLUGIN_PROVIDER_DIR     BABELTRACE_PLUGIN_PROVIDERS_DIR
 #define PYTHON_PLUGIN_PROVIDER_SYM_NAME        bt_plugin_python_create_all_from_file
 #define PYTHON_PLUGIN_PROVIDER_SYM_NAME_STR    G_STRINGIFY(PYTHON_PLUGIN_PROVIDER_SYM_NAME)
 
@@ -64,7 +66,7 @@ int (*bt_plugin_python_create_all_from_file_sym)(
        bt_plugin_python_create_all_from_file;
 
 static
-void init_python_plugin_provider(void)
+enum bt_plugin_status init_python_plugin_provider(void)
 {
 }
 #else /* BT_BUILT_IN_PYTHON_PLUGIN_SUPPORT */
@@ -76,14 +78,34 @@ int (*bt_plugin_python_create_all_from_file_sym)(
                 struct bt_plugin_set **plugin_set_out);
 
 static
-void init_python_plugin_provider(void) {
+int init_python_plugin_provider(void) {
+       int status = BT_FUNC_STATUS_OK;
+       const char *provider_dir_envvar;
+       static const char * const provider_dir_envvar_name = "LIBBABELTRACE2_PLUGIN_PROVIDER_DIR";
+       char *provider_path = NULL;
+
        if (bt_plugin_python_create_all_from_file_sym != NULL) {
-               return;
+               goto end;
        }
 
        BT_LOGI_STR("Loading Python plugin provider module.");
-       python_plugin_provider_module =
-               g_module_open(PYTHON_PLUGIN_PROVIDER_FILENAME, 0);
+
+       provider_dir_envvar = getenv(provider_dir_envvar_name);
+       if (provider_dir_envvar) {
+               provider_path = g_build_filename(provider_dir_envvar,
+                       PYTHON_PLUGIN_PROVIDER_FILENAME, NULL);
+               BT_LOGI("Using `%s` environment variable to find the Python "
+                       "plugin provider: path=\"%s\"", provider_dir_envvar_name,
+                       provider_path);
+       } else {
+               provider_path = g_build_filename(PYTHON_PLUGIN_PROVIDER_DIR,
+                       PYTHON_PLUGIN_PROVIDER_FILENAME, NULL);
+               BT_LOGI("Using default path (`%s` environment variable is not "
+                       "set) to find the Python plugin provider: path=\"%s\"",
+                       provider_dir_envvar_name, provider_path);
+       }
+
+       python_plugin_provider_module = g_module_open(provider_path, 0);
        if (!python_plugin_provider_module) {
                /*
                 * This is not an error. The whole point of having an
@@ -91,8 +113,8 @@ void init_python_plugin_provider(void) {
                 * missing and the Babeltrace library still works.
                 */
                BT_LOGI("Cannot open `%s`: %s: continuing without Python plugin support.",
-                       PYTHON_PLUGIN_PROVIDER_FILENAME, g_module_error());
-               return;
+                       provider_path, g_module_error());
+               goto end;
        }
 
        if (!g_module_symbol(python_plugin_provider_module,
@@ -103,17 +125,24 @@ void init_python_plugin_provider(void) {
                 * plugin provider shared object, we expect this symbol
                 * to exist.
                 */
-               BT_LOGE("Cannot find the Python plugin provider loading symbol: "
+               BT_LIB_LOGE_APPEND_CAUSE(
+                       "Cannot find the Python plugin provider loading symbol: "
                        "%s: continuing without Python plugin support: "
                        "file=\"%s\", symbol=\"%s\"",
                        g_module_error(),
-                       PYTHON_PLUGIN_PROVIDER_FILENAME,
+                       provider_path,
                        PYTHON_PLUGIN_PROVIDER_SYM_NAME_STR);
-               return;
+               status = BT_FUNC_STATUS_ERROR;
+               goto end;
        }
 
        BT_LOGI("Loaded Python plugin provider module: addr=%p",
                python_plugin_provider_module);
+
+end:
+       g_free(provider_path);
+
+       return status;
 }
 
 __attribute__((destructor)) static
@@ -122,6 +151,10 @@ void fini_python_plugin_provider(void) {
                BT_LOGI("Unloading Python plugin provider module.");
 
                if (!g_module_close(python_plugin_provider_module)) {
+                       /*
+                        * This occurs when the library is finalized: do
+                        * NOT append an error cause.
+                        */
                        BT_LOGE("Failed to close the Python plugin provider module: %s.",
                                g_module_error());
                }
@@ -131,17 +164,17 @@ void fini_python_plugin_provider(void) {
 }
 #endif
 
-uint64_t bt_plugin_set_get_plugin_count(struct bt_plugin_set *plugin_set)
+uint64_t bt_plugin_set_get_plugin_count(const struct bt_plugin_set *plugin_set)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin_set, "Plugin set");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin_set, "Plugin set");
        return (uint64_t) plugin_set->plugins->len;
 }
 
 const struct bt_plugin *bt_plugin_set_borrow_plugin_by_index_const(
                const struct bt_plugin_set *plugin_set, uint64_t index)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin_set, "Plugin set");
-       BT_ASSERT_PRE_VALID_INDEX(index, plugin_set->plugins->len);
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin_set, "Plugin set");
+       BT_ASSERT_PRE_DEV_VALID_INDEX(index, plugin_set->plugins->len);
        return g_ptr_array_index(plugin_set->plugins, index);
 }
 
@@ -180,8 +213,17 @@ enum bt_plugin_find_all_from_file_status bt_plugin_find_all_from_file(
        BT_ASSERT(!*plugin_set_out);
 
        /* Try Python plugins if support is available */
-       init_python_plugin_provider();
+       status = init_python_plugin_provider();
+       if (status < 0) {
+               /* init_python_plugin_provider() logs errors */
+               goto end;
+       }
+
+       BT_ASSERT(status == BT_FUNC_STATUS_OK);
+       status = BT_FUNC_STATUS_NOT_FOUND;
+
        if (bt_plugin_python_create_all_from_file_sym) {
+               /* Python plugin provider exists */
                status = bt_plugin_python_create_all_from_file_sym(path,
                        fail_on_load_error, (void *) plugin_set_out);
                if (status == BT_FUNC_STATUS_OK) {
@@ -189,6 +231,11 @@ enum bt_plugin_find_all_from_file_status bt_plugin_find_all_from_file(
                        BT_ASSERT((*plugin_set_out)->plugins->len > 0);
                        goto end;
                } else if (status < 0) {
+                       /*
+                        * bt_plugin_python_create_all_from_file_sym()
+                        * handles `fail_on_load_error` itself, so this
+                        * is a "real" error.
+                        */
                        BT_ASSERT(!*plugin_set_out);
                        goto end;
                }
@@ -235,7 +282,7 @@ enum bt_plugin_find_status bt_plugin_find(const char *plugin_name,
                "name=\"%s\"", plugin_name);
        dirs = g_ptr_array_new_with_free_func((GDestroyNotify) destroy_gstring);
        if (!dirs) {
-               BT_LOGE_STR("Failed to allocate a GPtrArray.");
+               BT_LIB_LOGE_APPEND_CAUSE("Failed to allocate a GPtrArray.");
                status = BT_FUNC_STATUS_MEMORY_ERROR;
                goto end;
        }
@@ -258,7 +305,8 @@ enum bt_plugin_find_status bt_plugin_find(const char *plugin_name,
        if (envvar) {
                ret = bt_common_append_plugin_path_dirs(envvar, dirs);
                if (ret) {
-                       BT_LOGE_STR("Failed to append plugin path to array of directories.");
+                       BT_LIB_LOGE_APPEND_CAUSE(
+                               "Failed to append plugin path to array of directories.");
                        status = BT_FUNC_STATUS_MEMORY_ERROR;
                        goto end;
                }
@@ -269,7 +317,7 @@ enum bt_plugin_find_status bt_plugin_find(const char *plugin_name,
                GString *home_plugin_dir_str = g_string_new(home_plugin_dir);
 
                if (!home_plugin_dir_str) {
-                       BT_LOGE_STR("Failed to allocate a GString.");
+                       BT_LIB_LOGE_APPEND_CAUSE("Failed to allocate a GString.");
                        status = BT_FUNC_STATUS_MEMORY_ERROR;
                        goto end;
                }
@@ -283,7 +331,7 @@ enum bt_plugin_find_status bt_plugin_find(const char *plugin_name,
                        g_string_new(system_plugin_dir);
 
                if (!system_plugin_dir_str) {
-                       BT_LOGE_STR("Failed to allocate a GString.");
+                       BT_LIB_LOGE_APPEND_CAUSE("Failed to allocate a GString.");
                        status = BT_FUNC_STATUS_MEMORY_ERROR;
                        goto end;
                }
@@ -501,6 +549,10 @@ int bt_plugin_create_append_all_from_dir(struct bt_plugin_set *plugin_set,
                BT_LOGW_ERRNO("Cannot open directory",
                        ": path=\"%s\", recurse=%d",
                        path, recurse);
+               (void) BT_CURRENT_THREAD_ERROR_APPEND_CAUSE_FROM_UNKNOWN(
+                       "Babeltrace library",
+                       "Cannot open directory: path=\"%s\", recurse=%d",
+                       path, recurse);
                status = BT_FUNC_STATUS_ERROR;
                goto end;
        }
@@ -516,7 +568,7 @@ int bt_plugin_create_append_all_from_dir(struct bt_plugin_set *plugin_set,
        status = append_all_from_dir_info.status;
        pthread_mutex_unlock(&append_all_from_dir_info.lock);
        if (ret) {
-               BT_LOGW_ERRNO("Failed to walk directory",
+               BT_LIB_LOGW_APPEND_CAUSE("Failed to walk directory",
                        ": path=\"%s\", recurse=%d",
                        path, recurse);
                status = BT_FUNC_STATUS_ERROR;
@@ -548,7 +600,7 @@ enum bt_plugin_find_all_from_dir_status bt_plugin_find_all_from_dir(
                path, recurse);
        *plugin_set_out = bt_plugin_set_create();
        if (!*plugin_set_out) {
-               BT_LOGE_STR("Cannot create empty plugin set.");
+               BT_LIB_LOGE_APPEND_CAUSE("Cannot create empty plugin set.");
                status = BT_FUNC_STATUS_MEMORY_ERROR;
                goto error;
        }
@@ -564,7 +616,8 @@ enum bt_plugin_find_all_from_dir_status bt_plugin_find_all_from_dir(
                 * bt_plugin_create_append_all_from_dir() handles
                 * `fail_on_load_error`, so this is a "real" error.
                 */
-               BT_LOGW("Cannot append plugins found in directory: "
+               BT_LIB_LOGE_APPEND_CAUSE(
+                       "Cannot append plugins found in directory: "
                        "path=\"%s\", status=%s",
                        path, bt_common_func_status_string(status));
                goto error;
@@ -594,31 +647,31 @@ end:
 
 const char *bt_plugin_get_name(const struct bt_plugin *plugin)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
        return plugin->info.name_set ? plugin->info.name->str : NULL;
 }
 
 const char *bt_plugin_get_author(const struct bt_plugin *plugin)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
        return plugin->info.author_set ? plugin->info.author->str : NULL;
 }
 
 const char *bt_plugin_get_license(const struct bt_plugin *plugin)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
        return plugin->info.license_set ? plugin->info.license->str : NULL;
 }
 
 const char *bt_plugin_get_path(const struct bt_plugin *plugin)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
        return plugin->info.path_set ? plugin->info.path->str : NULL;
 }
 
 const char *bt_plugin_get_description(const struct bt_plugin *plugin)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
        return plugin->info.description_set ?
                plugin->info.description->str : NULL;
 }
@@ -630,7 +683,7 @@ enum bt_property_availability bt_plugin_get_version(const struct bt_plugin *plug
        enum bt_property_availability avail =
                BT_PROPERTY_AVAILABILITY_AVAILABLE;
 
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
 
        if (!plugin->info.version_set) {
                BT_LIB_LOGD("Plugin's version is not set: %!+l", plugin);
@@ -660,19 +713,19 @@ end:
 
 uint64_t bt_plugin_get_source_component_class_count(const struct bt_plugin *plugin)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
        return (uint64_t) plugin->src_comp_classes->len;
 }
 
 uint64_t bt_plugin_get_filter_component_class_count(const struct bt_plugin *plugin)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
        return (uint64_t) plugin->flt_comp_classes->len;
 }
 
 uint64_t bt_plugin_get_sink_component_class_count(const struct bt_plugin *plugin)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
        return (uint64_t) plugin->sink_comp_classes->len;
 }
 
@@ -681,8 +734,8 @@ struct bt_component_class *borrow_component_class_by_index(
                const struct bt_plugin *plugin, GPtrArray *comp_classes,
                uint64_t index)
 {
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
-       BT_ASSERT_PRE_VALID_INDEX(index, comp_classes->len);
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_VALID_INDEX(index, comp_classes->len);
        return g_ptr_array_index(comp_classes, index);
 }
 
@@ -718,8 +771,8 @@ struct bt_component_class *borrow_component_class_by_name(
        struct bt_component_class *comp_class = NULL;
        size_t i;
 
-       BT_ASSERT_PRE_NON_NULL(plugin, "Plugin");
-       BT_ASSERT_PRE_NON_NULL(name, "Name");
+       BT_ASSERT_PRE_DEV_NON_NULL(plugin, "Plugin");
+       BT_ASSERT_PRE_DEV_NON_NULL(name, "Name");
 
        for (i = 0; i < comp_classes->len; i++) {
                struct bt_component_class *comp_class_candidate =
This page took 0.029482 seconds and 4 git commands to generate.