linux-stable.git/include/linux/nfs_fs.h, branch linux-2.6.29.y

optimize attribute timeouts for "noac" and "actimeo=0"

2008-12-23T20:21:56+00:00

Hi.

I've been looking at a bugzilla which describes a problem where
a customer was advised to use either the "noac" or "actimeo=0"
mount options to solve a consistency problem that they were
seeing in the file attributes.  It turned out that this solution
did not work reliably for them because sometimes, the local
attribute cache was believed to be valid and not timed out.
(With an attribute cache timeout of 0, the cache should always
appear to be timed out.)

In looking at this situation, it appears to me that the problem
is that the attribute cache timeout code has an off-by-one
error in it.  It is assuming that the cache is valid in the
region, [read_cache_jiffies, read_cache_jiffies + attrtimeo].  The
cache should be considered valid only in the region,
[read_cache_jiffies, read_cache_jiffies + attrtimeo).  With this
change, the options, "noac" and "actimeo=0", work as originally
expected.

This problem was previously addressed by special casing the
attrtimeo == 0 case.  However, since the problem is only an off-
by-one error, the cleaner solution is address the off-by-one
error and thus, not require the special case.

    Thanx...

        ps

Signed-off-by: Peter Staubach 
Signed-off-by: Trond Myklebust

NFSv4: Convert the open and close ops to use fmode

2008-12-23T20:21:56+00:00

Signed-off-by: Trond Myklebust

NFSv4: Convert delegation->type field to fmode_t

2008-12-23T20:21:53+00:00

Signed-off-by: Trond Myklebust

NFS: Move declaration of nfs_mount() to fs/nfs/internal.h

2008-12-23T20:21:34+00:00

Clean up:  The nfs_mount() function is not to be used outside of the
NFS client.  Move its public declaration to fs/nfs/internal.h.

Signed-off-by: Chuck Lever 
Signed-off-by: Trond Myklebust

Make nfs_file_cred more robust.

2008-10-17T17:06:45+00:00

As not all files have an associated open_context (e.g. device special
files), it is safest to test for the existence of the open context
before de-referencing it.

Signed-off-by: NeilBrown 
Signed-off-by: Trond Myklebust

NFS: Fix the resolution problem with nfs_inode_attrs_need_update()

2008-10-14T23:23:17+00:00

It appears that 'jiffies' timestamps do not have high enough resolution for
nfs_inode_attrs_need_update(). One problem is that a GETATTR can be
launched within < 1 jiffy of the last operation that updated the attribute.
Another problem is that RPC calls can take < 1 jiffy to execute.

We can fix this by switching the variables to use a simple global counter
that gets incremented every time we start another GETATTR call.

Signed-off-by: Trond Myklebust

NFS: Allow concurrent inode revalidation

2008-10-07T21:59:43+00:00

Currently, if two processes are both trying to revalidate metadata for the
same inode, they will find themselves being serialised. There is no good
justification for this now that we have improved our ability to detect
stale attribute data, so we should remove that serialisation.

Signed-off-by: Trond Myklebust

[PATCH] get rid of indirect users of namei.h

2008-07-27T00:53:42+00:00

fs.h needs path.h, not namei.h; nfs_fs.h doesn't need it at all.
Several places in the tree needed direct include.

Signed-off-by: Al Viro

[PATCH] sanitize ->permission() prototype

2008-07-27T00:53:14+00:00

* kill nameidata * argument; map the 3 bits in ->flags anybody cares
  about to new MAY_... ones and pass with the mask.
* kill redundant gfs2_iop_permission()
* sanitize ecryptfs_permission()
* fix remaining places where ->permission() instances might barf on new
  MAY_... found in mask.

The obvious next target in that direction is permission(9)

folded fix for nfs_permission() breakage from Miklos Szeredi 

Signed-off-by: Al Viro

NFS: use documenting macro constants for initializing ac{reg, dir}{min, max}

2008-07-09T16:09:40+00:00

Clean up.

Signed-off-by: Chuck Lever 
Signed-off-by: Trond Myklebust