proc: use seq_puts() everywhere

Alexey Dobriyan · torvalds · commit 08b55775133b · 2019-03-05T21:07:22.000-08:00
seq_printf() without format specifiers == faster seq_puts() Link: http://lkml.kernel.org/r/20190114200545.GC9680@avx2 Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
diff --git a/fs/proc/array.c b/fs/proc/array.c
@@ -343,28 +343,28 @@ static inline void task_seccomp(struct seq_file *m, struct task_struct *p)
 #ifdef CONFIG_SECCOMP
 	seq_put_decimal_ull(m, "\nSeccomp:\t", p->seccomp.mode);
 #endif
-	seq_printf(m, "\nSpeculation_Store_Bypass:\t");
+	seq_puts(m, "\nSpeculation_Store_Bypass:\t");
 	switch (arch_prctl_spec_ctrl_get(p, PR_SPEC_STORE_BYPASS)) {
 	case -EINVAL:
-		seq_printf(m, "unknown");
+		seq_puts(m, "unknown");
 		break;
 	case PR_SPEC_NOT_AFFECTED:
-		seq_printf(m, "not vulnerable");
+		seq_puts(m, "not vulnerable");
 		break;
 	case PR_SPEC_PRCTL | PR_SPEC_FORCE_DISABLE:
-		seq_printf(m, "thread force mitigated");
+		seq_puts(m, "thread force mitigated");
 		break;
 	case PR_SPEC_PRCTL | PR_SPEC_DISABLE:
-		seq_printf(m, "thread mitigated");
+		seq_puts(m, "thread mitigated");
 		break;
 	case PR_SPEC_PRCTL | PR_SPEC_ENABLE:
-		seq_printf(m, "thread vulnerable");
+		seq_puts(m, "thread vulnerable");
 		break;
 	case PR_SPEC_DISABLE:
-		seq_printf(m, "globally mitigated");
+		seq_puts(m, "globally mitigated");
 		break;
 	default:
-		seq_printf(m, "vulnerable");
+		seq_puts(m, "vulnerable");
 		break;
 	}
 	seq_putc(m, '\n');
diff --git a/fs/proc/base.c b/fs/proc/base.c
@@ -456,7 +456,7 @@ static int proc_pid_schedstat(struct seq_file *m, struct pid_namespace *ns,
 			      struct pid *pid, struct task_struct *task)
 {
 	if (unlikely(!sched_info_on()))
-		seq_printf(m, "0 0 0\n");
+		seq_puts(m, "0 0 0\n");
 	else
 		seq_printf(m, "%llu %llu %lu\n",
 		   (unsigned long long)task->se.sum_exec_runtime,
diff --git a/fs/proc/task_nommu.c b/fs/proc/task_nommu.c
@@ -178,7 +178,7 @@ static int nommu_vma_show(struct seq_file *m, struct vm_area_struct *vma)
 		seq_file_path(m, file, "");
 	} else if (mm && is_stack(vma)) {
 		seq_pad(m, ' ');
-		seq_printf(m, "[stack]");
+		seq_puts(m, "[stack]");
 	}
 
 	seq_putc(m, '\n');

Original file line number	Diff line number	Diff line change
`@@ -456,7 +456,7 @@ static int proc_pid_schedstat(struct seq_file m, struct pid_namespace ns,`
`456`	`456`	`struct pid pid, struct task_struct task)`
`457`	`457`	`{`
`458`	`458`	`if (unlikely(!sched_info_on()))`
`459`		`- seq_printf(m, "0 0 0\n");`
	`459`	`+ seq_puts(m, "0 0 0\n");`
`460`	`460`	`else`
`461`	`461`	`seq_printf(m, "%llu %llu %lu\n",`
`462`	`462`	`(unsigned long long)task->se.sum_exec_runtime,`
Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ static int nommu_vma_show(struct seq_file m, struct vm_area_struct vma)`
`178`	`178`	`seq_file_path(m, file, "");`
`179`	`179`	`} else if (mm && is_stack(vma)) {`
`180`	`180`	`seq_pad(m, ' ');`
`181`		`- seq_printf(m, "[stack]");`
	`181`	`+ seq_puts(m, "[stack]");`
`182`	`182`	`}`
`183`	`183`
`184`	`184`	`seq_putc(m, '\n');`