On Sep 8, 2011, at 11:21 AM, J. Bruce Fields wrote: > On Fri, Sep 02, 2011 at 12:38:42PM -0400, Chuck Lever wrote: >> From: Trond Myklebust <Trond.Myklebust@xxxxxxxxxx> >> >> Signed-off-by: Trond Myklebust <Trond.Myklebust@xxxxxxxxxx> >> [ cel: since this is server-side, use nfsd4_ prefix instead of nfs4_ prefix. ] >> [ cel: implement S_ISVTX filter in bfields-normal form ] >> Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx> > > OK, and this is all it takes to get entries for these added to the > export cache, so we're done? The two important future NFSD work items are: 1. Make the "refer=" mountd reply rich enough to return fs_locations_info data 2. Server-side support for fs_locations_info operations in NFSv4 compounds But neither of these are necessary for basic junction support. > > --b. > >> --- >> >> fs/nfsd/nfsd.h | 7 +++++++ >> fs/nfsd/vfs.c | 16 ++++++++++++++++ >> 2 files changed, 23 insertions(+), 0 deletions(-) >> >> diff --git a/fs/nfsd/nfsd.h b/fs/nfsd/nfsd.h >> index 7ecfa24..d314812 100644 >> --- a/fs/nfsd/nfsd.h >> +++ b/fs/nfsd/nfsd.h >> @@ -335,6 +335,13 @@ static inline u32 nfsd_suppattrs2(u32 minorversion) >> #define NFSD_SUPPATTR_EXCLCREAT_WORD2 \ >> NFSD_WRITEABLE_ATTRS_WORD2 >> >> +extern int nfsd4_is_junction(struct dentry *dentry); >> +#else >> +static inline int nfsd4_is_junction(struct dentry *dentry) >> +{ >> + return 0; >> +} >> + >> #endif /* CONFIG_NFSD_V4 */ >> >> #endif /* LINUX_NFSD_NFSD_H */ >> diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c >> index fd0acca..1f2d5bf 100644 >> --- a/fs/nfsd/vfs.c >> +++ b/fs/nfsd/vfs.c >> @@ -168,6 +168,8 @@ int nfsd_mountpoint(struct dentry *dentry, struct svc_export *exp) >> { >> if (d_mountpoint(dentry)) >> return 1; >> + if (nfsd4_is_junction(dentry)) >> + return 1; >> if (!(exp->ex_flags & NFSEXP_V4ROOT)) >> return 0; >> return dentry->d_inode != NULL; >> @@ -592,6 +594,20 @@ nfsd4_get_nfs4_acl(struct svc_rqst *rqstp, struct dentry *dentry, struct nfs4_ac >> return error; >> } >> >> +#define NFSD_XATTR_JUNCTION_PREFIX XATTR_TRUSTED_PREFIX "junction." >> +#define NFSD_XATTR_JUNCTION_TYPE NFSD_XATTR_JUNCTION_PREFIX "type" >> +int nfsd4_is_junction(struct dentry *dentry) >> +{ >> + struct inode *inode = dentry->d_inode; >> + >> + if (inode == NULL) >> + return 0; >> + if (!(inode->i_mode & S_ISVTX)) >> + return 0; >> + if (vfs_getxattr(dentry, NFSD_XATTR_JUNCTION_TYPE, NULL, 0) <= 0) >> + return 0; >> + return 1; >> +} >> #endif /* defined(CONFIG_NFSD_V4) */ >> >> #ifdef CONFIG_NFSD_V3 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html -- Chuck Lever chuck[dot]lever[at]oracle[dot]com -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html