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 9D922A0562; Tue, 23 Mar 2021 12:08:12 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8741B140DB9; Tue, 23 Mar 2021 12:08:12 +0100 (CET) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mails.dpdk.org (Postfix) with ESMTP id 0BD31140DB2 for ; Tue, 23 Mar 2021 12:08:10 +0100 (CET) IronPort-SDR: HO0ebqG49PaPR97kBz88eyYV7EQ+TUvbeOeGYVQACXJnclB8YVH4dQ2YmkaSptaR/6aGHby0vd HW9mTPlvCg8Q== X-IronPort-AV: E=McAfee;i="6000,8403,9931"; a="210530072" X-IronPort-AV: E=Sophos;i="5.81,271,1610438400"; d="scan'208";a="210530072" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2021 04:08:08 -0700 IronPort-SDR: ZPA6ZFMEmmbMTJLp6iokYAJHeqh7CULBxQGmqgng633hibs/+dFWRc/x1A08l5tF+4BoKwEqEB zo6Y4DYdv4yw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.81,271,1610438400"; d="scan'208";a="441633109" Received: from fmsmsx603.amr.corp.intel.com ([10.18.126.83]) by fmsmga002.fm.intel.com with ESMTP; 23 Mar 2021 04:08:08 -0700 Received: from shsmsx606.ccr.corp.intel.com (10.109.6.216) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Tue, 23 Mar 2021 04:08:07 -0700 Received: from shsmsx601.ccr.corp.intel.com (10.109.6.141) by SHSMSX606.ccr.corp.intel.com (10.109.6.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Tue, 23 Mar 2021 19:08:05 +0800 Received: from shsmsx601.ccr.corp.intel.com ([10.109.6.141]) by SHSMSX601.ccr.corp.intel.com ([10.109.6.141]) with mapi id 15.01.2106.013; Tue, 23 Mar 2021 19:08:05 +0800 From: "Zhang, Qi Z" To: "Yigit, Ferruh" CC: "dev@dpdk.org" , "thomas@monjalon.net" , "Richardson, Bruce" , "Wang, Xiao W" , "Wu, Jingjing" , "Xing, Beilei" , "Guo, Jia" , "Yang, Qiming" , "Wang, Haiyue" Thread-Topic: [PATCH v5 0/8] ether: refine debug build option Thread-Index: AQHXH9RADcmqLUYcgESK0vvjenWLtaqRaVxw Date: Tue, 23 Mar 2021 11:08:05 +0000 Message-ID: References: <20210310121626.2019863-1-qi.z.zhang@intel.com> <20210323110749.2306452-1-qi.z.zhang@intel.com> In-Reply-To: <20210323110749.2306452-1-qi.z.zhang@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-reaction: no-action dlp-version: 11.5.1.3 dlp-product: dlpe-windows x-originating-ip: [10.239.127.36] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v5 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" Sorry, forgot adding change log.... v5: - Remove RTE_ETHDEV_DEBUG - Alias RTE_LIBRTE_ETHDEV_DEBUG to RTE_ETHDEV_DEBUG_RX|TX for backward comp= atibility. 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. > -----Original Message----- > From: Zhang, Qi Z > Sent: Tuesday, March 23, 2021 7:08 PM > To: Yigit, Ferruh > Cc: dev@dpdk.org; thomas@monjalon.net; Richardson, Bruce > ; Wang, Xiao W ; Wu, > Jingjing ; Xing, Beilei ; G= uo, Jia > ; Yang, Qiming ; Wang, Haiyue > ; Zhang, Qi Z > Subject: [PATCH v5 0/8] ether: refine debug build option >=20 > 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. >=20 > Patch 1/8: introduces new compile options for data path debug in ether la= yer, > PMD can choose to reuse them to avoid maintain their own. >=20 > - RTE_ETHDEV_DEBUG_RX > - RTE_ETHDEV_DEBUG_TX >=20 > The undocumented RTE_LIBRTE_ETHDEV_DEBUG will a alias to both > RTE_ETHDEV_DEBUG_RX and RTE_ETHDEV_DEBUG_TX for backward > compatibility. >=20 > Patch 2/8 - 8/8: refine the build option for Intel PMDs by using the new = build > options >=20 > 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 >=20 > doc/guides/nics/build_and_test.rst | 16 ++++++++ > 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 | 22 ++++++---- > 23 files changed, 137 insertions(+), 193 deletions(-) >=20 > -- > 2.26.2