gfs2: remove redundant check in gfs2_rgrp_go_lock
authorBob Peterson <rpeterso@redhat.com>
Mon, 13 Sep 2021 17:34:38 +0000 (12:34 -0500)
committerAndreas Gruenbacher <agruenba@redhat.com>
Mon, 25 Oct 2021 06:42:17 +0000 (08:42 +0200)
Before this patch, function gfs2_rgrp_go_lock checked if GL_SKIP and
ar_rgrplvb were both true. However, GL_SKIP is only set for rgrps if
ar_rgrplvb is true (see gfs2_inplace_reserve). This patch simply removes
the redundant check.

Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
fs/gfs2/rgrp.c

index c3b00ba92ed2e037e3b0d2aa1c9cb7305e02b5e7..7a13a687e4f20615bfd1f7d36015673f210d26ab 100644 (file)
@@ -1291,9 +1291,8 @@ static int update_rgrp_lvb(struct gfs2_rgrpd *rgd)
 int gfs2_rgrp_go_lock(struct gfs2_holder *gh)
 {
        struct gfs2_rgrpd *rgd = gh->gh_gl->gl_object;
-       struct gfs2_sbd *sdp = rgd->rd_sbd;
 
-       if (gh->gh_flags & GL_SKIP && sdp->sd_args.ar_rgrplvb)
+       if (gh->gh_flags & GL_SKIP)
                return 0;
        return gfs2_rgrp_bh_get(rgd);
 }