Commit baf57a09 authored by Trond Myklebust's avatar Trond Myklebust

NFS: Optimise the readdir searches

If we're going through the loop in nfs_readdir() more than once, we usually
do not want to restart searching from the beginning of the pages cache.

We only want to do that if the previous search failed...
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent d1bacf9e
...@@ -491,10 +491,6 @@ static inline ...@@ -491,10 +491,6 @@ static inline
int readdir_search_pagecache(nfs_readdir_descriptor_t *desc) int readdir_search_pagecache(nfs_readdir_descriptor_t *desc)
{ {
int res = -EAGAIN; int res = -EAGAIN;
desc->page_index = 0;
if (*desc->dir_cookie == 0)
desc->cache_entry_index = 0;
while (1) { while (1) {
res = find_cache_page(desc); res = find_cache_page(desc);
...@@ -589,6 +585,7 @@ int uncached_readdir(nfs_readdir_descriptor_t *desc, void *dirent, ...@@ -589,6 +585,7 @@ int uncached_readdir(nfs_readdir_descriptor_t *desc, void *dirent,
goto out_release; goto out_release;
} }
desc->page_index = 0;
desc->page = page; desc->page = page;
status = nfs_do_filldir(desc, dirent, filldir); status = nfs_do_filldir(desc, dirent, filldir);
...@@ -653,6 +650,7 @@ static int nfs_readdir(struct file *filp, void *dirent, filldir_t filldir) ...@@ -653,6 +650,7 @@ static int nfs_readdir(struct file *filp, void *dirent, filldir_t filldir)
if (res == -ETOOSMALL && desc->plus) { if (res == -ETOOSMALL && desc->plus) {
clear_bit(NFS_INO_ADVISE_RDPLUS, &NFS_I(inode)->flags); clear_bit(NFS_INO_ADVISE_RDPLUS, &NFS_I(inode)->flags);
nfs_zap_caches(inode); nfs_zap_caches(inode);
desc->page_index = 0;
desc->plus = 0; desc->plus = 0;
desc->eof = 0; desc->eof = 0;
continue; continue;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment