diff options
| author |   Bob Peterson <rpeterso@redhat.com> | 2021-09-13 12:54:16 -0500 |
|---|---|---|
| committer |   Andreas Gruenbacher <agruenba@redhat.com> | 2021-10-25 08:42:17 +0200 |
| commit | c1442f6b53d887d08a098ec23056be7cfea1e3c7 (patch) | |
| tree | 0c6ddb607a901f3cef47fcd2d86989999b2eef87 /fs/gfs2/rgrp.c | |
| parent | gfs2: Add GL_SKIP holder flag to dump_holder (diff) | |
| download | linux-dev-c1442f6b53d887d08a098ec23056be7cfea1e3c7.tar.xz linux-dev-c1442f6b53d887d08a098ec23056be7cfea1e3c7.zip | |
gfs2: move GL_SKIP check from glops to do_promote
Before this patch, each individual "go_lock" glock operation (glop)
checked the GL_SKIP flag, and if set, would skip further processing.
This patch changes the logic so the go_lock caller, function go_promote,
checks the GL_SKIP flag before calling the go_lock op in the first place.
This avoids having to unnecessarily unlock gl_lockref.lock only to
re-lock it again.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Diffstat (limited to 'fs/gfs2/rgrp.c')
| -rw-r--r-- | fs/gfs2/rgrp.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c index 7a13a687e4f2..1fb66f6e6a0c 100644 --- a/fs/gfs2/rgrp.c +++ b/fs/gfs2/rgrp.c @@ -1292,8 +1292,6 @@ int gfs2_rgrp_go_lock(struct gfs2_holder *gh) { struct gfs2_rgrpd *rgd = gh->gh_gl->gl_object; - if (gh->gh_flags & GL_SKIP) - return 0; return gfs2_rgrp_bh_get(rgd); } |