diff options
| author |   Bob Peterson <rpeterso@redhat.com> | 2016-04-01 13:18:38 -0400 |
|---|---|---|
| committer | Bob Peterson <rpeterso@redhat.com> | 2016-04-05 11:59:18 -0400 |
| commit | 73cc86252bd8a547349d2856497eceaf4f2d1766 (patch) | |
| tree | 6dda7db9bc8ed1be8e252219e58b1f84539a6302 /fs/gfs2 | |
| parent | GFS2: ignore unlock failures after withdraw (diff) | |
| download | linux-dev-73cc86252bd8a547349d2856497eceaf4f2d1766.tar.xz linux-dev-73cc86252bd8a547349d2856497eceaf4f2d1766.zip | |
GFS2: Get rid of dead code in inode_go_demote_ok
Function inode_go_demote_ok had some code that was only executed
if gl_holders was not empty. However, if gl_holders was not empty,
the only caller, demote_ok(), returns before inode_go_demote_ok
would ever be called. Therefore, it's dead code, so I removed it.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Acked-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2')
| -rw-r--r-- | fs/gfs2/glops.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/fs/gfs2/glops.c b/fs/gfs2/glops.c index 437fd73e381e..5db59d444838 100644 --- a/fs/gfs2/glops.c +++ b/fs/gfs2/glops.c @@ -286,17 +286,10 @@ static void inode_go_inval(struct gfs2_glock *gl, int flags) static int inode_go_demote_ok(const struct gfs2_glock *gl) { struct gfs2_sbd *sdp = gl->gl_name.ln_sbd; - struct gfs2_holder *gh; if (sdp->sd_jindex == gl->gl_object || sdp->sd_rindex == gl->gl_object) return 0; - if (!list_empty(&gl->gl_holders)) { - gh = list_entry(gl->gl_holders.next, struct gfs2_holder, gh_list); - if (gh->gh_list.next != &gl->gl_holders) - return 0; - } - return 1; } |