actions: improve logging in lttng_action_create_from_buffer

Small improvements in the logging messages.  Use "Create from buffer"
instead of "Deserializing", since that's the term used in the function
names.

Change-Id: I7e688df766602cfb73bc40d87d224591c0f29534
Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/src/common/actions/action.c b/src/common/actions/action.c
index 6894036cb779d09ced9c5acee4dacf9a2e99d876..4cad01b743189ff5318fa9e388ed2bd1d82f933a 100644 (file)
--- a/src/common/actions/action.c
+++ b/src/common/actions/action.c
@@ -10,6 +10,18 @@
 #include <common/error.h>
 #include <assert.h>
 
+static const char *lttng_action_type_string(enum lttng_action_type action_type)
+{
+       switch (action_type) {
+       case LTTNG_ACTION_TYPE_UNKNOWN:
+               return "UNKNOWN";
+       case LTTNG_ACTION_TYPE_NOTIFY:
+               return "NOTIFY";
+       default:
+               return "???";
+       }
+}
+
 enum lttng_action_type lttng_action_get_type(struct lttng_action *action)
 {
        return action ? action->type : LTTNG_ACTION_TYPE_UNKNOWN;
@@ -106,12 +118,18 @@ ssize_t lttng_action_create_from_buffer(const struct lttng_buffer_view *view,
 
        action_comm = (const struct lttng_action_comm *) view->data;
 
-       DBG("Deserializing action from buffer");
+       DBG("Create action from buffer: action-type=%s",
+                       lttng_action_type_string(action_comm->action_type));
+
        switch (action_comm->action_type) {
        case LTTNG_ACTION_TYPE_NOTIFY:
                create_from_buffer_cb = lttng_action_notify_create_from_buffer;
                break;
        default:
+               ERR("Failed to create action from buffer, unhandled action type: action-type=%u (%s)",
+                               action_comm->action_type,
+                               lttng_action_type_string(
+                                               action_comm->action_type));
                consumed_len = -1;
                goto end;
        }