From: jim.cromie@gmail.com
To: jbaron@redhat.com, rusty@rustcorp.com.au
Cc: linux-kernel@vger.kernel.org
Subject: [00/11] pr_debug during module initialization
Date: Wed, 14 Mar 2012 17:01:55 -0600 [thread overview]
Message-ID: <1331766126-11674-1-git-send-email-jim.cromie@gmail.com> (raw)
This is 3rd revision of the dyndbg modinit patches, previously sent
Dec 11. Patches 1-17/25 sent then were added to driver-core-next,
this set reworks the remainder.
It implements the "fake" module param approach proposed by
Thomas Renninger, back in https://lkml.org/lkml/2010/9/15/397
This set is on top of linux-next, since that includes Pawel Moll's
initcall-level params patch. Im not using this feature, but I didnt
know that when I started.
Rusty Russell did a partial review of 2nd rev (sent off-list), here:
http://thread.gmane.org/gmane.linux.kernel/1262934
This revision incorporates my understanding of his feedback.
Core of patchset is 2 callbacks, called from parse_args:
For builtin modules, dynamic_debug_init() calls parse_args(...,
&ddebug_dyndbg_boot_param_cb) to handle dyndbg and $module.dyndbg
params. For loadable modules, load_module() calls parse_args(...,
&ddebug_dyndbg_module_param_cb) to handle dyndbg args given by
modprobe. Both callbacks call ddebug_exec_queries() to activate the
specified pr_debug callsites.
0001-init-trivial-tweaks-to-initcall_levels.patch
This adds pr_info for each initcall-level, yielding 7 lines in dmesg
like:
initlevel:6=device, 172 registered initcalls
I dropped the pr_debug added in 2nd rev, which was noisy and not
useful, init_debug provides similar info and more. This patch is
decoration only, and can be dropped.
0002-dynamic_debug-fix-leading-spaces.patch
whitespace
0003-dynamic_debug-replace-if-verbose-pr_info-with-macro-.patch
I should have done this previously. Ive disregarded a complaint about
wrapping complex macro args in (), since theres plenty of similar code
in include/*
0004-dynamic_debug-change-ddebug_query-core-param-to-dynd.patch
Change "ddebug_query" to "dyndbg", to match new $module.dyndbg added
by this patchset.
0005-params-add-param-name-to-parse_one-s-pr_debug.patch
Current message doesnt ID the param name being handled.
0006-params-add-3rd-arg-to-option-handler-callback-signat.patch
Add "doing" to lower levels of param handling callchain, supplying it
to callback added next.
0007-dynamic_debug-make-dynamic-debug-work-during-module-.patch
main patch of set.
0008-pnp-if-CONFIG_DYNAMIC_DEBUG-use-pnp.dyndbg-instead-o.patch
Adjust pnp for CONFIG_PNP_DEBUG_MESSAGES. Unchanged since Dec 11th
0009-dynamic_debug-add-modname-arg-to-exec_query-callchai.patch
This allows parsing $modname.dyndbg=" func foo +p ; func bar +p"
without repeating "module $modname" 2x in the value
0010-dynamic_debug-update-Documentation-Kconfig.debug.patch
This gets a checkpatch complaint because of long lines in which I
tweaked flags. I chose to disregard this cuz the lines show real
output for ~out-of-tree code.
0011-dynamic_debug-init-with-core_initcall-not-arch_initc.patch
This works-for-me, but may be broken on some arches I dont have to
test on. I dont know why original design used arch_initcall, perhaps
constraints have changed. I also tested with early_initcall, that
also worked.
next reply other threads:[~2012-03-14 23:02 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-14 23:01 jim.cromie [this message]
2012-03-14 23:01 ` [PATCH 01/11] init: trivial tweaks to initcall_levels jim.cromie
2012-03-14 23:01 ` [PATCH 02/11] dynamic_debug: fix leading spaces jim.cromie
2012-03-14 23:01 ` [PATCH 03/11] dynamic_debug: replace if (verbose) pr_info with macro vpr_info jim.cromie
2012-03-14 23:01 ` [PATCH 04/11] dynamic_debug: change ddebug_query core param to dyndbg jim.cromie
2012-03-14 23:02 ` [PATCH 05/11] params: add param-name to parse_one's pr_debug() jim.cromie
2012-03-14 23:02 ` [PATCH 06/11] params: add 3rd arg to option handler callback signature jim.cromie
2012-03-14 23:02 ` [PATCH 07/11] dynamic_debug: make dynamic-debug work during module initialization jim.cromie
2012-03-14 23:02 ` [PATCH 08/11] pnp: if CONFIG_DYNAMIC_DEBUG, use pnp.dyndbg instead of pnp.debug jim.cromie
2012-03-14 23:02 ` [PATCH 09/11] dynamic_debug: add modname arg to exec_query callchain jim.cromie
2012-03-14 23:02 ` [PATCH 10/11] dynamic_debug: update Documentation/*, Kconfig.debug jim.cromie
2012-03-14 23:02 ` [PATCH 11/11] dynamic_debug: init with core_initcall, not arch_initcall jim.cromie
2012-03-19 2:04 ` [00/11] pr_debug during module initialization Rusty Russell
2012-03-19 6:17 ` Jim Cromie
2012-03-19 23:27 ` Rusty Russell
2012-03-23 20:27 ` Jason Baron
2012-03-24 0:12 ` Jim Cromie
2012-03-26 0:24 ` Rusty Russell
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=1331766126-11674-1-git-send-email-jim.cromie@gmail.com \
--to=jim.cromie@gmail.com \
--cc=jbaron@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rusty@rustcorp.com.au \
/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
all inboxes | Powered by JetHome®