From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752584AbbGENfR (ORCPT ); Sun, 5 Jul 2015 09:35:17 -0400 Received: from mout.web.de ([212.227.17.11]:61584 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750846AbbGENfN (ORCPT ); Sun, 5 Jul 2015 09:35:13 -0400 Message-ID: <5599328E.5010905@users.sourceforge.net> Date: Sun, 05 Jul 2015 15:35:10 +0200 From: SF Markus Elfring User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Joel Becker , Mark Fasheh , ocfs2-devel@oss.oracle.com CC: LKML , kernel-janitors@vger.kernel.org, Julia Lawall Subject: [PATCH 06/11] ocfs2: Two checks less in ocfs2_local_alloc_slide_window() References: <5307CAA2.8060406@users.sourceforge.net> <530A086E.8010901@users.sourceforge.net> <530A72AA.3000601@users.sourceforge.net> <530B5FB6.6010207@users.sourceforge.net> <530C5E18.1020800@users.sourceforge.net> <530CD2C4.4050903@users.sourceforge.net> <530CF8FF.8080600@users.sourceforge.net> <530DD06F.4090703@users.sourceforge.net> <5317A59D.4@users.sourceforge.net> <5455FC05.4010606@users.sourceforge.net> <55992DF0.5030205@users.sourceforge.net> In-Reply-To: <55992DF0.5030205@users.sourceforge.net> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:HASN7MSgWjhyXum9Bzi5hgTprhOF3wIZN1CzYyEfk4PwzvErlkT JIp07f4a79m0/36vJVP+zaqj4Q3T1TzqOcTU1sMm38jJaPlvL5lh4TwZ2s9IaMbENXqq/eY AYmRJH5E2k/rpZkoFfz6GITWrMA6WKNcLfnthc5jK3nuoyfymmvUGUGLb/B+igDZv0+61mu 99sSECj7M3/+6pBUmGa/w== X-UI-Out-Filterresults: notjunk:1;V01:K0:ZcP00RR/nBI=:wi/GgE/EQjnPlywjEJNIxM FBzHM/UJfkEAuQrElafIKXdo+47ZSDEekTX1PbIiwUlwLfbdDDfMvo5/kBjmf/hscZSgr4FPJ Qy5HGyQg+DwUuxb9jsX/lYdJhMELaWOzEkEEF+4FytsRCx57YizfuPW+eWXpk4T+F7tqKZZdU 7UgfY491POv0FUr1M7up0LoQOOZ5XS9Tfh+BCr6MY1ae+Oq4KEdpbMOU5sDNxR7B25DugZHpO bs4q7Goycz9qYCK1qkobUQFveVnOaxGVDd67aWnMu11bTORvOXv0vjgcd//xFY4+T0BblRGkP UYPxFqQJwcDydCrlUlkBlj2Eom48DEBh9BCmCRc+WQmXvW0/sIkkHdt+kj+P31WpX54klAUT7 /ysIrv4ur2pOde3HyfFON/hW88OEXCz5vLjy2tf7RQBEv5mX7Jv6z0NXpLg/R6fvhndF1Kubz hq/TlE33E+sxBeZlb2xTPeCmZ2Lo6mZ0AbrglvRk07Ib48ZS96t30Onk1fa79BqhfjDqsh41u dfytbV7j++SwTAK8DhYPrdT669i3tBV2P8Q/vdd/E/DWaa/BVPFRX/beRSzIi20+j96oaHs/n 15Wl23HA3ETt1qWwQX6Wup00GG8+54NlD5xI+U+RgzGhBs/YggWB8a2VpQWreEz8INkUjNs0/ SukWxhisoinz0ldb/qrH9zLZONFVlURxUFtXQUVFO7tEV47Enx4cRVX2aI2T1su7RzY0= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sat, 4 Jul 2015 18:18:23 +0200 * Return directly from the ocfs2_local_alloc_slide_window() function if a call of the ocfs2_local_alloc_reserve_for_window() function failed at the beginning. * Rename jump labels according to the Linux coding style convention. * Drop an unnecessary initialisation for the variable "handle" then. * Delete a call of the mlog_errno() function at the end. Signed-off-by: Markus Elfring --- fs/ocfs2/localalloc.c | 25 ++++++++++--------------- 1 file changed, 10 insertions(+), 15 deletions(-) diff --git a/fs/ocfs2/localalloc.c b/fs/ocfs2/localalloc.c index a614037..16579ed 100644 --- a/fs/ocfs2/localalloc.c +++ b/fs/ocfs2/localalloc.c @@ -1248,7 +1248,7 @@ static int ocfs2_local_alloc_slide_window(struct ocfs2_super *osb, int status = 0; struct buffer_head *main_bm_bh = NULL; struct inode *main_bm_inode = NULL; - handle_t *handle = NULL; + handle_t *handle; struct ocfs2_dinode *alloc; struct ocfs2_dinode *alloc_copy = NULL; struct ocfs2_alloc_context *ac = NULL; @@ -1263,15 +1263,14 @@ static int ocfs2_local_alloc_slide_window(struct ocfs2_super *osb, if (status < 0) { if (status != -ENOSPC) mlog_errno(status); - goto bail; + return status; } handle = ocfs2_start_trans(osb, OCFS2_WINDOW_MOVE_CREDITS); if (IS_ERR(handle)) { status = PTR_ERR(handle); - handle = NULL; mlog_errno(status); - goto bail; + goto clean_up_main; } alloc = (struct ocfs2_dinode *) osb->local_alloc_bh->b_data; @@ -1285,7 +1284,7 @@ static int ocfs2_local_alloc_slide_window(struct ocfs2_super *osb, if (!alloc_copy) { status = -ENOMEM; mlog_errno(status); - goto bail; + goto commit; } memcpy(alloc_copy, alloc, osb->local_alloc_bh->b_size); @@ -1295,7 +1294,7 @@ static int ocfs2_local_alloc_slide_window(struct ocfs2_super *osb, OCFS2_JOURNAL_ACCESS_WRITE); if (status < 0) { mlog_errno(status); - goto bail; + goto commit; } ocfs2_clear_local_alloc(alloc); @@ -1305,31 +1304,27 @@ static int ocfs2_local_alloc_slide_window(struct ocfs2_super *osb, main_bm_inode, main_bm_bh); if (status < 0) { mlog_errno(status); - goto bail; + goto commit; } status = ocfs2_local_alloc_new_window(osb, handle, ac); if (status < 0) { if (status != -ENOSPC) mlog_errno(status); - goto bail; + goto commit; } atomic_inc(&osb->alloc_stats.moves); -bail: - if (handle) - ocfs2_commit_trans(osb, handle); - +commit: + ocfs2_commit_trans(osb, handle); +clean_up_main: brelse(main_bm_bh); iput(main_bm_inode); kfree(alloc_copy); if (ac) ocfs2_free_alloc_context(ac); - - if (status) - mlog_errno(status); return status; } -- 2.4.5