No need to copy values in intermediate variables.
Just use the right trace point emitters.

Signed-off-by: David Marchand <david.march...@redhat.com>
---
Changes since v2:
- split this change into multiple changes,
  only kept trivial parts in this patch,

---
 lib/dmadev/rte_dmadev_trace.h | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/lib/dmadev/rte_dmadev_trace.h b/lib/dmadev/rte_dmadev_trace.h
index c5e4babe15..ddf60b9649 100644
--- a/lib/dmadev/rte_dmadev_trace.h
+++ b/lib/dmadev/rte_dmadev_trace.h
@@ -46,11 +46,10 @@ RTE_TRACE_POINT(
        const struct rte_dma_conf __dev_conf = {0};
        dev_conf = &__dev_conf;
 #endif /* _RTE_TRACE_POINT_REGISTER_H_ */
-       int enable_silent = (int)dev_conf->enable_silent;
        rte_trace_point_emit_i16(dev_id);
        rte_trace_point_emit_u16(dev_conf->nb_vchans);
        rte_trace_point_emit_u16(dev_conf->priority);
-       rte_trace_point_emit_int(enable_silent);
+       rte_trace_point_emit_u8(dev_conf->enable_silent);
        rte_trace_point_emit_int(ret);
 )
 
@@ -83,16 +82,13 @@ RTE_TRACE_POINT(
        const struct rte_dma_vchan_conf __conf = {0};
        conf = &__conf;
 #endif /* _RTE_TRACE_POINT_REGISTER_H_ */
-       int src_port_type = conf->src_port.port_type;
-       int dst_port_type = conf->dst_port.port_type;
-       int direction = conf->direction;
        rte_trace_point_emit_i16(dev_id);
        rte_trace_point_emit_u16(vchan);
-       rte_trace_point_emit_int(direction);
+       rte_trace_point_emit_int(conf->direction);
        rte_trace_point_emit_u16(conf->nb_desc);
-       rte_trace_point_emit_int(src_port_type);
+       rte_trace_point_emit_int(conf->src_port.port_type);
        rte_trace_point_emit_blob(&conf->src_port.pcie, sizeof(uint64_t));
-       rte_trace_point_emit_int(dst_port_type);
+       rte_trace_point_emit_int(conf->dst_port.port_type);
        rte_trace_point_emit_blob(&conf->dst_port.pcie, sizeof(uint64_t));
        rte_trace_point_emit_ptr(conf->auto_free.m2d.pool);
        rte_trace_point_emit_int(ret);
-- 
2.48.1

Reply via email to