Adapt plugin system to use unified reference counting
[babeltrace.git] / plugins / component-factory.c
index 74c6b72c30f02c69d245a57b25f5be38930fec48..03e0148595065ee1a14e4e2ce17673e9ed275ed9 100644 (file)
 #include <babeltrace/plugin/component-factory-internal.h>
 #include <babeltrace/babeltrace-internal.h>
 #include <babeltrace/compiler.h>
+#include <babeltrace/ref.h>
 #include <unistd.h>
 #include <stdlib.h>
 #include <sys/stat.h>
+#include <gmodule.h>
 
 #define NATIVE_PLUGIN_SUFFIX ".so"
 #define NATIVE_PLUGIN_SUFFIX_LEN sizeof(NATIVE_PLUGIN_SUFFIX)
 #define PLUGIN_SUFFIX_LEN max_t(size_t, sizeof(NATIVE_PLUGIN_SUFFIX), \
                sizeof(LIBTOOL_PLUGIN_SUFFIX))
 
-static
-void component_destroy(gpointer data)
-{
-        g_free(data);
-}
-
-static
-void plugin_destroy(gpointer data)
-{
-       struct plugin *plugin = data;
-
-       if (plugin->module && g_module_close(plugin->module)) {
-               printf_error("Failed to close plugin");
-       }
-
-       if (plugin->components) {
-               g_ptr_array_free(plugin->components, TRUE);
-       }
-
-       g_free(plugin);
-}
-
-static
-struct plugin *plugin_create(void)
-{
-       struct plugin *plugin = g_new0(struct plugin, 1);
-
-       if (!plugin) {
-               goto end;
-       }
-
-       plugin->components = g_ptr_array_new_with_free_func(component_destroy);
-       if (!plugin->components) {
-               g_free(plugin);
-               plugin = NULL;
-               goto end;
-       }
-end:
-       return plugin;
-}
-
 /* Allocate dirent as recommended by READDIR(3), NOTES on readdir_r */
 static
 struct dirent *alloc_dirent(const char *path)
@@ -99,21 +60,16 @@ struct dirent *alloc_dirent(const char *path)
        return entry;
 }
 
-static
-struct plugin *load_plugin(GModule *module)
-{
-       return NULL;
-}
-
 static
 enum bt_component_factory_status
 bt_component_factory_load_file(struct bt_component_factory *factory,
                const char *path)
 {
        enum bt_component_factory_status ret = BT_COMPONENT_FACTORY_STATUS_OK;
+       enum bt_component_status component_status;
        size_t path_len;
        GModule *module;
-       struct plugin *plugin;
+       struct bt_plugin *plugin;
 
        if (!factory || !path) {
                ret = BT_COMPONENT_FACTORY_STATUS_INVAL;
@@ -132,11 +88,11 @@ bt_component_factory_load_file(struct bt_component_factory *factory,
         * or .la on Linux).
         */
        if (strncmp(NATIVE_PLUGIN_SUFFIX,
-               path + path_len - NATIVE_PLUGIN_SUFFIX_LEN,
-               NATIVE_PLUGIN_SUFFIX_LEN) &&
-               strncmp(LIBTOOL_PLUGIN_SUFFIX,
-               path + path_len - LIBTOOL_PLUGIN_SUFFIX_LEN,
-               LIBTOOL_PLUGIN_SUFFIX_LEN)) {
+                       path + path_len - NATIVE_PLUGIN_SUFFIX_LEN,
+                       NATIVE_PLUGIN_SUFFIX_LEN) &&
+                       strncmp(LIBTOOL_PLUGIN_SUFFIX,
+                       path + path_len - LIBTOOL_PLUGIN_SUFFIX_LEN,
+                       LIBTOOL_PLUGIN_SUFFIX_LEN)) {
                /* Name indicates that this is not a plugin file. */
                ret = BT_COMPONENT_FACTORY_STATUS_INVAL;
                goto end;
@@ -149,17 +105,33 @@ bt_component_factory_load_file(struct bt_component_factory *factory,
                goto end;
        }
 
-       /* Check if the module defines the appropriate entry points */
-       plugin = load_plugin(module);
+       /* Load plugin and make sure it defines the required entry points */
+       plugin = bt_plugin_create(module);
        if (!plugin) {
-               /* Not a Babeltrace plugin */
-               ret = BT_COMPONENT_FACTORY_STATUS_INVAL;
+               ret = BT_COMPONENT_FACTORY_STATUS_INVAL_PLUGIN;
                if (!g_module_close(module)) {
                        printf_error("Module close error: %s",
-                               g_module_error());
+                               g_module_error());
+               }
+               goto end;
+       }
+
+       component_status = bt_plugin_register_component_classes(plugin,
+                       factory);
+       if (component_status != BT_COMPONENT_STATUS_OK) {
+               switch (component_status) {
+               case BT_COMPONENT_STATUS_NOMEM:
+                       ret = BT_COMPONENT_FACTORY_STATUS_NOMEM;
+                       break;
+               default:
+                       ret = BT_COMPONENT_FACTORY_STATUS_ERROR;
+                       break;
                }
+
+               BT_PUT(plugin);
                goto end;
        }
+       g_ptr_array_add(factory->plugins, plugin);
 end:
        return ret;
 }
@@ -220,7 +192,7 @@ bt_component_factory_load_dir_recursive(struct bt_component_factory *factory,
 
                if (S_ISDIR(st.st_mode)) {
                        ret = bt_component_factory_load_dir_recursive(factory,
-                               file_path);
+                                       file_path);
                        if (ret != BT_COMPONENT_FACTORY_STATUS_OK) {
                                goto end;
                        }
@@ -234,17 +206,24 @@ end:
        return ret;
 }
 
-void bt_component_factory_destroy(struct bt_component_factory *factory)
+static
+void bt_component_factory_destroy(struct bt_object *obj)
 {
-       if (!factory) {
-               return;
-       }
+       struct bt_component_factory *factory = NULL;
+
+       assert(obj);
+       factory = container_of(obj, struct bt_component_factory, base);
 
+       if (factory->plugins) {
+               g_ptr_array_free(factory->plugins, TRUE);
+       }
+       if (factory->components) {
+               g_ptr_array_free(factory->components, TRUE);
+       }
        g_free(factory);
 }
 
-struct bt_component_factory *
-bt_component_factory_create(void)
+struct bt_component_factory *bt_component_factory_create(void)
 {
        struct bt_component_factory *factory;
 
@@ -253,51 +232,48 @@ bt_component_factory_create(void)
                goto end;
        }
 
-       factory->plugins = g_ptr_array_new_with_free_func(plugin_destroy);
+       bt_object_init(factory, bt_component_factory_destroy);
+       factory->plugins = g_ptr_array_new_with_free_func(
+               (GDestroyNotify) bt_put);
        if (!factory->plugins) {
                goto error;
        }
+       factory->components = g_ptr_array_new_with_free_func(
+               (GDestroyNotify) bt_put);
+       if (!factory->components) {
+               goto error;
+       }
 end:
        return factory;
 error:
-       bt_component_factory_destroy(factory);
-       return NULL;
+        BT_PUT(factory);
+       return factory;
 }
 
 enum bt_component_factory_status bt_component_factory_load(
                struct bt_component_factory *factory, const char *path)
 {
        enum bt_component_factory_status ret = BT_COMPONENT_FACTORY_STATUS_OK;
-       DIR *directory = NULL;
 
        if (!factory || !path) {
                ret = BT_COMPONENT_FACTORY_STATUS_INVAL;
                goto end;
        }
 
-       directory = opendir(path) ;
-       if (!directory) {
-               switch (errno) {
-               case ENOTDIR:
-                       /* Try loading as a file. */
-                       break;
-               case ENOENT:
-                       ret = BT_COMPONENT_FACTORY_STATUS_NOENT;
-                       goto end;
-               default:
-                       ret = BT_COMPONENT_FACTORY_STATUS_IO;
-                       goto end;
-               }
+       if (!g_file_test(path, G_FILE_TEST_EXISTS)) {
+               ret = BT_COMPONENT_FACTORY_STATUS_NOENT;
+               goto end;
        }
 
-       if (directory) {
+       if (g_file_test(path, G_FILE_TEST_IS_DIR)) {
                ret = bt_component_factory_load_dir_recursive(factory, path);
-       } else {
+       } else if (g_file_test(path, G_FILE_TEST_IS_REGULAR) ||
+                       g_file_test(path, G_FILE_TEST_IS_SYMLINK)) {
                ret = bt_component_factory_load_file(factory, path);
+       } else {
+               ret = BT_COMPONENT_FACTORY_STATUS_INVAL;
+               goto end;
        }
 end:
-       if (directory) {
-               closedir(directory);
-       }
        return ret;
 }
This page took 0.036676 seconds and 4 git commands to generate.