From 9c9bc136aa6f301685aaee681af3681057d229fb Mon Sep 17 00:00:00 2001 From: Trond Myklebust Date: Fri, 10 Jun 2011 13:30:22 -0400 Subject: [PATCH] --- yaml --- r: 253943 b: refs/heads/master c: 8f7d5efbef8718a774ac5e347b4ec069f17fd9b4 h: refs/heads/master i: 253941: 623ac3f51edd3b2004563049f501bfe503b85200 253939: 26af03020ae221affecc0267bc91039fbc4cb3c2 253935: bfd94632eecc5325f1775371fae8f3951714c29b v: v3 --- [refs] | 2 +- trunk/fs/nfs/objlayout/objio_osd.c | 3 +++ trunk/fs/nfs/pnfs.c | 12 +++++++----- 3 files changed, 11 insertions(+), 6 deletions(-) diff --git a/[refs] b/[refs] index a91ac9822fb8..d0759093d9e1 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 19345cb299e8234006c5125151ab723e851a1d24 +refs/heads/master: 8f7d5efbef8718a774ac5e347b4ec069f17fd9b4 diff --git a/trunk/fs/nfs/objlayout/objio_osd.c b/trunk/fs/nfs/objlayout/objio_osd.c index eb4aafa9f521..8ff2ea3f10ef 100644 --- a/trunk/fs/nfs/objlayout/objio_osd.c +++ b/trunk/fs/nfs/objlayout/objio_osd.c @@ -1000,6 +1000,9 @@ static bool objio_pg_test(struct nfs_pageio_descriptor *pgio, if (!pnfs_generic_pg_test(pgio, prev, req)) return false; + if (pgio->pg_lseg == NULL) + return true; + return pgio->pg_count + req->wb_bytes <= OBJIO_LSEG(pgio->pg_lseg)->max_io_size; } diff --git a/trunk/fs/nfs/pnfs.c b/trunk/fs/nfs/pnfs.c index 539b94cb6c0f..0f42e02436ee 100644 --- a/trunk/fs/nfs/pnfs.c +++ b/trunk/fs/nfs/pnfs.c @@ -1064,19 +1064,21 @@ pnfs_generic_pg_test(struct nfs_pageio_descriptor *pgio, struct nfs_page *prev, gfp_flags = GFP_NOFS; } - if (pgio->pg_count == prev->wb_bytes) { + if (pgio->pg_lseg == NULL) { + if (pgio->pg_count != prev->wb_bytes) + return true; /* This is first coelesce call for a series of nfs_pages */ pgio->pg_lseg = pnfs_update_layout(pgio->pg_inode, prev->wb_context, - req_offset(req), + req_offset(prev), pgio->pg_count, access_type, gfp_flags); - return true; + if (pgio->pg_lseg == NULL) + return true; } - if (pgio->pg_lseg && - req_offset(req) > end_offset(pgio->pg_lseg->pls_range.offset, + if (req_offset(req) > end_offset(pgio->pg_lseg->pls_range.offset, pgio->pg_lseg->pls_range.length)) return false;