From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id F1675A034F; Wed, 31 Mar 2021 11:49:13 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8BE66140E28; Wed, 31 Mar 2021 11:49:13 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mails.dpdk.org (Postfix) with ESMTP id 3C375406A3 for ; Wed, 31 Mar 2021 11:49:12 +0200 (CEST) IronPort-SDR: fkYaRDMe5vsrlJa/2tdNaQZgVTFchN2tG+K1Kw7a/JM1r/kxTq+ULVntlqKmf4FKJOQfOaVjBC 1cx7bi8av1kA== X-IronPort-AV: E=McAfee;i="6000,8403,9939"; a="191448487" X-IronPort-AV: E=Sophos;i="5.81,293,1610438400"; d="scan'208";a="191448487" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 31 Mar 2021 02:49:10 -0700 IronPort-SDR: kg5kWvXhnsBPUBKdLukUJeS2db3XQIibXeDREbryFjUtoJswSm8GiQBgiRUlCXqwx3Nar2JRX8 Y3GCILsgvp3g== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.81,293,1610438400"; d="scan'208";a="445603368" Received: from dpdk51.sh.intel.com ([10.67.111.142]) by fmsmga002.fm.intel.com with ESMTP; 31 Mar 2021 02:49:08 -0700 From: Qi Zhang To: ferruh.yigit@intel.com Cc: dev@dpdk.org, thomas@monjalon.net, bruce.richardson@intel.com, xiao.w.wang@intel.com, jingjing.wu@intel.com, beilei.xing@intel.com, jia.guo@intel.com, qiming.yang@intel.com, haiyue.wang@intel.com, Qi Zhang Date: Wed, 31 Mar 2021 17:52:46 +0800 Message-Id: <20210331095254.2454439-1-qi.z.zhang@intel.com> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20210310121626.2019863-1-qi.z.zhang@intel.com> References: <20210310121626.2019863-1-qi.z.zhang@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v6 0/8] ether: refine debug build option X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" PMDs use RTE_LIBRTE__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 doc/guides/nics/build_and_test.rst | 16 ++++++++ drivers/net/e1000/e1000_logs.c | 18 +++------ drivers/net/e1000/e1000_logs.h | 13 +----- drivers/net/e1000/em_rxtx.c | 38 +++++++++--------- drivers/net/e1000/igb_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/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 +++++++++++++++--------------- lib/librte_ethdev/rte_ethdev.h | 22 ++++++---- 23 files changed, 138 insertions(+), 193 deletions(-) -- 2.26.2