* [PATCH net-next] drivers/acpi/acpica/utmisc.c: Use printk extension %pV @ 2010-07-20 5:06 Joe Perches 2010-07-20 5:16 ` David Miller 2010-07-20 16:08 ` Bjorn Helgaas 0 siblings, 2 replies; 5+ messages in thread From: Joe Perches @ 2010-07-20 5:06 UTC (permalink / raw) To: Len Brown; +Cc: linux-acpi, linux-kernel, netdev, David Miller Consolidates the printk messages to a single call so the messages can not be interleaved. Reduces text a bit. $ size drivers/acpi/acpica/utmisc.o.* text data bss dec hex filename 7822 56 1832 9710 25ee drivers/acpi/acpica/utmisc.o.old 7748 56 1736 9540 2544 drivers/acpi/acpica/utmisc.o.new Depends on net-next commit 7db6f5fb65a82af03229eef104dc9899c5eecf33 (vsprintf: Recursive vsnprintf: Add "%pV", struct va_format) Compile tested only Signed-off-by: Joe Perches <joe@perches.com> --- drivers/acpi/acpica/utmisc.c | 78 ++++++++++++++++++++++++++++-------------- 1 files changed, 52 insertions(+), 26 deletions(-) diff --git a/drivers/acpi/acpica/utmisc.c b/drivers/acpi/acpica/utmisc.c index e8d0724..d9a4a64 100644 --- a/drivers/acpi/acpica/utmisc.c +++ b/drivers/acpi/acpica/utmisc.c @@ -53,8 +53,8 @@ ACPI_MODULE_NAME("utmisc") /* * Common suffix for messages */ -#define ACPI_COMMON_MSG_SUFFIX \ - acpi_os_printf(" (%8.8X/%s-%u)\n", ACPI_CA_VERSION, module_name, line_number) +#define ACPI_COMMON_MSG_SUFFIX_FMT " (%8.8X/%s-%u)" +#define ACPI_COMMON_MSG_SUFFIX_ARGS ACPI_CA_VERSION, module_name, line_number /******************************************************************************* * * FUNCTION: acpi_ut_validate_exception @@ -1063,12 +1063,16 @@ void ACPI_INTERNAL_VAR_XFACE acpi_error(const char *module_name, u32 line_number, const char *format, ...) { va_list args; - - acpi_os_printf("ACPI Error: "); + struct va_format vaf; va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + + vaf.fmt = format; + vaf.va = &args; + + acpi_os_printf("ACPI Error: %pV" ACPI_COMMON_MSG_SUFFIX_FMT "\n", + &vaf, ACPI_COMMON_MSG_SUFFIX_ARGS); + va_end(args); } @@ -1077,12 +1081,18 @@ acpi_exception(const char *module_name, u32 line_number, acpi_status status, const char *format, ...) { va_list args; - - acpi_os_printf("ACPI Exception: %s, ", acpi_format_exception(status)); + struct va_format vaf; va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + + vaf.fmt = format; + vaf.va = &args; + + acpi_os_printf("ACPI Exception: %s, %pV" + ACPI_COMMON_MSG_SUFFIX_FMT "\n", + acpi_format_exception(status), &vaf, + ACPI_COMMON_MSG_SUFFIX_ARGS); + va_end(args); } @@ -1090,12 +1100,16 @@ void ACPI_INTERNAL_VAR_XFACE acpi_warning(const char *module_name, u32 line_number, const char *format, ...) { va_list args; - - acpi_os_printf("ACPI Warning: "); + struct va_format vaf; va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + + vaf.fmt = format; + vaf.va = &args; + + acpi_os_printf("ACPI Warning: %pV" ACPI_COMMON_MSG_SUFFIX_FMT "\n", + &vaf, ACPI_COMMON_MSG_SUFFIX_ARGS); + va_end(args); } @@ -1103,12 +1117,15 @@ void ACPI_INTERNAL_VAR_XFACE acpi_info(const char *module_name, u32 line_number, const char *format, ...) { va_list args; - - acpi_os_printf("ACPI: "); + struct va_format vaf; va_start(args, format); - acpi_os_vprintf(format, args); - acpi_os_printf("\n"); + + vaf.fmt = format; + vaf.va = &args; + + acpi_os_printf("ACPI: %pV\n", &vaf); + va_end(args); } @@ -1143,6 +1160,7 @@ acpi_ut_predefined_warning(const char *module_name, u8 node_flags, const char *format, ...) { va_list args; + struct va_format vaf; /* * Warning messages for this method/object will be disabled after the @@ -1152,11 +1170,15 @@ acpi_ut_predefined_warning(const char *module_name, return; } - acpi_os_printf("ACPI Warning for %s: ", pathname); - va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + + vaf.fmt = format; + vaf.va = &args; + + acpi_os_printf("ACPI Warning for %s: %pV" + ACPI_COMMON_MSG_SUFFIX_FMT "\n", + pathname, &vaf, ACPI_COMMON_MSG_SUFFIX_ARGS); + va_end(args); } @@ -1185,6 +1207,7 @@ acpi_ut_predefined_info(const char *module_name, char *pathname, u8 node_flags, const char *format, ...) { va_list args; + struct va_format vaf; /* * Warning messages for this method/object will be disabled after the @@ -1194,10 +1217,13 @@ acpi_ut_predefined_info(const char *module_name, return; } - acpi_os_printf("ACPI Info for %s: ", pathname); - va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + + vaf.fmt = format; + vaf.va = &args; + + acpi_os_printf("ACPI Info for %s: %pV" ACPI_COMMON_MSG_SUFFIX_FMT "\n", + pathname, &vaf, ACPI_COMMON_MSG_SUFFIX_ARGS); + va_end(args); } ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH net-next] drivers/acpi/acpica/utmisc.c: Use printk extension %pV 2010-07-20 5:06 [PATCH net-next] drivers/acpi/acpica/utmisc.c: Use printk extension %pV Joe Perches @ 2010-07-20 5:16 ` David Miller 2010-07-20 5:21 ` Joe Perches 2010-07-20 16:08 ` Bjorn Helgaas 1 sibling, 1 reply; 5+ messages in thread From: David Miller @ 2010-07-20 5:16 UTC (permalink / raw) To: joe; +Cc: lenb, linux-acpi, linux-kernel, netdev From: Joe Perches <joe@perches.com> Date: Mon, 19 Jul 2010 22:06:07 -0700 > Consolidates the printk messages to a single > call so the messages can not be interleaved. > > Reduces text a bit. > > $ size drivers/acpi/acpica/utmisc.o.* > text data bss dec hex filename > 7822 56 1832 9710 25ee drivers/acpi/acpica/utmisc.o.old > 7748 56 1736 9540 2544 drivers/acpi/acpica/utmisc.o.new > > Depends on net-next commit 7db6f5fb65a82af03229eef104dc9899c5eecf33 > (vsprintf: Recursive vsnprintf: Add "%pV", struct va_format) > > Compile tested only > > Signed-off-by: Joe Perches <joe@perches.com> Joe, I really can't keep merging stuff like this into the networking tree. The driver generic bits, since the netdev print macros needed this indirectly and you used %pV primarily for networking stuff, that's fine. But you're going to have to find another way to merge uses outside of networking that you want integrated. Thanks. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH net-next] drivers/acpi/acpica/utmisc.c: Use printk extension %pV 2010-07-20 5:16 ` David Miller @ 2010-07-20 5:21 ` Joe Perches 0 siblings, 0 replies; 5+ messages in thread From: Joe Perches @ 2010-07-20 5:21 UTC (permalink / raw) To: David Miller; +Cc: lenb, linux-acpi, linux-kernel, netdev On Mon, 2010-07-19 at 22:16 -0700, David Miller wrote: > Joe, I really can't keep merging stuff like this into the networking > tree. Oh no worries David. The patch is more like a preliminary notification that at some point after the vsprintf stuff gets merged into Linus' tree after 2.6.36-rc1 that this change to acpi could be useful. > But you're going to have to find another way to merge uses outside of > networking that you want integrated. Hey, I'm a patient guy, there's no hurry. cheers, Joe ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH net-next] drivers/acpi/acpica/utmisc.c: Use printk extension %pV 2010-07-20 5:06 [PATCH net-next] drivers/acpi/acpica/utmisc.c: Use printk extension %pV Joe Perches 2010-07-20 5:16 ` David Miller @ 2010-07-20 16:08 ` Bjorn Helgaas 2010-07-20 18:16 ` Joe Perches 1 sibling, 1 reply; 5+ messages in thread From: Bjorn Helgaas @ 2010-07-20 16:08 UTC (permalink / raw) To: Joe Perches; +Cc: Len Brown, linux-acpi, linux-kernel, netdev, David Miller On Monday, July 19, 2010 11:06:07 pm Joe Perches wrote: > Consolidates the printk messages to a single > call so the messages can not be interleaved. > > Reduces text a bit. > > $ size drivers/acpi/acpica/utmisc.o.* > text data bss dec hex filename > 7822 56 1832 9710 25ee drivers/acpi/acpica/utmisc.o.old > 7748 56 1736 9540 2544 drivers/acpi/acpica/utmisc.o.new > > Depends on net-next commit 7db6f5fb65a82af03229eef104dc9899c5eecf33 > (vsprintf: Recursive vsnprintf: Add "%pV", struct va_format) drivers/acpi/acpica/utmisc.c is part of the ACPI CA and is used in several different OSes, but %pV sounds like a Linux-specific feature, so I don't see how this patch can work. Bjorn ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH net-next] drivers/acpi/acpica/utmisc.c: Use printk extension %pV 2010-07-20 16:08 ` Bjorn Helgaas @ 2010-07-20 18:16 ` Joe Perches 0 siblings, 0 replies; 5+ messages in thread From: Joe Perches @ 2010-07-20 18:16 UTC (permalink / raw) To: Bjorn Helgaas; +Cc: Len Brown, linux-acpi, linux-kernel, netdev, David Miller On Tue, 2010-07-20 at 10:08 -0600, Bjorn Helgaas wrote: > On Monday, July 19, 2010 11:06:07 pm Joe Perches wrote: > > Consolidates the printk messages to a single > > call so the messages can not be interleaved. > > > > Reduces text a bit. > > > > $ size drivers/acpi/acpica/utmisc.o.* > > text data bss dec hex filename > > 7822 56 1832 9710 25ee drivers/acpi/acpica/utmisc.o.old > > 7748 56 1736 9540 2544 drivers/acpi/acpica/utmisc.o.new > > > > Depends on net-next commit 7db6f5fb65a82af03229eef104dc9899c5eecf33 > > (vsprintf: Recursive vsnprintf: Add "%pV", struct va_format) > > drivers/acpi/acpica/utmisc.c is part of the ACPI CA and is used in > several different OSes, but %pV sounds like a Linux-specific feature, > so I don't see how this patch can work. Well, maybe something like this then. It still reduces overall size a little bit, but less. $ size drivers/acpi/acpica/utmisc.o text data bss dec hex filename 7816 56 1784 9656 25b8 drivers/acpi/acpica/utmisc.o I'm not sure that utmisc.c is the right place for an #ifdef though. --- drivers/acpi/acpica/utmisc.c | 76 +++++++++++++++++++++++++++--------------- 1 files changed, 49 insertions(+), 27 deletions(-) diff --git a/drivers/acpi/acpica/utmisc.c b/drivers/acpi/acpica/utmisc.c index e8d0724..874e4d2 100644 --- a/drivers/acpi/acpica/utmisc.c +++ b/drivers/acpi/acpica/utmisc.c @@ -51,10 +51,45 @@ ACPI_MODULE_NAME("utmisc") /* - * Common suffix for messages + * Single line messages, adds a newline after message */ -#define ACPI_COMMON_MSG_SUFFIX \ - acpi_os_printf(" (%8.8X/%s-%u)\n", ACPI_CA_VERSION, module_name, line_number) + +static void ACPI_PRINTF_LIKE(5) +acpi_put_message(const char *module_name, u32 line_number, + const char *fmt, va_list *args, const char *message, ...) +{ +#ifdef __KERNEL__ + va_list message_args; + struct va_format vaf1; + struct va_format vaf2; + + va_start(message_args, message); + vaf1.fmt = message; + vaf1.va = &message_args; + vaf2.fmt = fmt; + vaf2.va = args; + if (module_name) + acpi_os_printf("%pV%pV (%8.8X/%s-%u)\n", + &vaf1, &vaf2, + ACPI_CA_VERSION, module_name, line_number); + else + acpi_os_printf("%pV%pV\n", &vaf1, &vaf2); + va_end(message_args); +#else + va_list message_args; + + va_start(message_args, message); + acpi_os_vprintf(message, message_args); + acpi_os_vprintf(fmt, *args); + if (module_name) + acpi_os_printf(" (%8.8X/%s-%u)\n", + ACPI_CA_VERSION, module_name, line_number); + else + acpi_os_printf("\n"); + va_end(message_args); +#endif +} + /******************************************************************************* * * FUNCTION: acpi_ut_validate_exception @@ -1064,11 +1099,9 @@ acpi_error(const char *module_name, u32 line_number, const char *format, ...) { va_list args; - acpi_os_printf("ACPI Error: "); - va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + acpi_put_message(module_name, line_number, format, &args, + "ACPI Error: "); va_end(args); } @@ -1078,11 +1111,9 @@ acpi_exception(const char *module_name, { va_list args; - acpi_os_printf("ACPI Exception: %s, ", acpi_format_exception(status)); - va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + acpi_put_message(module_name, line_number, format, &args, + "ACPI Exception: %s, ", acpi_format_exception(status)); va_end(args); } @@ -1091,11 +1122,9 @@ acpi_warning(const char *module_name, u32 line_number, const char *format, ...) { va_list args; - acpi_os_printf("ACPI Warning: "); - va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + acpi_put_message(module_name, line_number, format, &args, + "ACPI Warning: "); va_end(args); } @@ -1104,11 +1133,8 @@ acpi_info(const char *module_name, u32 line_number, const char *format, ...) { va_list args; - acpi_os_printf("ACPI: "); - va_start(args, format); - acpi_os_vprintf(format, args); - acpi_os_printf("\n"); + acpi_put_message(NULL, 0, format, &args, "ACPI: "); va_end(args); } @@ -1152,11 +1178,9 @@ acpi_ut_predefined_warning(const char *module_name, return; } - acpi_os_printf("ACPI Warning for %s: ", pathname); - va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + acpi_put_message(module_name, line_number, format, &args, + "ACPI Warning for %s: ", pathname); va_end(args); } @@ -1194,10 +1218,8 @@ acpi_ut_predefined_info(const char *module_name, return; } - acpi_os_printf("ACPI Info for %s: ", pathname); - va_start(args, format); - acpi_os_vprintf(format, args); - ACPI_COMMON_MSG_SUFFIX; + acpi_put_message(module_name, line_number, format, &args, + "ACPI Info for %s: ", pathname); va_end(args); } ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-07-20 18:16 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2010-07-20 5:06 [PATCH net-next] drivers/acpi/acpica/utmisc.c: Use printk extension %pV Joe Perches 2010-07-20 5:16 ` David Miller 2010-07-20 5:21 ` Joe Perches 2010-07-20 16:08 ` Bjorn Helgaas 2010-07-20 18:16 ` Joe Perches
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).