Make these generic names more specific to the printk subsystem and allow these variables and functions to become non-static.
Rename log_text to printk_log_text. Rename log_dict to printk_log_dict. Rename log_from_idx to printk_log_from_idx. Rename log_next to printk_log_next. Rename log_store to printk_log_store. Signed-off-by: Joe Perches <j...@perches.com> --- kernel/printk/printk.c | 100 ++++++++++++++++++++++++------------------------ 1 files changed, 50 insertions(+), 50 deletions(-) diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index 602a1ab..992c064 100644 --- a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -252,19 +252,19 @@ static u32 printk_log_buf_len = __PRINTK_LOG_BUF_LEN; static volatile unsigned int logbuf_cpu = UINT_MAX; /* human readable text of the record */ -static char *log_text(const struct printk_log *msg) +static char *printk_log_text(const struct printk_log *msg) { return (char *)msg + sizeof(struct printk_log); } /* optional key/value pair dictionary attached to the record */ -static char *log_dict(const struct printk_log *msg) +static char *printk_log_dict(const struct printk_log *msg) { return (char *)msg + sizeof(struct printk_log) + msg->text_len; } /* get record by index; idx must point to valid msg */ -static struct printk_log *log_from_idx(u32 idx) +static struct printk_log *printk_log_from_idx(u32 idx) { struct printk_log *msg = (struct printk_log *)(printk_log_buf + idx); @@ -278,7 +278,7 @@ static struct printk_log *log_from_idx(u32 idx) } /* get next record; idx must point to valid msg */ -static u32 log_next(u32 idx) +static u32 printk_log_next(u32 idx) { struct printk_log *msg = (struct printk_log *)(printk_log_buf + idx); @@ -296,10 +296,10 @@ static u32 log_next(u32 idx) } /* insert record into the buffer, discard old ones, update heads */ -static void log_store(int facility, int level, - enum log_flags flags, u64 ts_nsec, - const char *dict, u16 dict_len, - const char *text, u16 text_len) +static void printk_log_store(int facility, int level, + enum log_flags flags, u64 ts_nsec, + const char *dict, u16 dict_len, + const char *text, u16 text_len) { struct printk_log *msg; u32 size, pad_len; @@ -321,7 +321,7 @@ static void log_store(int facility, int level, break; /* drop old messages until we have enough contiuous space */ - printk_log_first_idx = log_next(printk_log_first_idx); + printk_log_first_idx = printk_log_next(printk_log_first_idx); printk_log_first_seq++; } @@ -337,9 +337,9 @@ static void log_store(int facility, int level, /* fill message */ msg = (struct printk_log *)(printk_log_buf + printk_log_next_idx); - memcpy(log_text(msg), text, text_len); + memcpy(printk_log_text(msg), text, text_len); msg->text_len = text_len; - memcpy(log_dict(msg), dict, dict_len); + memcpy(printk_log_dict(msg), dict, dict_len); msg->dict_len = dict_len; msg->facility = facility; msg->level = level & 7; @@ -348,7 +348,7 @@ static void log_store(int facility, int level, msg->ts_nsec = ts_nsec; else msg->ts_nsec = local_clock(); - memset(log_dict(msg) + dict_len, 0, pad_len); + memset(printk_log_dict(msg) + dict_len, 0, pad_len); msg->len = sizeof(struct printk_log) + text_len + dict_len + pad_len; /* insert message */ @@ -463,7 +463,7 @@ static ssize_t devkmsg_read(struct file *file, char __user *buf, goto out; } - msg = log_from_idx(user->idx); + msg = printk_log_from_idx(user->idx); ts_usec = msg->ts_nsec; do_div(ts_usec, 1000); @@ -488,7 +488,7 @@ static ssize_t devkmsg_read(struct file *file, char __user *buf, /* escape non-printable characters */ for (i = 0; i < msg->text_len; i++) { - unsigned char c = log_text(msg)[i]; + unsigned char c = printk_log_text(msg)[i]; if (c < ' ' || c >= 127 || c == '\\') len += sprintf(user->buf + len, "\\x%02x", c); @@ -501,7 +501,7 @@ static ssize_t devkmsg_read(struct file *file, char __user *buf, bool line = true; for (i = 0; i < msg->dict_len; i++) { - unsigned char c = log_dict(msg)[i]; + unsigned char c = printk_log_dict(msg)[i]; if (line) { user->buf[len++] = ' '; @@ -524,7 +524,7 @@ static ssize_t devkmsg_read(struct file *file, char __user *buf, user->buf[len++] = '\n'; } - user->idx = log_next(user->idx); + user->idx = printk_log_next(user->idx); user->seq++; raw_spin_unlock_irq(&logbuf_lock); @@ -875,7 +875,7 @@ static size_t print_prefix(const struct printk_log *msg, bool syslog, char *buf) static size_t msg_print_text(const struct printk_log *msg, enum log_flags prev, bool syslog, char *buf, size_t size) { - const char *text = log_text(msg); + const char *text = printk_log_text(msg); size_t text_size = msg->text_len; bool prefix = true; bool newline = true; @@ -959,12 +959,12 @@ static int syslog_print(char __user *buf, int size) } skip = syslog_partial; - msg = log_from_idx(syslog_idx); + msg = printk_log_from_idx(syslog_idx); n = msg_print_text(msg, syslog_prev, true, text, LOG_LINE_MAX + PREFIX_MAX); if (n - syslog_partial <= size) { /* message fits into buffer, move forward */ - syslog_idx = log_next(syslog_idx); + syslog_idx = printk_log_next(syslog_idx); syslog_seq++; syslog_prev = msg->flags; n -= syslog_partial; @@ -1025,11 +1025,11 @@ static int syslog_print_all(char __user *buf, int size, bool clear) idx = clear_idx; prev = 0; while (seq < printk_log_next_seq) { - struct printk_log *msg = log_from_idx(idx); + struct printk_log *msg = printk_log_from_idx(idx); len += msg_print_text(msg, prev, true, NULL, 0); prev = msg->flags; - idx = log_next(idx); + idx = printk_log_next(idx); seq++; } @@ -1038,11 +1038,11 @@ static int syslog_print_all(char __user *buf, int size, bool clear) idx = clear_idx; prev = 0; while (len > size && seq < printk_log_next_seq) { - struct printk_log *msg = log_from_idx(idx); + struct printk_log *msg = printk_log_from_idx(idx); len -= msg_print_text(msg, prev, true, NULL, 0); prev = msg->flags; - idx = log_next(idx); + idx = printk_log_next(idx); seq++; } @@ -1052,7 +1052,7 @@ static int syslog_print_all(char __user *buf, int size, bool clear) len = 0; prev = 0; while (len >= 0 && seq < next_seq) { - struct printk_log *msg = log_from_idx(idx); + struct printk_log *msg = printk_log_from_idx(idx); int textlen; textlen = msg_print_text(msg, prev, true, text, @@ -1061,7 +1061,7 @@ static int syslog_print_all(char __user *buf, int size, bool clear) len = textlen; break; } - idx = log_next(idx); + idx = printk_log_next(idx); seq++; prev = msg->flags; @@ -1198,10 +1198,10 @@ int do_syslog(int type, char __user *buf, int len, bool from_file) error = 0; while (seq < printk_log_next_seq) { - struct printk_log *msg = log_from_idx(idx); + struct printk_log *msg = printk_log_from_idx(idx); error += msg_print_text(msg, prev, true, NULL, 0); - idx = log_next(idx); + idx = printk_log_next(idx); seq++; prev = msg->flags; } @@ -1400,8 +1400,8 @@ static void cont_flush(enum log_flags flags) * console; wait for the console to pick up the rest of the * line. LOG_NOCONS suppresses a duplicated output. */ - log_store(cont.facility, cont.level, flags | LOG_NOCONS, - cont.ts_nsec, NULL, 0, cont.buf, cont.len); + printk_log_store(cont.facility, cont.level, flags | LOG_NOCONS, + cont.ts_nsec, NULL, 0, cont.buf, cont.len); cont.flags = flags; cont.flushed = true; } else { @@ -1409,8 +1409,8 @@ static void cont_flush(enum log_flags flags) * If no fragment of this line ever reached the console, * just submit it to the store and free the buffer. */ - log_store(cont.facility, cont.level, flags, 0, - NULL, 0, cont.buf, cont.len); + printk_log_store(cont.facility, cont.level, flags, 0, + NULL, 0, cont.buf, cont.len); cont.len = 0; } } @@ -1522,8 +1522,8 @@ asmlinkage int vprintk_emit(int facility, int level, recursion_bug = 0; printed_len += strlen(recursion_msg); /* emit KERN_CRIT message */ - log_store(0, 2, LOG_PREFIX|LOG_NEWLINE, 0, - NULL, 0, recursion_msg, printed_len); + printk_log_store(0, 2, LOG_PREFIX|LOG_NEWLINE, 0, + NULL, 0, recursion_msg, printed_len); } /* @@ -1574,8 +1574,8 @@ asmlinkage int vprintk_emit(int facility, int level, /* buffer line if possible, otherwise store it right away */ if (!cont_add(facility, level, text, text_len)) - log_store(facility, level, lflags | LOG_CONT, 0, - dict, dictlen, text, text_len); + printk_log_store(facility, level, lflags | LOG_CONT, 0, + dict, dictlen, text, text_len); } else { bool stored = false; @@ -1592,8 +1592,8 @@ asmlinkage int vprintk_emit(int facility, int level, } if (!stored) - log_store(facility, level, lflags, 0, - dict, dictlen, text, text_len); + printk_log_store(facility, level, lflags, 0, + dict, dictlen, text, text_len); } printed_len += text_len; @@ -1699,8 +1699,8 @@ static struct cont { u8 level; bool flushed:1; } cont; -static struct printk_log *log_from_idx(u32 idx) { return NULL; } -static u32 log_next(u32 idx) { return 0; } +static struct printk_log *printk_log_from_idx(u32 idx) { return NULL; } +static u32 printk_log_next(u32 idx) { return 0; } static void call_console_drivers(int level, const char *text, size_t len) {} static size_t msg_print_text(const struct printk_log *msg, enum log_flags prev, bool syslog, char *buf, size_t size) { return 0; } @@ -2045,13 +2045,13 @@ skip: if (console_seq == printk_log_next_seq) break; - msg = log_from_idx(console_idx); + msg = printk_log_from_idx(console_idx); if (msg->flags & LOG_NOCONS) { /* * Skip record we have buffered and already printed * directly to the console when we received it. */ - console_idx = log_next(console_idx); + console_idx = printk_log_next(console_idx); console_seq++; /* * We will get here again when we register a new @@ -2066,7 +2066,7 @@ skip: level = msg->level; len = msg_print_text(msg, console_prev, false, text, sizeof(text)); - console_idx = log_next(console_idx); + console_idx = printk_log_next(console_idx); console_seq++; console_prev = msg->flags; raw_spin_unlock(&logbuf_lock); @@ -2618,10 +2618,10 @@ bool kmsg_dump_get_line_nolock(struct kmsg_dumper *dumper, bool syslog, if (dumper->cur_seq >= printk_log_next_seq) goto out; - msg = log_from_idx(dumper->cur_idx); + msg = printk_log_from_idx(dumper->cur_idx); l = msg_print_text(msg, 0, syslog, line, size); - dumper->cur_idx = log_next(dumper->cur_idx); + dumper->cur_idx = printk_log_next(dumper->cur_idx); dumper->cur_seq++; ret = true; out: @@ -2713,10 +2713,10 @@ bool kmsg_dump_get_buffer(struct kmsg_dumper *dumper, bool syslog, idx = dumper->cur_idx; prev = 0; while (seq < dumper->next_seq) { - struct printk_log *msg = log_from_idx(idx); + struct printk_log *msg = printk_log_from_idx(idx); l += msg_print_text(msg, prev, true, NULL, 0); - idx = log_next(idx); + idx = printk_log_next(idx); seq++; prev = msg->flags; } @@ -2726,10 +2726,10 @@ bool kmsg_dump_get_buffer(struct kmsg_dumper *dumper, bool syslog, idx = dumper->cur_idx; prev = 0; while (l > size && seq < dumper->next_seq) { - struct printk_log *msg = log_from_idx(idx); + struct printk_log *msg = printk_log_from_idx(idx); l -= msg_print_text(msg, prev, true, NULL, 0); - idx = log_next(idx); + idx = printk_log_next(idx); seq++; prev = msg->flags; } @@ -2741,10 +2741,10 @@ bool kmsg_dump_get_buffer(struct kmsg_dumper *dumper, bool syslog, l = 0; prev = 0; while (seq < dumper->next_seq) { - struct printk_log *msg = log_from_idx(idx); + struct printk_log *msg = printk_log_from_idx(idx); l += msg_print_text(msg, prev, syslog, buf + l, size - l); - idx = log_next(idx); + idx = printk_log_next(idx); seq++; prev = msg->flags; } -- 1.7.8.112.g3fd21 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/