Commit dba39448 authored by Steven Rostedt (Red Hat)'s avatar Steven Rostedt (Red Hat) Committed by Steven Rostedt

tracing: Remove return values of most trace_seq_*() functions

The trace_seq_printf() and friends are used to store strings into a buffer
that can be passed around from function to function. If the trace_seq buffer
fills up, it will not print any more. The return values were somewhat
inconsistant and using trace_seq_has_overflowed() was a better way to know
if the write to the trace_seq buffer succeeded or not.

Now that all users have removed reading the return value of the printf()
type functions, they can safely return void and keep future users of them
from reading the inconsistent values as well.

Link: http://lkml.kernel.org/r/20141114011411.992510720@goodmis.orgSigned-off-by: default avatarSteven Rostedt <rostedt@goodmis.org>
parent 183742f0
...@@ -57,40 +57,37 @@ static inline bool trace_seq_has_overflowed(struct trace_seq *s) ...@@ -57,40 +57,37 @@ static inline bool trace_seq_has_overflowed(struct trace_seq *s)
*/ */
#ifdef CONFIG_TRACING #ifdef CONFIG_TRACING
extern __printf(2, 3) extern __printf(2, 3)
int trace_seq_printf(struct trace_seq *s, const char *fmt, ...); void trace_seq_printf(struct trace_seq *s, const char *fmt, ...);
extern __printf(2, 0) extern __printf(2, 0)
int trace_seq_vprintf(struct trace_seq *s, const char *fmt, va_list args); void trace_seq_vprintf(struct trace_seq *s, const char *fmt, va_list args);
extern int extern void
trace_seq_bprintf(struct trace_seq *s, const char *fmt, const u32 *binary); trace_seq_bprintf(struct trace_seq *s, const char *fmt, const u32 *binary);
extern int trace_print_seq(struct seq_file *m, struct trace_seq *s); extern int trace_print_seq(struct seq_file *m, struct trace_seq *s);
extern int trace_seq_to_user(struct trace_seq *s, char __user *ubuf, extern int trace_seq_to_user(struct trace_seq *s, char __user *ubuf,
int cnt); int cnt);
extern int trace_seq_puts(struct trace_seq *s, const char *str); extern void trace_seq_puts(struct trace_seq *s, const char *str);
extern int trace_seq_putc(struct trace_seq *s, unsigned char c); extern void trace_seq_putc(struct trace_seq *s, unsigned char c);
extern int trace_seq_putmem(struct trace_seq *s, const void *mem, unsigned int len); extern void trace_seq_putmem(struct trace_seq *s, const void *mem, unsigned int len);
extern int trace_seq_putmem_hex(struct trace_seq *s, const void *mem, extern void trace_seq_putmem_hex(struct trace_seq *s, const void *mem,
unsigned int len); unsigned int len);
extern int trace_seq_path(struct trace_seq *s, const struct path *path); extern int trace_seq_path(struct trace_seq *s, const struct path *path);
extern int trace_seq_bitmask(struct trace_seq *s, const unsigned long *maskp, extern void trace_seq_bitmask(struct trace_seq *s, const unsigned long *maskp,
int nmaskbits); int nmaskbits);
#else /* CONFIG_TRACING */ #else /* CONFIG_TRACING */
static inline int trace_seq_printf(struct trace_seq *s, const char *fmt, ...) static inline void trace_seq_printf(struct trace_seq *s, const char *fmt, ...)
{ {
return 0;
} }
static inline int static inline void
trace_seq_bprintf(struct trace_seq *s, const char *fmt, const u32 *binary) trace_seq_bprintf(struct trace_seq *s, const char *fmt, const u32 *binary)
{ {
return 0;
} }
static inline int static inline void
trace_seq_bitmask(struct trace_seq *s, const unsigned long *maskp, trace_seq_bitmask(struct trace_seq *s, const unsigned long *maskp,
int nmaskbits) int nmaskbits)
{ {
return 0;
} }
static inline int trace_print_seq(struct seq_file *m, struct trace_seq *s) static inline int trace_print_seq(struct seq_file *m, struct trace_seq *s)
...@@ -102,23 +99,19 @@ static inline int trace_seq_to_user(struct trace_seq *s, char __user *ubuf, ...@@ -102,23 +99,19 @@ static inline int trace_seq_to_user(struct trace_seq *s, char __user *ubuf,
{ {
return 0; return 0;
} }
static inline int trace_seq_puts(struct trace_seq *s, const char *str) static inline void trace_seq_puts(struct trace_seq *s, const char *str)
{ {
return 0;
} }
static inline int trace_seq_putc(struct trace_seq *s, unsigned char c) static inline void trace_seq_putc(struct trace_seq *s, unsigned char c)
{ {
return 0;
} }
static inline int static inline void
trace_seq_putmem(struct trace_seq *s, const void *mem, unsigned int len) trace_seq_putmem(struct trace_seq *s, const void *mem, unsigned int len)
{ {
return 0;
} }
static inline int trace_seq_putmem_hex(struct trace_seq *s, const void *mem, static inline void trace_seq_putmem_hex(struct trace_seq *s, const void *mem,
unsigned int len) unsigned int len)
{ {
return 0;
} }
static inline int trace_seq_path(struct trace_seq *s, const struct path *path) static inline int trace_seq_path(struct trace_seq *s, const struct path *path)
{ {
......
...@@ -69,20 +69,15 @@ int trace_print_seq(struct seq_file *m, struct trace_seq *s) ...@@ -69,20 +69,15 @@ int trace_print_seq(struct seq_file *m, struct trace_seq *s)
* trace_seq_printf() is used to store strings into a special * trace_seq_printf() is used to store strings into a special
* buffer (@s). Then the output may be either used by * buffer (@s). Then the output may be either used by
* the sequencer or pulled into another buffer. * the sequencer or pulled into another buffer.
*
* Returns 1 if we successfully written all the contents to
* the buffer.
* Returns 0 if we the length to write is bigger than the
* reserved buffer space. In this case, nothing gets written.
*/ */
int trace_seq_printf(struct trace_seq *s, const char *fmt, ...) void trace_seq_printf(struct trace_seq *s, const char *fmt, ...)
{ {
unsigned int len = TRACE_SEQ_BUF_LEFT(s); unsigned int len = TRACE_SEQ_BUF_LEFT(s);
va_list ap; va_list ap;
int ret; int ret;
if (s->full || !len) if (s->full || !len)
return 0; return;
va_start(ap, fmt); va_start(ap, fmt);
ret = vsnprintf(s->buffer + s->len, len, fmt, ap); ret = vsnprintf(s->buffer + s->len, len, fmt, ap);
...@@ -91,12 +86,10 @@ int trace_seq_printf(struct trace_seq *s, const char *fmt, ...) ...@@ -91,12 +86,10 @@ int trace_seq_printf(struct trace_seq *s, const char *fmt, ...)
/* If we can't write it all, don't bother writing anything */ /* If we can't write it all, don't bother writing anything */
if (ret >= len) { if (ret >= len) {
s->full = 1; s->full = 1;
return 0; return;
} }
s->len += ret; s->len += ret;
return 1;
} }
EXPORT_SYMBOL_GPL(trace_seq_printf); EXPORT_SYMBOL_GPL(trace_seq_printf);
...@@ -107,25 +100,18 @@ EXPORT_SYMBOL_GPL(trace_seq_printf); ...@@ -107,25 +100,18 @@ EXPORT_SYMBOL_GPL(trace_seq_printf);
* @nmaskbits: The number of bits that are valid in @maskp * @nmaskbits: The number of bits that are valid in @maskp
* *
* Writes a ASCII representation of a bitmask string into @s. * Writes a ASCII representation of a bitmask string into @s.
*
* Returns 1 if we successfully written all the contents to
* the buffer.
* Returns 0 if we the length to write is bigger than the
* reserved buffer space. In this case, nothing gets written.
*/ */
int trace_seq_bitmask(struct trace_seq *s, const unsigned long *maskp, void trace_seq_bitmask(struct trace_seq *s, const unsigned long *maskp,
int nmaskbits) int nmaskbits)
{ {
unsigned int len = TRACE_SEQ_BUF_LEFT(s); unsigned int len = TRACE_SEQ_BUF_LEFT(s);
int ret; int ret;
if (s->full || !len) if (s->full || !len)
return 0; return;
ret = bitmap_scnprintf(s->buffer + s->len, len, maskp, nmaskbits); ret = bitmap_scnprintf(s->buffer + s->len, len, maskp, nmaskbits);
s->len += ret; s->len += ret;
return 1;
} }
EXPORT_SYMBOL_GPL(trace_seq_bitmask); EXPORT_SYMBOL_GPL(trace_seq_bitmask);
...@@ -139,28 +125,24 @@ EXPORT_SYMBOL_GPL(trace_seq_bitmask); ...@@ -139,28 +125,24 @@ EXPORT_SYMBOL_GPL(trace_seq_bitmask);
* trace_seq_printf is used to store strings into a special * trace_seq_printf is used to store strings into a special
* buffer (@s). Then the output may be either used by * buffer (@s). Then the output may be either used by
* the sequencer or pulled into another buffer. * the sequencer or pulled into another buffer.
*
* Returns how much it wrote to the buffer.
*/ */
int trace_seq_vprintf(struct trace_seq *s, const char *fmt, va_list args) void trace_seq_vprintf(struct trace_seq *s, const char *fmt, va_list args)
{ {
unsigned int len = TRACE_SEQ_BUF_LEFT(s); unsigned int len = TRACE_SEQ_BUF_LEFT(s);
int ret; int ret;
if (s->full || !len) if (s->full || !len)
return 0; return;
ret = vsnprintf(s->buffer + s->len, len, fmt, args); ret = vsnprintf(s->buffer + s->len, len, fmt, args);
/* If we can't write it all, don't bother writing anything */ /* If we can't write it all, don't bother writing anything */
if (ret >= len) { if (ret >= len) {
s->full = 1; s->full = 1;
return 0; return;
} }
s->len += ret; s->len += ret;
return len;
} }
EXPORT_SYMBOL_GPL(trace_seq_vprintf); EXPORT_SYMBOL_GPL(trace_seq_vprintf);
...@@ -178,28 +160,24 @@ EXPORT_SYMBOL_GPL(trace_seq_vprintf); ...@@ -178,28 +160,24 @@ EXPORT_SYMBOL_GPL(trace_seq_vprintf);
* *
* This function will take the format and the binary array and finish * This function will take the format and the binary array and finish
* the conversion into the ASCII string within the buffer. * the conversion into the ASCII string within the buffer.
*
* Returns how much it wrote to the buffer.
*/ */
int trace_seq_bprintf(struct trace_seq *s, const char *fmt, const u32 *binary) void trace_seq_bprintf(struct trace_seq *s, const char *fmt, const u32 *binary)
{ {
unsigned int len = TRACE_SEQ_BUF_LEFT(s); unsigned int len = TRACE_SEQ_BUF_LEFT(s);
int ret; int ret;
if (s->full || !len) if (s->full || !len)
return 0; return;
ret = bstr_printf(s->buffer + s->len, len, fmt, binary); ret = bstr_printf(s->buffer + s->len, len, fmt, binary);
/* If we can't write it all, don't bother writing anything */ /* If we can't write it all, don't bother writing anything */
if (ret >= len) { if (ret >= len) {
s->full = 1; s->full = 1;
return 0; return;
} }
s->len += ret; s->len += ret;
return len;
} }
EXPORT_SYMBOL_GPL(trace_seq_bprintf); EXPORT_SYMBOL_GPL(trace_seq_bprintf);
...@@ -212,25 +190,21 @@ EXPORT_SYMBOL_GPL(trace_seq_bprintf); ...@@ -212,25 +190,21 @@ EXPORT_SYMBOL_GPL(trace_seq_bprintf);
* copy to user routines. This function records a simple string * copy to user routines. This function records a simple string
* into a special buffer (@s) for later retrieval by a sequencer * into a special buffer (@s) for later retrieval by a sequencer
* or other mechanism. * or other mechanism.
*
* Returns how much it wrote to the buffer.
*/ */
int trace_seq_puts(struct trace_seq *s, const char *str) void trace_seq_puts(struct trace_seq *s, const char *str)
{ {
unsigned int len = strlen(str); unsigned int len = strlen(str);
if (s->full) if (s->full)
return 0; return;
if (len > TRACE_SEQ_BUF_LEFT(s)) { if (len > TRACE_SEQ_BUF_LEFT(s)) {
s->full = 1; s->full = 1;
return 0; return;
} }
memcpy(s->buffer + s->len, str, len); memcpy(s->buffer + s->len, str, len);
s->len += len; s->len += len;
return len;
} }
EXPORT_SYMBOL_GPL(trace_seq_puts); EXPORT_SYMBOL_GPL(trace_seq_puts);
...@@ -243,22 +217,18 @@ EXPORT_SYMBOL_GPL(trace_seq_puts); ...@@ -243,22 +217,18 @@ EXPORT_SYMBOL_GPL(trace_seq_puts);
* copy to user routines. This function records a simple charater * copy to user routines. This function records a simple charater
* into a special buffer (@s) for later retrieval by a sequencer * into a special buffer (@s) for later retrieval by a sequencer
* or other mechanism. * or other mechanism.
*
* Returns how much it wrote to the buffer.
*/ */
int trace_seq_putc(struct trace_seq *s, unsigned char c) void trace_seq_putc(struct trace_seq *s, unsigned char c)
{ {
if (s->full) if (s->full)
return 0; return;
if (TRACE_SEQ_BUF_LEFT(s) < 1) { if (TRACE_SEQ_BUF_LEFT(s) < 1) {
s->full = 1; s->full = 1;
return 0; return;
} }
s->buffer[s->len++] = c; s->buffer[s->len++] = c;
return 1;
} }
EXPORT_SYMBOL_GPL(trace_seq_putc); EXPORT_SYMBOL_GPL(trace_seq_putc);
...@@ -271,23 +241,19 @@ EXPORT_SYMBOL_GPL(trace_seq_putc); ...@@ -271,23 +241,19 @@ EXPORT_SYMBOL_GPL(trace_seq_putc);
* There may be cases where raw memory needs to be written into the * There may be cases where raw memory needs to be written into the
* buffer and a strcpy() would not work. Using this function allows * buffer and a strcpy() would not work. Using this function allows
* for such cases. * for such cases.
*
* Returns how much it wrote to the buffer.
*/ */
int trace_seq_putmem(struct trace_seq *s, const void *mem, unsigned int len) void trace_seq_putmem(struct trace_seq *s, const void *mem, unsigned int len)
{ {
if (s->full) if (s->full)
return 0; return;
if (len > TRACE_SEQ_BUF_LEFT(s)) { if (len > TRACE_SEQ_BUF_LEFT(s)) {
s->full = 1; s->full = 1;
return 0; return;
} }
memcpy(s->buffer + s->len, mem, len); memcpy(s->buffer + s->len, mem, len);
s->len += len; s->len += len;
return len;
} }
EXPORT_SYMBOL_GPL(trace_seq_putmem); EXPORT_SYMBOL_GPL(trace_seq_putmem);
...@@ -303,20 +269,17 @@ EXPORT_SYMBOL_GPL(trace_seq_putmem); ...@@ -303,20 +269,17 @@ EXPORT_SYMBOL_GPL(trace_seq_putmem);
* This is similar to trace_seq_putmem() except instead of just copying the * This is similar to trace_seq_putmem() except instead of just copying the
* raw memory into the buffer it writes its ASCII representation of it * raw memory into the buffer it writes its ASCII representation of it
* in hex characters. * in hex characters.
*
* Returns how much it wrote to the buffer.
*/ */
int trace_seq_putmem_hex(struct trace_seq *s, const void *mem, void trace_seq_putmem_hex(struct trace_seq *s, const void *mem,
unsigned int len) unsigned int len)
{ {
unsigned char hex[HEX_CHARS]; unsigned char hex[HEX_CHARS];
const unsigned char *data = mem; const unsigned char *data = mem;
unsigned int start_len; unsigned int start_len;
int i, j; int i, j;
int cnt = 0;
if (s->full) if (s->full)
return 0; return;
while (len) { while (len) {
start_len = min(len, HEX_CHARS - 1); start_len = min(len, HEX_CHARS - 1);
...@@ -335,9 +298,8 @@ int trace_seq_putmem_hex(struct trace_seq *s, const void *mem, ...@@ -335,9 +298,8 @@ int trace_seq_putmem_hex(struct trace_seq *s, const void *mem,
len -= j / 2; len -= j / 2;
hex[j++] = ' '; hex[j++] = ' ';
cnt += trace_seq_putmem(s, hex, j); trace_seq_putmem(s, hex, j);
} }
return cnt;
} }
EXPORT_SYMBOL_GPL(trace_seq_putmem_hex); EXPORT_SYMBOL_GPL(trace_seq_putmem_hex);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment