Re: [PATCH] procfs: fix the output format in /proc/PID/wchan

From: Andrew Morton
Date: Wed Nov 21 2018 - 22:28:48 EST


On Mon, 19 Nov 2018 19:17:52 +0800 Yafang Shao <laoar.shao@xxxxxxxxx> wrote:

> Just add the missing newline.
>
> ...
>
> --- a/fs/proc/base.c
> +++ b/fs/proc/base.c
> @@ -370,11 +370,12 @@ static int proc_pid_wchan(struct seq_file *m, struct pid_namespace *ns,
> wchan = get_wchan(task);
> if (wchan && !lookup_symbol_name(wchan, symname)) {
> seq_puts(m, symname);
> + seq_putc(m, '\n');
> return 0;
> }
>
> print0:
> - seq_putc(m, '0');
> + seq_puts(m, "0\n");
> return 0;
> }
> #endif /* CONFIG_KALLSYMS */

What is presently wrong with the wchan output? The changelog
should explain such things, please.

Providing example output with the patch unapplied and then with the
patch applied would help us to understand the patch's effect.

Thanks.