mbox series

[v3,0/9] ether: refine debug build option

Message ID 20210317111551.2215332-1-qi.z.zhang@intel.com (mailing list archive)
Headers
Series ether: refine debug build option |

Message

Qi Zhang March 17, 2021, 11:15 a.m. UTC
  PMDs use RTE_LIBRTE_<PMD_NAME>_DEBUG_RX|TX as build option to wrap
data path debug code. As .config has been removed since the meson build,
It is not friendly for new DPDK users to notice those debug options.

Patch 1/9: introduces new compile options for data path debug in ether
layer, PMD can choose to reuse them to avoid maintain their own.

- RTE_ETHDEV_DEBUG
- RTE_ETHDEV_DEBUG_RX
- RTE_ETHDEV_DEBUG_TX

Patch 2/9: replace all RTE_LIBRTE_ETHDEV_DEBUG with RTE_ETHDEV_DEBUG
in all PMDs

Patch 3/9 - 9/9: refine the build option for Intel PMDs by using the
new build options

v3:
- rename "RTE_LIBRTE_ETHDEV_xxx" to "RTE_ETHDEV_xxx"
- move document from "overview.rst" page to "build_and_test.rst"
- fix grammar issues.

v2:
- fix some coding style issues.

Qi Zhang (9):
  ether: refine debug build option
  net: replace build option
  net/fm10k: refine debug build option
  net/e1000: refine debug build option
  net/i40e: refine debug build option
  net/iavf: refine debug build option
  net/ice: refine debug build option
  net/ixgbe: refine debug build option
  net/igc: refine debug build option

 doc/guides/nics/build_and_test.rst           | 20 ++++++
 drivers/net/ark/ark_logs.h                   |  2 +-
 drivers/net/atlantic/atl_rxtx.c              |  2 +-
 drivers/net/e1000/e1000_logs.c               | 17 ++----
 drivers/net/e1000/e1000_logs.h               | 13 +---
 drivers/net/e1000/em_rxtx.c                  | 38 ++++++------
 drivers/net/e1000/igb_rxtx.c                 |  2 +-
 drivers/net/ena/ena_ethdev.c                 |  2 +-
 drivers/net/enic/enic_rxtx.c                 |  2 +-
 drivers/net/fm10k/fm10k_ethdev.c             |  7 +--
 drivers/net/fm10k/fm10k_logs.h               | 13 +---
 drivers/net/fm10k/fm10k_rxtx.c               |  8 +--
 drivers/net/hinic/hinic_pmd_tx.c             |  2 +-
 drivers/net/hns3/hns3_rxtx.c                 |  4 +-
 drivers/net/i40e/i40e_ethdev.c               |  7 +--
 drivers/net/i40e/i40e_logs.h                 | 13 +---
 drivers/net/i40e/i40e_rxtx.c                 | 16 ++---
 drivers/net/iavf/iavf_ethdev.c               |  7 +--
 drivers/net/iavf/iavf_log.h                  | 13 +---
 drivers/net/iavf/iavf_rxtx.c                 |  8 +--
 drivers/net/ice/ice_ethdev.c                 |  7 +--
 drivers/net/ice/ice_logs.h                   | 13 +---
 drivers/net/ice/ice_rxtx.c                   | 20 +++---
 drivers/net/igc/igc_logs.h                   |  4 +-
 drivers/net/igc/igc_txrx.c                   |  2 +-
 drivers/net/ixgbe/ixgbe_ethdev.c             |  7 +--
 drivers/net/ixgbe/ixgbe_logs.h               | 13 +---
 drivers/net/ixgbe/ixgbe_rxtx.c               | 64 ++++++++++----------
 drivers/net/qede/qede_rxtx.c                 |  4 +-
 drivers/net/softnic/rte_eth_softnic.c        |  2 +-
 drivers/net/softnic/rte_eth_softnic_thread.c |  2 +-
 drivers/net/txgbe/txgbe_rxtx.c               |  2 +-
 drivers/net/virtio/virtio_rxtx.c             |  2 +-
 drivers/net/vmxnet3/vmxnet3_rxtx.c           |  2 +-
 lib/librte_ethdev/rte_ethdev.h               | 16 ++---
 35 files changed, 149 insertions(+), 207 deletions(-)