From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f179.google.com (mail-wi0-f179.google.com [209.85.212.179]) by dpdk.org (Postfix) with ESMTP id 4F0365927 for ; Sat, 24 Oct 2015 15:11:00 +0200 (CEST) Received: by wicll6 with SMTP id ll6so62219693wic.1 for ; Sat, 24 Oct 2015 06:11:00 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding:content-type; bh=S7le4ai7QKN0BTI4gkD9GaZ3BPVF6CqsFpOeTSOw08A=; b=O0zxDs/Y5ckjUyG0RmoVh6ro0zLFY/InWMmxo+EtkgBRoHH+cZog4o+Nay+0B1SNBB JfG6Fz9BwpuEkHaagFS9HVpmYzvO/WO3llPQoSHeRyW+5R4ViAVvu74QNjOq3MkwlEKc OiC6WqS4TcuG2VNyfHyYStsckjZz3hq4RFcMAzkvPLgX4EyTcPxlRplGfCO4rsaLcug7 2HLribFTdazSKKUV77o8awotzwHIfIfPRZY/qGh96h/tBxvNPkZvTJM9b7ynn9gXZ54M cyf6EtZDzL9gshbT+UzLluL0hQEFYCe48Q/9tzxlHgZ6+opsAFFBsmot8PGSj0UCfEKv 84qQ== X-Gm-Message-State: ALoCoQmzeWRuvR5Ei+DKXBQ4InoeUezh8OqFaxbjJCIqgy+WCYgYLqjOpC0npXjaGTLZRCLyheh0 X-Received: by 10.194.62.137 with SMTP id y9mr29285577wjr.136.1445692259997; Sat, 24 Oct 2015 06:10:59 -0700 (PDT) Received: from xps13.localnet (APoitiers-657-1-74-252.w90-45.abo.wanadoo.fr. [90.45.221.252]) by smtp.gmail.com with ESMTPSA id b1sm14425621wjx.12.2015.10.24.06.10.58 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 24 Oct 2015 06:10:59 -0700 (PDT) From: Thomas Monjalon To: Harish Patil Date: Sat, 24 Oct 2015 15:09:33 +0200 Message-ID: <6673500.WzqtfFB45U@xps13> Organization: 6WIND User-Agent: KMail/4.14.10 (Linux/4.1.6-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: References: <1442018576-19981-1-git-send-email-rasesh.mody@qlogic.com> <3670820.DENFCmAgGV@xps13> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: dev@dpdk.org, Sony Chacko Subject: Re: [dpdk-dev] [PATCH 3/5] bnx2x: Add RTE_LIBRTE_BNX2X_REG_ACCESS config option X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 24 Oct 2015 13:11:00 -0000 2015-10-23 22:28, Harish Patil: > >2015-09-11 17:42, Rasesh Mody: > >> From: Harish Patil > >> --- a/config/common_linuxapp > >> +++ b/config/common_linuxapp > >> @@ -220,6 +220,7 @@ CONFIG_RTE_LIBRTE_BNX2X_DEBUG_INIT=n > >> CONFIG_RTE_LIBRTE_BNX2X_DEBUG_RX=n > >> CONFIG_RTE_LIBRTE_BNX2X_DEBUG_TX=n > >> CONFIG_RTE_LIBRTE_BNX2X_MF_SUPPORT=n > >> +CONFIG_RTE_LIBRTE_BNX2X_REG_ACCESS=n > > > >Why do you need this option? > > Currently logging of register accesses is using (wrongly) the generic > driver debug option of PMD_DRV_LOG (which gets enabled if > CONFIG_RTE_LIBRTE_BNX2X_DEBUG=y). > This floods the console due to excessive printing of register > reads/writes. So creating a new debug option in order to capture only the > register read/writes (if required) and also does not interfere with the > informational/user-level debug. You mean there is not enough log levels? #define RTE_LOG_EMERG 1U /**< System is unusable. */ #define RTE_LOG_ALERT 2U /**< Action must be taken immediately. */ #define RTE_LOG_CRIT 3U /**< Critical conditions. */ #define RTE_LOG_ERR 4U /**< Error conditions. */ #define RTE_LOG_WARNING 5U /**< Warning conditions. */ #define RTE_LOG_NOTICE 6U /**< Normal but significant condition. */ #define RTE_LOG_INFO 7U /**< Informational. */ #define RTE_LOG_DEBUG 8U /**< Debug-level messages. */ Either other "debug" logs can be put as INFO level, or you need a level 9 for verbose debug. But I don't really understand what is the problem of a verbose debug. It is not critical for the acceptance of this patch. Just wanted to discuss it as we have many config options to tune the debug. Please, at least, use the DEBUG word in the option name as done for CONFIG_RTE_LIBRTE_BNX2X_DEBUG_RX.