[1/8] cryptodev: fix trace points registration

Message ID 20200503203135.6493-2-david.marchand@redhat.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series Traces cleanup for rc2 |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-nxp-Performance success Performance Testing PASS
ci/Intel-compilation success Compilation OK
ci/iol-testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS

Commit Message

David Marchand May 3, 2020, 8:31 p.m. UTC
  Those trace points are defined but not registered.

Fixes: 4cf30e3f3c35 ("cryptodev: add tracepoints")

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 lib/librte_cryptodev/cryptodev_trace_points.c | 6 ++++++
 1 file changed, 6 insertions(+)
  

Comments

Sunil Kumar Kori May 4, 2020, 7:41 a.m. UTC | #1
>-----Original Message-----
>From: David Marchand <david.marchand@redhat.com>
>Sent: Monday, May 4, 2020 2:01 AM
>To: dev@dpdk.org
>Cc: thomas@monjalon.net; Declan Doherty <declan.doherty@intel.com>;
>Sunil Kumar Kori <skori@marvell.com>
>Subject: [EXT] [PATCH 1/8] cryptodev: fix trace points registration
>
>External Email
>
>----------------------------------------------------------------------
>Those trace points are defined but not registered.
>
>Fixes: 4cf30e3f3c35 ("cryptodev: add tracepoints")
>
>Signed-off-by: David Marchand <david.marchand@redhat.com>
>---
> lib/librte_cryptodev/cryptodev_trace_points.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
>diff --git a/lib/librte_cryptodev/cryptodev_trace_points.c
>b/lib/librte_cryptodev/cryptodev_trace_points.c
>index 9df213e25b..7d03c93882 100644
>--- a/lib/librte_cryptodev/cryptodev_trace_points.c
>+++ b/lib/librte_cryptodev/cryptodev_trace_points.c
>@@ -61,6 +61,12 @@ RTE_INIT(cryptodev_trace_init)
>
>	RTE_TRACE_POINT_REGISTER(rte_cryptodev_trace_asym_session_init
>,
> 		lib.cryptodev.asym.init);
>
>+
>	RTE_TRACE_POINT_REGISTER(rte_cryptodev_trace_sym_session_clear
>,
>+		lib.cryptodev.sym.clear);
>+
>+
>	RTE_TRACE_POINT_REGISTER(rte_cryptodev_trace_asym_session_cle
>ar,
>+		lib.cryptodev.asym.clear);
>+
> 	RTE_TRACE_POINT_REGISTER(rte_cryptodev_trace_enqueue_burst,
> 		lib.cryptodev.enq.burst);
>
>--

Acked-by: Sunil Kumar Kori <skori@marvell.com>

>2.23.0
  

Patch

diff --git a/lib/librte_cryptodev/cryptodev_trace_points.c b/lib/librte_cryptodev/cryptodev_trace_points.c
index 9df213e25b..7d03c93882 100644
--- a/lib/librte_cryptodev/cryptodev_trace_points.c
+++ b/lib/librte_cryptodev/cryptodev_trace_points.c
@@ -61,6 +61,12 @@  RTE_INIT(cryptodev_trace_init)
 	RTE_TRACE_POINT_REGISTER(rte_cryptodev_trace_asym_session_init,
 		lib.cryptodev.asym.init);
 
+	RTE_TRACE_POINT_REGISTER(rte_cryptodev_trace_sym_session_clear,
+		lib.cryptodev.sym.clear);
+
+	RTE_TRACE_POINT_REGISTER(rte_cryptodev_trace_asym_session_clear,
+		lib.cryptodev.asym.clear);
+
 	RTE_TRACE_POINT_REGISTER(rte_cryptodev_trace_enqueue_burst,
 		lib.cryptodev.enq.burst);