Re: [PATCH 2/2] trace_workqueues: fix empty line's output

[Frederic Weisbecker <fweisbec@gmail.com>]

On Wed, Mar 25, 2009 at 04:59:18PM +0800, Lai Jiangshan wrote:
> 
> Empty lines separate cpus stat. After previous
> fix(trace_stat: keep original order) applied, the empty lines
> are displayed at incorrect position.
> 
> Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
> ---
> diff --git a/kernel/trace/trace_workqueue.c b/kernel/trace/trace_workqueue.c
> index 9ab035b..797201e 100644
> --- a/kernel/trace/trace_workqueue.c
> +++ b/kernel/trace/trace_workqueue.c
> @@ -196,6 +196,11 @@ static int workqueue_stat_show(struct seq_file *s, void *p)
>  	struct pid *pid;
>  	struct task_struct *tsk;
>  
> +	spin_lock_irqsave(&workqueue_cpu_stat(cpu)->lock, flags);
> +	if (&cws->list == workqueue_cpu_stat(cpu)->list.next)
> +		seq_printf(s, "\n");
> +	spin_unlock_irqrestore(&workqueue_cpu_stat(cpu)->lock, flags);
> +
>  	pid = find_get_pid(cws->pid);
>  	if (pid) {
>  		tsk = get_pid_task(pid, PIDTYPE_PID);
> @@ -208,18 +213,13 @@ static int workqueue_stat_show(struct seq_file *s, void *p)
>  		put_pid(pid);
>  	}
>  
> -	spin_lock_irqsave(&workqueue_cpu_stat(cpu)->lock, flags);
> -	if (&cws->list == workqueue_cpu_stat(cpu)->list.next)
> -		seq_printf(s, "\n");
> -	spin_unlock_irqrestore(&workqueue_cpu_stat(cpu)->lock, flags);
> -
>  	return 0;
>  }
>  
>  static int workqueue_stat_headers(struct seq_file *s)
>  {
>  	seq_printf(s, "# CPU  INSERTED  EXECUTED   NAME\n");
> -	seq_printf(s, "# |      |         |          |\n\n");
> +	seq_printf(s, "# |      |         |          |\n");
>  	return 0;
>  }
>  
> 
> 

Looks good.

Acked-by: Frederic Weisbecker <fweisbec@gmail.com>