From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161170AbcGZTNe (ORCPT ); Tue, 26 Jul 2016 15:13:34 -0400 Received: from mout.web.de ([212.227.15.4]:54179 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932499AbcGZTNa (ORCPT ); Tue, 26 Jul 2016 15:13:30 -0400 Subject: [PATCH 11/12] staging: lustre: Move an assignment for the variable "eof" in mgc_process_recover_log() To: devel@driverdev.osuosl.org, lustre-devel@lists.lustre.org, Andreas Dilger , Greg Kroah-Hartman , Oleg Drokin References: <566ABCD9.1060404@users.sourceforge.net> <566D7733.1030102@users.sourceforge.net> <56784D83.7080108@users.sourceforge.net> <56784F0C.6040007@users.sourceforge.net> <20151221234857.GA27079@kroah.com> <59d94e70-7476-728e-5f63-013557ec2db9@users.sourceforge.net> Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall , Bhumika Goyal From: SF Markus Elfring Message-ID: <83a0671d-5c02-0bd4-2a97-47dea50585c3@users.sourceforge.net> Date: Tue, 26 Jul 2016 21:13:23 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2 MIME-Version: 1.0 In-Reply-To: <59d94e70-7476-728e-5f63-013557ec2db9@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:LJB/l6Q9Ky/j6VKF6Iepjbx0cnabCvyf5ldVz8PDTp8KKh6wgZl ns46Ef+BFXSePTLe6Wrk475q3euvgIbLgtBOLHmrN7EvdfI0iZf6sM8/hYWf5hDwtCYNPi0 YQMuxm3PckLchOgGpTxSRy44+yl/3iKvC46/JNshNgqrXKw3HT9V/XH/w0Zf5eZImS8b42U VNTgN3UxcrTMeh6DYnIXA== X-UI-Out-Filterresults: notjunk:1;V01:K0:DeeSg/Uo0Hc=:nQI9Cwf7hawq/aDAvsbeRG zkfpVzjr2ktCg4YEEzE0Prc7wW+RXQPc8ixCvFqpjxdkKoLtFoeKTQJ5mbYSm/jOJpEsEhQNE 9f+LAt3nYq6rswwL5BMXg6KS3mCqIQRV3SBuw/cDd50hWKAlQLdkFfO0ABBAjOOuhp8o5hp3S E+548wiHxQIpSp7dliWuXe12d8dZAJmZXeAqLAbk8MzfCGoEMexOTojtfVxdHNH/MJa3n5PhV fKIYY4orkARZL2fi9kz7GzdZ2zw/vMWfCv5w6EMEDn4t10oi3nqJyVF/bQHFnC/LQ4Livq8Qs gQNl1uj6f2G/AMMZTr5/1m/RDEkf4AW5annJqiWRC9hKWCMQKi43TO6P8moDptQ5+LeUyOVrn 2T3mgZp9zz5n9vHK9g8Kzl1DwsKCHd1oDHUWy2LOcB8CJ9KSLWXTE8ZU0UTfi7nkVABNVYmcl OTeA4dop/wYt5hm5IFtDfSe0A3TNb9Vgx+y2fAdLIsVKaPTb9cNSViiNZlP+emq99jcb7rayG oD9gzr4ZtVt6WgOnDyukQxqYOaBSoj+KCQ+DaWv42XHL3Nse32bswza3nshatsf0Hutt5kkIh HRnv9QAXbTo4SgU/TCKtgrgSyt7N4tbEckF1OTb7PlaORIuPUKkh53qNLQNoYhikpkygijTaZ JdEej+QQ4QzvX5ba7XUi0iHSegbSyZ/2sL99qwH1dGUMcCmMbX6/STL2i7ED54vaROSRI3EmW 78jjyfaTOs3KIJJKdyV3HA6fi2F1bs9Ew3XI3p2oHSC19yD4D5XORWItMj0EtD9tPar30kkF+ wo/OpNF Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Tue, 26 Jul 2016 19:40:28 +0200 Move the assignment for the local variable "eof" behind the source code for memory allocations by this function. Signed-off-by: Markus Elfring --- drivers/staging/lustre/lustre/mgc/mgc_request.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/mgc/mgc_request.c b/drivers/staging/lustre/lustre/mgc/mgc_request.c index ff60b9b..8f5db79 100644 --- a/drivers/staging/lustre/lustre/mgc/mgc_request.c +++ b/drivers/staging/lustre/lustre/mgc/mgc_request.c @@ -1322,7 +1322,7 @@ static int mgc_process_recover_log(struct obd_device *obd, struct ptlrpc_bulk_desc *desc; struct page **pages; int nrpages; - bool eof = true; + bool eof; bool mne_swab; int i; int ealen; @@ -1350,6 +1350,7 @@ static int mgc_process_recover_log(struct obd_device *obd, } } + eof = true; again: LASSERT(cld_is_recover(cld)); LASSERT(mutex_is_locked(&cld->cld_lock)); -- 2.9.2