On 9/29/22 13:29, Ankur Dwivedi wrote:
This series adds trace points for functions in the ethdev library.
The trace points are added in ethdev, flow, mtr and tm files.

v2:
  - Made rte_eth_trace_call_rx_callbacks and rte_eth_trace_call_tx_callbacks
    as fastpath trace functions.
  - Moved some trace functions call to the end to capture return value and
    data.

Looking at the patch series I still see many-many places where
tracing happens on entry point. Intentional?

  - Removed rte_eth_trace_tx_queue_setup as it is not required.
    rte_ethdev_trace_txq_setup() is already present in
    rte_eth_tx_queue_setup().
  - Removed trace in ethdev_driver.c
  - Removed trace function check patch.

Ankur Dwivedi (4):
   ethdev: add trace points
   ethdev: add trace points for flow
   ethdev: add trace points for mtr
   ethdev: add trace points for tm

  lib/ethdev/ethdev_private.c      |    3 +
  lib/ethdev/ethdev_trace_points.c |  699 ++++++++++
  lib/ethdev/rte_ethdev.c          |  148 +++
  lib/ethdev/rte_ethdev_trace.h    | 2062 ++++++++++++++++++++++++++++++
  lib/ethdev/rte_ethdev_trace_fp.h |   19 +
  lib/ethdev/rte_flow.c            |   54 +
  lib/ethdev/rte_mtr.c             |   27 +
  lib/ethdev/rte_tm.c              |   40 +
  lib/ethdev/version.map           |  232 ++++
  9 files changed, 3284 insertions(+)


Reply via email to