From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v4 2/4] kcp: add kernel control path kernel module Date: Wed, 2 Mar 2016 11:05:55 +0000 Message-ID: <56D6C913.8080806@intel.com> References: <1456495841-15749-1-git-send-email-ferruh.yigit@intel.com> <1456846920-28770-1-git-send-email-ferruh.yigit@intel.com> <1456846920-28770-3-git-send-email-ferruh.yigit@intel.com> <20160301150631.4cf29e47@xeon-e3> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Stephen Hemminger Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 1B3AA2617 for ; Wed, 2 Mar 2016 12:05:57 +0100 (CET) In-Reply-To: <20160301150631.4cf29e47@xeon-e3> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 3/1/2016 11:06 PM, Stephen Hemminger wrote: > On Tue, 1 Mar 2016 15:41:58 +0000 > Ferruh Yigit wrote: > >> +#ifdef RTE_KCP_KO_DEBUG >> +#define KCP_DBG(args...) pr_info(args) >> +#else >> +#define KCP_DBG(args...) >> +#en > > Why not use pr_debug() which is compile enabled already and supports dynamic > enabling as well. > Because of dynamic debug control, it is more complex and less known. But I will update it to pr_debug. Thanks, ferruh