Revert "kmsg: add predefined _PID, _TID, _COMM keywords to kmsg* log dict"
authorMarek Szyprowski <m.szyprowski@samsung.com>
Wed, 25 Apr 2018 09:51:26 +0000 (11:51 +0200)
committerMarek Szyprowski <m.szyprowski@samsung.com>
Wed, 25 Apr 2018 09:51:26 +0000 (11:51 +0200)
This reverts commit 3d93c8208381e7efeeb7428853ca5b0a8cf8f8e5.

kernel/printk/printk.c

index 0d3c4b232155b2f0b2517dbf858a9d1986d21111..6857def266b025237e44088146c3550e219c91a3 100644 (file)
@@ -787,34 +787,7 @@ void log_buf_release(struct kref *ref)
        kfree(log_b);
 }
 
-#define MAX_PID_LEN    20
-#define MAX_TID_LEN    20
-/*
- * Fromat below describes dict appended to message written from userspace:
- * "_PID=<pid>\0_TID=<tid>\0_COMM=<comm>"
- * KMSG_DICT_MAX_LEN definition represents maximal length of this dict.
- */
-#define KMSG_DICT_MAX_LEN      (5 + MAX_PID_LEN + 1 + \
-                                5 + MAX_TID_LEN + 1 + \
-                                6 + TASK_COMM_LEN)
-
-static size_t set_kmsg_dict(char *buf)
-{
-       size_t len;
-
-       len = sprintf(buf, "_PID=%d", task_tgid_nr(current)) + 1;
-       len += sprintf(buf + len, "_TID=%d", task_pid_nr(current)) + 1;
-       memcpy(buf + len, "_COMM=", 6);
-       len += 6;
-       get_task_comm(buf + len, current);
-       while (buf[len] != '\0')
-               len++;
-       return len;
-}
-
-static int kmsg_sys_write(int minor, int level,
-                         const char *dict, size_t dictlen,
-                         const char *fmt, ...)
+static int kmsg_sys_write(int minor, int level, const char *fmt, ...)
 {
        va_list args;
        int ret = -ENXIO;
@@ -829,7 +802,7 @@ static int kmsg_sys_write(int minor, int level,
 
                va_start(args, fmt);
                log_format_and_store(log_b, 1 /* LOG_USER */, level,
-                                    dict, dictlen, fmt, args);
+                                    NULL, 0, fmt, args);
                va_end(args);
                wake_up_interruptible(&log_b->wait);
 
@@ -849,8 +822,6 @@ static ssize_t devkmsg_write(struct kiocb *iocb, struct iov_iter *from)
        int level = default_message_loglevel;
        int facility = 1;       /* LOG_USER */
        size_t len = iov_iter_count(from);
-       char dict[KMSG_DICT_MAX_LEN];
-       size_t dictlen;
        ssize_t ret = len;
        int minor = iminor(iocb->ki_filp->f_inode);
 
@@ -890,13 +861,10 @@ static ssize_t devkmsg_write(struct kiocb *iocb, struct iov_iter *from)
                }
        }
 
-       dictlen = set_kmsg_dict(dict);
-
        if (minor == log_buf.minor) {
-               printk_emit(facility, level, dict, dictlen, "%s", line);
+               printk_emit(facility, level, NULL, 0, "%s", line);
        } else {
-               int error = kmsg_sys_write(minor, level, dict, dictlen,
-                                          "%s", line);
+               int error = kmsg_sys_write(minor, level, "%s", line);
 
                if (error)
                        ret = error;