Re: Debug prints mac80211 drivers

[Joe Perches <joe@perches.com>]

On Sat, 2016-04-23 at 02:32 +0530, Krishna Chaitanya wrote:
> On Sat, Apr 23, 2016 at 12:59 AM, Joe Perches <joe@perches.com> wrote:
> > 
> > 
> > On Fri, 2016-04-22 at 17:51 +0530, Krishna Chaitanya wrote:
> > > 
> > > What is the recommended method for adding
> > > debug prints in mac80211 based drivers.
> > > 
> > > 1) -DDEBUG + pr_debug ==> used by mac80211, brcm80211
> > > 2) -DDEBUG + dev_dbg ==> zd1201
> > > 3) dev_printk(KERN_DEBUG) ==> used by iwlwifi
> > > 4) printk(KERN_DEBUG) ==> Just to complete the list.
> > wiphy_dbg -> netif_dbg -> netdev_dbg -> dev_dbg -> pr_debug
> Ok, thats what checpatch --strict throws. but still different vendors
> follow
> different standards, so wanted to check if we should go strictly with
> checkpatch (or) is there any rationale behind choose each of the
> variant.

Generally the variants are used to produce sufficient
logging information to identify the appropriate device.

Most all debugging printks shouldn't be emitted unless
actually debugging.

> > and CONFIG_DYNAMIC_DEBUG, no -DDEBUG required
> Yes, i understand. Till now we had this enabled, so pr_debug
> works just fine, but now it is disabled hence the question.
> 
> Also there are pros and cons to having control using dyndbg,
> user can disable dyndbg, there be missing imp

imp?

>  debugs, in this case
> having module level (-DDEBUG) helps but if we want entire system
> to run in non-debug mode, disabling dyndbg helps.

Confused:

dynamic debug printks aren't emitted by default
unless DEBUG is also defined or specifically
enabled by the user.