diff --git a/config/kernel-create-nameidata.m4 b/config/kernel-create-nameidata.m4 index 9aad46fec..a71490a00 100644 --- a/config/kernel-create-nameidata.m4 +++ b/config/kernel-create-nameidata.m4 @@ -2,7 +2,7 @@ dnl # dnl # 3.6 API change dnl # AC_DEFUN([ZFS_AC_KERNEL_CREATE_NAMEIDATA], [ - AC_MSG_CHECKING([whether iops->create() takes struct nameidata]) + AC_MSG_CHECKING([whether iops->create() passes nameidata]) ZFS_LINUX_TRY_COMPILE([ #include @@ -22,7 +22,7 @@ AC_DEFUN([ZFS_AC_KERNEL_CREATE_NAMEIDATA], [ ],[ AC_MSG_RESULT(yes) AC_DEFINE(HAVE_CREATE_NAMEIDATA, 1, - [iops->create() operation takes nameidata]) + [iops->create() passes nameidata]) ],[ AC_MSG_RESULT(no) ]) diff --git a/config/kernel-follow-link-nameidata.m4 b/config/kernel-follow-link-nameidata.m4 new file mode 100644 index 000000000..88c85accb --- /dev/null +++ b/config/kernel-follow-link-nameidata.m4 @@ -0,0 +1,24 @@ +dnl # +dnl # 4.2 API change +dnl # This kernel retired the nameidata structure which forced the +dnl # restructuring of the follow_link() prototype and how it is called. +dnl # We check for the new interface rather than detecting the old one. +dnl # +AC_DEFUN([ZFS_AC_KERNEL_FOLLOW_LINK], [ + AC_MSG_CHECKING([whether iops->follow_link() passes nameidata]) + ZFS_LINUX_TRY_COMPILE([ + #include + const char *follow_link(struct dentry *de, void **cookie) + { return "symlink"; } + static struct inode_operations iops __attribute__ ((unused)) = { + .follow_link = follow_link, + }; + ],[ + ],[ + AC_MSG_RESULT(no) + ],[ + AC_MSG_RESULT(yes) + AC_DEFINE(HAVE_FOLLOW_LINK_NAMEIDATA, 1, + [iops->follow_link() nameidata]) + ]) +]) diff --git a/config/kernel-lookup-nameidata.m4 b/config/kernel-lookup-nameidata.m4 index 645560398..43f5fb4cb 100644 --- a/config/kernel-lookup-nameidata.m4 +++ b/config/kernel-lookup-nameidata.m4 @@ -2,7 +2,7 @@ dnl # dnl # 3.6 API change dnl # AC_DEFUN([ZFS_AC_KERNEL_LOOKUP_NAMEIDATA], [ - AC_MSG_CHECKING([whether iops->lookup() takes struct nameidata]) + AC_MSG_CHECKING([whether iops->lookup() passes nameidata]) ZFS_LINUX_TRY_COMPILE([ #include @@ -18,7 +18,7 @@ AC_DEFUN([ZFS_AC_KERNEL_LOOKUP_NAMEIDATA], [ ],[ AC_MSG_RESULT(yes) AC_DEFINE(HAVE_LOOKUP_NAMEIDATA, 1, - [iops->lookup() operation takes nameidata]) + [iops->lookup() passes nameidata]) ],[ AC_MSG_RESULT(no) ]) diff --git a/config/kernel-put-link-nameidata.m4 b/config/kernel-put-link-nameidata.m4 new file mode 100644 index 000000000..0181ae515 --- /dev/null +++ b/config/kernel-put-link-nameidata.m4 @@ -0,0 +1,23 @@ +dnl # +dnl # 4.2 API change +dnl # This kernel retired the nameidata structure which forced the +dnl # restructuring of the put_link() prototype and how it is called. +dnl # We check for the new interface rather than detecting the old one. +dnl # +AC_DEFUN([ZFS_AC_KERNEL_PUT_LINK], [ + AC_MSG_CHECKING([whether iops->put_link() passes nameidata]) + ZFS_LINUX_TRY_COMPILE([ + #include + void put_link(struct inode *ip, void *cookie) { return; } + static struct inode_operations iops __attribute__ ((unused)) = { + .put_link = put_link, + }; + ],[ + ],[ + AC_MSG_RESULT(no) + ],[ + AC_MSG_RESULT(yes) + AC_DEFINE(HAVE_PUT_LINK_NAMEIDATA, 1, + [iops->put_link() nameidata]) + ]) +]) diff --git a/config/kernel.m4 b/config/kernel.m4 index 806c5747a..5c97659c7 100644 --- a/config/kernel.m4 +++ b/config/kernel.m4 @@ -70,6 +70,8 @@ AC_DEFUN([ZFS_AC_CONFIG_KERNEL], [ ZFS_AC_KERNEL_MKDIR_UMODE_T ZFS_AC_KERNEL_LOOKUP_NAMEIDATA ZFS_AC_KERNEL_CREATE_NAMEIDATA + ZFS_AC_KERNEL_FOLLOW_LINK + ZFS_AC_KERNEL_PUT_LINK ZFS_AC_KERNEL_TRUNCATE_RANGE ZFS_AC_KERNEL_AUTOMOUNT ZFS_AC_KERNEL_ENCODE_FH_WITH_INODE diff --git a/module/zfs/zpl_inode.c b/module/zfs/zpl_inode.c index 31251e730..70b5e1239 100644 --- a/module/zfs/zpl_inode.c +++ b/module/zfs/zpl_inode.c @@ -348,8 +348,13 @@ zpl_symlink(struct inode *dir, struct dentry *dentry, const char *name) return (error); } +#ifdef HAVE_FOLLOW_LINK_NAMEIDATA static void * zpl_follow_link(struct dentry *dentry, struct nameidata *nd) +#else +const char * +zpl_follow_link(struct dentry *dentry, void **symlink_cookie) +#endif { cred_t *cr = CRED(); struct inode *ip = dentry->d_inode; @@ -372,17 +377,28 @@ zpl_follow_link(struct dentry *dentry, struct nameidata *nd) cookie = spl_fstrans_mark(); error = -zfs_readlink(ip, &uio, cr); spl_fstrans_unmark(cookie); - if (error) { + + if (error) kmem_free(link, MAXPATHLEN); - nd_set_link(nd, ERR_PTR(error)); - } else { - nd_set_link(nd, link); - } crfree(cr); + +#ifdef HAVE_FOLLOW_LINK_NAMEIDATA + if (error) + nd_set_link(nd, ERR_PTR(error)); + else + nd_set_link(nd, link); + return (NULL); +#else + if (error) + return (ERR_PTR(error)); + else + return (*symlink_cookie = link); +#endif } +#ifdef HAVE_PUT_LINK_NAMEIDATA static void zpl_put_link(struct dentry *dentry, struct nameidata *nd, void *ptr) { @@ -391,6 +407,13 @@ zpl_put_link(struct dentry *dentry, struct nameidata *nd, void *ptr) if (!IS_ERR(link)) kmem_free(link, MAXPATHLEN); } +#else +static void +zpl_put_link(struct inode *unused, void *symlink_cookie) +{ + kmem_free(symlink_cookie, MAXPATHLEN); +} +#endif static int zpl_link(struct dentry *old_dentry, struct inode *dir, struct dentry *dentry)