[dpdk-dev] [PATCH v6 0/8] ether: refine debug build option

[Ferruh Yigit]

On 3/31/2021 10:52 AM, Qi Zhang wrote:
> 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/8: 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_RX
> - RTE_ETHDEV_DEBUG_TX
> 
> The undocumented RTE_LIBRTE_ETHDEV_DEBUG will a alias to
> both RTE_ETHDEV_DEBUG_RX and RTE_ETHDEV_DEBUG_TX for backward
> compatibility.
> 
> Patch 2/8 - 8/8: refine the build option for Intel PMDs by using the
> new build options
> 
> v6:
> - fix build issue for -Dc_args="-DRTE_LIBRTE_ETHDEV_DEBUG=1"
> - s/compile/build/
> 
> v5:
> - Remove RTE_ETHDEV_DEBUG
> - Alias RTE_LIBRTE_ETHDEV_DEBUG to RTE_ETHDEV_DEBUG_RX|TX for backward
>    compatibility.
> 
> v4:
> - Alias RTE_LIBRTE_ETHDEV_DEBUG to RTE_ETHDEV_DEBUG in ethdev library
>    for backward compatibility.
> - Not replace RTE_LIBRTE_ETHDEV_DEBUG in all PMDs.
> 
> 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 (8):
>    ether: refine debug 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
> 

For series,
Reviewed-by: Ferruh Yigit <ferruh.yigit at intel.com>

Series applied to dpdk-next-net/main, thanks.