diff options
author | Joe Perches <joe@perches.com> | 2010-06-26 21:02:33 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-07-04 13:40:17 -0400 |
commit | 7db6f5fb65a82af03229eef104dc9899c5eecf33 (patch) | |
tree | fb7a83297d8bf2680df9616ebf58bec89229ff3b | |
parent | e490c1defec4236a6a131fe2d13bf7ba787c02f8 (diff) |
vsprintf: Recursive vsnprintf: Add "%pV", struct va_format
Add the ability to print a format and va_list from a structure pointer
Allows __dev_printk to be implemented as a single printk while
minimizing string space duplication.
%pV should not be used without some mechanism to verify the
format and argument use ala __attribute__(format (printf(...))).
Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Greg Kroah-Hartman <gregkh@suse.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | include/linux/kernel.h | 5 | ||||
-rw-r--r-- | lib/vsprintf.c | 9 |
2 files changed, 14 insertions, 0 deletions
diff --git a/include/linux/kernel.h b/include/linux/kernel.h index 8317ec4b9f3b..01dfc05ef4ac 100644 --- a/include/linux/kernel.h +++ b/include/linux/kernel.h | |||
@@ -171,6 +171,11 @@ static inline void might_fault(void) | |||
171 | } | 171 | } |
172 | #endif | 172 | #endif |
173 | 173 | ||
174 | struct va_format { | ||
175 | const char *fmt; | ||
176 | va_list *va; | ||
177 | }; | ||
178 | |||
174 | extern struct atomic_notifier_head panic_notifier_list; | 179 | extern struct atomic_notifier_head panic_notifier_list; |
175 | extern long (*panic_blink)(long time); | 180 | extern long (*panic_blink)(long time); |
176 | NORET_TYPE void panic(const char * fmt, ...) | 181 | NORET_TYPE void panic(const char * fmt, ...) |
diff --git a/lib/vsprintf.c b/lib/vsprintf.c index b8a2f549ab0e..4ee19d0d3910 100644 --- a/lib/vsprintf.c +++ b/lib/vsprintf.c | |||
@@ -980,6 +980,11 @@ char *uuid_string(char *buf, char *end, const u8 *addr, | |||
980 | * [0][1][2][3]-[4][5]-[6][7]-[8][9]-[10][11][12][13][14][15] | 980 | * [0][1][2][3]-[4][5]-[6][7]-[8][9]-[10][11][12][13][14][15] |
981 | * little endian output byte order is: | 981 | * little endian output byte order is: |
982 | * [3][2][1][0]-[5][4]-[7][6]-[8][9]-[10][11][12][13][14][15] | 982 | * [3][2][1][0]-[5][4]-[7][6]-[8][9]-[10][11][12][13][14][15] |
983 | * - 'V' For a struct va_format which contains a format string * and va_list *, | ||
984 | * call vsnprintf(->format, *->va_list). | ||
985 | * Implements a "recursive vsnprintf". | ||
986 | * Do not use this feature without some mechanism to verify the | ||
987 | * correctness of the format string and va_list arguments. | ||
983 | * | 988 | * |
984 | * Note: The difference between 'S' and 'F' is that on ia64 and ppc64 | 989 | * Note: The difference between 'S' and 'F' is that on ia64 and ppc64 |
985 | * function pointers are really function descriptors, which contain a | 990 | * function pointers are really function descriptors, which contain a |
@@ -1025,6 +1030,10 @@ char *pointer(const char *fmt, char *buf, char *end, void *ptr, | |||
1025 | break; | 1030 | break; |
1026 | case 'U': | 1031 | case 'U': |
1027 | return uuid_string(buf, end, ptr, spec, fmt); | 1032 | return uuid_string(buf, end, ptr, spec, fmt); |
1033 | case 'V': | ||
1034 | return buf + vsnprintf(buf, end - buf, | ||
1035 | ((struct va_format *)ptr)->fmt, | ||
1036 | *(((struct va_format *)ptr)->va)); | ||
1028 | } | 1037 | } |
1029 | spec.flags |= SMALL; | 1038 | spec.flags |= SMALL; |
1030 | if (spec.field_width == -1) { | 1039 | if (spec.field_width == -1) { |