From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932310AbdAAQgz (ORCPT ); Sun, 1 Jan 2017 11:36:55 -0500 Received: from mout.web.de ([212.227.15.4]:62834 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932264AbdAAQgx (ORCPT ); Sun, 1 Jan 2017 11:36:53 -0500 Subject: [PATCH 2/5] staging/lustre/mgc: Combine two seq_printf() calls into one call in lprocfs_mgc_rd_ir_state() To: devel@driverdev.osuosl.org, lustre-devel@lists.lustre.org, Andreas Dilger , Ben Evans , Bobi Jam , Emoly Liu , Fan Yong , Greg Kroah-Hartman , Gregoire Pichon , Henri Doreau , James Simmons , Jinshan Xiong , "John L. Hammond" , "Kirill A. Shutemov" , Lai Siyao , Oleg Drokin , Sebastien Buisson , Stephen Champion , wang di References: <45ae8939-a775-865b-ade0-9e38a7ff2b5d@users.sourceforge.net> Cc: LKML , kernel-janitors@vger.kernel.org From: SF Markus Elfring Message-ID: <25b07069-c624-cdd4-ca7b-85aaae51e79e@users.sourceforge.net> Date: Sun, 1 Jan 2017 17:35:25 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <45ae8939-a775-865b-ade0-9e38a7ff2b5d@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:frDbsFuRiJoU2EHWzslZMjOjjfDTYPMbfxjDkyOyFu+JRMAqNoX m45fGQ+moRB7+LyoRB+qg5UNGT4Ac4VKuueyYQ9RLEENNRrTyjubph2yjv0EFFOFhBTg3dU a3qF0bHGrJ0CgzywlrayuVN9SCBXWP5zSWjHoktoExzp5DsemhtA/4bEo14ypbemqvHVSsx mxRMuNcbKX3ggp58lRUHQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:hBm3fx8C5fs=:jc669MaKuKR282aOTn3sfs rGq34DG0FaPHz1jfJEWsr7tVRojPnTIlxtiqeV2+UlbqhPaZg3S9QNqOSuyzg82e2jxV8IAvO Da1Ha6xohCHUQY4j0ZRLdbJwVoehntULu3ZfeVyx7x5zdCvlvcNtUIbf3tv5JzBnxzHYE1Czx Or+lgiBtTZvq0UhYev0WGJVn0AVRg46Vojbyj0AGdkvP/t3Y4bA6Zj+d53n4MjRBGwVR9cyFa mjjJR2wEIbi2IoLtoI+QOquYB/Wrx6JdVrUsxEIBeJqTqKisPbl8LDSO6WFCkCalkaq/rUa/a wu0Ui7co4iv+az5BZV/dhAiLE3nVf1XWEnVaxf4so91ZxlqYRRQIwSEWnXY3O3fwCMY6h6WfP KOMZauhmOye1HBi4T8myQwzenxv+80nlF+sKZ+gO6QP1yhnHVYTafjVNZ83q7e8grs+9OrUDk dFlaFGLKvBqu3VThEonsh3imjEyqY/b7tR395u0UXA1P1fkSGN3QSICWlVB6jnJAlBRp4r9Sy JY9FJpLvLFUcb9jwemQmI+W05+1wkINvEKxYkyVWcpcstUan1z63Jas94AHynhbb/zSJQRirE wQUhmppwSm7biG5zEBO3Hy0mmmBSZeKw6bceunKo+bY5tb25N7BSzGcAzFUjQey163at3vMFL lUmRbvcFEqOTrMRQIZz1e2FEME6KD7DbhbzVqDqfB5aSOa0Qk87EHZR+7TFdA3lrHXfkXDBwm +CnsY8yoQgyy6kBwNmZOl2Op33KyEKMlb4shj/LYCTxYJBRSYmtzu/Sn5ATSI7HmjD3SgJYh+ nbRHzMO Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sun, 1 Jan 2017 15:40:29 +0100 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 --- drivers/staging/lustre/lustre/mgc/mgc_request.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/mgc/mgc_request.c b/drivers/staging/lustre/lustre/mgc/mgc_request.c index b9c522a3c7a4..a6ca48d7e96b 100644 --- a/drivers/staging/lustre/lustre/mgc/mgc_request.c +++ b/drivers/staging/lustre/lustre/mgc/mgc_request.c @@ -460,11 +460,8 @@ int lprocfs_mgc_rd_ir_state(struct seq_file *m, void *data) imp = obd->u.cli.cl_import; ocd = &imp->imp_connect_data; - - seq_printf(m, "imperative_recovery: %s\n", + seq_printf(m, "imperative_recovery: %s\nclient_state:\n", OCD_HAS_FLAG(ocd, IMP_RECOV) ? "ENABLED" : "DISABLED"); - seq_printf(m, "client_state:\n"); - spin_lock(&config_list_lock); list_for_each_entry(cld, &config_llog_list, cld_list_chain) { if (!cld->cld_recover) -- 2.11.0