Caching files in nfsd was Re: [patch 12/16] fix race between writeback and unlink

[Andi Kleen]

Linus Torvalds <[e-mail]> writes:

> I _think_ that right now nfsd doesn't cache file opens (only inodes), so
> this could be a performance issue for nfsd, but it might be possible to
> change how nfsd acts. And it would be a _lot_ cleaner to do it at the file
> level.

Yes.

Fixing this would also help XFS (which I hope will be merged in 2.5 as
it works very well for a lot of people). It manages its extent
preallocation per file and flushes extents on closes. Currently it has
to maintain an ugly private nfs reference cache to avoid flushing an
extent after every NFS write operation (and killing write performance
this way)

Also letting nfsd know about the filemap.c readahead window information in 
struct file (that is what it currently caches in the racache) is really ugly
and a kind of layering violation...

I guess it is not too uncommon for other file systems too to hold state 
per open file.

-Andi