From: Andrew W Elble <aweits@rit.edu>
Date: Tue, 06 Oct 2015 14:31:05 -0400
Subject: [Cluster-devel] [GFS2 PATCH 13/15] gfs2: Use new variable
	i_gl	instead of ip->i_gl
In-Reply-To: <1444059397-4705-14-git-send-email-rpeterso@redhat.com> (Bob
	Peterson's message of "Mon, 5 Oct 2015 10:36:35 -0500")
References: <1444059397-4705-1-git-send-email-rpeterso@redhat.com>
	<1444059397-4705-14-git-send-email-rpeterso@redhat.com>
Message-ID: <m2k2qzu9w6.fsf@discipline.rit.edu>
List-Id: <cluster-devel.redhat.com>
To: cluster-devel.redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit


Bob Peterson <rpeterso@redhat.com> writes:

> This patch adds a new variable to function gfs2_evict_inode that
> simplifies the references to ip->i_gl. This is just for readability
> and to clarify future patches.
>
> Signed-off-by: Bob Peterson <rpeterso@redhat.com>
> ---
>  fs/gfs2/super.c | 13 +++++++------
>  1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c
> index 06bd72b..79ee54b 100644
> --- a/fs/gfs2/super.c
> +++ b/fs/gfs2/super.c

<snip>

> @@ -1616,7 +1617,7 @@ out:
>  	ip->i_gl->gl_object = NULL;
>  	flush_delayed_work(&ip->i_gl->gl_work);
>  	gfs2_glock_add_to_lru(ip->i_gl);
> -	gfs2_glock_put(ip->i_gl);

        if (i_gl)

> +	gfs2_glock_put(i_gl);
>  	ip->i_gl = NULL;
>  	if (ip->i_iopen_gh.gh_gl) {
>  		ip->i_iopen_gh.gh_gl->gl_object = NULL;

-- 
Andrew W. Elble
aweits at discipline.rit.edu
Infrastructure Engineer, Communications Technical Lead
Rochester Institute of Technology
PGP: BFAD 8461 4CCF DC95 DA2C B0EB 965B 082E 863E C912