After these two changes: 1a34c8c9a49e NFS: Support larger readdir buffers and 85aa8ddc3818 NFS: Trigger the "ls -l" readdir heuristic sooner .. we've disadvantaged/regressed some users workloads where the user is forced to use NFSv3 (such that we cannot prime the dcache with d_type on READDIR as we do for NFSv4), and where the workload is a search through a large directory tree. Previously, the smaller buffers and/or the larger heuristic size worked to their advantage to optimize the client toward the READDIRPLUS path. We've been able to relieve some pain temporarily by putting a knob into sysfs to allow them to set the operation to one of: - force READDIRPLUS - disable READDIRPLUS - use the default heurstic That's exactly the solution they're looking for and we have most of that in mount options today with the "rdirplus" and "nordirplus". Missing only is the option to force the client to always use READDIR plus. What follows is a patch to do that with a new mount option. I'll follow this up with a man page update to nfs-utils. The global heuristic is always going to be wrong for some workloads and we cannnot depend on something like a mount-wide option to always work well for mixed workloads on the same mount. A realistic long-term solution involves allowing the applications to signal their intended behavior. So, I am going to look at plumbing in a posix_fadvise() flag for READDIRPLUS. There's been some interest expressed already for other filesystems[1]. I plan to send along patches for this and face the discussions. However, that work will take some time to filter up into the utilities and into distros. In the meantime, I hope we can add this simpler mount option to help the folks that need the old behaviors in recent kernels. [1]: https://lore.kernel.org/linux-fsdevel/CAOQ4uxhBWV3DfqaE=reuPjh8w92wwujA6Abj=Gt0YvapR4m_1g@xxxxxxxxxxxxxx/ Benjamin Coddington (1): NFS: New mount option force_rdirplus fs/nfs/dir.c | 2 ++ fs/nfs/fs_context.c | 14 ++++++++++++-- include/linux/nfs_fs_sb.h | 1 + 3 files changed, 15 insertions(+), 2 deletions(-) base-commit: 2408a807bfc3f738850ef5ad5e3fd59d66168996 -- 2.47.0
