From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH 1/3] net/e1000: optimize Rx/Tx log message level Date: Tue, 6 Dec 2016 14:41:44 -0800 Message-ID: <20161206144144.1d68d40b@xeon-e3> References: <1480761783-36467-1-git-send-email-qiming.yang@intel.com> <1480761783-36467-2-git-send-email-qiming.yang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Qiming Yang Return-path: Received: from mail-pg0-f51.google.com (mail-pg0-f51.google.com [74.125.83.51]) by dpdk.org (Postfix) with ESMTP id B20642BAF for ; Tue, 6 Dec 2016 23:41:52 +0100 (CET) Received: by mail-pg0-f51.google.com with SMTP id p66so153961148pga.2 for ; Tue, 06 Dec 2016 14:41:52 -0800 (PST) In-Reply-To: <1480761783-36467-2-git-send-email-qiming.yang@intel.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 Sat, 3 Dec 2016 18:43:01 +0800 Qiming Yang wrote: > > +#ifdef RTE_LIBRTE_E1000_DEBUG_RX_FREE > +#define PMD_RX_FREE_LOG(level, fmt, args...) \ > + RTE_LOG(level, PMD, "%s(): " fmt "\n", __func__, ## args) > +#else > +#define PMD_RX_FREE_LOG(level, fmt, args...) do { } while (0) > +#endif If you have to copy/paste same code in more than one driver, that looks like a missing interface. I.e Do not Repeat Yourself.