Commit 7de41d36 authored Dec 04, 2013 by Steven Whitehouse

GFS2: Remove test which is always true



Since gfs2_inplace_reserve() is always called with a valid
alloc parms structure, there is no need to test for this
within the function itself - and in any case, after we've
all ready dereferenced it anyway.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>

parent 7aed98fb

fs/gfs2/rgrp.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1943,7 +1943,7 @@ int gfs2_inplace_reserve(struct gfs2_inode ip, const struct gfs2_alloc_parms a
		/* Skip unuseable resource groups */
		if ((rs->rs_rbm.rgd->rd_flags & (GFS2_RGF_NOALLOC \|
		GFS2_RDF_ERROR)) \|\|
		(ap && (ap->target > rs->rs_rbm.rgd->rd_extfail_pt)))
		(ap->target > rs->rs_rbm.rgd->rd_extfail_pt))
		goto skip_rgrp;

		if (sdp->sd_args.ar_rgrplvb)

Admin message