Thread (12 messages) 12 messages, 4 authors, 2011-08-10

Re: [PATCH 4/4] nfsd: vfs_llseek() with 32 or 64 bit offsets (hashes)

From: Bernd Schubert <hidden>
Date: 2011-08-10 19:13:13
Also in: linux-fsdevel, linux-nfs

On 08/09/2011 07:33 PM, J. Bruce Fields wrote:
On Mon, Aug 08, 2011 at 05:38:13PM +0200, Bernd Schubert wrote:
quoted
Use 32-bit or 64-bit llseek() hashes for directory offsets depending on
the NFS version. NFSv2 gets 32-bit hashes only.

NOTE: This patch got rather complex as Christoph asked to set the
filp->f_mode flag in the open call or immediatly after dentry_open()
in nfsd_open() to avoid races.
Personally I still do not see a reason for that and in my opinion
FMODE_32BITHASH/FMODE_64BITHASH flags could be set nfsd_readdir(), as it
follows directly after nfsd_open() without a chance of races.
The bulk of the patch seems to be just an access->may_flags rename.
Could you please split that into a separate patch?
Ok, shall I resend the entire patch series, but already remove the 
32-bit nfsd_readdir() cookie patch? Or only just this patch split into 
to parts?


Thanks,
Bernd
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help