]> git.proxmox.com Git - mirror_zfs.git/blobdiff - module/zfs/zfs_acl.c
Fix typo/etc in module/zfs/zfs_ctldir.c
[mirror_zfs.git] / module / zfs / zfs_acl.c
index df690b364d06e0d0dd956b48e313e8bedef7ac32..b1af4da2f4a5f8e0fd8e816e94e376717eaf6f08 100644 (file)
  */
 /*
  * Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2013 by Delphix. All rights reserved.
  */
 
 
 #include <sys/types.h>
 #include <sys/param.h>
 #include <sys/time.h>
-#include <sys/systm.h>
 #include <sys/sysmacros.h>
-#include <sys/resource.h>
 #include <sys/vfs.h>
 #include <sys/vnode.h>
 #include <sys/sid.h>
@@ -37,7 +36,6 @@
 #include <sys/kmem.h>
 #include <sys/cmn_err.h>
 #include <sys/errno.h>
-#include <sys/unistd.h>
 #include <sys/sdt.h>
 #include <sys/fs/zfs.h>
 #include <sys/mode.h>
@@ -51,7 +49,8 @@
 #include <sys/dnode.h>
 #include <sys/zap.h>
 #include <sys/sa.h>
-#include "fs/fs_subr.h"
+#include <sys/trace_acl.h>
+#include <sys/zpl.h>
 
 #define        ALLOW   ACE_ACCESS_ALLOWED_ACE_TYPE
 #define        DENY    ACE_ACCESS_DENIED_ACE_TYPE
@@ -95,6 +94,8 @@
 
 #define        ALL_MODE_EXECS (S_IXUSR | S_IXGRP | S_IXOTH)
 
+#define        IDMAP_WK_CREATOR_OWNER_UID      2147483648U
+
 static uint16_t
 zfs_ace_v0_get_type(void *acep)
 {
@@ -171,18 +172,18 @@ zfs_ace_v0_data(void *acep, void **datap)
 }
 
 static acl_ops_t zfs_acl_v0_ops = {
-       zfs_ace_v0_get_mask,
-       zfs_ace_v0_set_mask,
-       zfs_ace_v0_get_flags,
-       zfs_ace_v0_set_flags,
-       zfs_ace_v0_get_type,
-       zfs_ace_v0_set_type,
-       zfs_ace_v0_get_who,
-       zfs_ace_v0_set_who,
-       zfs_ace_v0_size,
-       zfs_ace_v0_abstract_size,
-       zfs_ace_v0_mask_off,
-       zfs_ace_v0_data
+       .ace_mask_get = zfs_ace_v0_get_mask,
+       .ace_mask_set = zfs_ace_v0_set_mask,
+       .ace_flags_get = zfs_ace_v0_get_flags,
+       .ace_flags_set = zfs_ace_v0_set_flags,
+       .ace_type_get = zfs_ace_v0_get_type,
+       .ace_type_set = zfs_ace_v0_set_type,
+       .ace_who_get = zfs_ace_v0_get_who,
+       .ace_who_set = zfs_ace_v0_set_who,
+       .ace_size = zfs_ace_v0_size,
+       .ace_abstract_size = zfs_ace_v0_abstract_size,
+       .ace_mask_off = zfs_ace_v0_mask_off,
+       .ace_data = zfs_ace_v0_data
 };
 
 static uint16_t
@@ -307,18 +308,18 @@ zfs_ace_fuid_data(void *acep, void **datap)
 }
 
 static acl_ops_t zfs_acl_fuid_ops = {
-       zfs_ace_fuid_get_mask,
-       zfs_ace_fuid_set_mask,
-       zfs_ace_fuid_get_flags,
-       zfs_ace_fuid_set_flags,
-       zfs_ace_fuid_get_type,
-       zfs_ace_fuid_set_type,
-       zfs_ace_fuid_get_who,
-       zfs_ace_fuid_set_who,
-       zfs_ace_fuid_size,
-       zfs_ace_fuid_abstract_size,
-       zfs_ace_fuid_mask_off,
-       zfs_ace_fuid_data
+       .ace_mask_get = zfs_ace_fuid_get_mask,
+       .ace_mask_set = zfs_ace_fuid_set_mask,
+       .ace_flags_get = zfs_ace_fuid_get_flags,
+       .ace_flags_set = zfs_ace_fuid_set_flags,
+       .ace_type_get = zfs_ace_fuid_get_type,
+       .ace_type_set = zfs_ace_fuid_set_type,
+       .ace_who_get = zfs_ace_fuid_get_who,
+       .ace_who_set = zfs_ace_fuid_set_who,
+       .ace_size = zfs_ace_fuid_size,
+       .ace_abstract_size = zfs_ace_fuid_abstract_size,
+       .ace_mask_off = zfs_ace_fuid_mask_off,
+       .ace_data = zfs_ace_fuid_data
 };
 
 /*
@@ -368,23 +369,23 @@ static int
 zfs_acl_znode_info(znode_t *zp, int *aclsize, int *aclcount,
     zfs_acl_phys_t *aclphys)
 {
-       zfs_sb_t *zsb = ZTOZSB(zp);
+       zfsvfs_t *zfsvfs = ZTOZSB(zp);
        uint64_t acl_count;
        int size;
        int error;
 
        ASSERT(MUTEX_HELD(&zp->z_acl_lock));
        if (zp->z_is_sa) {
-               if ((error = sa_size(zp->z_sa_hdl, SA_ZPL_DACL_ACES(zsb),
+               if ((error = sa_size(zp->z_sa_hdl, SA_ZPL_DACL_ACES(zfsvfs),
                    &size)) != 0)
                        return (error);
                *aclsize = size;
-               if ((error = sa_lookup(zp->z_sa_hdl, SA_ZPL_DACL_COUNT(zsb),
+               if ((error = sa_lookup(zp->z_sa_hdl, SA_ZPL_DACL_COUNT(zfsvfs),
                    &acl_count, sizeof (acl_count))) != 0)
                        return (error);
                *aclcount = acl_count;
        } else {
-               if ((error = sa_lookup(zp->z_sa_hdl, SA_ZPL_ZNODE_ACL(zsb),
+               if ((error = sa_lookup(zp->z_sa_hdl, SA_ZPL_ZNODE_ACL(zfsvfs),
                    aclphys, sizeof (*aclphys))) != 0)
                        return (error);
 
@@ -648,7 +649,7 @@ zfs_ace_walk(void *datap, uint64_t cookie, int aclcnt,
  * ACE FUIDs will be created later.
  */
 int
-zfs_copy_ace_2_fuid(zfs_sb_t *zsb, umode_t obj_mode, zfs_acl_t *aclp,
+zfs_copy_ace_2_fuid(zfsvfs_t *zfsvfs, umode_t obj_mode, zfs_acl_t *aclp,
     void *datap, zfs_ace_t *z_acl, uint64_t aclcnt, size_t *size,
     zfs_fuid_info_t **fuidp, cred_t *cr)
 {
@@ -666,7 +667,7 @@ zfs_copy_ace_2_fuid(zfs_sb_t *zsb, umode_t obj_mode, zfs_acl_t *aclp,
                entry_type = aceptr->z_hdr.z_flags & ACE_TYPE_FLAGS;
                if (entry_type != ACE_OWNER && entry_type != OWNING_GROUP &&
                    entry_type != ACE_EVERYONE) {
-                       aceptr->z_fuid = zfs_fuid_create(zsb, acep->a_who,
+                       aceptr->z_fuid = zfs_fuid_create(zfsvfs, acep->a_who,
                            cr, (entry_type == 0) ?
                            ZFS_ACE_USER : ZFS_ACE_GROUP, fuidp);
                }
@@ -676,7 +677,7 @@ zfs_copy_ace_2_fuid(zfs_sb_t *zsb, umode_t obj_mode, zfs_acl_t *aclp,
                 */
                if (zfs_ace_valid(obj_mode, aclp, aceptr->z_hdr.z_type,
                    aceptr->z_hdr.z_flags) != B_TRUE)
-                       return (EINVAL);
+                       return (SET_ERROR(EINVAL));
 
                switch (acep->a_type) {
                case ACE_ACCESS_ALLOWED_OBJECT_ACE_TYPE:
@@ -710,7 +711,7 @@ zfs_copy_ace_2_fuid(zfs_sb_t *zsb, umode_t obj_mode, zfs_acl_t *aclp,
  * Copy ZFS ACEs to fixed size ace_t layout
  */
 static void
-zfs_copy_fuid_2_ace(zfs_sb_t *zsb, zfs_acl_t *aclp, cred_t *cr,
+zfs_copy_fuid_2_ace(zfsvfs_t *zfsvfs, zfs_acl_t *aclp, cred_t *cr,
     void *datap, int filter)
 {
        uint64_t who;
@@ -753,7 +754,7 @@ zfs_copy_fuid_2_ace(zfs_sb_t *zsb, zfs_acl_t *aclp, cred_t *cr,
                if ((entry_type != ACE_OWNER &&
                    entry_type != OWNING_GROUP &&
                    entry_type != ACE_EVERYONE)) {
-                       acep->a_who = zfs_fuid_map_id(zsb, who,
+                       acep->a_who = zfs_fuid_map_id(zfsvfs, who,
                            cr, (entry_type & ACE_IDENTIFIER_GROUP) ?
                            ZFS_ACE_GROUP : ZFS_ACE_USER);
                } else {
@@ -783,7 +784,7 @@ zfs_copy_ace_2_oldace(umode_t obj_mode, zfs_acl_t *aclp, ace_t *acep,
                 */
                if (zfs_ace_valid(obj_mode, aclp, aceptr->z_type,
                    aceptr->z_flags) != B_TRUE)
-                       return (EINVAL);
+                       return (SET_ERROR(EINVAL));
        }
        *size = (caddr_t)aceptr - (caddr_t)z_acl;
        return (0);
@@ -1051,13 +1052,13 @@ zfs_mode_compute(uint64_t fmode, zfs_acl_t *aclp,
  * Read an external acl object.  If the intent is to modify, always
  * create a new acl and leave any cached acl in place.
  */
-static int
-zfs_acl_node_read(znode_t *zp, boolean_t have_lock, zfs_acl_t **aclpp,
+int
+zfs_acl_node_read(struct znode *zp, boolean_t have_lock, zfs_acl_t **aclpp,
     boolean_t will_modify)
 {
        zfs_acl_t       *aclp;
-       int             aclsize;
-       int             acl_count;
+       int             aclsize = 0;
+       int             acl_count = 0;
        zfs_acl_node_t  *aclnode;
        zfs_acl_phys_t  znode_acl;
        int             version;
@@ -1117,7 +1118,7 @@ zfs_acl_node_read(znode_t *zp, boolean_t have_lock, zfs_acl_t **aclpp,
                zfs_acl_node_free(aclnode);
                /* convert checksum errors into IO errors */
                if (error == ECKSUM)
-                       error = EIO;
+                       error = SET_ERROR(EIO);
                goto done;
        }
 
@@ -1155,12 +1156,30 @@ zfs_acl_chown_setattr(znode_t *zp)
        int error;
        zfs_acl_t *aclp;
 
+       if (ZTOZSB(zp)->z_acl_type == ZFS_ACLTYPE_POSIXACL)
+               return (0);
+
        ASSERT(MUTEX_HELD(&zp->z_lock));
        ASSERT(MUTEX_HELD(&zp->z_acl_lock));
 
-       if ((error = zfs_acl_node_read(zp, B_TRUE, &aclp, B_FALSE)) == 0)
-               zp->z_mode = zfs_mode_compute(zp->z_mode, aclp,
-                   &zp->z_pflags, zp->z_uid, zp->z_gid);
+       error = zfs_acl_node_read(zp, B_TRUE, &aclp, B_FALSE);
+       if (error == 0 && aclp->z_acl_count > 0)
+               zp->z_mode = ZTOI(zp)->i_mode =
+                   zfs_mode_compute(zp->z_mode, aclp,
+                   &zp->z_pflags, KUID_TO_SUID(ZTOI(zp)->i_uid),
+                   KGID_TO_SGID(ZTOI(zp)->i_gid));
+
+       /*
+        * Some ZFS implementations (ZEVO) create neither a ZNODE_ACL
+        * nor a DACL_ACES SA in which case ENOENT is returned from
+        * zfs_acl_node_read() when the SA can't be located.
+        * Allow chown/chgrp to succeed in these cases rather than
+        * returning an error that makes no sense in the context of
+        * the caller.
+        */
+       if (error == ENOENT)
+               return (0);
+
        return (error);
 }
 
@@ -1295,25 +1314,26 @@ int
 zfs_aclset_common(znode_t *zp, zfs_acl_t *aclp, cred_t *cr, dmu_tx_t *tx)
 {
        int                     error;
-       zfs_sb_t                *zsb = ZTOZSB(zp);
+       zfsvfs_t                *zfsvfs = ZTOZSB(zp);
        dmu_object_type_t       otype;
        zfs_acl_locator_cb_t    locate = { 0 };
        uint64_t                mode;
        sa_bulk_attr_t          bulk[5];
        uint64_t                ctime[2];
        int                     count = 0;
+       zfs_acl_phys_t          acl_phys;
 
        mode = zp->z_mode;
 
        mode = zfs_mode_compute(mode, aclp, &zp->z_pflags,
-           zp->z_uid, zp->z_gid);
+           KUID_TO_SUID(ZTOI(zp)->i_uid), KGID_TO_SGID(ZTOI(zp)->i_gid));
 
-       zp->z_mode = mode;
-       SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_MODE(zsb), NULL,
+       zp->z_mode = ZTOI(zp)->i_mode = mode;
+       SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_MODE(zfsvfs), NULL,
            &mode, sizeof (mode));
-       SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_FLAGS(zsb), NULL,
+       SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_FLAGS(zfsvfs), NULL,
            &zp->z_pflags, sizeof (zp->z_pflags));
-       SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_CTIME(zsb), NULL,
+       SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_CTIME(zfsvfs), NULL,
            &ctime, sizeof (ctime));
 
        if (zp->z_acl_cached) {
@@ -1324,11 +1344,11 @@ zfs_aclset_common(znode_t *zp, zfs_acl_t *aclp, cred_t *cr, dmu_tx_t *tx)
        /*
         * Upgrade needed?
         */
-       if (!zsb->z_use_fuids) {
+       if (!zfsvfs->z_use_fuids) {
                otype = DMU_OT_OLDACL;
        } else {
                if ((aclp->z_version == ZFS_ACL_VERSION_INITIAL) &&
-                   (zsb->z_version >= ZPL_VERSION_FUID))
+                   (zfsvfs->z_version >= ZPL_VERSION_FUID))
                        zfs_acl_xform(zp, aclp, cr);
                ASSERT(aclp->z_version >= ZFS_ACL_VERSION_FUID);
                otype = DMU_OT_ACL;
@@ -1341,17 +1361,16 @@ zfs_aclset_common(znode_t *zp, zfs_acl_t *aclp, cred_t *cr, dmu_tx_t *tx)
 
        if (zp->z_is_sa) { /* the easy case, just update the ACL attribute */
                locate.cb_aclp = aclp;
-               SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_DACL_ACES(zsb),
+               SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_DACL_ACES(zfsvfs),
                    zfs_acl_data_locator, &locate, aclp->z_acl_bytes);
-               SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_DACL_COUNT(zsb),
+               SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_DACL_COUNT(zfsvfs),
                    NULL, &aclp->z_acl_count, sizeof (uint64_t));
        } else { /* Painful legacy way */
                zfs_acl_node_t *aclnode;
                uint64_t off = 0;
-               zfs_acl_phys_t acl_phys;
                uint64_t aoid;
 
-               if ((error = sa_lookup(zp->z_sa_hdl, SA_ZPL_ZNODE_ACL(zsb),
+               if ((error = sa_lookup(zp->z_sa_hdl, SA_ZPL_ZNODE_ACL(zfsvfs),
                    &acl_phys, sizeof (acl_phys))) != 0)
                        return (error);
 
@@ -1365,20 +1384,20 @@ zfs_aclset_common(znode_t *zp, zfs_acl_t *aclp, cred_t *cr, dmu_tx_t *tx)
                         */
                        if (aoid &&
                            aclp->z_version != acl_phys.z_acl_version) {
-                               error = dmu_object_free(zsb->z_os, aoid, tx);
+                               error = dmu_object_free(zfsvfs->z_os, aoid, tx);
                                if (error)
                                        return (error);
                                aoid = 0;
                        }
                        if (aoid == 0) {
-                               aoid = dmu_object_alloc(zsb->z_os,
+                               aoid = dmu_object_alloc(zfsvfs->z_os,
                                    otype, aclp->z_acl_bytes,
                                    otype == DMU_OT_ACL ?
                                    DMU_OT_SYSACL : DMU_OT_NONE,
                                    otype == DMU_OT_ACL ?
-                                   DN_MAX_BONUSLEN : 0, tx);
+                                   DN_OLD_MAX_BONUSLEN : 0, tx);
                        } else {
-                               (void) dmu_object_set_blocksize(zsb->z_os,
+                               (void) dmu_object_set_blocksize(zfsvfs->z_os,
                                    aoid, aclp->z_acl_bytes, 0, tx);
                        }
                        acl_phys.z_acl_extern_obj = aoid;
@@ -1386,7 +1405,7 @@ zfs_aclset_common(znode_t *zp, zfs_acl_t *aclp, cred_t *cr, dmu_tx_t *tx)
                            aclnode = list_next(&aclp->z_acl, aclnode)) {
                                if (aclnode->z_ace_count == 0)
                                        continue;
-                               dmu_write(zsb->z_os, aoid, off,
+                               dmu_write(zfsvfs->z_os, aoid, off,
                                    aclnode->z_size, aclnode->z_acldata, tx);
                                off += aclnode->z_size;
                        }
@@ -1396,7 +1415,7 @@ zfs_aclset_common(znode_t *zp, zfs_acl_t *aclp, cred_t *cr, dmu_tx_t *tx)
                         * Migrating back embedded?
                         */
                        if (acl_phys.z_acl_extern_obj) {
-                               error = dmu_object_free(zsb->z_os,
+                               error = dmu_object_free(zfsvfs->z_os,
                                    acl_phys.z_acl_extern_obj, tx);
                                if (error)
                                        return (error);
@@ -1425,7 +1444,7 @@ zfs_aclset_common(znode_t *zp, zfs_acl_t *aclp, cred_t *cr, dmu_tx_t *tx)
                }
                acl_phys.z_acl_version = aclp->z_version;
 
-               SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_ZNODE_ACL(zsb), NULL,
+               SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_ZNODE_ACL(zfsvfs), NULL,
                    &acl_phys, sizeof (acl_phys));
        }
 
@@ -1439,12 +1458,12 @@ zfs_aclset_common(znode_t *zp, zfs_acl_t *aclp, cred_t *cr, dmu_tx_t *tx)
        if (ace_trivial_common(aclp, 0, zfs_ace_walk) == 0)
                zp->z_pflags |= ZFS_ACL_TRIVIAL;
 
-       zfs_tstamp_update_setup(zp, STATE_CHANGED, NULL, ctime, B_TRUE);
+       zfs_tstamp_update_setup(zp, STATE_CHANGED, NULL, ctime);
        return (sa_bulk_update(zp->z_sa_hdl, bulk, count, tx));
 }
 
 static void
-zfs_acl_chmod(zfs_sb_t *zsb, uint64_t mode, zfs_acl_t *aclp)
+zfs_acl_chmod(zfsvfs_t *zfsvfs, uint64_t mode, zfs_acl_t *aclp)
 {
        void            *acep = NULL;
        uint64_t        who;
@@ -1472,7 +1491,8 @@ zfs_acl_chmod(zfs_sb_t *zsb, uint64_t mode, zfs_acl_t *aclp)
                zacep = (void *)((uintptr_t)zacep + abstract_size);
                new_count++;
                new_bytes += abstract_size;
-       } if (deny1) {
+       }
+       if (deny1) {
                zfs_set_ace(aclp, zacep, deny1, DENY, -1, ACE_OWNER);
                zacep = (void *)((uintptr_t)zacep + abstract_size);
                new_count++;
@@ -1516,7 +1536,7 @@ zfs_acl_chmod(zfs_sb_t *zsb, uint64_t mode, zfs_acl_t *aclp)
                         * Limit permissions to be no greater than
                         * group permissions
                         */
-                       if (zsb->z_acl_inherit == ZFS_ACL_RESTRICTED) {
+                       if (zfsvfs->z_acl_inherit == ZFS_ACL_RESTRICTED) {
                                if (!(mode & S_IRGRP))
                                        access_mask &= ~ACE_READ_DATA;
                                if (!(mode & S_IWGRP))
@@ -1568,11 +1588,11 @@ zfs_acl_chmod_setattr(znode_t *zp, zfs_acl_t **aclp, uint64_t mode)
  * strip off write_owner and write_acl
  */
 static void
-zfs_restricted_update(zfs_sb_t *zsb, zfs_acl_t *aclp, void *acep)
+zfs_restricted_update(zfsvfs_t *zfsvfs, zfs_acl_t *aclp, void *acep)
 {
        uint32_t mask = aclp->z_ops->ace_mask_get(acep);
 
-       if ((zsb->z_acl_inherit == ZFS_ACL_RESTRICTED) &&
+       if ((zfsvfs->z_acl_inherit == ZFS_ACL_RESTRICTED) &&
            (aclp->z_ops->ace_type_get(acep) == ALLOW)) {
                mask &= ~RESTRICTED_CLEAR;
                aclp->z_ops->ace_mask_set(acep, mask);
@@ -1599,7 +1619,7 @@ zfs_ace_can_use(umode_t obj_mode, uint16_t acep_flags)
  * inherit inheritable ACEs from parent
  */
 static zfs_acl_t *
-zfs_acl_inherit(zfs_sb_t *zsb, umode_t obj_mode, zfs_acl_t *paclp,
+zfs_acl_inherit(zfsvfs_t *zfsvfs, umode_t obj_mode, zfs_acl_t *paclp,
     uint64_t mode, boolean_t *need_chmod)
 {
        void            *pacep;
@@ -1617,16 +1637,16 @@ zfs_acl_inherit(zfs_sb_t *zsb, umode_t obj_mode, zfs_acl_t *paclp,
        boolean_t       passthrough, passthrough_x, noallow;
 
        passthrough_x =
-           zsb->z_acl_inherit == ZFS_ACL_PASSTHROUGH_X;
+           zfsvfs->z_acl_inherit == ZFS_ACL_PASSTHROUGH_X;
        passthrough = passthrough_x ||
-           zsb->z_acl_inherit == ZFS_ACL_PASSTHROUGH;
+           zfsvfs->z_acl_inherit == ZFS_ACL_PASSTHROUGH;
        noallow =
-           zsb->z_acl_inherit == ZFS_ACL_NOALLOW;
+           zfsvfs->z_acl_inherit == ZFS_ACL_NOALLOW;
 
        *need_chmod = B_TRUE;
        pacep = NULL;
        aclp = zfs_acl_alloc(paclp->z_version);
-       if (zsb->z_acl_inherit == ZFS_ACL_DISCARD || S_ISLNK(obj_mode))
+       if (zfsvfs->z_acl_inherit == ZFS_ACL_DISCARD || S_ISLNK(obj_mode))
                return (aclp);
        while ((pacep = zfs_acl_next_ace(paclp, pacep, &who,
            &access_mask, &iflags, &type))) {
@@ -1690,7 +1710,7 @@ zfs_acl_inherit(zfs_sb_t *zsb, umode_t obj_mode, zfs_acl_t *paclp,
                        newflags &= ~ALL_INHERIT;
                        aclp->z_ops->ace_flags_set(acep,
                            newflags|ACE_INHERITED_ACE);
-                       zfs_restricted_update(zsb, aclp, acep);
+                       zfs_restricted_update(zfsvfs, aclp, acep);
                        continue;
                }
 
@@ -1723,11 +1743,9 @@ zfs_acl_ids_create(znode_t *dzp, int flag, vattr_t *vap, cred_t *cr,
     vsecattr_t *vsecp, zfs_acl_ids_t *acl_ids)
 {
        int             error;
-       zfs_sb_t        *zsb = ZTOZSB(dzp);
+       zfsvfs_t        *zfsvfs = ZTOZSB(dzp);
        zfs_acl_t       *paclp;
-#ifdef HAVE_KSID
-       gid_t           gid;
-#endif /* HAVE_KSID */
+       gid_t           gid = vap->va_gid;
        boolean_t       need_chmod = B_TRUE;
        boolean_t       inherited = B_FALSE;
 
@@ -1735,7 +1753,7 @@ zfs_acl_ids_create(znode_t *dzp, int flag, vattr_t *vap, cred_t *cr,
        acl_ids->z_mode = vap->va_mode;
 
        if (vsecp)
-               if ((error = zfs_vsec_2_aclp(zsb, vap->va_mode, vsecp,
+               if ((error = zfs_vsec_2_aclp(zfsvfs, vap->va_mode, vsecp,
                    cr, &acl_ids->z_fuidp, &acl_ids->z_aclp)) != 0)
                        return (error);
 
@@ -1745,23 +1763,23 @@ zfs_acl_ids_create(znode_t *dzp, int flag, vattr_t *vap, cred_t *cr,
        /*
         * Determine uid and gid.
         */
-       if ((flag & IS_ROOT_NODE) || zsb->z_replay ||
+       if ((flag & IS_ROOT_NODE) || zfsvfs->z_replay ||
            ((flag & IS_XATTR) && (S_ISDIR(vap->va_mode)))) {
-               acl_ids->z_fuid = zfs_fuid_create(zsb, (uint64_t)vap->va_uid,
+               acl_ids->z_fuid = zfs_fuid_create(zfsvfs, (uint64_t)vap->va_uid,
                    cr, ZFS_OWNER, &acl_ids->z_fuidp);
-               acl_ids->z_fgid = zfs_fuid_create(zsb, (uint64_t)vap->va_gid,
+               acl_ids->z_fgid = zfs_fuid_create(zfsvfs, (uint64_t)vap->va_gid,
                    cr, ZFS_GROUP, &acl_ids->z_fuidp);
                gid = vap->va_gid;
        } else {
-               acl_ids->z_fuid = zfs_fuid_create_cred(zsb, ZFS_OWNER,
+               acl_ids->z_fuid = zfs_fuid_create_cred(zfsvfs, ZFS_OWNER,
                    cr, &acl_ids->z_fuidp);
                acl_ids->z_fgid = 0;
                if (vap->va_mask & AT_GID)  {
-                       acl_ids->z_fgid = zfs_fuid_create(zsb,
+                       acl_ids->z_fgid = zfs_fuid_create(zfsvfs,
                            (uint64_t)vap->va_gid,
                            cr, ZFS_GROUP, &acl_ids->z_fuidp);
                        gid = vap->va_gid;
-                       if (acl_ids->z_fgid != dzp->z_gid &&
+                       if (acl_ids->z_fgid != KGID_TO_SGID(ZTOI(dzp)->i_gid) &&
                            !groupmember(vap->va_gid, cr) &&
                            secpolicy_vnode_create_gid(cr) != 0)
                                acl_ids->z_fgid = 0;
@@ -1771,14 +1789,15 @@ zfs_acl_ids_create(znode_t *dzp, int flag, vattr_t *vap, cred_t *cr,
                                char            *domain;
                                uint32_t        rid;
 
-                               acl_ids->z_fgid = dzp->z_gid;
-                               gid = zfs_fuid_map_id(zsb, acl_ids->z_fgid,
+                               acl_ids->z_fgid = KGID_TO_SGID(
+                                   ZTOI(dzp)->i_gid);
+                               gid = zfs_fuid_map_id(zfsvfs, acl_ids->z_fgid,
                                    cr, ZFS_GROUP);
 
-                               if (zsb->z_use_fuids &&
+                               if (zfsvfs->z_use_fuids &&
                                    IS_EPHEMERAL(acl_ids->z_fgid)) {
                                        domain = zfs_fuid_idx_domain(
-                                           &zsb->z_fuid_idx,
+                                           &zfsvfs->z_fuid_idx,
                                            FUID_INDEX(acl_ids->z_fgid));
                                        rid = FUID_RID(acl_ids->z_fgid);
                                        zfs_fuid_node_add(&acl_ids->z_fuidp,
@@ -1787,7 +1806,7 @@ zfs_acl_ids_create(znode_t *dzp, int flag, vattr_t *vap, cred_t *cr,
                                            acl_ids->z_fgid, ZFS_GROUP);
                                }
                        } else {
-                               acl_ids->z_fgid = zfs_fuid_create_cred(zsb,
+                               acl_ids->z_fgid = zfs_fuid_create_cred(zfsvfs,
                                    ZFS_GROUP, cr, &acl_ids->z_fuidp);
                                gid = crgetgid(cr);
                        }
@@ -1819,7 +1838,7 @@ zfs_acl_ids_create(znode_t *dzp, int flag, vattr_t *vap, cred_t *cr,
                    !(dzp->z_pflags & ZFS_XATTR)) {
                        VERIFY(0 == zfs_acl_node_read(dzp, B_TRUE,
                            &paclp, B_FALSE));
-                       acl_ids->z_aclp = zfs_acl_inherit(zsb,
+                       acl_ids->z_aclp = zfs_acl_inherit(zfsvfs,
                            vap->va_mode, paclp, acl_ids->z_mode, &need_chmod);
                        inherited = B_TRUE;
                } else {
@@ -1832,7 +1851,7 @@ zfs_acl_ids_create(znode_t *dzp, int flag, vattr_t *vap, cred_t *cr,
                if (need_chmod) {
                        acl_ids->z_aclp->z_hints |= S_ISDIR(vap->va_mode) ?
                            ZFS_ACL_AUTO_INHERIT : 0;
-                       zfs_acl_chmod(zsb, acl_ids->z_mode, acl_ids->z_aclp);
+                       zfs_acl_chmod(zfsvfs, acl_ids->z_mode, acl_ids->z_aclp);
                }
        }
 
@@ -1862,14 +1881,16 @@ zfs_acl_ids_free(zfs_acl_ids_t *acl_ids)
 }
 
 boolean_t
-zfs_acl_ids_overquota(zfs_sb_t *zsb, zfs_acl_ids_t *acl_ids)
+zfs_acl_ids_overquota(zfsvfs_t *zv, zfs_acl_ids_t *acl_ids, uint64_t projid)
 {
-       return (zfs_fuid_overquota(zsb, B_FALSE, acl_ids->z_fuid) ||
-           zfs_fuid_overquota(zsb, B_TRUE, acl_ids->z_fgid));
+       return (zfs_id_overquota(zv, DMU_USERUSED_OBJECT, acl_ids->z_fuid) ||
+           zfs_id_overquota(zv, DMU_GROUPUSED_OBJECT, acl_ids->z_fgid) ||
+           (projid != ZFS_DEFAULT_PROJID && projid != ZFS_INVALID_PROJID &&
+           zfs_id_overquota(zv, DMU_PROJECTUSED_OBJECT, projid)));
 }
 
 /*
- * Retrieve a files ACL
+ * Retrieve a file's ACL
  */
 int
 zfs_getacl(znode_t *zp, vsecattr_t *vsecp, boolean_t skipaclchk, cred_t *cr)
@@ -1884,7 +1905,7 @@ zfs_getacl(znode_t *zp, vsecattr_t *vsecp, boolean_t skipaclchk, cred_t *cr)
            VSA_ACE_ACLFLAGS | VSA_ACE_ALLTYPES);
 
        if (mask == 0)
-               return (ENOSYS);
+               return (SET_ERROR(ENOSYS));
 
        if ((error = zfs_zaccess(zp, ACE_READ_ACL, 0, skipaclchk, cr)))
                return (error);
@@ -1969,7 +1990,7 @@ zfs_getacl(znode_t *zp, vsecattr_t *vsecp, boolean_t skipaclchk, cred_t *cr)
 }
 
 int
-zfs_vsec_2_aclp(zfs_sb_t *zsb, umode_t obj_mode,
+zfs_vsec_2_aclp(zfsvfs_t *zfsvfs, umode_t obj_mode,
     vsecattr_t *vsecp, cred_t *cr, zfs_fuid_info_t **fuidp, zfs_acl_t **zaclp)
 {
        zfs_acl_t *aclp;
@@ -1978,9 +1999,9 @@ zfs_vsec_2_aclp(zfs_sb_t *zsb, umode_t obj_mode,
        int error;
 
        if (vsecp->vsa_aclcnt > MAX_ACL_ENTRIES || vsecp->vsa_aclcnt <= 0)
-               return (EINVAL);
+               return (SET_ERROR(EINVAL));
 
-       aclp = zfs_acl_alloc(zfs_acl_version(zsb->z_version));
+       aclp = zfs_acl_alloc(zfs_acl_version(zfsvfs->z_version));
 
        aclp->z_hints = 0;
        aclnode = zfs_acl_node_alloc(aclcnt * sizeof (zfs_object_ace_t));
@@ -1993,7 +2014,7 @@ zfs_vsec_2_aclp(zfs_sb_t *zsb, umode_t obj_mode,
                        return (error);
                }
        } else {
-               if ((error = zfs_copy_ace_2_fuid(zsb, obj_mode, aclp,
+               if ((error = zfs_copy_ace_2_fuid(zfsvfs, obj_mode, aclp,
                    vsecp->vsa_aclentp, aclnode->z_acldata, aclcnt,
                    &aclnode->z_size, fuidp, cr)) != 0) {
                        zfs_acl_free(aclp);
@@ -2024,13 +2045,13 @@ zfs_vsec_2_aclp(zfs_sb_t *zsb, umode_t obj_mode,
 }
 
 /*
- * Set a files ACL
+ * Set a file's ACL
  */
 int
 zfs_setacl(znode_t *zp, vsecattr_t *vsecp, boolean_t skipaclchk, cred_t *cr)
 {
-       zfs_sb_t        *zsb = ZTOZSB(zp);
-       zilog_t         *zilog = zsb->z_log;
+       zfsvfs_t        *zfsvfs = ZTOZSB(zp);
+       zilog_t         *zilog = zfsvfs->z_log;
        ulong_t         mask = vsecp->vsa_mask & (VSA_ACE | VSA_ACECNT);
        dmu_tx_t        *tx;
        int             error;
@@ -2040,15 +2061,15 @@ zfs_setacl(znode_t *zp, vsecattr_t *vsecp, boolean_t skipaclchk, cred_t *cr)
        uint64_t        acl_obj;
 
        if (mask == 0)
-               return (ENOSYS);
+               return (SET_ERROR(ENOSYS));
 
        if (zp->z_pflags & ZFS_IMMUTABLE)
-               return (EPERM);
+               return (SET_ERROR(EPERM));
 
        if ((error = zfs_zaccess(zp, ACE_WRITE_ACL, 0, skipaclchk, cr)))
                return (error);
 
-       error = zfs_vsec_2_aclp(zsb, ZTOI(zp)->i_mode, vsecp, cr, &fuidp,
+       error = zfs_vsec_2_aclp(zfsvfs, ZTOI(zp)->i_mode, vsecp, cr, &fuidp,
            &aclp);
        if (error)
                return (error);
@@ -2065,13 +2086,13 @@ top:
        mutex_enter(&zp->z_acl_lock);
        mutex_enter(&zp->z_lock);
 
-       tx = dmu_tx_create(zsb->z_os);
+       tx = dmu_tx_create(zfsvfs->z_os);
 
        dmu_tx_hold_sa(tx, zp->z_sa_hdl, B_TRUE);
 
-       fuid_dirtied = zsb->z_fuid_dirty;
+       fuid_dirtied = zfsvfs->z_fuid_dirty;
        if (fuid_dirtied)
-               zfs_fuid_txhold(zsb, tx);
+               zfs_fuid_txhold(zfsvfs, tx);
 
        /*
         * If old version and ACL won't fit in bonus and we aren't
@@ -2079,7 +2100,7 @@ top:
         */
 
        if ((acl_obj = zfs_external_acl(zp)) != 0) {
-               if (zsb->z_version >= ZPL_VERSION_FUID &&
+               if (zfsvfs->z_version >= ZPL_VERSION_FUID &&
                    zfs_znode_acl_version(zp) <= ZFS_ACL_VERSION_INITIAL) {
                        dmu_tx_hold_free(tx, acl_obj, 0,
                            DMU_OBJECT_END);
@@ -2114,7 +2135,7 @@ top:
        zp->z_acl_cached = aclp;
 
        if (fuid_dirtied)
-               zfs_fuid_sync(zsb, tx);
+               zfs_fuid_sync(zfsvfs, tx);
 
        zfs_log_acl(zilog, tx, zp, vsecp, fuidp);
 
@@ -2139,7 +2160,7 @@ zfs_zaccess_dataset_check(znode_t *zp, uint32_t v4_mode)
        if ((v4_mode & WRITE_MASK) && (zfs_is_readonly(ZTOZSB(zp))) &&
            (!S_ISDEV(ZTOI(zp)->i_mode) ||
            (S_ISDEV(ZTOI(zp)->i_mode) && (v4_mode & WRITE_MASK_ATTRS)))) {
-               return (EROFS);
+               return (SET_ERROR(EROFS));
        }
 
        /*
@@ -2150,17 +2171,17 @@ zfs_zaccess_dataset_check(znode_t *zp, uint32_t v4_mode)
            (zp->z_pflags & (ZFS_READONLY | ZFS_IMMUTABLE))) ||
            (S_ISDIR(ZTOI(zp)->i_mode) &&
            (zp->z_pflags & ZFS_IMMUTABLE)))) {
-               return (EPERM);
+               return (SET_ERROR(EPERM));
        }
 
        if ((v4_mode & (ACE_DELETE | ACE_DELETE_CHILD)) &&
            (zp->z_pflags & ZFS_NOUNLINK)) {
-               return (EPERM);
+               return (SET_ERROR(EPERM));
        }
 
        if (((v4_mode & (ACE_READ_DATA|ACE_EXECUTE)) &&
            (zp->z_pflags & ZFS_AV_QUARANTINED))) {
-               return (EACCES);
+               return (SET_ERROR(EACCES));
        }
 
        return (0);
@@ -2181,7 +2202,7 @@ zfs_zaccess_dataset_check(znode_t *zp, uint32_t v4_mode)
  * placed into the working_mode, giving the caller a mask of denied
  * accesses.  Returns:
  *     0               if all AoI granted
- *     EACCESS         if the denied mask is non-zero
+ *     EACCES          if the denied mask is non-zero
  *     other error     if abnormal failure (e.g., IO error)
  *
  * A secondary usage of the function is to determine if any of the
@@ -2195,7 +2216,7 @@ static int
 zfs_zaccess_aces_check(znode_t *zp, uint32_t *working_mode,
     boolean_t anyaccess, cred_t *cr)
 {
-       zfs_sb_t        *zsb = ZTOZSB(zp);
+       zfsvfs_t        *zfsvfs = ZTOZSB(zp);
        zfs_acl_t       *aclp;
        int             error;
        uid_t           uid = crgetuid(cr);
@@ -2250,7 +2271,7 @@ zfs_zaccess_aces_check(znode_t *zp, uint32_t *working_mode,
                        who = gowner;
                        /*FALLTHROUGH*/
                case ACE_IDENTIFIER_GROUP:
-                       checkit = zfs_groupmember(zsb, who, cr);
+                       checkit = zfs_groupmember(zfsvfs, who, cr);
                        break;
                case ACE_EVERYONE:
                        checkit = B_TRUE;
@@ -2261,7 +2282,7 @@ zfs_zaccess_aces_check(znode_t *zp, uint32_t *working_mode,
                        if (entry_type == 0) {
                                uid_t newid;
 
-                               newid = zfs_fuid_map_id(zsb, who, cr,
+                               newid = zfs_fuid_map_id(zfsvfs, who, cr,
                                    ZFS_ACE_USER);
                                if (newid != IDMAP_WK_CREATOR_OWNER_UID &&
                                    uid == newid)
@@ -2269,7 +2290,7 @@ zfs_zaccess_aces_check(znode_t *zp, uint32_t *working_mode,
                                break;
                        } else {
                                mutex_exit(&zp->z_acl_lock);
-                               return (EIO);
+                               return (SET_ERROR(EIO));
                        }
                }
 
@@ -2303,7 +2324,7 @@ zfs_zaccess_aces_check(znode_t *zp, uint32_t *working_mode,
        /* Put the found 'denies' back on the working mode */
        if (deny_mask) {
                *working_mode |= deny_mask;
-               return (EACCES);
+               return (SET_ERROR(EACCES));
        } else if (*working_mode) {
                return (-1);
        }
@@ -2323,7 +2344,8 @@ zfs_has_access(znode_t *zp, cred_t *cr)
        if (zfs_zaccess_aces_check(zp, &have, B_TRUE, cr) != 0) {
                uid_t owner;
 
-               owner = zfs_fuid_map_id(ZTOZSB(zp), zp->z_uid, cr, ZFS_OWNER);
+               owner = zfs_fuid_map_id(ZTOZSB(zp),
+                   KUID_TO_SUID(ZTOI(zp)->i_uid), cr, ZFS_OWNER);
                return (secpolicy_vnode_any_access(cr, ZTOI(zp), owner) == 0);
        }
        return (B_TRUE);
@@ -2333,7 +2355,7 @@ static int
 zfs_zaccess_common(znode_t *zp, uint32_t v4_mode, uint32_t *working_mode,
     boolean_t *check_privs, boolean_t skipaclchk, cred_t *cr)
 {
-       zfs_sb_t *zsb = ZTOZSB(zp);
+       zfsvfs_t *zfsvfs = ZTOZSB(zp);
        int err;
 
        *working_mode = v4_mode;
@@ -2342,7 +2364,7 @@ zfs_zaccess_common(znode_t *zp, uint32_t v4_mode, uint32_t *working_mode,
        /*
         * Short circuit empty requests
         */
-       if (v4_mode == 0 || zsb->z_replay) {
+       if (v4_mode == 0 || zfsvfs->z_replay) {
                *working_mode = 0;
                return (0);
        }
@@ -2370,7 +2392,7 @@ zfs_zaccess_append(znode_t *zp, uint32_t *working_mode, boolean_t *check_privs,
     cred_t *cr)
 {
        if (*working_mode != ACE_WRITE_DATA)
-               return (EACCES);
+               return (SET_ERROR(EACCES));
 
        return (zfs_zaccess_common(zp, ACE_APPEND_DATA, working_mode,
            check_privs, B_FALSE, cr));
@@ -2386,7 +2408,7 @@ zfs_fastaccesschk_execute(znode_t *zdp, cred_t *cr)
        int error;
 
        if (zdp->z_pflags & ZFS_AV_QUARANTINED)
-               return (EACCES);
+               return (SET_ERROR(EACCES));
 
        is_attr = ((zdp->z_pflags & ZFS_XATTR) &&
            (S_ISDIR(ZTOI(zdp)->i_mode)));
@@ -2401,12 +2423,13 @@ zfs_fastaccesschk_execute(znode_t *zdp, cred_t *cr)
                return (0);
        }
 
-       if (FUID_INDEX(zdp->z_uid) != 0 || FUID_INDEX(zdp->z_gid) != 0) {
+       if (KUID_TO_SUID(ZTOI(zdp)->i_uid) != 0 ||
+           KGID_TO_SGID(ZTOI(zdp)->i_gid) != 0) {
                mutex_exit(&zdp->z_acl_lock);
                goto slow;
        }
 
-       if (uid == zdp->z_uid) {
+       if (uid == KUID_TO_SUID(ZTOI(zdp)->i_uid)) {
                owner = B_TRUE;
                if (zdp->z_mode & S_IXUSR) {
                        mutex_exit(&zdp->z_acl_lock);
@@ -2416,7 +2439,7 @@ zfs_fastaccesschk_execute(znode_t *zdp, cred_t *cr)
                        goto slow;
                }
        }
-       if (groupmember(zdp->z_gid, cr)) {
+       if (groupmember(KGID_TO_SGID(ZTOI(zdp)->i_gid), cr)) {
                groupmbr = B_TRUE;
                if (zdp->z_mode & S_IXGRP) {
                        mutex_exit(&zdp->z_acl_lock);
@@ -2445,7 +2468,8 @@ slow:
 
 /*
  * Determine whether Access should be granted/denied.
- * The least priv subsytem is always consulted as a basic privilege
+ *
+ * The least priv subsystem is always consulted as a basic privilege
  * can define any form of access.
  */
 int
@@ -2454,9 +2478,9 @@ zfs_zaccess(znode_t *zp, int mode, int flags, boolean_t skipaclchk, cred_t *cr)
        uint32_t        working_mode;
        int             error;
        int             is_attr;
-       boolean_t       check_privs;
+       boolean_t       check_privs;
        znode_t         *xzp;
-       znode_t         *check_zp = zp;
+       znode_t         *check_zp = zp;
        mode_t          needed_bits;
        uid_t           owner;
 
@@ -2466,15 +2490,8 @@ zfs_zaccess(znode_t *zp, int mode, int flags, boolean_t skipaclchk, cred_t *cr)
         * If attribute then validate against base file
         */
        if (is_attr) {
-               uint64_t        parent;
-
-               if ((error = sa_lookup(zp->z_sa_hdl,
-                   SA_ZPL_PARENT(ZTOZSB(zp)), &parent,
-                   sizeof (parent))) != 0)
-                       return (error);
-
                if ((error = zfs_zget(ZTOZSB(zp),
-                   parent, &xzp)) != 0)        {
+                   zp->z_xattr_parent, &xzp)) != 0) {
                        return (error);
                }
 
@@ -2495,7 +2512,8 @@ zfs_zaccess(znode_t *zp, int mode, int flags, boolean_t skipaclchk, cred_t *cr)
                }
        }
 
-       owner = zfs_fuid_map_id(ZTOZSB(zp), zp->z_uid, cr, ZFS_OWNER);
+       owner = zfs_fuid_map_id(ZTOZSB(zp), KUID_TO_SUID(ZTOI(zp)->i_uid),
+           cr, ZFS_OWNER);
        /*
         * Map the bits required to the standard inode flags
         * S_IRUSR|S_IWUSR|S_IXUSR in the needed_bits.  Map the bits
@@ -2582,7 +2600,7 @@ zfs_zaccess(znode_t *zp, int mode, int flags, boolean_t skipaclchk, cred_t *cr)
                         * for are still present.  If so then return EACCES
                         */
                        if (working_mode & ~(ZFS_CHECKED_MASKS)) {
-                               error = EACCES;
+                               error = SET_ERROR(EACCES);
                        }
                }
        } else if (error == 0) {
@@ -2590,7 +2608,6 @@ zfs_zaccess(znode_t *zp, int mode, int flags, boolean_t skipaclchk, cred_t *cr)
                    needed_bits, needed_bits);
        }
 
-
        if (is_attr)
                iput(ZTOI(xzp));
 
@@ -2625,7 +2642,8 @@ zfs_delete_final_check(znode_t *zp, znode_t *dzp,
        int error;
        uid_t downer;
 
-       downer = zfs_fuid_map_id(ZTOZSB(dzp), dzp->z_uid, cr, ZFS_OWNER);
+       downer = zfs_fuid_map_id(ZTOZSB(dzp), KUID_TO_SUID(ZTOI(dzp)->i_uid),
+           cr, ZFS_OWNER);
 
        error = secpolicy_vnode_access2(cr, ZTOI(dzp),
            downer, available_perms, S_IWUSR|S_IXUSR);
@@ -2640,7 +2658,6 @@ zfs_delete_final_check(znode_t *zp, znode_t *dzp,
  * Determine whether Access should be granted/deny, without
  * consulting least priv subsystem.
  *
- *
  * The following chart is the recommended NFSv4 enforcement for
  * ability to delete an object.
  *
@@ -2696,7 +2713,7 @@ zfs_zaccess_delete(znode_t *dzp, znode_t *zp, cred_t *cr)
         */
 
        if (zp->z_pflags & (ZFS_IMMUTABLE | ZFS_NOUNLINK))
-               return (EPERM);
+               return (SET_ERROR(EPERM));
 
        /*
         * First row
@@ -2763,7 +2780,7 @@ zfs_zaccess_rename(znode_t *sdzp, znode_t *szp, znode_t *tdzp,
        int error;
 
        if (szp->z_pflags & ZFS_AV_QUARANTINED)
-               return (EACCES);
+               return (SET_ERROR(EACCES));
 
        add_perm = S_ISDIR(ZTOI(szp)->i_mode) ?
            ACE_ADD_SUBDIRECTORY : ACE_ADD_FILE;