Introduce struct lttng_kernel_tracepoint_class, enum probe_desc field

[lttng-modules.git] / src / probes / lttng-kretprobes.c

diff --git a/src/probes/lttng-kretprobes.c b/src/probes/lttng-kretprobes.c
index 2677f8ab8abe77ce35adcf44a322a0df53546b76..7d0db416908f6ba95ff21600d115374a6f9dc477 100644 (file)
--- a/src/probes/lttng-kretprobes.c
+++ b/src/probes/lttng-kretprobes.c
@@ -112,8 +112,19 @@ int lttng_kretprobes_handler_exit(struct kretprobe_instance *krpi,
        return _lttng_kretprobes_handler(krpi, regs, EVENT_EXIT);
 }
 
-static const struct lttng_kernel_type_common *event_type =
-       lttng_kernel_static_type_integer_from_type(unsigned long, __BYTE_ORDER, 16);
+static const struct lttng_kernel_event_field *event_fields[] = {
+       lttng_kernel_static_event_field("ip",
+               lttng_kernel_static_type_integer_from_type(unsigned long, __BYTE_ORDER, 16),
+               false, false, false),
+       lttng_kernel_static_event_field("parent_ip",
+               lttng_kernel_static_type_integer_from_type(unsigned long, __BYTE_ORDER, 16),
+               false, false, false),
+};
+
+static const struct lttng_kernel_tracepoint_class tp_class = {
+       .nr_fields = ARRAY_SIZE(event_fields),
+       .fields = event_fields,
+};
 
 /*
  * Create event description
@@ -122,8 +133,6 @@ static
 int lttng_create_kprobe_event(const char *name, struct lttng_kernel_event_recorder *event_recorder,
                              enum lttng_kretprobe_type type)
 {
-       const struct lttng_kernel_event_field **fieldp_array;
-       struct lttng_kernel_event_field *field;
        struct lttng_kernel_event_desc *desc;
        char *alloc_name;
        size_t name_len;
@@ -151,43 +160,12 @@ int lttng_create_kprobe_event(const char *name, struct lttng_kernel_event_record
        strcpy(alloc_name, name);
        strcat(alloc_name, suffix);
        desc->event_name = alloc_name;
-       desc->nr_fields = 2;
-       fieldp_array = kzalloc(desc->nr_fields * sizeof(struct lttng_kernel_event_field *), GFP_KERNEL);
-       if (!fieldp_array) {
-               ret = -ENOMEM;
-               goto error_fieldp_array;
-       }
-       desc->fields = fieldp_array;
-
-       field = kzalloc(sizeof(struct lttng_kernel_event_field), GFP_KERNEL);
-       if (!field) {
-               ret = -ENOMEM;
-               goto error_field0;
-       }
-       field->name = "ip";
-       field->type = event_type;
-       desc->fields[0] = field;
-
-       field = kzalloc(sizeof(struct lttng_kernel_event_field), GFP_KERNEL);
-       if (!field) {
-               ret = -ENOMEM;
-               goto error_field1;
-       }
-       field->name = "parent_ip";
-       field->type = event_type;
-       desc->fields[1] = field;
-
+       desc->tp_class = &tp_class;
        desc->owner = THIS_MODULE;
        event_recorder->priv->parent.desc = desc;
 
        return 0;
 
-error_field1:
-       kfree(desc->fields[0]);
-error_field0:
-       kfree(fieldp_array);
-error_fieldp_array:
-       kfree(desc->event_name);
 error_str:
        kfree(desc);
        return ret;
@@ -268,15 +246,9 @@ register_error:
 name_error:
        kfree(lttng_krp);
 krp_error:
-       kfree(event_recorder_exit->priv->parent.desc->fields[0]);
-       kfree(event_recorder_exit->priv->parent.desc->fields[1]);
-       kfree(event_recorder_exit->priv->parent.desc->fields);
        kfree(event_recorder_exit->priv->parent.desc->event_name);
        kfree(event_recorder_exit->priv->parent.desc);
 event_exit_error:
-       kfree(event_recorder_entry->priv->parent.desc->fields[0]);
-       kfree(event_recorder_entry->priv->parent.desc->fields[1]);
-       kfree(event_recorder_entry->priv->parent.desc->fields);
        kfree(event_recorder_entry->priv->parent.desc->event_name);
        kfree(event_recorder_entry->priv->parent.desc);
 error:
@@ -309,9 +281,6 @@ void _lttng_kretprobes_release(struct kref *kref)
 
 void lttng_kretprobes_destroy_private(struct lttng_kernel_event_recorder *event_recorder)
 {
-       kfree(event_recorder->priv->parent.desc->fields[0]);
-       kfree(event_recorder->priv->parent.desc->fields[1]);
-       kfree(event_recorder->priv->parent.desc->fields);
        kfree(event_recorder->priv->parent.desc->event_name);
        kfree(event_recorder->priv->parent.desc);
        kref_put(&event_recorder->priv->parent.u.kretprobe.lttng_krp->kref_alloc,