From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH] ethdev: decrease log level for successful API Date: Thu, 2 Aug 2018 08:14:44 -0700 Message-ID: <20180802081444.1211cf79@xeon-e3> References: <1533213223-24028-1-git-send-email-ktraynor@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Kevin Traynor Return-path: Received: from mail-pg1-f194.google.com (mail-pg1-f194.google.com [209.85.215.194]) by dpdk.org (Postfix) with ESMTP id 881D51B4CE for ; Thu, 2 Aug 2018 17:14:47 +0200 (CEST) Received: by mail-pg1-f194.google.com with SMTP id a11-v6so1348105pgw.6 for ; Thu, 02 Aug 2018 08:14:47 -0700 (PDT) In-Reply-To: <1533213223-24028-1-git-send-email-ktraynor@redhat.com> 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 Thu, 2 Aug 2018 13:33:43 +0100 Kevin Traynor wrote: > Change log level of messages from ERR back to DEBUG where > existing API indicates that the condition is a success. > > This means applications using the API in its current form will > not get new ERR logs. > > Fixes: bea1e0c70cfc ("ethdev: convert static log type usage to dynamic") The expectations about level come from syslog: LOG_ERR error conditions LOG_WARNING warning conditions LOG_NOTICE normal, but significant, condition LOG_INFO informational message LOG_DEBUG debug-level message Therefore messages where an API did something wrong should be NOTICE and messages where an API's normal action was successful should be INFO or DEBUG. All the "already XXX" messages should be NOTICE.