RE: [EXTERNAL] [PATCH v4 4/5] trace: support expression for blob length

[Sunil Kumar Kori <skori@marvell.com>]

> Support any expression as a blob length by using an intermediate variable in
> the trace point emitter itself.
> This also avoids any side effect on the passed variable.
> 
> On the "register" side, prefix the length variable in the trace metadata with
> the name of the emitted argument.
> 
> With this, we can update ethdev traces and avoid intermediate variables.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
> Changes since v2:
> - fixed length field in CTF metadata,
> 
> Changes since v1:
> - removed code relying on arguments in "registering"
>   rte_trace_point_emit_blob implementation (see patch 3),
> - moved build check in non registering __rte_trace_point_emit,
> 
> ---
>  lib/eal/include/rte_trace_point.h          | 12 ++++++----
>  lib/eal/include/rte_trace_point_register.h |  6 ++++-
>  lib/ethdev/ethdev_trace.h                  | 27 ++++++----------------
>  3 files changed, 19 insertions(+), 26 deletions(-)
>
> 2.48.1
Acked-by: Sunil Kumar Kori <skori@marvell.com>