Enforce ABI size checks
authorMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Thu, 23 Nov 2023 19:32:02 +0000 (14:32 -0500)
committerMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Thu, 23 Nov 2023 19:32:02 +0000 (14:32 -0500)
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
include/side/endian.h
include/side/macros-bkp.h [new file with mode: 0644]
include/side/macros.h
include/side/trace.h
src/.tracer.c.swp [new file with mode: 0644]
src/tracer.c

index de8ccd716aaea99d1a84ccefbfe732e146da1e39..c89d486c22decba87bf5204287b99b2ee94ed00b 100644 (file)
 #ifndef _SIDE_ENDIAN_H
 #define _SIDE_ENDIAN_H
 
-#include <side/macros.h>
 #include <math.h>
 
+#if defined(__SIZEOF_LONG__)
+# define SIDE_BITS_PER_LONG    (__SIZEOF_LONG__ * 8)
+#elif defined(_LP64)
+# define SIDE_BITS_PER_LONG    64
+#else
+# define SIDE_BITS_PER_LONG    32
+#endif
+
 #if (defined(__linux__) || defined(__CYGWIN__))
 #include <endian.h>
 #include <byteswap.h>
diff --git a/include/side/macros-bkp.h b/include/side/macros-bkp.h
new file mode 100644 (file)
index 0000000..3c9f6b7
--- /dev/null
@@ -0,0 +1,71 @@
+// SPDX-License-Identifier: MIT
+/*
+ * Copyright 2022 Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
+ */
+
+#ifndef _SIDE_MACROS_H
+#define _SIDE_MACROS_H
+
+#include <stddef.h>
+#include <limits.h>
+
+/* Helper macros */
+
+#define SIDE_ARRAY_SIZE(arr)   (sizeof(arr) / sizeof((arr)[0]))
+
+/*
+ * Compound literals with static storage are needed by SIDE
+ * instrumentation.
+ * Compound literals are part of the C99 and C11 standards, but not
+ * part of the C++ standards. They are supported by most C++ compilers
+ * though.
+ *
+ * Example use:
+ * static struct mystruct *var = LTTNG_UST_COMPOUND_LITERAL(struct mystruct, { 1, 2, 3 });
+ */
+#define SIDE_COMPOUND_LITERAL(type, ...)   (type[]) { __VA_ARGS__ }
+
+#define side_likely(x)         __builtin_expect(!!(x), 1)
+#define side_unlikely(x)       __builtin_expect(!!(x), 0)
+
+#define SIDE_PARAM(...)        __VA_ARGS__
+
+/* Select arg1 in list of arguments. */
+#define SIDE_PARAM_SELECT_ARG1(_arg0, _arg1, ...) _arg1
+
+/*
+ * Use the default parameter if no additional arguments are provided,
+.* else use the following arguments. Use inside macros to implement
+ * optional last macro argument.
+ */
+#define SIDE_PARAM_DEFAULT(_default, ...)      \
+       SIDE_PARAM_SELECT_ARG1(dummy, ##__VA_ARGS__, _default)
+
+/*
+ * side_container_of - Get the address of an object containing a field.
+ *
+ * @ptr: pointer to the field.
+ * @type: type of the object.
+ * @member: name of the field within the object.
+ */
+#define side_container_of(ptr, type, member)                           \
+       __extension__                                                   \
+       ({                                                              \
+               const __typeof__(((type *) NULL)->member) * __ptr = (ptr); \
+               (type *)((char *)__ptr - offsetof(type, member));       \
+       })
+
+#define side_struct_field_sizeof(_struct, _field) \
+       sizeof(((_struct * )NULL)->_field)
+
+#if defined(__SIZEOF_LONG__)
+#define SIDE_BITS_PER_LONG     (__SIZEOF_LONG__ * 8)
+#elif defined(_LP64)
+#define SIDE_BITS_PER_LONG     64
+#else
+#define SIDE_BITS_PER_LONG     32
+#endif
+
+#define SIDE_PACKED    __attribute__((packed))
+
+#endif /* _SIDE_MACROS_H */
index 391537439e84741e0341f6b5d190f3c73a3ba641..26fc0799975fb9ed410b198bcd8002fcf7e48002 100644 (file)
@@ -8,6 +8,8 @@
 
 #include <stddef.h>
 #include <limits.h>
+#include <stdint.h>
+#include <side/endian.h>
 
 /* Helper macros */
 
  */
 #define SIDE_PARAM_SELECT_ARG1(_arg0, _arg1, ...) _arg1
 
+/*
+ * Compile time assertion.
+ * - predicate: boolean expression to evaluate,
+ * - msg: string to print to the user on failure when `static_assert()` is
+ *   supported,
+ * - c_identifier_msg: message to be included in the typedef to emulate a
+ *   static assertion. This parameter must be a valid C identifier as it will
+ *   be used as a typedef name.
+ */
+#ifdef __cplusplus
+#define side_static_assert(predicate, msg, c_identifier_msg)  \
+       static_assert(predicate, msg)
+#elif defined(__STDC_VERSION__) && (__STDC_VERSION__ >= 201112L)
+#define side_static_assert(predicate, msg, c_identifier_msg)  \
+       _Static_assert(predicate, msg)
+#else
+/*
+ * Evaluates the predicate and emit a compilation error on failure.
+ *
+ * If the predicate evaluates to true, this macro emits a function
+ * prototype with an argument type which is an array of size 0.
+ *
+ * If the predicate evaluates to false, this macro emits a function
+ * prototype with an argument type which is an array of negative size
+ * which is invalid in C and forces a compiler error. The
+ * c_identifier_msg parameter is used as the argument identifier so it
+ * is printed to the user when the error is reported.
+ */
+#define side_static_assert(predicate, msg, c_identifier_msg)  \
+       void side_static_assert_proto(char c_identifier_msg[2*!!(predicate)-1])
+#endif
+
 /*
  * side_container_of - Get the address of an object containing a field.
  *
 #define side_struct_field_sizeof(_struct, _field) \
        sizeof(((_struct * )NULL)->_field)
 
-#if defined(__SIZEOF_LONG__)
-# define SIDE_BITS_PER_LONG    (__SIZEOF_LONG__ * 8)
-#elif defined(_LP64)
-# define SIDE_BITS_PER_LONG    64
+#define SIDE_PACKED    __attribute__((packed))
+
+#define side_padding(bytes)    char padding[bytes]
+
+#define side_check_size(_type, _len)                           \
+       side_static_assert(sizeof(_type) == (_len),             \
+               "Unexpected size for type: `" #_type "`",       \
+               unexpected_size_for_type_##_type)
+
+#if (SIDE_BYTE_ORDER == SIDE_LITTLE_ENDIAN)
+#define SIDE_ENDIAN_ORDER(_low, _high)         _low, _high
 #else
-# define SIDE_BITS_PER_LONG    32
+#define SIDE_ENDIAN_ORDER(_low, _high)         _high, _low
 #endif
 
-#define SIDE_PACKED    __attribute__((packed))
-
 /*
  * The side_ptr macros allow defining a pointer type which is suitable
- * for use by 32-bit and 64-bit kernels without compatibility code,
- * while preserving information about the pointer type.
+ * for use by 32-bit, 64-bit and 128-bit kernels without compatibility
+ * code, while preserving information about the pointer type.
  *
- * Those pointers are stored as 64-bit integers, and the type of the
- * actual pointer is kept alongside with the 64-bit pointer value in a
+ * Those pointers are stored as 128-bit integers, and the type of the
+ * actual pointer is kept alongside with the 128-bit pointer value in a
  * 0-len array within a union.
- *
- * uintptr_t will fit within a uint64_t except on architectures with
- * 128-bit pointers. This provides fixed-size pointers on architectures
- * with pointer size of 64-bit or less. Architectures with larger
- * pointer size will have to handle the ABI offset specifics explicitly.
  */
 #if (__SIZEOF_POINTER__ <= 8)
-# define side_ptr_t(_type)                                     \
+# define side_raw_ptr_t(_type)                                 \
        union {                                                 \
-               uint64_t v;                                     \
-               _type *t[0];                                    \
+               struct {                                        \
+                       uint64_t SIDE_ENDIAN_ORDER(low, high);  \
+               } v;                                            \
+               struct {                                        \
+                       _type t[0];                             \
+               } SIDE_PACKED s;                                \
+               side_padding(16);                               \
        }
 # define side_ptr_get(_field)                                  \
-       ((__typeof__((_field).t[0]))(uintptr_t)(_field).v)
+       ((__typeof__((_field).s.t[0]))(uintptr_t)(_field).v.low)
 # define side_ptr_set(_field, _ptr)                            \
        do {                                                    \
-               (_field).v = (uint64_t)(uintptr_t)(_ptr);       \
+               (_field).v.low = (uint64_t)(uintptr_t)(_ptr);   \
+               (_field).v.high = 0;                            \
        } while (0)
-#else
-# define side_ptr_t(_type)                                     \
+
+/* Keep the correct field init order to make old g++ happy. */
+# if (SIDE_BYTE_ORDER == SIDE_LITTLE_ENDIAN)
+#  define SIDE_PTR_INIT(...)                                   \
+       {                                                       \
+               .v = {                                          \
+                       .low = (uintptr_t) (__VA_ARGS__),       \
+                       .high = 0,                              \
+               },                                              \
+       }
+# else
+#  define SIDE_PTR_INIT(...)                                   \
+       {                                                       \
+               .v = {                                          \
+                       .high = 0,                              \
+                       .low = (uintptr_t) (__VA_ARGS__),       \
+               },                                              \
+       }
+# endif
+#elif (__SIZEOF_POINTER__ == 16)
+# define side_raw_ptr_t(_type)                                 \
        union {                                                 \
                uintptr_t v;                                    \
-               _type *t[0];                                    \
+               struct {                                        \
+                       _type t[0];                             \
+               } SIDE_PACKED s;                                \
+               side_padding(16);                               \
        }
 # define side_ptr_get(_field)                                  \
-       ((__typeof__((_field).t[0]))(_field).v)
+       ((__typeof__((_field).s.t[0]))(_field).v)
 # define side_ptr_set(_field, _ptr)                            \
        do {                                                    \
                (_field).v = (uintptr_t)(_ptr);                 \
        } while (0)
+# define SIDE_PTR_INIT(...)    { .v = (uintptr_t) (__VA_ARGS__) }
+#else
+# error "Unsupported pointer size"
 #endif
 
-#define SIDE_PTR_INIT(...)     { .v = (uintptr_t) (__VA_ARGS__) }
+#define side_ptr_t(_type)      side_raw_ptr_t(_type *)
+#define side_func_ptr_t(_type) side_raw_ptr_t(_type)
+
+side_static_assert(sizeof(side_ptr_t(int)) == 16,
+       "Unexpected size for side_ptr_t",
+       unexpected_size_side_ptr_t);
 
 /*
  * side_enum_t allows defining fixed-sized enumerations while preserving
 #define side_enum_t(_enum_type, _size_type)                    \
        union {                                                 \
                _size_type v;                                   \
-               _enum_type t[0];                                \
+               struct {                                        \
+                       _enum_type t[0];                        \
+               } SIDE_PACKED s;                                \
        }
 
 #define side_enum_get(_field)                                  \
-       ((__typeof__((_field).t[0]))(_field).v)
+       ((__typeof__((_field).s.t[0]))(_field).v)
 
 #define side_enum_set(_field, _v)                              \
        do {                                                    \
index fad7c2bf233b1148e91f2663454232a2dd9e44bb..329bec194376e656c4e73b54974015d02c2fa85a 100644 (file)
 /*
  * SIDE stands for "Software Instrumentation Dynamically Enabled"
  *
- * This is an instrumentation API for Linux user-space, which exposes an
+ * This is an instrumentation ABI for Linux user-space, which exposes an
  * instrumentation type system and facilities allowing a kernel or
  * user-space tracer to consume user-space instrumentation.
  *
- * This instrumentation API exposes 3 type systems:
+ * This instrumentation ABI exposes 3 type systems:
  *
  * * Stack-copy type system: This is the core type system which can
  *   represent all supported types and into which all other type systems
  *   Those dynamic types can be either used as arguments to a variadic
  *   field list, or as on-stack instrumentation argument for a static
  *   type SIDE_TYPE_DYNAMIC place holder in the stack-copy type system.
+ *
+ * The extensibility scheme for the SIDE ABI is as follows:
+ *
+ * * Existing field types are never changed nor extended. Field types
+ *   can be added to the ABI by reserving a label within
+ *   enum side_type_label.
+ * * Existing attribute types are never changed nor extended. Attribute
+ *   types can be added to the ABI by reserving a label within
+ *   enum side_attr_type.
+ * * Each union part of the ABI has an explicit side defined by a
+ *   side_padding() member. Each structure and union have a static
+ *   assert validating its size.
+ *
+ * Handling of unknown types by the tracers:
+ *
+ * * A tracer may choose to support only a subset of the types supported
+ *   by libside. When encountering an unknown or unsupported type, the
+ *   tracer has the option to either disallow the entire event or skip
+ *   over the unknown type, both at event registration and when
+ *   receiving the side_call arguments.
+ *
+ * TODO: extend event description with new fields ?
  */
 
 //TODO: as those structures will be ABI, we need to either consider them
@@ -85,6 +107,7 @@ struct side_arg_dynamic_struct;
 struct side_events_register_handle;
 struct side_arg_variant;
 struct side_event_state;
+struct side_callback;
 
 enum side_type_label {
        /* Stack-copy basic types */
@@ -226,6 +249,12 @@ typedef enum side_visitor_status (*side_visitor_func)(
 typedef enum side_visitor_status (*side_dynamic_struct_visitor_func)(
                const struct side_tracer_dynamic_struct_visitor_ctx *tracer_ctx,
                void *app_ctx);
+typedef enum side_visitor_status (*side_write_elem_func)(
+               const struct side_tracer_visitor_ctx *tracer_ctx,
+               const struct side_arg *elem);
+typedef enum side_visitor_status (*side_write_field_func)(
+               const struct side_tracer_dynamic_struct_visitor_ctx *tracer_ctx,
+               const struct side_arg_dynamic_field *dynamic_field);
 
 union side_integer_value {
        uint8_t side_u8;
@@ -237,14 +266,18 @@ union side_integer_value {
        int32_t side_s32;
        int64_t side_s64;
        uintptr_t side_uptr;
+       side_padding(32);
 } SIDE_PACKED;
+side_check_size(union side_integer_value, 32);
 
 union side_bool_value {
        uint8_t side_bool8;
        uint16_t side_bool16;
        uint32_t side_bool32;
        uint64_t side_bool64;
+       side_padding(32);
 } SIDE_PACKED;
+side_check_size(union side_bool_value, 32);
 
 union side_float_value {
 #if __HAVE_FLOAT16
@@ -259,13 +292,16 @@ union side_float_value {
 #if __HAVE_FLOAT128
        _Float128 side_float_binary128;
 #endif
+       side_padding(32);
 } SIDE_PACKED;
+side_check_size(union side_float_value, 32);
 
 struct side_type_raw_string {
        side_ptr_t(const void) p;       /* pointer to string */
        uint8_t unit_size;              /* 1, 2, or 4 bytes */
        side_enum_t(enum side_type_label_byte_order, uint8_t) byte_order;
 } SIDE_PACKED;
+side_check_size(struct side_type_raw_string, 18);
 
 struct side_attr_value {
        side_enum_t(enum side_attr_type, uint32_t) type;
@@ -274,20 +310,24 @@ struct side_attr_value {
                struct side_type_raw_string string_value;
                union side_integer_value integer_value;
                union side_float_value float_value;
+               side_padding(32);
        } SIDE_PACKED u;
 };
+side_check_size(struct side_attr_value, 36);
 
 /* User attributes. */
 struct side_attr {
        const struct side_type_raw_string key;
        const struct side_attr_value value;
 } SIDE_PACKED;
+side_check_size(struct side_attr, 54);
 
 /* Type descriptions */
 struct side_type_null {
        side_ptr_t(const struct side_attr) attr;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_type_null, 20);
 
 struct side_type_bool {
        side_ptr_t(const struct side_attr) attr;
@@ -296,11 +336,13 @@ struct side_type_bool {
        uint16_t len_bits;              /* bits. 0 for (bool_size * CHAR_BITS) */
        side_enum_t(enum side_type_label_byte_order, uint8_t) byte_order;
 } SIDE_PACKED;
+side_check_size(struct side_type_bool, 25);
 
 struct side_type_byte {
        side_ptr_t(const struct side_attr) attr;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_type_byte, 20);
 
 struct side_type_string {
        side_ptr_t(const struct side_attr) attr;
@@ -308,6 +350,7 @@ struct side_type_string {
        uint8_t unit_size;              /* 1, 2, or 4 bytes */
        side_enum_t(enum side_type_label_byte_order, uint8_t) byte_order;
 } SIDE_PACKED;
+side_check_size(struct side_type_string, 22);
 
 struct side_type_integer {
        side_ptr_t(const struct side_attr) attr;
@@ -317,6 +360,7 @@ struct side_type_integer {
        uint8_t signedness;             /* true/false */
        side_enum_t(enum side_type_label_byte_order, uint8_t) byte_order;
 } SIDE_PACKED;
+side_check_size(struct side_type_integer, 26);
 
 struct side_type_float {
        side_ptr_t(const struct side_attr) attr;
@@ -324,12 +368,14 @@ struct side_type_float {
        uint16_t float_size;            /* bytes */
        side_enum_t(enum side_type_label_byte_order, uint8_t) byte_order;
 } SIDE_PACKED;
+side_check_size(struct side_type_float, 23);
 
 struct side_enum_mapping {
        int64_t range_begin;
        int64_t range_end;
        struct side_type_raw_string label;
 } SIDE_PACKED;
+side_check_size(struct side_enum_mapping, 16 + sizeof(struct side_type_raw_string));
 
 struct side_enum_mappings {
        side_ptr_t(const struct side_enum_mapping) mappings;
@@ -337,12 +383,14 @@ struct side_enum_mappings {
        uint32_t nr_mappings;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_enum_mappings, 40);
 
 struct side_enum_bitmap_mapping {
        uint64_t range_begin;
        uint64_t range_end;
        struct side_type_raw_string label;
 } SIDE_PACKED;
+side_check_size(struct side_enum_bitmap_mapping, 16 + sizeof(struct side_type_raw_string));
 
 struct side_enum_bitmap_mappings {
        side_ptr_t(const struct side_enum_bitmap_mapping) mappings;
@@ -350,6 +398,7 @@ struct side_enum_bitmap_mappings {
        uint32_t nr_mappings;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_enum_bitmap_mappings, 40);
 
 struct side_type_struct {
        side_ptr_t(const struct side_event_field) fields;
@@ -357,6 +406,7 @@ struct side_type_struct {
        uint32_t nr_fields;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_type_struct, 40);
 
 struct side_type_array {
        side_ptr_t(const struct side_type) elem_type;
@@ -364,87 +414,100 @@ struct side_type_array {
        uint32_t length;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_type_array, 40);
 
 struct side_type_vla {
        side_ptr_t(const struct side_type) elem_type;
        side_ptr_t(const struct side_attr) attr;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_type_vla, 36);
 
 struct side_type_vla_visitor {
        side_ptr_t(const struct side_type) elem_type;
-       side_visitor_func visitor;
+       side_func_ptr_t(side_visitor_func) visitor;
        side_ptr_t(const struct side_attr) attr;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_type_vla_visitor, 52);
 
 struct side_type_enum {
        side_ptr_t(const struct side_enum_mappings) mappings;
        side_ptr_t(const struct side_type) elem_type;
 } SIDE_PACKED;
+side_check_size(struct side_type_enum, 32);
 
 struct side_type_enum_bitmap {
        side_ptr_t(const struct side_enum_bitmap_mappings) mappings;
        side_ptr_t(const struct side_type) elem_type;
 } SIDE_PACKED;
+side_check_size(struct side_type_enum_bitmap, 32);
 
 struct side_type_gather_bool {
        uint64_t offset;        /* bytes */
+       uint16_t offset_bits;   /* bits */
        uint8_t access_mode;    /* enum side_type_gather_access_mode */
        struct side_type_bool type;
-       uint16_t offset_bits;   /* bits */
 } SIDE_PACKED;
+side_check_size(struct side_type_gather_bool, 11 + sizeof(struct side_type_bool));
 
 struct side_type_gather_byte {
        uint64_t offset;        /* bytes */
        uint8_t access_mode;    /* enum side_type_gather_access_mode */
        struct side_type_byte type;
 } SIDE_PACKED;
+side_check_size(struct side_type_gather_byte, 9 + sizeof(struct side_type_byte));
 
 struct side_type_gather_integer {
        uint64_t offset;        /* bytes */
+       uint16_t offset_bits;   /* bits */
        uint8_t access_mode;    /* enum side_type_gather_access_mode */
        struct side_type_integer type;
-       uint16_t offset_bits;   /* bits */
 } SIDE_PACKED;
+side_check_size(struct side_type_gather_integer, 11 + sizeof(struct side_type_integer));
 
 struct side_type_gather_float {
        uint64_t offset;        /* bytes */
        uint8_t access_mode;    /* enum side_type_gather_access_mode */
        struct side_type_float type;
 } SIDE_PACKED;
+side_check_size(struct side_type_gather_float, 9 + sizeof(struct side_type_float));
 
 struct side_type_gather_string {
        uint64_t offset;        /* bytes */
        uint8_t access_mode;    /* enum side_type_gather_access_mode */
        struct side_type_string type;
 } SIDE_PACKED;
+side_check_size(struct side_type_gather_string, 9 + sizeof(struct side_type_string));
 
 struct side_type_gather_enum {
        side_ptr_t(const struct side_enum_mappings) mappings;
        side_ptr_t(const struct side_type) elem_type;
 } SIDE_PACKED;
+side_check_size(struct side_type_gather_enum, 32);
 
 struct side_type_gather_struct {
+       side_ptr_t(const struct side_type_struct) type;
        uint64_t offset;        /* bytes */
        uint8_t access_mode;    /* enum side_type_gather_access_mode */
-       side_ptr_t(const struct side_type_struct) type;
        uint32_t size;          /* bytes */
 } SIDE_PACKED;
+side_check_size(struct side_type_gather_struct, 29);
 
 struct side_type_gather_array {
        uint64_t offset;        /* bytes */
        uint8_t access_mode;    /* enum side_type_gather_access_mode */
        struct side_type_array type;
 } SIDE_PACKED;
+side_check_size(struct side_type_gather_array, 9 + sizeof(struct side_type_array));
 
 struct side_type_gather_vla {
        side_ptr_t(const struct side_type) length_type; /* side_length() */
-
        uint64_t offset;        /* bytes */
        uint8_t access_mode;    /* enum side_type_gather_access_mode */
        struct side_type_vla type;
 } SIDE_PACKED;
+side_check_size(struct side_type_gather_vla, 25 + sizeof(struct side_type_vla));
 
 struct side_type_gather {
        union {
@@ -457,11 +520,13 @@ struct side_type_gather {
                struct side_type_gather_array side_array;
                struct side_type_gather_vla side_vla;
                struct side_type_gather_struct side_struct;
+               side_padding(61);
        } SIDE_PACKED u;
 } SIDE_PACKED;
+side_check_size(struct side_type_gather, 61);
 
 struct side_type {
-       side_enum_t(enum side_type_label, uint32_t) type;
+       side_enum_t(enum side_type_label, uint16_t) type;
        union {
                /* Stack-copy basic types */
                struct side_type_null side_null;
@@ -484,45 +549,37 @@ struct side_type {
 
                /* Gather types */
                struct side_type_gather side_gather;
+               side_padding(62);
        } SIDE_PACKED u;
 } SIDE_PACKED;
+side_check_size(struct side_type, 64);
 
 struct side_variant_option {
        int64_t range_begin;
        int64_t range_end;
        const struct side_type side_type;
 } SIDE_PACKED;
+side_check_size(struct side_variant_option, 16 + sizeof(const struct side_type));
 
 struct side_type_variant {
-       const struct side_type selector;
        side_ptr_t(const struct side_variant_option) options;
        side_ptr_t(const struct side_attr) attr;
        uint32_t nr_options;
        uint32_t nr_attr;
+       const struct side_type selector;
 } SIDE_PACKED;
+side_check_size(struct side_type_variant, 40 + sizeof(const struct side_type));
 
 struct side_event_field {
        side_ptr_t(const char) field_name;
        struct side_type side_type;
 } SIDE_PACKED;
+side_check_size(struct side_event_field, 16 + sizeof(struct side_type));
 
 enum side_event_flags {
        SIDE_EVENT_FLAG_VARIADIC = (1 << 0),
 };
 
-struct side_callback {
-       union {
-               void (*call)(const struct side_event_description *desc,
-                       const struct side_arg_vec *side_arg_vec,
-                       void *priv);
-               void (*call_variadic)(const struct side_event_description *desc,
-                       const struct side_arg_vec *side_arg_vec,
-                       const struct side_arg_dynamic_struct *var_struct,
-                       void *priv);
-       } SIDE_PACKED u;
-       void *priv;
-} SIDE_PACKED;
-
 union side_arg_static {
        /* Stack-copy basic types */
        union side_bool_value bool_value;
@@ -552,7 +609,9 @@ union side_arg_static {
                side_ptr_t(const void) ptr;
                side_ptr_t(const void) length_ptr;
        } SIDE_PACKED side_vla_gather;
+       side_padding(32);
 } SIDE_PACKED;
+side_check_size(union side_arg_static, 32);
 
 struct side_arg_dynamic_vla {
        side_ptr_t(const struct side_arg) sav;
@@ -560,6 +619,7 @@ struct side_arg_dynamic_vla {
        uint32_t len;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_arg_dynamic_vla, 40);
 
 struct side_arg_dynamic_struct {
        side_ptr_t(const struct side_arg_dynamic_field) fields;
@@ -567,20 +627,23 @@ struct side_arg_dynamic_struct {
        uint32_t len;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_arg_dynamic_struct, 40);
 
 struct side_dynamic_struct_visitor {
-       void *app_ctx;
-       side_dynamic_struct_visitor_func visitor;
+       side_func_ptr_t(side_dynamic_struct_visitor_func) visitor;
+       side_ptr_t(void) app_ctx;
        side_ptr_t(const struct side_attr) attr;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_dynamic_struct_visitor, 52);
 
 struct side_dynamic_vla_visitor {
-       void *app_ctx;
-       side_visitor_func visitor;
+       side_func_ptr_t(side_visitor_func) visitor;
+       side_ptr_t(void) app_ctx;
        side_ptr_t(const struct side_attr) attr;
        uint32_t nr_attr;
 } SIDE_PACKED;
+side_check_size(struct side_dynamic_vla_visitor, 52);
 
 union side_arg_dynamic {
        /* Dynamic basic types */
@@ -612,45 +675,38 @@ union side_arg_dynamic {
 
        struct side_dynamic_struct_visitor side_dynamic_struct_visitor;
        struct side_dynamic_vla_visitor side_dynamic_vla_visitor;
+
+       side_padding(58);
 } SIDE_PACKED;
+side_check_size(union side_arg_dynamic, 58);
 
 struct side_arg {
-       side_enum_t(enum side_type_label, uint32_t) type;
+       side_enum_t(enum side_type_label, uint16_t) type;
        union {
                union side_arg_static side_static;
                union side_arg_dynamic side_dynamic;
+               side_padding(62);
        } SIDE_PACKED u;
 } SIDE_PACKED;
+side_check_size(struct side_arg, 64);
 
 struct side_arg_variant {
        struct side_arg selector;
        struct side_arg option;
 } SIDE_PACKED;
+side_check_size(struct side_arg_variant, 128);
 
 struct side_arg_vec {
        side_ptr_t(const struct side_arg) sav;
        uint32_t len;
 } SIDE_PACKED;
+side_check_size(struct side_arg_vec, 20);
 
 struct side_arg_dynamic_field {
        side_ptr_t(const char) field_name;
        const struct side_arg elem;
 } SIDE_PACKED;
-
-/* The visitor pattern is a double-dispatch visitor. */
-struct side_tracer_visitor_ctx {
-       enum side_visitor_status (*write_elem)(
-                       const struct side_tracer_visitor_ctx *tracer_ctx,
-                       const struct side_arg *elem);
-       void *priv;             /* Private tracer context. */
-} SIDE_PACKED;
-
-struct side_tracer_dynamic_struct_visitor_ctx {
-       enum side_visitor_status (*write_field)(
-                       const struct side_tracer_dynamic_struct_visitor_ctx *tracer_ctx,
-                       const struct side_arg_dynamic_field *dynamic_field);
-       void *priv;             /* Private tracer context. */
-} SIDE_PACKED;
+side_check_size(struct side_arg_dynamic_field, 16 + sizeof(const struct side_arg));
 
 struct side_event_description {
        side_ptr_t(struct side_event_state) state;
@@ -1100,7 +1156,7 @@ struct side_event_state {
                .u = { \
                        .side_vla_visitor = { \
                                .elem_type = SIDE_PTR_INIT(_elem_type), \
-                               .visitor = _visitor, \
+                               .visitor = SIDE_PTR_INIT(_visitor), \
                                .attr = SIDE_PTR_INIT(SIDE_PARAM_SELECT_ARG1(_, ##_attr, side_attr_list())), \
                                .nr_attr = SIDE_ARRAY_SIZE(SIDE_PARAM_SELECT_ARG1(_, ##_attr, side_attr_list())), \
                        }, \
@@ -1694,8 +1750,8 @@ struct side_event_state {
                .u = { \
                        .side_dynamic = { \
                                .side_dynamic_vla_visitor = { \
-                                       .app_ctx = _ctx, \
-                                       .visitor = _dynamic_vla_visitor, \
+                                       .app_ctx = SIDE_PTR_INIT(_ctx), \
+                                       .visitor = SIDE_PTR_INIT(_dynamic_vla_visitor), \
                                        .attr = SIDE_PTR_INIT(SIDE_PARAM_SELECT_ARG1(_, ##_attr, side_attr_list())), \
                                        .nr_attr = SIDE_ARRAY_SIZE(SIDE_PARAM_SELECT_ARG1(_, ##_attr, side_attr_list())), \
                                }, \
@@ -1719,8 +1775,8 @@ struct side_event_state {
                .u = { \
                        .side_dynamic = { \
                                .side_dynamic_struct_visitor = { \
-                                       .app_ctx = _ctx, \
-                                       .visitor = _dynamic_struct_visitor, \
+                                       .app_ctx = SIDE_PTR_INIT(_ctx), \
+                                       .visitor = SIDE_PTR_INIT(_dynamic_struct_visitor), \
                                        .attr = SIDE_PTR_INIT(SIDE_PARAM_SELECT_ARG1(_, ##_attr, side_attr_list())), \
                                        .nr_attr = SIDE_ARRAY_SIZE(SIDE_PARAM_SELECT_ARG1(_, ##_attr, side_attr_list())), \
                                }, \
@@ -1863,6 +1919,30 @@ struct side_event_state {
 extern "C" {
 #endif
 
+struct side_callback {
+       union {
+               void (*call)(const struct side_event_description *desc,
+                       const struct side_arg_vec *side_arg_vec,
+                       void *priv);
+               void (*call_variadic)(const struct side_event_description *desc,
+                       const struct side_arg_vec *side_arg_vec,
+                       const struct side_arg_dynamic_struct *var_struct,
+                       void *priv);
+       } SIDE_PACKED u;
+       void *priv;
+} SIDE_PACKED;
+
+/* The visitor pattern is a double-dispatch visitor. */
+struct side_tracer_visitor_ctx {
+       side_write_elem_func write_elem;
+       void *priv;             /* Private tracer context. */
+} SIDE_PACKED;
+
+struct side_tracer_dynamic_struct_visitor_ctx {
+       side_write_field_func write_field;
+       void *priv;             /* Private tracer context. */
+} SIDE_PACKED;
+
 extern const struct side_callback side_empty_callback;
 
 void side_call(const struct side_event_state *state,
diff --git a/src/.tracer.c.swp b/src/.tracer.c.swp
new file mode 100644 (file)
index 0000000..362d5e1
Binary files /dev/null and b/src/.tracer.c.swp differ
index 486dc1968a6a5b39c64c0a27255e6ffb8b73d860..33c699093b83858617d6bb5cea80ad352dd0f194 100644 (file)
@@ -1517,12 +1517,14 @@ void tracer_print_vla_visitor(const struct side_type *type_desc, void *app_ctx)
                .write_elem = tracer_write_elem_cb,
                .priv = &tracer_priv,
        };
+       side_visitor_func func;
 
        print_attributes("attr", ":", side_ptr_get(type_desc->u.side_vla_visitor.attr), type_desc->u.side_vla_visitor.nr_attr);
        printf("%s", type_desc->u.side_vla_visitor.nr_attr ? ", " : "");
        printf("elements: ");
        printf("[ ");
-       status = type_desc->u.side_vla_visitor.visitor(&tracer_ctx, app_ctx);
+       func = side_ptr_get(type_desc->u.side_vla_visitor.visitor);
+       status = func(&tracer_ctx, app_ctx);
        switch (status) {
        case SIDE_VISITOR_STATUS_OK:
                break;
@@ -1580,13 +1582,13 @@ void tracer_print_dynamic_struct_visitor(const struct side_arg *item)
                .write_field = tracer_dynamic_struct_write_elem_cb,
                .priv = &tracer_priv,
        };
-       void *app_ctx = item->u.side_dynamic.side_dynamic_struct_visitor.app_ctx;
+       void *app_ctx = side_ptr_get(item->u.side_dynamic.side_dynamic_struct_visitor.app_ctx);
 
        print_attributes("attr", "::", side_ptr_get(item->u.side_dynamic.side_dynamic_struct_visitor.attr), item->u.side_dynamic.side_dynamic_struct_visitor.nr_attr);
        printf("%s", item->u.side_dynamic.side_dynamic_struct_visitor.nr_attr ? ", " : "");
        printf("fields:: ");
        printf("[ ");
-       status = item->u.side_dynamic.side_dynamic_struct_visitor.visitor(&tracer_ctx, app_ctx);
+       status = side_ptr_get(item->u.side_dynamic.side_dynamic_struct_visitor.visitor)(&tracer_ctx, app_ctx);
        switch (status) {
        case SIDE_VISITOR_STATUS_OK:
                break;
@@ -1642,13 +1644,13 @@ void tracer_print_dynamic_vla_visitor(const struct side_arg *item)
                .write_elem = tracer_dynamic_vla_write_elem_cb,
                .priv = &tracer_priv,
        };
-       void *app_ctx = item->u.side_dynamic.side_dynamic_vla_visitor.app_ctx;
+       void *app_ctx = side_ptr_get(item->u.side_dynamic.side_dynamic_vla_visitor.app_ctx);
 
        print_attributes("attr", "::", side_ptr_get(item->u.side_dynamic.side_dynamic_vla_visitor.attr), item->u.side_dynamic.side_dynamic_vla_visitor.nr_attr);
        printf("%s", item->u.side_dynamic.side_dynamic_vla_visitor.nr_attr ? ", " : "");
        printf("elements:: ");
        printf("[ ");
-       status = item->u.side_dynamic.side_dynamic_vla_visitor.visitor(&tracer_ctx, app_ctx);
+       status = side_ptr_get(item->u.side_dynamic.side_dynamic_vla_visitor.visitor)(&tracer_ctx, app_ctx);
        switch (status) {
        case SIDE_VISITOR_STATUS_OK:
                break;
This page took 0.034958 seconds and 4 git commands to generate.