From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bob Peterson Date: Mon, 5 Oct 2015 10:36:37 -0500 Subject: [Cluster-devel] [GFS2 PATCH 15/15] GFS2: Rework gfs2_evict_inode to prevent collisions with openers In-Reply-To: <1444059397-4705-1-git-send-email-rpeterso@redhat.com> References: <1444059397-4705-1-git-send-email-rpeterso@redhat.com> Message-ID: <1444059397-4705-16-git-send-email-rpeterso@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit This patch mainly changes function gfs2_evict_inode so that it takes out glock references and sets a DELETING bit to prevent collisions from inode and glock creations from interfering. This is necessary to ensure a smooth transition from unlinked to deleted status for dinodes. Signed-off-by: Bob Peterson --- fs/gfs2/incore.h | 1 + fs/gfs2/meta_io.c | 6 ++++++ fs/gfs2/super.c | 40 ++++++++++++++++++++++++++++++++-------- 3 files changed, 39 insertions(+), 8 deletions(-) diff --git a/fs/gfs2/incore.h b/fs/gfs2/incore.h index c8f9bb9..effa924 100644 --- a/fs/gfs2/incore.h +++ b/fs/gfs2/incore.h @@ -381,6 +381,7 @@ enum { GIF_SW_PAGED = 3, GIF_ORDERED = 4, GIF_FREE_VFS_INODE = 5, + GIF_AM_DELETING = 6, }; struct gfs2_inode { diff --git a/fs/gfs2/meta_io.c b/fs/gfs2/meta_io.c index c20d7ca..040eb15 100644 --- a/fs/gfs2/meta_io.c +++ b/fs/gfs2/meta_io.c @@ -128,6 +128,12 @@ struct buffer_head *gfs2_getbuf(struct gfs2_glock *gl, u64 blkno, int create) index = blkno >> shift; /* convert block to page */ bufnum = blkno - (index << shift); /* block buf index within page */ + if ((gl->gl_name.ln_type == LM_TYPE_INODE) && gl->gl_object) { + struct gfs2_inode *ip = gl->gl_object; + struct inode *inode = &ip->i_inode; + if (!test_bit(GIF_AM_DELETING, &ip->i_flags)) + BUG_ON(inode->i_state & I_FREEING); + } if (create) { for (;;) { page = grab_cache_page(mapping, index); diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c index 79ee54b..80168b5 100644 --- a/fs/gfs2/super.c +++ b/fs/gfs2/super.c @@ -1511,7 +1511,7 @@ static void gfs2_evict_inode(struct inode *inode) struct gfs2_sbd *sdp = sb->s_fs_info; struct gfs2_inode *ip = GFS2_I(inode); struct gfs2_holder gh; - struct gfs2_glock *i_gl = NULL; + struct gfs2_glock *io_gl = NULL, *i_gl = NULL; struct address_space *metamapping; int error; @@ -1523,6 +1523,11 @@ static void gfs2_evict_inode(struct inode *inode) if (inode->i_nlink || (sb->s_flags & MS_RDONLY)) goto out; + error = gfs2_glock_get(sdp, ip->i_no_addr, &gfs2_inode_glops, + CREATE, &i_gl); + if (unlikely(error)) + goto out; + /* Must not read inode block until block type has been verified */ error = gfs2_glock_nq_init(i_gl, LM_ST_EXCLUSIVE, GL_SKIP, &gh); if (unlikely(error)) { @@ -1532,10 +1537,13 @@ static void gfs2_evict_inode(struct inode *inode) goto out; } + set_bit(GIF_AM_DELETING, &ip->i_flags); if (!test_bit(GIF_ALLOC_FAILED, &ip->i_flags)) { error = gfs2_check_blk_type(sdp, ip->i_no_addr, GFS2_BLKST_UNLINKED); - if (error) - goto out_truncate; + if (error) { + gfs2_glock_dq_uninit(&gh); + goto out; + } } if (test_bit(GIF_INVALID, &ip->i_flags)) { @@ -1544,10 +1552,22 @@ static void gfs2_evict_inode(struct inode *inode) goto out_truncate; } + /* The iopen glock may have been released prior to this, but we need + it back. */ + error = gfs2_glock_get(sdp, ip->i_no_addr, &gfs2_iopen_glops, CREATE, + &io_gl); + if (error) + goto out_truncate; + ip->i_iopen_gh.gh_flags |= GL_NOCACHE; - gfs2_glock_dq_wait(&ip->i_iopen_gh); - gfs2_holder_reinit(LM_ST_EXCLUSIVE, LM_FLAG_TRY_1CB | GL_NOCACHE, &ip->i_iopen_gh); - error = gfs2_glock_nq(&ip->i_iopen_gh); + if (ip->i_iopen_gh.gh_gl && + test_bit(HIF_HOLDER, &ip->i_iopen_gh.gh_iflags)) { + gfs2_glock_dq_wait(&ip->i_iopen_gh); + gfs2_holder_uninit(&ip->i_iopen_gh); + } + + error = gfs2_glock_nq_init(io_gl, LM_ST_EXCLUSIVE, LM_FLAG_TRY_1CB | + GL_NOCACHE, &ip->i_iopen_gh); if (error) goto out_truncate; @@ -1599,11 +1619,15 @@ out_unlock: if (gfs2_rs_active(ip->i_res)) gfs2_rs_deltree(ip->i_res); - if (test_bit(HIF_HOLDER, &ip->i_iopen_gh.gh_iflags)) { + if (ip->i_iopen_gh.gh_gl && + test_bit(HIF_HOLDER, &ip->i_iopen_gh.gh_iflags)) { ip->i_iopen_gh.gh_flags |= GL_NOCACHE; gfs2_glock_dq_wait(&ip->i_iopen_gh); + gfs2_holder_uninit(&ip->i_iopen_gh); } - gfs2_holder_uninit(&ip->i_iopen_gh); + if (io_gl) + gfs2_glock_put(io_gl); + gfs2_glock_dq_uninit(&gh); if (error && error != GLR_TRYFAILED && error != -EROFS) fs_warn(sdp, "gfs2_evict_inode: %d\n", error); -- 2.4.3