trace: add size_t field emitter
authorDmitry Kozlyuk <dmitry.kozliuk@gmail.com>
Mon, 15 Jun 2020 00:43:48 +0000 (03:43 +0300)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 15 Jun 2020 17:27:00 +0000 (19:27 +0200)
It is not guaranteed that sizeof(long) == sizeof(size_t). On Windows,
sizeof(long) == 4 and sizeof(size_t) == 8 for 64-bit programs.
Tracepoints using "long" field emitter are therefore invalid there.
Add dedicated field emitter for size_t and use it to store size_t values
in all existing tracepoints.

Signed-off-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
lib/librte_eal/include/rte_eal_trace.h
lib/librte_eal/include/rte_trace_point.h
lib/librte_mempool/rte_mempool_trace.h

index 1ebb290..bcfef0c 100644 (file)
@@ -143,7 +143,7 @@ RTE_TRACE_POINT(
        RTE_TRACE_POINT_ARGS(const char *type, size_t size, unsigned int align,
                int socket, void *ptr),
        rte_trace_point_emit_string(type);
-       rte_trace_point_emit_long(size);
+       rte_trace_point_emit_size_t(size);
        rte_trace_point_emit_u32(align);
        rte_trace_point_emit_int(socket);
        rte_trace_point_emit_ptr(ptr);
@@ -154,7 +154,7 @@ RTE_TRACE_POINT(
        RTE_TRACE_POINT_ARGS(const char *type, size_t size, unsigned int align,
                int socket, void *ptr),
        rte_trace_point_emit_string(type);
-       rte_trace_point_emit_long(size);
+       rte_trace_point_emit_size_t(size);
        rte_trace_point_emit_u32(align);
        rte_trace_point_emit_int(socket);
        rte_trace_point_emit_ptr(ptr);
@@ -164,7 +164,7 @@ RTE_TRACE_POINT(
        rte_eal_trace_mem_realloc,
        RTE_TRACE_POINT_ARGS(size_t size, unsigned int align, int socket,
                void *ptr),
-       rte_trace_point_emit_long(size);
+       rte_trace_point_emit_size_t(size);
        rte_trace_point_emit_u32(align);
        rte_trace_point_emit_int(socket);
        rte_trace_point_emit_ptr(ptr);
@@ -183,7 +183,7 @@ RTE_TRACE_POINT(
                unsigned int flags, unsigned int align, unsigned int bound,
                const void *mz),
        rte_trace_point_emit_string(name);
-       rte_trace_point_emit_long(len);
+       rte_trace_point_emit_size_t(len);
        rte_trace_point_emit_int(socket_id);
        rte_trace_point_emit_u32(flags);
        rte_trace_point_emit_u32(align);
index b451712..377c241 100644 (file)
@@ -138,6 +138,8 @@ _tp _args \
 #define rte_trace_point_emit_int(val)
 /** Tracepoint function payload for long datatype */
 #define rte_trace_point_emit_long(val)
+/** Tracepoint function payload for size_t datatype */
+#define rte_trace_point_emit_size_t(val)
 /** Tracepoint function payload for float datatype */
 #define rte_trace_point_emit_float(val)
 /** Tracepoint function payload for double datatype */
@@ -395,6 +397,7 @@ do { \
 #define rte_trace_point_emit_i8(in) __rte_trace_point_emit(in, int8_t)
 #define rte_trace_point_emit_int(in) __rte_trace_point_emit(in, int32_t)
 #define rte_trace_point_emit_long(in) __rte_trace_point_emit(in, long)
+#define rte_trace_point_emit_size_t(in) __rte_trace_point_emit(in, size_t)
 #define rte_trace_point_emit_float(in) __rte_trace_point_emit(in, float)
 #define rte_trace_point_emit_double(in) __rte_trace_point_emit(in, double)
 #define rte_trace_point_emit_ptr(in) __rte_trace_point_emit(in, uintptr_t)
index e776df0..087c913 100644 (file)
@@ -72,7 +72,7 @@ RTE_TRACE_POINT(
        rte_trace_point_emit_string(mempool->name);
        rte_trace_point_emit_ptr(vaddr);
        rte_trace_point_emit_u64(iova);
-       rte_trace_point_emit_long(len);
+       rte_trace_point_emit_size_t(len);
        rte_trace_point_emit_ptr(free_cb);
        rte_trace_point_emit_ptr(opaque);
 )
@@ -84,8 +84,8 @@ RTE_TRACE_POINT(
        rte_trace_point_emit_ptr(mempool);
        rte_trace_point_emit_string(mempool->name);
        rte_trace_point_emit_ptr(addr);
-       rte_trace_point_emit_long(len);
-       rte_trace_point_emit_long(pg_sz);
+       rte_trace_point_emit_size_t(len);
+       rte_trace_point_emit_size_t(pg_sz);
        rte_trace_point_emit_ptr(free_cb);
        rte_trace_point_emit_ptr(opaque);
 )
@@ -126,7 +126,7 @@ RTE_TRACE_POINT(
        RTE_TRACE_POINT_ARGS(struct rte_mempool *mempool, size_t pg_sz),
        rte_trace_point_emit_ptr(mempool);
        rte_trace_point_emit_string(mempool->name);
-       rte_trace_point_emit_long(pg_sz);
+       rte_trace_point_emit_size_t(pg_sz);
 )
 
 RTE_TRACE_POINT(
@@ -139,7 +139,7 @@ RTE_TRACE_POINT(
        rte_trace_point_emit_u32(max_objs);
        rte_trace_point_emit_ptr(vaddr);
        rte_trace_point_emit_u64(iova);
-       rte_trace_point_emit_long(len);
+       rte_trace_point_emit_size_t(len);
        rte_trace_point_emit_ptr(obj_cb);
        rte_trace_point_emit_ptr(obj_cb_arg);
 )