Code

Let objects be named.
[sysdb.git] / src / core / plugin.c
index 7a8483f237abb70bc04f5d840e573191fdbd36c4..f771d70936e5eda9fbbc2f2fbfa7c735b3e0605a 100644 (file)
@@ -27,9 +27,9 @@
 
 #include "sysdb.h"
 #include "core/plugin.h"
-#include "utils/error.h"
+#include "core/error.h"
+#include "core/time.h"
 #include "utils/llist.h"
-#include "utils/time.h"
 
 #include <assert.h>
 
@@ -65,18 +65,16 @@ struct sdb_plugin_info {
 
 typedef struct {
        sdb_object_t super;
-       char cb_name[64];
        void *cb_callback;
        sdb_object_t *cb_user_data;
        sdb_plugin_ctx_t cb_ctx;
 } sdb_plugin_cb_t;
-#define SDB_PLUGIN_CB_INIT { SDB_OBJECT_INIT, /* name = */ "", \
+#define SDB_PLUGIN_CB_INIT { SDB_OBJECT_INIT, \
        /* callback = */ NULL, /* user_data = */ NULL, \
        SDB_PLUGIN_CTX_INIT }
 
 typedef struct {
        sdb_plugin_cb_t super;
-#define ccb_name super.cb_name
 #define ccb_callback super.cb_callback
 #define ccb_user_data super.cb_user_data
 #define ccb_ctx super.cb_ctx
@@ -100,6 +98,7 @@ static sdb_llist_t      *config_list = NULL;
 static sdb_llist_t      *init_list = NULL;
 static sdb_llist_t      *collector_list = NULL;
 static sdb_llist_t      *shutdown_list = NULL;
+static sdb_llist_t      *log_list = NULL;
 
 /*
  * private helper functions
@@ -140,16 +139,6 @@ sdb_plugin_ctx_create(void)
        return ctx;
 } /* sdb_plugin_ctx_create */
 
-static int
-sdb_plugin_cmp_name(const sdb_object_t *a, const sdb_object_t *b)
-{
-       const sdb_plugin_cb_t *cb1 = (const sdb_plugin_cb_t *)a;
-       const sdb_plugin_cb_t *cb2 = (const sdb_plugin_cb_t *)b;
-
-       assert(cb1 && cb2);
-       return strcasecmp(cb1->cb_name, cb2->cb_name);
-} /* sdb_plugin_cmp_name */
-
 static int
 sdb_plugin_cmp_next_update(const sdb_object_t *a, const sdb_object_t *b)
 {
@@ -168,7 +157,7 @@ sdb_plugin_cmp_next_update(const sdb_object_t *a, const sdb_object_t *b)
 static sdb_plugin_cb_t *
 sdb_plugin_find_by_name(sdb_llist_t *list, const char *name)
 {
-       sdb_plugin_cb_t tmp = SDB_PLUGIN_CB_INIT;
+       sdb_object_t tmp = SDB_OBJECT_INIT;
 
        sdb_object_t *obj;
        assert(name);
@@ -176,38 +165,39 @@ sdb_plugin_find_by_name(sdb_llist_t *list, const char *name)
        if (! list)
                return NULL;
 
-       snprintf(tmp.cb_name, sizeof(tmp.cb_name), "%s", name);
-       tmp.cb_name[sizeof(tmp.cb_name) - 1] = '\0';
-       obj = sdb_llist_search(list, SDB_OBJ(&tmp), sdb_plugin_cmp_name);
+       tmp.name = strdup(name);
+       if (! tmp.name)
+               return NULL;
+       obj = sdb_llist_search(list, &tmp, sdb_object_cmp_by_name);
+       free(tmp.name);
        if (! obj)
                return NULL;
        return SDB_PLUGIN_CB(obj);
 } /* sdb_plugin_find_by_name */
 
+/*
+ * private types
+ */
+
 static int
 sdb_plugin_cb_init(sdb_object_t *obj, va_list ap)
 {
        sdb_llist_t **list = va_arg(ap, sdb_llist_t **);
-       const char  *type = va_arg(ap, const char *);
-       const char  *name = va_arg(ap, const char *);
-       void    *callback = va_arg(ap, void *);
+       const char   *type = va_arg(ap, const char *);
+       void     *callback = va_arg(ap, void *);
        sdb_object_t   *ud = va_arg(ap, sdb_object_t *);
 
        assert(list);
        assert(type);
        assert(obj);
 
-       if (sdb_plugin_find_by_name(*list, name)) {
+       if (sdb_plugin_find_by_name(*list, obj->name)) {
                sdb_log(SDB_LOG_WARNING, "plugin: %s callback '%s' "
                                "has already been registered. Ignoring newly "
-                               "registered version.", type, name);
+                               "registered version.", type, obj->name);
                return -1;
        }
 
-       snprintf(SDB_PLUGIN_CB(obj)->cb_name,
-                       sizeof(SDB_PLUGIN_CB(obj)->cb_name),
-                       "%s", name);
-       SDB_PLUGIN_CB(obj)->cb_name[sizeof(SDB_PLUGIN_CB(obj)->cb_name) - 1] = '\0';
        SDB_PLUGIN_CB(obj)->cb_callback = callback;
        SDB_PLUGIN_CB(obj)->cb_ctx      = sdb_plugin_get_ctx();
 
@@ -223,6 +213,22 @@ sdb_plugin_cb_destroy(sdb_object_t *obj)
        sdb_object_deref(SDB_PLUGIN_CB(obj)->cb_user_data);
 } /* sdb_plugin_cb_destroy */
 
+static sdb_type_t sdb_plugin_cb_type = {
+       sizeof(sdb_plugin_cb_t),
+
+       sdb_plugin_cb_init,
+       sdb_plugin_cb_destroy,
+       /* clone = */ NULL
+};
+
+static sdb_type_t sdb_plugin_collector_cb_type = {
+       sizeof(sdb_plugin_collector_cb_t),
+
+       sdb_plugin_cb_init,
+       sdb_plugin_cb_destroy,
+       /* clone = */ NULL
+};
+
 static int
 sdb_plugin_add_callback(sdb_llist_t **list, const char *type,
                const char *name, void *callback, sdb_object_t *user_data)
@@ -239,8 +245,8 @@ sdb_plugin_add_callback(sdb_llist_t **list, const char *type,
        if (! *list)
                return -1;
 
-       obj = sdb_object_create(sizeof(sdb_plugin_cb_t), sdb_plugin_cb_init,
-                       sdb_plugin_cb_destroy, list, type, name, callback, user_data);
+       obj = sdb_object_create(name, sdb_plugin_cb_type,
+                       list, type, callback, user_data);
        if (! obj)
                return -1;
 
@@ -421,6 +427,14 @@ sdb_plugin_register_shutdown(const char *name, sdb_plugin_shutdown_cb callback,
                        callback, user_data);
 } /* sdb_plugin_register_shutdown */
 
+int
+sdb_plugin_register_log(const char *name, sdb_plugin_log_cb callback,
+               sdb_object_t *user_data)
+{
+       return sdb_plugin_add_callback(&log_list, "log", name, callback,
+                       user_data);
+} /* sdb_plugin_register_log */
+
 int
 sdb_plugin_register_collector(const char *name, sdb_plugin_collector_cb callback,
                const sdb_time_t *interval, sdb_object_t *user_data)
@@ -435,9 +449,8 @@ sdb_plugin_register_collector(const char *name, sdb_plugin_collector_cb callback
        if (! collector_list)
                return -1;
 
-       obj = sdb_object_create(sizeof(sdb_plugin_collector_cb_t),
-                       sdb_plugin_cb_init, sdb_plugin_cb_destroy,
-                       &collector_list, "collector", name, callback, user_data);
+       obj = sdb_object_create(name, sdb_plugin_collector_cb_type,
+                       &collector_list, "collector", callback, user_data);
        if (! obj)
                return -1;
 
@@ -561,13 +574,20 @@ sdb_plugin_init_all(void)
                }
                sdb_plugin_set_ctx(old_ctx);
        }
+       sdb_llist_iter_destroy(iter);
        return 0;
 } /* sdb_plugin_init_all */
 
 int
 sdb_plugin_collector_loop(sdb_plugin_loop_t *loop)
 {
-       if ((! collector_list) || (! loop))
+       if (! collector_list) {
+               sdb_log(SDB_LOG_WARNING, "plugin: No collectors registered. "
+                               "Quiting main loop.");
+               return -1;
+       }
+
+       if (! loop)
                return -1;
 
        while (loop->do_loop) {
@@ -619,7 +639,7 @@ sdb_plugin_collector_loop(sdb_plugin_loop_t *loop)
                if (! interval) {
                        sdb_log(SDB_LOG_WARNING, "plugin: No interval configured "
                                        "for plugin '%s'; skipping any further "
-                                       "iterations.", SDB_PLUGIN_CCB(obj)->ccb_name);
+                                       "iterations.", obj->name);
                        sdb_object_deref(obj);
                        continue;
                }
@@ -636,7 +656,7 @@ sdb_plugin_collector_loop(sdb_plugin_loop_t *loop)
                if (now > SDB_PLUGIN_CCB(obj)->ccb_next_update) {
                        sdb_log(SDB_LOG_WARNING, "plugin: Plugin '%s' took too "
                                        "long; skipping iterations to keep up.",
-                                       SDB_PLUGIN_CCB(obj)->ccb_name);
+                                       obj->name);
                        SDB_PLUGIN_CCB(obj)->ccb_next_update = now;
                }
 
@@ -645,7 +665,7 @@ sdb_plugin_collector_loop(sdb_plugin_loop_t *loop)
                        sdb_log(SDB_LOG_ERR, "plugin: Failed to re-insert "
                                        "plugin '%s' into collector list. Unable to further "
                                        "use the plugin.",
-                                       SDB_PLUGIN_CCB(obj)->ccb_name);
+                                       obj->name);
                        sdb_object_deref(obj);
                        return -1;
                }
@@ -656,5 +676,31 @@ sdb_plugin_collector_loop(sdb_plugin_loop_t *loop)
        return 0;
 } /* sdb_plugin_read_loop */
 
+int
+sdb_plugin_log(int prio, const char *msg)
+{
+       sdb_llist_iter_t *iter;
+       int ret = -1;
+
+       if (! log_list)
+               return fprintf(stderr, "[%s] %s\n", SDB_LOG_PRIO_TO_STRING(prio), msg);
+
+       iter = sdb_llist_get_iter(log_list);
+       while (sdb_llist_iter_has_next(iter)) {
+               sdb_plugin_log_cb callback;
+               int tmp;
+
+               sdb_object_t *obj = sdb_llist_iter_get_next(iter);
+               assert(obj);
+
+               callback = SDB_PLUGIN_CB(obj)->cb_callback;
+               tmp = callback(prio, msg, SDB_PLUGIN_CB(obj)->cb_user_data);
+               if (tmp > ret)
+                       ret = tmp;
+       }
+       sdb_llist_iter_destroy(iter);
+       return ret;
+} /* sdb_plugin_log */
+
 /* vim: set tw=78 sw=4 ts=4 noexpandtab : */