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 20C61A0561; Thu, 18 Mar 2021 02:38:55 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id DBD14140F5C; Thu, 18 Mar 2021 02:38:54 +0100 (CET) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mails.dpdk.org (Postfix) with ESMTP id E3DDB40698 for ; Thu, 18 Mar 2021 02:38:52 +0100 (CET) IronPort-SDR: t+/smyDjr5atMkgoiHVgFo2l4uhAgOKLR4E+Xf/vfZxMcRgGTObZYNkRoUreFbOXH7XO8ZyguX yJeC9mUvNRgw== X-IronPort-AV: E=McAfee;i="6000,8403,9926"; a="253591024" X-IronPort-AV: E=Sophos;i="5.81,257,1610438400"; d="scan'208";a="253591024" Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Mar 2021 18:38:51 -0700 IronPort-SDR: j7uKjt3G4xHRv09IO3XW7cnD7UdFcjQiwpjQymDVlwbgznXDJ4Rd+SUtX/334Zd4ZSf3IhkfBq H4pyqQ6oL/Yg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.81,257,1610438400"; d="scan'208";a="602471406" Received: from dpdk51.sh.intel.com ([10.67.111.142]) by fmsmga006.fm.intel.com with ESMTP; 17 Mar 2021 18:38:47 -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: Thu, 18 Mar 2021 09:42:26 +0800 Message-Id: <20210318014234.2255366-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 v4 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 - RTE_ETHDEV_DEBUG_RX - RTE_ETHDEV_DEBUG_TX The undocumented RTE_LIBRTE_ETHDEV_DEBUG will a alias to RTE_ETHDEV_DEBUG for backward compatibility. Patch 2/8 - 8/8: refine the build option for Intel PMDs by using the new build options 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 | 20 ++++++++++ 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/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 | 21 ++++++---- 23 files changed, 140 insertions(+), 193 deletions(-) -- 2.26.2