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 A24D7A054F; Tue, 16 Mar 2021 14:05:31 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 2E1E04069F; Tue, 16 Mar 2021 14:05:31 +0100 (CET) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mails.dpdk.org (Postfix) with ESMTP id C16244003D for ; Tue, 16 Mar 2021 14:05:28 +0100 (CET) IronPort-SDR: qoFAJ0FjIXuiFSQxiQ7cuyryjFw1g1cYtq+7Bju5RlsX4n0RfBXks/nnExAdlZgehcHE2yHKQ9 JNNSOcJrmvVQ== X-IronPort-AV: E=McAfee;i="6000,8403,9924"; a="185892506" X-IronPort-AV: E=Sophos;i="5.81,251,1610438400"; d="scan'208";a="185892506" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Mar 2021 06:05:27 -0700 IronPort-SDR: x7sdGqYyQ18a63JCJ6yWWNuI/Sc2I5UH9ZIXrNS4ebUwf3+Ifud0zqCJBqPS8g3ci62kTos7FF HRUnPa8SjhHw== X-IronPort-AV: E=Sophos;i="5.81,251,1610438400"; d="scan'208";a="405537804" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.252.11.157]) ([10.252.11.157]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Mar 2021 06:05:24 -0700 To: Qi Zhang , Thomas Monjalon , Andrew Rybchenko Cc: dev@dpdk.org, 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 References: <20210310121626.2019863-1-qi.z.zhang@intel.com> <20210312121223.2028029-1-qi.z.zhang@intel.com> <20210312121223.2028029-2-qi.z.zhang@intel.com> From: Ferruh Yigit X-User: ferruhy Message-ID: <93fd4fce-a961-8ccf-fd32-0386461e4e99@intel.com> Date: Tue, 16 Mar 2021 13:05:19 +0000 MIME-Version: 1.0 In-Reply-To: <20210312121223.2028029-2-qi.z.zhang@intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2 1/8] ether: refine debug compile 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" On 3/12/2021 12:12 PM, Qi Zhang wrote: > PMDs use RTE_LIBRTE__DEBUG_RX|TX as compile 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. > > The patch introduces below compile options for specific Rx/Tx data path > debug, so PMD can choose to reuse them to avoid maintain their own. > > - RTE_LIBRTE_ETHDEV_DEBUG_RX > - RTE_LIBRTE_ETHDEV_DEBUG_TX > > Also, all the compile options are documented on the overview page, so > users can easily find them. > > Signed-off-by: Qi Zhang > --- > doc/guides/nics/overview.rst | 20 ++++++++++++++++++++ > lib/librte_ethdev/rte_ethdev.h | 16 ++++++++-------- > 2 files changed, 28 insertions(+), 8 deletions(-) > > diff --git a/doc/guides/nics/overview.rst b/doc/guides/nics/overview.rst > index 20cd52b097..20cf54ef32 100644 > --- a/doc/guides/nics/overview.rst > +++ b/doc/guides/nics/overview.rst > @@ -32,3 +32,23 @@ More details about features can be found in :doc:`features`. > > Features marked with "P" are partially supported. Refer to the appropriate > NIC guide in the following sections for details. > + > +The ethdev layer support below compile options for debug purpose: > + > +- ``RTE_LIBRTE_ETHDEV_DEBUG`` (default **disabled**) > + > + Compile with debug code on data path. > + > +- ``RTE_LIBRTE_ETHDEV_DEBUG_RX`` (default **disabled**) > + > + Compile with debug code on Rx data path. > + > +- ``RTE_LIBRTE_ETHDEV_DEBUG_TX`` (default **disabled**) > + > + Compile with debug code on Tx data path. > + > +.. Note:: > + > + The lib_ethdev use above options to wrap debug code to trace invalid parameters on > + data path APIs, so performance downgrade is expected when enable those options. > + Each PMD can decide to reuse them to wrap their own debug code in the Rx/Tx path. Hi Qi, Overall patch looks good to me, but not sure about adding the documentation to the NIC overview page. What do you think about moving the doc to next chapter, under "3.1. Driver Compilation"?