Re: + remove-current-defines-and-uses-of-pr_err-add-pr_emerg.patchadded to -mm tree

From: Jan Engelhardt
Date: Sat Aug 04 2007 - 05:43:43 EST



On Aug 3 2007 15:16, Joe Perches wrote:
>On Fri, 2007-08-03 at 17:05 +0200, Jean Delvare wrote:
>> Fine with me, but this first patch should still be correct per se.
>
>Add new pr_<level> printk(KERN_<level> fmt "\n", ##arg) to kernel.h
>pr_info and pr_debug are unchanged
>Remove local pr_err #defines
>Convert current uses of pr_err
>
>Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>
>
>diff --git a/include/linux/kernel.h b/include/linux/kernel.h
>index 4300bb4..6447072 100644
>--- a/include/linux/kernel.h
>+++ b/include/linux/kernel.h
>@@ -229,20 +229,24 @@ extern void print_hex_dump_bytes(const char *prefix_str, int prefix_type,
> void *buf, size_t len);
> #define hex_asc(x) "0123456789abcdef"[x]
>
>+#define pr_emerg(fmt, arg...) printk(KERN_EMERG fmt "\n", ##arg)
>+#define pr_alert(fmt, arg...) printk(KERN_ALERT fmt "\n", ##arg)
>+#define pr_crit(fmt, arg...) printk(KERN_CRIT fmt "\n", ##arg)
>+#define pr_err(fmt, arg...) printk(KERN_ERR fmt "\n", ##arg)
>+#define pr_warn(fmt, arg...) printk(KERN_WARNING fmt "\n", ##arg)
>+#define pr_notice(fmt, arg...) printk(KERN_NOTICE fmt "\n", ##arg)
>+#define pr_info(fmt, arg...) printk(KERN_INFO fmt, ##arg)
>+

Ugh. What do we have printk for then? I do not like this.
For pr_debug() it makes sense because its semantics change with
-DDEBUG and -UDEBUG, but for these pr_()s it does not seem so.


> if (omap_request_gpio(15)) {
>- pr_err("can't request GPIO 15\n");
>+ pr_err(PFX "can't request GPIO 15");
> omap_free_gpio(14);
> r = -1;
> goto exit;
>
>
>-
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to majordomo@xxxxxxxxxxxxxxx
>More majordomo info at http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at http://www.tux.org/lkml/
>

Jan
--
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/