From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [PATCH 01/10] dynamic_debug: Add __dynamic_dev_dbg Date: Wed, 06 Jul 2011 14:46:43 -0700 Message-ID: <1309988803.1710.10.camel@Joe-Laptop> References: <6d176b4a9fa316e5130455b6a6b47ba1dd0501d5.1309967232.git.root@dhcp-100-18-164.bos.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: gregkh@suse.de, jim.cromie@gmail.com, bvanassche@acm.org, linux-kernel@vger.kernel.org, davem@davemloft.net, aloisio.almeida@openbossa.org, netdev@vger.kernel.org To: Jason Baron Return-path: In-Reply-To: <6d176b4a9fa316e5130455b6a6b47ba1dd0501d5.1309967232.git.root@dhcp-100-18-164.bos.redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Wed, 2011-07-06 at 13:24 -0400, Jason Baron wrote: > Unlike dynamic_pr_debug, dynamic uses of dev_dbg can not > currently add task_pid/KBUILD_MODNAME/__func__/__LINE__ > to selected debug output. > Add a new function similar to dynamic_pr_debug to > optionally emit these prefixes. [] > diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c [] > @@ -456,6 +457,43 @@ int __dynamic_pr_debug(struct _ddebug *descriptor, const char *fmt, ...) [] > +int __dynamic_dev_dbg(struct _ddebug *descriptor, > + const struct device *dev, const char *fmt, ...) [] > + res += __dev_printk("", dev, &vaf); I suppose that should more properly be written as: res += __dev_printk(KERN_CONT, dev, &vaf);