From: Joe Perches <joe@perches.com>
To: LKML <linux-kernel@vger.kernel.org>
Cc: Andrew Morton <akpm@linux-foundation.org>,
Jean Delvare <khali@linux-fr.org>,
Guenter Roeck <guenter.roeck@ericsson.com>
Subject: [RFC PATCH] include/linux/kernel.h: Add config option for pr_fmt(fmt)
Date: Thu, 21 Oct 2010 19:19:42 -0700 [thread overview]
Message-ID: <1287713982.16920.39.camel@Joe-Laptop> (raw)
Change the default #define pr_fmt(fmt) from:
- #define pr_fmt(fmt) fmt
to:
- #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
This will standard use of prefixes and prevent the
addition of new #defines when using pr_<level>.
Adds a config option to use the old style if desired.
This adds prefixes to 326 strings in an x86 defconfig.
Broken out by subsystem/module, the prefixes are:
10 acpi
2 act_api
2 af_inet
6 af_packet
22 apic
1 apic_noop
2 blktrace
1 boot
4 calibrate
1 centaur
2 cleanup
1 common
2 dev
1 devinet
17 edac_mce_amd
1 ehci_hcd
1 fb
13 generic
9 hibernate
2 i2c_algo_bit
1 io_delay
1 ip6table_filter
1 iptable_filter
2 iptable_nat
3 ipv6
2 irq
8 kexec
5 libphy
19 main
2 manage
11 mce
2 mount
1 msr
5 nf_conntrack_ipv4
7 nf_conntrack_ipv6
4 nf_conntrack_netlink
2 nfnetlink
2 nfnetlink_log
1 ohci_hcd
5 oom_kill
1 pcieportdrv
9 percpu
21 perf_event
1 pid
2 platform
2 probe_32
1 rtc_cmos
2 setup
1 setup_bus
1 skbuff
1 sleep
8 smpboot
4 snapshot
3 suspend
6 swap
1 tcp_ipv4
8 trace
14 trace_events
26 trace_kprobe
3 trace_sched_switch
1 trace_stat
3 tsc_sync
10 usbcore
2 usb_storage
10 vgaarb
1 xt_state
Some of these add useful info, others are redundant
and can and will be trivially fixed by adding either
a new line to #define pr_fmt(fmt) fmt to the sources
or by removing the internal prefix strings in the
existing pr_<level> calls.
Signed-off-by: Joe Perches <joe@perches.com>
---
include/linux/kernel.h | 4 ++++
init/Kconfig | 10 ++++++++++
2 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/include/linux/kernel.h b/include/linux/kernel.h
index 1759ba5..9cb2e8f 100644
--- a/include/linux/kernel.h
+++ b/include/linux/kernel.h
@@ -409,8 +409,12 @@ static inline char *pack_hex_byte(char *buf, u8
byte)
extern int hex_to_bin(char ch);
#ifndef pr_fmt
+#ifdef CONFIG_PR_FMT_IS_KBUILD_MODNAME
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+#else
#define pr_fmt(fmt) fmt
#endif
+#endif
#define pr_emerg(fmt, ...) \
printk(KERN_EMERG pr_fmt(fmt), ##__VA_ARGS__)
diff --git a/init/Kconfig b/init/Kconfig
index 7b920aa..a406d1c 100644
--- a/init/Kconfig
+++ b/init/Kconfig
@@ -897,6 +897,16 @@ config PRINTK
very difficult to diagnose system problems, saying N here is
strongly discouraged.
+config PR_FMT_IS_KBUILD_MODNAME
+ default y
+ depends on PRINTK
+ bool "Use KBUILD_MODNAME as prefix for pr_<level>"
+ help
+ This option sets the default pr_fmt to KBUILD_MODNAME.
+ This will prefix all pr_<level> logging message calls with the
+ build system defined value unless there is an existing define
+ of pr_fmt in the source code.
+
config BUG
bool "BUG() support" if EMBEDDED
default y
next reply other threads:[~2010-10-22 2:19 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-22 2:19 Joe Perches [this message]
[not found] ` <alpine.LFD.2.00.1010220131320.7016@localhost.localdomain>
2010-10-22 22:23 ` [PATCH] drivers/acpi: Add and use pr_fmt(fmt) Joe Perches
2010-10-26 9:03 ` [RFC PATCH] include/linux/kernel.h: Add config option for pr_fmt(fmt) Jean Delvare
2010-10-27 17:41 ` Joe Perches
2010-10-28 4:28 ` Guenter Roeck
2010-10-28 7:35 ` Jean Delvare
2010-10-28 7:54 ` Joe Perches
2010-10-28 8:43 ` Jean Delvare
2010-10-29 22:10 ` Joe Perches
2010-11-08 12:33 ` Jean Delvare
2010-11-08 16:55 ` Guenter Roeck
2010-11-08 17:16 ` Joe Perches
2010-11-09 3:07 ` Guenter Roeck
2010-11-09 8:42 ` Jean Delvare
2010-11-09 16:16 ` Guenter Roeck
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=1287713982.16920.39.camel@Joe-Laptop \
--to=joe@perches.com \
--cc=akpm@linux-foundation.org \
--cc=guenter.roeck@ericsson.com \
--cc=khali@linux-fr.org \
--cc=linux-kernel@vger.kernel.org \
/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