From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Sat, 22 Oct 2016 19:58:51 +0000 Subject: [PATCH 6/6] IA64-sn2_smp: Combine two seq_printf() calls into one call in sn2_ptc_seq_show() Message-Id: List-Id: References: <8daf03e1-5f6d-fc3b-c2db-397e6d4a9a1c@users.sourceforge.net> In-Reply-To: <8daf03e1-5f6d-fc3b-c2db-397e6d4a9a1c@users.sourceforge.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-ia64@vger.kernel.org, Al Viro , Borislav Petkov , Fenghua Yu , Greg Kroah-Hartman , Hans-Christian Noren Egtvedt , Ingo Molnar , =?UTF-8?B?SsO2cmcgUsO2ZGVs?= , Krzysztof Kozlowski , Robert Richter , Robin Murphy , Tony Luck , Toshi Kani , Vineet Gupta Cc: LKML , kernel-janitors@vger.kernel.org From: Markus Elfring Date: Sat, 22 Oct 2016 21:19:03 +0200 Some data were printed into a sequence by two separate function calls. Print the same data by a single function call instead. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- arch/ia64/sn/kernel/sn2/sn2_smp.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/arch/ia64/sn/kernel/sn2/sn2_smp.c b/arch/ia64/sn/kernel/sn2/sn2_smp.c index c98dc96..1beac3c 100644 --- a/arch/ia64/sn/kernel/sn2/sn2_smp.c +++ b/arch/ia64/sn/kernel/sn2/sn2_smp.c @@ -494,12 +494,11 @@ static int sn2_ptc_seq_show(struct seq_file *file, void *data) int cpu; cpu = *(loff_t *) data; - - if (!cpu) { + if (!cpu) seq_printf(file, - "# cpu ptc_l newrid ptc_flushes nodes_flushed deadlocks lock_nsec shub_nsec shub_nsec_max not_my_mm deadlock2 ipi_fluches ipi_nsec\n"); - seq_printf(file, "# ptctest %d, flushopt %d\n", sn2_ptctest, sn2_flush_opt); - } + "# cpu ptc_l newrid ptc_flushes nodes_flushed deadlocks lock_nsec shub_nsec shub_nsec_max not_my_mm deadlock2 ipi_fluches ipi_nsec\n" + "# ptctest %d, flushopt %d\n", + sn2_ptctest, sn2_flush_opt); if (cpu < nr_cpu_ids && cpu_online(cpu)) { stat = &per_cpu(ptcstats, cpu); -- 2.10.1