linux-toradex.git/include/linux/nfsd, branch v2.6.25.17

Use struct path in struct svc_expkey

2008-02-15T05:17:08+00:00

I'm embedding struct path into struct svc_expkey.

Signed-off-by: Jan Blunck 
Cc: Al Viro 
Acked-by: "J. Bruce Fields" 
Cc: Neil Brown 
Cc: Christoph Hellwig 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

Use struct path in struct svc_export

2008-02-15T05:17:08+00:00

I'm embedding struct path into struct svc_export.

[akpm@linux-foundation.org: coding-style fixes]
[ezk@cs.sunysb.edu: NFSD: fix wrong mnt_writer count in rename]
Signed-off-by: Jan Blunck 
Acked-by: J. Bruce Fields 
Acked-by: Christoph Hellwig 
Cc: Al Viro 
Cc: "J. Bruce Fields" 
Cc: Neil Brown 
Cc: Trond Myklebust 
Signed-off-by: Erez Zadok 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

nfsd: Allow AIX client to read dir containing mountpoints

2008-02-01T21:42:06+00:00

This patch addresses a compatibility issue with a Linux NFS server and
AIX NFS client.

I have exported /export as fsid=0 with sec=krb5:krb5i
I have mount --bind /home onto /export/home
I have exported /export/home with sec=krb5i

The AIX client mounts / -o sec=krb5:krb5i onto /mnt

If I do an ls /mnt, the AIX client gets a permission error. Looking at
the network traceIwe see a READDIR looking for attributes
FATTR4_RDATTR_ERROR and FATTR4_MOUNTED_ON_FILEID. The response gives a
NFS4ERR_WRONGSEC which the AIX client is not expecting.

Since the AIX client is only asking for an attribute that is an
attribute of the parent file system (pseudo root in my example), it
seems reasonable that there should not be an error.

In discussing this issue with Bruce Fields, I initially proposed
ignoring the error in nfsd4_encode_dirent_fattr() if all that was being
asked for was FATTR4_RDATTR_ERROR and FATTR4_MOUNTED_ON_FILEID, however,
Bruce suggested that we avoid calling cross_mnt() if only these
attributes are requested.

The following patch implements bypassing cross_mnt() if only
FATTR4_RDATTR_ERROR and FATTR4_MOUNTED_ON_FILEID are called. Since there
is some complexity in the code in nfsd4_encode_fattr(), I didn't want to
duplicate code (and introduce a maintenance nightmare), so I added a
parameter to nfsd4_encode_fattr() that indicates whether it should
ignore cross mounts and simply fill in the attribute using the passed in
dentry as opposed to it's parent.

Signed-off-by: Frank Filz 
Signed-off-by: J. Bruce Fields

nfsd: move nfsd/auth.h into fs/nfsd

2008-02-01T21:42:05+00:00

This header is used only in a few places in fs/nfsd, so there seems to
be little point to having it in include/.  (Thanks to Robert Day for
pointing this out.)

Cc: Robert P. J. Day 
Acked-by: NeilBrown 
Signed-off-by: J. Bruce Fields

knfsd: allow cache_register to return error on failure

2008-02-01T21:42:05+00:00

Newer server features such as nfsv4 and gss depend on proc to work, so a
failure to initialize the proc files they need should be treated as
fatal.

Thanks to Andrew Morton for style fix and compile fix in case where
CONFIG_NFSD_V4 is undefined.

Cc: Andrew Morton 
Acked-by: NeilBrown 
Signed-off-by: J. Bruce Fields

nfsd: fail module init on reply cache init failure

2008-02-01T21:42:04+00:00

If the reply cache initialization fails due to a kmalloc failure,
currently we try to soldier on with a reduced (or nonexistant) reply
cache.

Better to just fail immediately: the failure is then much easier to
understand and debug, and it could save us complexity in some later
code.  (But actually, it doesn't help currently because the cache is
also turned off in some odd failure cases; we should probably find a
better way to handle those failure cases some day.)

Fix some minor style problems while we're at it, and rename
nfsd_cache_init() to remove the need for a comment describing it.

Acked-by: NeilBrown 
Signed-off-by: J. Bruce Fields

NFSD: Path name length signage in nfsd request argument structures

2008-02-01T21:42:03+00:00

Clean up: For consistency, store the length of path name strings in nfsd
argument structures as unsigned integers.

Signed-off-by: Chuck Lever 
Acked-By: NeilBrown 
Signed-off-by: J. Bruce Fields

NFSD: Adjust filename length argument of nfsd_lookup

2008-02-01T21:42:03+00:00

Clean up: adjust the sign of the length argument of nfsd_lookup and
nfsd_lookup_dentry, for consistency with recent changes.  NFSD version
4 callers already pass an unsigned file name length.

Signed-off-by: Chuck Lever 
Acked-By: NeilBrown 
Signed-off-by: J. Bruce Fields

NFSD: File name length signage in nfsd request argument structures

2008-02-01T21:42:02+00:00

Clean up: For consistency, store the length of file name strings in nfsd
argument structures as unsigned integers.  This matches the XDR routines
and client argument structures for the same operation types.

Signed-off-by: Chuck Lever 
Acked-By: NeilBrown 
Signed-off-by: J. Bruce Fields

fs/nfsd/export.c: make 3 functions static

2007-10-16T16:43:10+00:00

This patch makes the following needlessly global functions static:
- exp_get_by_name()
- exp_parent()
- exp_find()

Signed-off-by: Adrian Bunk 
Cc: Neil Brown 
Cc: "J. Bruce Fields" 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds