X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Fllite%2Fvvp_page.c;h=18d7fa50d2545e70da6a2942ef13495f5b78e31e;hb=b15f3875f46eec3c5186fe6b84cf2cda7ceb8518;hp=7285045a2f5b641322d573decb86e5d5363d5f02;hpb=118bb27a7b2acce63cd0cf52af10f53b95f2a732;p=fs%2Flustre-release.git diff --git a/lustre/llite/vvp_page.c b/lustre/llite/vvp_page.c index 7285045..18d7fa5 100644 --- a/lustre/llite/vvp_page.c +++ b/lustre/llite/vvp_page.c @@ -78,8 +78,9 @@ static void vvp_page_fini(const struct lu_env *env, vvp_page_fini_common(cp); } -static void vvp_page_own(const struct lu_env *env, - const struct cl_page_slice *slice, struct cl_io *io) +static int vvp_page_own(const struct lu_env *env, + const struct cl_page_slice *slice, struct cl_io *io, + int nonblock) { struct ccc_page *vpg = cl2ccc_page(slice); cfs_page_t *vmpage = vpg->cpg_page; @@ -87,11 +88,24 @@ static void vvp_page_own(const struct lu_env *env, LASSERT(vmpage != NULL); + if (nonblock) { + if (TestSetPageLocked(vmpage)) + return -EAGAIN; + + if (unlikely(PageWriteback(vmpage))) { + /* Something gets wrong? */ + unlock_page(vmpage); + return -EAGAIN; + } + + return 0; + } + /* DEBUG CODE FOR #18881 */ while (TestSetPageLocked(vmpage)) { cfs_schedule_timeout(CFS_TASK_INTERRUPTIBLE, cfs_time_seconds(1)/10); - if (++count > 600) { + if (++count > 1200) { CL_PAGE_DEBUG(D_ERROR, env, cl_page_top(slice->cpl_page), "XXX page %p blocked on acquiring the" @@ -116,6 +130,7 @@ static void vvp_page_own(const struct lu_env *env, /* lock_page(vmpage); */ wait_on_page_writeback(vmpage); + return 0; } static void vvp_page_assume(const struct lu_env *env, @@ -277,14 +292,10 @@ static void vvp_page_completion_common(const struct lu_env *env, struct cl_page *clp = cp->cpg_cl.cpl_page; cfs_page_t *vmpage = cp->cpg_page; struct inode *inode = ccc_object_inode(clp->cp_obj); - struct cl_sync_io *anchor = cp->cpg_sync_io; LINVRNT(cl_page_is_vmlocked(env, clp)); - if (anchor != NULL) { - cp->cpg_sync_io = NULL; - cl_sync_io_note(anchor, ioret); - } else if (clp->cp_type == CPT_CACHEABLE) { + if (!clp->cp_sync_io && clp->cp_type == CPT_CACHEABLE) { /* * Only mark the page error only when it's a cacheable page * and NOT a sync io. @@ -469,11 +480,12 @@ static void vvp_transient_page_verify(const struct cl_page *page) /* LASSERT_SEM_LOCKED(&inode->i_alloc_sem); */ } -static void vvp_transient_page_own(const struct lu_env *env, - const struct cl_page_slice *slice, - struct cl_io *unused) +static int vvp_transient_page_own(const struct lu_env *env, + const struct cl_page_slice *slice, + struct cl_io *unused, int nonblock) { vvp_transient_page_verify(slice->cpl_page); + return 0; } static void vvp_transient_page_assume(const struct lu_env *env,