aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBrian Behlendorf <[email protected]>2016-08-02 10:50:52 -0700
committerBrian Behlendorf <[email protected]>2016-08-08 11:46:32 -0700
commite85a6396b0f7f3cb5a76cf88e47fc1334f9162b7 (patch)
treee6d18dd2f7fbe837442671db4b211309a9e481dc
parent64aefee1b847b747147016c039f5b0e276a9e1b7 (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.m439
-rw-r--r--config/kernel.m42
-rw-r--r--include/linux/vfs_compat.h13
-rw-r--r--module/zfs/zpl_xattr.c2
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: