Now that we have per-page cache validation, we can allow filling of the pagecache from arbitrary offsets. Store the pagecache cursor on the open directory context so it can be used to validate our entries the next time we enter nfs_readdir() without having to fill the pagecache from the beginning. The open_directory_context's dir_cookie and index_cookie will store the same value between calls to nfs_readdir(), so we can save a little room in the struct by dropping dir_cookie. Signed-off-by: Benjamin Coddington <bcodding@xxxxxxxxxx> --- fs/nfs/dir.c | 15 ++++++++------- include/linux/nfs_fs.h | 2 +- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/fs/nfs/dir.c b/fs/nfs/dir.c index 5fc22d97054a..4e21b21c75d0 100644 --- a/fs/nfs/dir.c +++ b/fs/nfs/dir.c @@ -81,7 +81,7 @@ static struct nfs_open_dir_context *alloc_nfs_open_dir_context(struct inode *dir if (ctx != NULL) { ctx->duped = 0; ctx->attr_gencount = nfsi->attr_gencount; - ctx->dir_cookie = 0; + memset(&ctx->pgc, 0, sizeof(struct nfs_dir_page_cursor)); ctx->dup_cookie = 0; spin_lock(&dir->i_lock); if (list_empty(&nfsi->open_files) && @@ -365,7 +365,8 @@ nfs_readdir_page_valid(struct page *page, unsigned int entry_index, u64 index_co ret = true; array = kmap_atomic(page); - if (array->size == 0 && array->last_cookie == index_cookie) + if ((array->size == 0 || array->size == entry_index) + && array->last_cookie == index_cookie) goto out_unmap; if (array->size > entry_index && @@ -1054,7 +1055,6 @@ static int readdir_search_pagecache(struct nfs_readdir_descriptor *desc) if (desc->pgc.page_index == 0) { desc->current_index = 0; desc->prev_index = 0; - desc->pgc.index_cookie = 0; } res = find_and_lock_cache_page(desc); } while (res == -EAGAIN); @@ -1192,7 +1192,8 @@ static int nfs_readdir(struct file *file, struct dir_context *ctx) desc->plus = nfs_use_readdirplus(inode, ctx); spin_lock(&file->f_lock); - desc->dir_cookie = dir_ctx->dir_cookie; + desc->dir_cookie = dir_ctx->pgc.index_cookie; + desc->pgc = dir_ctx->pgc; desc->dup_cookie = dir_ctx->dup_cookie; desc->duped = dir_ctx->duped; desc->attr_gencount = dir_ctx->attr_gencount; @@ -1231,7 +1232,7 @@ static int nfs_readdir(struct file *file, struct dir_context *ctx) } while (!desc->eof); spin_lock(&file->f_lock); - dir_ctx->dir_cookie = desc->dir_cookie; + dir_ctx->pgc = desc->pgc; dir_ctx->dup_cookie = desc->dup_cookie; dir_ctx->duped = desc->duped; dir_ctx->attr_gencount = desc->attr_gencount; @@ -1273,9 +1274,9 @@ static loff_t nfs_llseek_dir(struct file *filp, loff_t offset, int whence) if (offset != filp->f_pos) { filp->f_pos = offset; if (nfs_readdir_use_cookie(filp)) - dir_ctx->dir_cookie = offset; + dir_ctx->pgc.index_cookie = offset; else - dir_ctx->dir_cookie = 0; + dir_ctx->pgc.index_cookie = 0; if (offset == 0) memset(dir_ctx->verf, 0, sizeof(dir_ctx->verf)); dir_ctx->duped = 0; diff --git a/include/linux/nfs_fs.h b/include/linux/nfs_fs.h index 557e54b7d555..cd35137a935b 100644 --- a/include/linux/nfs_fs.h +++ b/include/linux/nfs_fs.h @@ -101,7 +101,7 @@ struct nfs_open_dir_context { struct list_head list; unsigned long attr_gencount; __be32 verf[NFS_DIR_VERIFIER_SIZE]; - __u64 dir_cookie; + struct nfs_dir_page_cursor pgc; __u64 dup_cookie; signed char duped; }; -- 2.25.4