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 C0647A0561; Wed, 17 Mar 2021 18:39:34 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 732E7140F0B; Wed, 17 Mar 2021 18:39:33 +0100 (CET) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mails.dpdk.org (Postfix) with ESMTP id E43DE140F06 for ; Wed, 17 Mar 2021 18:39:31 +0100 (CET) IronPort-SDR: JfuTlLZ2+TVXyU4b7K/a3csqNiRxBaL9WStmHw4I/F8DDjHEXZF3yBhqsU0WVMhIlEKmYFMEBF uvK7p8KMn7kQ== X-IronPort-AV: E=McAfee;i="6000,8403,9926"; a="250871522" X-IronPort-AV: E=Sophos;i="5.81,257,1610438400"; d="scan'208";a="250871522" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Mar 2021 10:39:30 -0700 IronPort-SDR: T7loLPJJELBYlgMMbm2eUS9gmr0bnKg8mf9zWP+oNCeY/ERDei7nWiO4AvAQdRM8UMwy9BOE0x ezq7EHbSqgFg== X-IronPort-AV: E=Sophos;i="5.81,257,1610438400"; d="scan'208";a="406016973" Received: from mmansoor-mobl.ger.corp.intel.com (HELO [10.252.22.40]) ([10.252.22.40]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Mar 2021 10:39:29 -0700 To: Thomas Monjalon , dev@dpdk.org Cc: Matan Azrad , Shahaf Shuler , Viacheslav Ovsiienko References: <20210308222855.823670-1-thomas@monjalon.net> <20210309094836.988763-1-thomas@monjalon.net> <20210309094836.988763-3-thomas@monjalon.net> From: Ferruh Yigit X-User: ferruhy Message-ID: <51a1b4b9-946d-95f4-632d-791a3553b281@intel.com> Date: Wed, 17 Mar 2021 17:39:25 +0000 MIME-Version: 1.0 In-Reply-To: <20210309094836.988763-3-thomas@monjalon.net> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2 2/4] common/mlx5: enable debug logs dynamically 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/9/2021 9:48 AM, Thomas Monjalon wrote: > Most debug logs are using DRV_LOG(DEBUG,) > but some were using DEBUG(). > The macro DEBUG is doing nothing if not compiled with > RTE_LIBRTE_MLX5_DEBUG. > > As it is not used in the data path, the macro DEBUG > can be replaced with DRV_LOG. > Then all debug logs can be enabled at runtime with: > --log-level pmd.net.mlx5:debug > > Signed-off-by: Thomas Monjalon Similar comment for the mlx4 one, copying here: Why 'RTE_LIBRTE_MLX5_DEBUG' exists at first place? It seems is is used both for data and control path, can you extend the patch for: 1- Remove #ifdef from control path 2- Replace with 'RTE_ETHDEV_DEBUG_RX' & 'RTE_ETHDEV_DEBUG_TX' for data path, please see: https://patches.dpdk.org/project/dpdk/list/?series=15738 3- Remove 'RTE_LIBRTE_MLX5_DEBUG' completely, if not removed document it in the driver documentation as supported config file Both for 'mlx4' and 'mlx5', I will continue with existing patch, but can it be possible to make additional patches to address above issues?