Refactoring: struct lttng_event_field
[lttng-ust.git] / include / lttng / ust-events.h
index 37e5575eb06f4599e005d772094d618713e5481c..acea99105a86492646723732f930cc8fe250ab85 100644 (file)
@@ -43,8 +43,7 @@ extern "C" {
 struct lttng_channel;
 struct lttng_session;
 struct lttng_ust_lib_ring_buffer_ctx;
-struct lttng_event_field;
-struct lttng_event_notifier;
+struct lttng_ust_event_field;
 struct lttng_event_notifier_group;
 
 /*
@@ -57,13 +56,9 @@ struct lttng_event_notifier_group;
 /* Update the astract_types name table in lttng-types.c along with this enum */
 enum lttng_abstract_types {
        atype_integer,
-       atype_enum,     /* legacy */
-       atype_array,    /* legacy */
-       atype_sequence, /* legacy */
        atype_string,
        atype_float,
        atype_dynamic,
-       atype_struct,   /* legacy */
        atype_enum_nestable,
        atype_array_nestable,
        atype_sequence_nestable,
@@ -162,29 +157,6 @@ struct lttng_float_type {
        char padding[LTTNG_UST_FLOAT_TYPE_PADDING];
 };
 
-/* legacy */
-#define LTTNG_UST_BASIC_TYPE_PADDING   128
-union _lttng_basic_type {
-       struct lttng_integer_type integer;      /* legacy */
-       struct {
-               const struct lttng_enum_desc *desc;     /* Enumeration mapping */
-               struct lttng_integer_type container_type;
-       } enumeration;                          /* legacy */
-       struct {
-               enum lttng_string_encodings encoding;
-       } string;                               /* legacy */
-       struct lttng_float_type _float;         /* legacy */
-       char padding[LTTNG_UST_BASIC_TYPE_PADDING];
-};
-
-/* legacy */
-struct lttng_basic_type {
-       enum lttng_abstract_types atype;
-       union {
-               union _lttng_basic_type basic;
-       } u;
-};
-
 #define LTTNG_UST_TYPE_PADDING 128
 struct lttng_type {
        enum lttng_abstract_types atype;
@@ -211,26 +183,10 @@ struct lttng_type {
                } sequence_nestable;
                struct {
                        unsigned int nr_fields;
-                       const struct lttng_event_field *fields; /* Array of fields. */
+                       const struct lttng_ust_event_field **fields; /* Array of pointers to fields. */
                        unsigned int alignment;
                } struct_nestable;
 
-               union {
-                       /* legacy provider ABI 1.0 */
-                       union _lttng_basic_type basic;  /* legacy */
-                       struct {
-                               struct lttng_basic_type elem_type;
-                               unsigned int length;            /* Num. elems. */
-                       } array;                        /* legacy */
-                       struct {
-                               struct lttng_basic_type length_type;
-                               struct lttng_basic_type elem_type;
-                       } sequence;                     /* legacy */
-                       struct {
-                               unsigned int nr_fields;
-                               struct lttng_event_field *fields;       /* Array of fields. */
-                       } _struct;                      /* legacy */
-               } legacy;
                char padding[LTTNG_UST_TYPE_PADDING];
        } u;
 };
@@ -249,19 +205,21 @@ struct lttng_enum_desc {
  * IMPORTANT: this structure is part of the ABI between the probe and
  * UST. Fields need to be only added at the end, never reordered, never
  * removed.
+ *
+ * The field @struct_size should be used to determine the size of the
+ * structure. It should be queried before using additional fields added
+ * at the end of the structure.
  */
 
-#define LTTNG_UST_EVENT_FIELD_PADDING  28
-struct lttng_event_field {
+struct lttng_ust_event_field {
+       uint32_t struct_size;
+
        const char *name;
        struct lttng_type type;
-       unsigned int nowrite;   /* do not write into trace */
-       union {
-               struct {
-                       unsigned int nofilter:1;        /* do not consider for filter */
-               } ext;
-               char padding[LTTNG_UST_EVENT_FIELD_PADDING];
-       } u;
+       unsigned int nowrite:1,         /* do not write into trace */
+               nofilter:1;             /* do not consider for filter */
+
+       /* End of base ABI. Fields below should be used after checking struct_size. */
 };
 
 enum lttng_ust_dynamic_type {
@@ -294,7 +252,7 @@ struct lttng_perf_counter_field;
 
 #define LTTNG_UST_CTX_FIELD_PADDING    40
 struct lttng_ctx_field {
-       struct lttng_event_field event_field;
+       struct lttng_ust_event_field event_field;
        size_t (*get_size)(struct lttng_ctx_field *field, size_t offset);
        void (*record)(struct lttng_ctx_field *field,
                       struct lttng_ust_lib_ring_buffer_ctx *ctx,
@@ -318,35 +276,51 @@ struct lttng_ctx {
        char padding[LTTNG_UST_CTX_PADDING];
 };
 
-#define LTTNG_UST_EVENT_DESC_PADDING   40
-struct lttng_event_desc {
+/*
+ * IMPORTANT: this structure is part of the ABI between the probe and
+ * UST. Fields need to be only added at the end, never reordered, never
+ * removed.
+ *
+ * The field @struct_size should be used to determine the size of the
+ * structure. It should be queried before using additional fields added
+ * at the end of the structure.
+ */
+struct lttng_ust_event_desc {
+       uint32_t struct_size;                   /* Size of this structure. */
        const char *name;
        void (*probe_callback)(void);
        const struct lttng_event_ctx *ctx;      /* context */
-       const struct lttng_event_field *fields; /* event payload */
+       const struct lttng_ust_event_field **fields;    /* event payload */
        unsigned int nr_fields;
        const int **loglevel;
-       const char *signature;  /* Argument types/names received */
-       union {
-               struct {
-                       const char **model_emf_uri;
-                       void (*event_notifier_callback)(void);
-               } ext;
-               char padding[LTTNG_UST_EVENT_DESC_PADDING];
-       } u;
+       const char *signature;                  /* Argument types/names received */
+       const char **model_emf_uri;
+
+       /* End of base ABI. Fields below should be used after checking struct_size. */
 };
 
-#define LTTNG_UST_PROBE_DESC_PADDING   12
-struct lttng_probe_desc {
+/*
+ * IMPORTANT: this structure is part of the ABI between the probe and
+ * UST. Fields need to be only added at the end, never reordered, never
+ * removed.
+ *
+ * The field @struct_size should be used to determine the size of the
+ * structure. It should be queried before using additional fields added
+ * at the end of the structure.
+ */
+struct lttng_ust_probe_desc {
+       uint32_t struct_size;                   /* Size of this structure. */
+
        const char *provider;
-       const struct lttng_event_desc **event_desc;
+       const struct lttng_ust_event_desc **event_desc;
        unsigned int nr_events;
        struct cds_list_head head;              /* chain registered probes */
        struct cds_list_head lazy_init_head;
        int lazy;                               /* lazy registration */
        uint32_t major;
        uint32_t minor;
-       char padding[LTTNG_UST_PROBE_DESC_PADDING];
+
+       /* End of base ABI. Fields below should be used after checking struct_size. */
 };
 
 /* Data structures used by the tracer. */
@@ -392,37 +366,97 @@ struct lttng_bytecode_runtime {
  * removed.
  */
 
-struct lttng_ust_event_private;
+struct lttng_ust_event_common_private;
 
-struct lttng_event {
-       uint32_t struct_size;                   /* Size of this structure. */
-       struct lttng_ust_event_private *priv;   /* Private event interface */
+enum lttng_ust_event_type {
+       LTTNG_UST_EVENT_TYPE_RECORDER = 0,
+       LTTNG_UST_EVENT_TYPE_NOTIFIER = 1,
+};
+
+/*
+ * IMPORTANT: this structure is part of the ABI between the probe and
+ * UST. Fields need to be only added at the end, never reordered, never
+ * removed.
+ *
+ * struct lttng_ust_event_common is the common ancestor of the various
+ * public event actions. Inheritance is done by composition: The parent
+ * has a pointer to its child, and the child has a pointer to its
+ * parent. Inheritance of those public structures is done by composition
+ * to ensure both parent and child structures can be extended.
+ *
+ * The field @struct_size should be used to determine the size of the
+ * structure. It should be queried before using additional fields added
+ * at the end of the structure.
+ */
+struct lttng_ust_event_common {
+       uint32_t struct_size;                           /* Size of this structure. */
+       struct lttng_ust_event_common_private *priv;    /* Private event interface */
+
+       enum lttng_ust_event_type type;
+       void *child;                                    /* Pointer to child, for inheritance by aggregation. */
 
-       unsigned int id;
        int enabled;
        int has_enablers_without_bytecode;
        /* list of struct lttng_bytecode_runtime, sorted by seqnum */
        struct cds_list_head filter_bytecode_runtime_head;
+
+       /* End of base ABI. Fields below should be used after checking struct_size. */
+};
+
+struct lttng_ust_event_recorder_private;
+
+/*
+ * IMPORTANT: this structure is part of the ABI between the probe and
+ * UST. Fields need to be only added at the end, never reordered, never
+ * removed.
+ *
+ * struct lttng_ust_event_recorder is the action for recording events
+ * into a ring buffer. It inherits from struct lttng_ust_event_common
+ * by composition to ensure both parent and child structure are
+ * extensible.
+ *
+ * The field @struct_size should be used to determine the size of the
+ * structure. It should be queried before using additional fields added
+ * at the end of the structure.
+ */
+struct lttng_ust_event_recorder {
+       uint32_t struct_size;                           /* Size of this structure. */
+       struct lttng_ust_event_common *parent;          /* Inheritance by aggregation. */
+       struct lttng_ust_event_recorder_private *priv;  /* Private event record interface */
+
+       unsigned int id;
        struct lttng_channel *chan;
        struct lttng_ctx *ctx;
+
+       /* End of base ABI. Fields below should be used after checking struct_size. */
 };
 
-struct lttng_event_notifier {
-       uint64_t user_token;
-       uint64_t error_counter_index;
-       int enabled;
-       int registered;                 /* has reg'd tracepoint probe */
-       size_t num_captures;            /* Needed to allocate the msgpack array. */
-       void (*notification_send)(struct lttng_event_notifier *event_notifier,
+struct lttng_ust_event_notifier_private;
+
+/*
+ * IMPORTANT: this structure is part of the ABI between the probe and
+ * UST. Fields need to be only added at the end, never reordered, never
+ * removed.
+ *
+ * struct lttng_ust_event_notifier is the action for sending
+ * notifications. It inherits from struct lttng_ust_event_common
+ * by composition to ensure both parent and child structure are
+ * extensible.
+ *
+ * The field @struct_size should be used to determine the size of the
+ * structure. It should be queried before using additional fields added
+ * at the end of the structure.
+ */
+struct lttng_ust_event_notifier {
+       uint32_t struct_size;                           /* Size of this structure. */
+       struct lttng_ust_event_common *parent;          /* Inheritance by aggregation. */
+       struct lttng_ust_event_notifier_private *priv;  /* Private event notifier interface */
+
+       void (*notification_send)(struct lttng_ust_event_notifier *event_notifier,
                const char *stack_data);
-       struct cds_list_head filter_bytecode_runtime_head;
        struct cds_list_head capture_bytecode_runtime_head;
-       int has_enablers_without_bytecode;
-       struct cds_list_head enablers_ref_head;
-       const struct lttng_event_desc *desc;
-       struct cds_hlist_node hlist;    /* hashtable of event_notifiers */
-       struct cds_list_head node;      /* event_notifier list in session */
-       struct lttng_event_notifier_group *group; /* weak ref */
+
+       /* End of base ABI. Fields below should be used after checking struct_size. */
 };
 
 struct lttng_enum {
@@ -496,7 +530,7 @@ struct lttng_channel {
 
        /* Channel ID */
        unsigned int id;
-       enum lttng_ust_chan_type type;
+       enum lttng_ust_abi_chan_type type;
        unsigned char uuid[LTTNG_UST_UUID_LEN]; /* Trace session unique ID */
        int tstate:1;                   /* Transient enable state */
 };
@@ -525,7 +559,7 @@ struct lttng_counter_ops {
 
 #define LTTNG_UST_STACK_CTX_PADDING    32
 struct lttng_stack_ctx {
-       struct lttng_event *event;
+       struct lttng_ust_event_recorder *event_recorder;
        struct lttng_ctx *chan_ctx;     /* RCU dereferenced. */
        struct lttng_ctx *event_ctx;    /* RCU dereferenced. */
        char padding[LTTNG_UST_STACK_CTX_PADDING];
@@ -557,15 +591,22 @@ struct lttng_ust_session_private;
  * IMPORTANT: this structure is part of the ABI between the probe and
  * UST. Fields need to be only added at the end, never reordered, never
  * removed.
+ *
+ * The field @struct_size should be used to determine the size of the
+ * structure. It should be queried before using additional fields added
+ * at the end of the structure.
  */
 struct lttng_session {
+       uint32_t struct_size;                   /* Size of this structure */
        struct lttng_ust_session_private *priv; /* Private session interface */
 
        int active;                             /* Is trace session active ? */
+
+       /* End of base ABI. Fields below should be used after checking struct_size. */
 };
 
-int lttng_probe_register(struct lttng_probe_desc *desc);
-void lttng_probe_unregister(struct lttng_probe_desc *desc);
+int lttng_ust_probe_register(struct lttng_ust_probe_desc *desc);
+void lttng_ust_probe_unregister(struct lttng_ust_probe_desc *desc);
 
 /*
  * Can be used by applications that change their procname to clear the ust cached value.
This page took 0.026727 seconds and 4 git commands to generate.