public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Thomas Renninger <trenn@suse.de>
To: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: Jason Baron <jbaron@redhat.com>, Greg KH <gregkh@suse.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	LKML <linux-kernel@vger.kernel.org>,
	bjorn.helgaas@hp.com
Subject: Re: Boot-time dynamic debugging?
Date: Mon, 2 Aug 2010 16:13:59 +0200	[thread overview]
Message-ID: <201008021614.00038.trenn@suse.de> (raw)
In-Reply-To: <AANLkTi=k5nkUk9LOS=EGHGSq=5Qe8_L-1d3_t9byxMrt@mail.gmail.com>

On Monday 02 August 2010 15:53:24 Pekka Enberg wrote:
> > On Sun, Aug 01, 2010 at 03:10:40PM +0300, Pekka Enberg wrote:
> >> Commit 346e15beb5343c2eb8216d820f2ed8f150822b08 ("driver core: basic
> >> infrastructure for per-module dynamic debug messages") removed
> >> "dynamic_printk" kernel parameter from the dynamic debugging subsystem
> >> (and forgot to update Documentation/kernel-parameters.txt). How can I
> >> use the new subsystem for enabling boot-time dev_dbg() calls for
> >> non-modular kernels?
> 
> On Mon, Aug 2, 2010 at 4:47 PM, Jason Baron <jbaron@redhat.com> wrote:
> > Right now, you can not enable them dynamically from the boot command
> > line (only via the debugfs file). However, Thomas Renninger, has posted
> > a series to add back the boot-time functionality, see:
> >
> > Subject:    [patch 0/3] Dynamic Debug providing early boot debug messages via
> > boot parameter
> > http://marc.info/?l=linux-kernel&m=127970474426120&w=2
> >
> > So I intend to queue those up for 2.6.36, and also update the
> > documentation.
> 
> Oh, great!
> 
> Acked-by: Pekka Enberg <penberg@cs.helsinki.fi>
Thanks.
> 
> Btw, can you easily enable all dev_dbg() calls with the boot parameter?
I expect yes, but never explicitly tried it, dev_dbg should result in
printk(KERN_DEBUG ...) after some macro/inline func processing.

Be aware that I sent another neat ddebug functionality:
Subject on LKML: [PATCH] Dynamic Debug: Introduce global fake module param module.ddebug

If, take the second post (same mail thread) which is the same patch
with some documentation added.
It's based on top of the patchseries mentioned above.
Would be great if someone could pick it up as well.
I was not sure about the fact that drivers must not use ddebug as parameter
anymore with this patch and how to document that, quick review and/or
comments appreciated.

Thanks,

      Thomas

  reply	other threads:[~2010-08-02 14:11 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-01 12:10 Boot-time dynamic debugging? Pekka Enberg
2010-08-02 13:47 ` Jason Baron
2010-08-02 13:53   ` Pekka Enberg
2010-08-02 14:13     ` Thomas Renninger [this message]
2010-08-06 14:03       ` Thomas Renninger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201008021614.00038.trenn@suse.de \
    --to=trenn@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=bjorn.helgaas@hp.com \
    --cc=gregkh@suse.de \
    --cc=jbaron@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=penberg@cs.helsinki.fi \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox