Re: [PATCH v11 00/20] nfs/nfsd: add support for localio

Christoph Hellwig
2024-07-04T05:49:32Z

On Wed, Jul 03, 2024 at 11:28:55AM -0400, Mike Snitzer wrote:
> Containers is a significant usecase, but also any client that might
> need to access local storage efficiently (e.g. GPU service running NFS
> client that needs to access NVMe on same host).

Please explain that in terms of who talks to whom concretely using
the actual Linux and/or NFS entities. The last sentence just sound
like a AI generated marketing whitepaper.

> I can tighten that up in the Documentation.

Please write up a coherent document for the use case and circle it
around. It's kinda pointless do code review if we don't have a
problem statement and use case.

> Using pNFS layout isn't viable because NFSv3 is very much in the mix
> (flexfiles layout) for Hammerspace.

Again, why and how. We have a codebase that works entirely inside the
Linux kernel, and requires new code to be merged. If we can't ask
people to use a the current protocol (where current means a 14 year
old RFC!), we have a problem.