diff options
author | Brian Behlendorf <[email protected]> | 2016-08-02 10:50:52 -0700 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2016-08-08 11:46:32 -0700 |
commit | e85a6396b0f7f3cb5a76cf88e47fc1334f9162b7 (patch) | |
tree | e6d18dd2f7fbe837442671db4b211309a9e481dc | |
parent | 64aefee1b847b747147016c039f5b0e276a9e1b7 (diff) |
Retire HAVE_CURRENT_UMASK and HAVE_POSIX_ACL_CACHING
Remove ZFS_AC_KERNEL_CURRENT_UMASK and ZFS_AC_KERNEL_POSIX_ACL_CACHING
configure checks, all supported kernel provide this functionality.
Signed-off-by: Brian Behlendorf <[email protected]>
Signed-off-by: Nikolay Borisov <[email protected]>
Signed-off-by: Chunwei Chen <[email protected]>
Closes #4922
-rw-r--r-- | config/kernel-acl.m4 | 39 | ||||
-rw-r--r-- | config/kernel.m4 | 2 | ||||
-rw-r--r-- | include/linux/vfs_compat.h | 13 | ||||
-rw-r--r-- | module/zfs/zpl_xattr.c | 2 |
4 files changed, 0 insertions, 56 deletions
diff --git a/config/kernel-acl.m4 b/config/kernel-acl.m4 index ffb6a0254..6ef80fe5e 100644 --- a/config/kernel-acl.m4 +++ b/config/kernel-acl.m4 @@ -77,27 +77,6 @@ AC_DEFUN([ZFS_AC_KERNEL_POSIX_ACL_CHMOD], [ ]) dnl # -dnl # 2.6.30 API change, -dnl # caching of ACL into the inode was added in this version. -dnl # -AC_DEFUN([ZFS_AC_KERNEL_POSIX_ACL_CACHING], [ - AC_MSG_CHECKING([whether inode has i_acl and i_default_acl]) - ZFS_LINUX_TRY_COMPILE([ - #include <linux/fs.h> - ],[ - struct inode ino; - ino.i_acl = NULL; - ino.i_default_acl = NULL; - ],[ - AC_MSG_RESULT(yes) - AC_DEFINE(HAVE_POSIX_ACL_CACHING, 1, - [inode contains i_acl and i_default_acl]) - ],[ - AC_MSG_RESULT(no) - ]) -]) - -dnl # dnl # 3.1 API change, dnl # posix_acl_equiv_mode now wants an umode_t* instead of a mode_t* dnl # @@ -246,21 +225,3 @@ AC_DEFUN([ZFS_AC_KERNEL_INODE_OPERATIONS_GET_ACL], [ AC_MSG_RESULT(no) ]) ]) - -dnl # -dnl # 2.6.30 API change, -dnl # current_umask exists only since this version. -dnl # -AC_DEFUN([ZFS_AC_KERNEL_CURRENT_UMASK], [ - AC_MSG_CHECKING([whether current_umask exists]) - ZFS_LINUX_TRY_COMPILE([ - #include <linux/fs.h> - ],[ - current_umask(); - ],[ - AC_MSG_RESULT(yes) - AC_DEFINE(HAVE_CURRENT_UMASK, 1, [current_umask() exists]) - ],[ - AC_MSG_RESULT(no) - ]) -]) diff --git a/config/kernel.m4 b/config/kernel.m4 index 2fa04706b..9eec504f8 100644 --- a/config/kernel.m4 +++ b/config/kernel.m4 @@ -42,14 +42,12 @@ AC_DEFUN([ZFS_AC_CONFIG_KERNEL], [ ZFS_AC_KERNEL_POSIX_ACL_FROM_XATTR_USERNS ZFS_AC_KERNEL_POSIX_ACL_RELEASE ZFS_AC_KERNEL_POSIX_ACL_CHMOD - ZFS_AC_KERNEL_POSIX_ACL_CACHING ZFS_AC_KERNEL_POSIX_ACL_EQUIV_MODE_WANTS_UMODE_T ZFS_AC_KERNEL_INODE_OPERATIONS_PERMISSION ZFS_AC_KERNEL_INODE_OPERATIONS_PERMISSION_WITH_NAMEIDATA ZFS_AC_KERNEL_INODE_OPERATIONS_CHECK_ACL ZFS_AC_KERNEL_INODE_OPERATIONS_CHECK_ACL_WITH_FLAGS ZFS_AC_KERNEL_INODE_OPERATIONS_GET_ACL - ZFS_AC_KERNEL_CURRENT_UMASK ZFS_AC_KERNEL_SHOW_OPTIONS ZFS_AC_KERNEL_FILE_INODE ZFS_AC_KERNEL_FSYNC diff --git a/include/linux/vfs_compat.h b/include/linux/vfs_compat.h index fa12ba95a..52c539933 100644 --- a/include/linux/vfs_compat.h +++ b/include/linux/vfs_compat.h @@ -203,9 +203,6 @@ lseek_execute( * At 60 seconds the kernel will also begin issuing RCU stall warnings. */ #include <linux/posix_acl.h> -#ifndef HAVE_POSIX_ACL_CACHING -#define ACL_NOT_CACHED ((void *)(-1)) -#endif /* HAVE_POSIX_ACL_CACHING */ #if defined(HAVE_POSIX_ACL_RELEASE) && !defined(HAVE_POSIX_ACL_RELEASE_GPL_ONLY) @@ -234,7 +231,6 @@ zpl_posix_acl_release(struct posix_acl *acl) static inline void zpl_set_cached_acl(struct inode *ip, int type, struct posix_acl *newer) { -#ifdef HAVE_POSIX_ACL_CACHING struct posix_acl *older = NULL; spin_lock(&ip->i_lock); @@ -256,7 +252,6 @@ zpl_set_cached_acl(struct inode *ip, int type, struct posix_acl *newer) { spin_unlock(&ip->i_lock); zpl_posix_acl_release(older); -#endif /* HAVE_POSIX_ACL_CACHING */ } static inline void @@ -323,14 +318,6 @@ typedef mode_t zpl_equivmode_t; #endif /* HAVE_POSIX_ACL_EQUIV_MODE_UMODE_T */ #endif /* CONFIG_FS_POSIX_ACL */ -#ifndef HAVE_CURRENT_UMASK -static inline int -current_umask(void) -{ - return (current->fs->umask); -} -#endif /* HAVE_CURRENT_UMASK */ - /* * 2.6.38 API change, * The is_owner_or_cap() function was renamed to inode_owner_or_capable(). diff --git a/module/zfs/zpl_xattr.c b/module/zfs/zpl_xattr.c index 23c63a587..6dab895e0 100644 --- a/module/zfs/zpl_xattr.c +++ b/module/zfs/zpl_xattr.c @@ -1017,11 +1017,9 @@ zpl_get_acl(struct inode *ip, int type) char *name; int size; -#ifdef HAVE_POSIX_ACL_CACHING acl = get_cached_acl(ip, type); if (acl != ACL_NOT_CACHED) return (acl); -#endif /* HAVE_POSIX_ACL_CACHING */ switch (type) { case ACL_TYPE_ACCESS: |