]> git.proxmox.com Git - mirror_zfs.git/blobdiff - module/zfs/zfs_vnops.c
Introduce read/write kstats per dataset
[mirror_zfs.git] / module / zfs / zfs_vnops.c
index 62241a46b1a65d0e3468027698046bcb822ef24f..4e163e2e3fe8864197987cbcdd5ea6a50f1ad8e6 100644 (file)
@@ -21,7 +21,7 @@
 
 /*
  * Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
- * Copyright (c) 2012, 2015 by Delphix. All rights reserved.
+ * Copyright (c) 2012, 2018 by Delphix. All rights reserved.
  * Copyright (c) 2015 by Chunwei Chen. All rights reserved.
  * Copyright 2017 Nexenta Systems, Inc.
  */
 #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/vfs_opreg.h>
 #include <sys/file.h>
 #include <sys/stat.h>
 #include <sys/kmem.h>
 #include <sys/uio.h>
 #include <sys/vmsystm.h>
 #include <sys/atomic.h>
-#include <vm/pvn.h>
 #include <sys/pathname.h>
 #include <sys/cmn_err.h>
 #include <sys/errno.h>
-#include <sys/unistd.h>
 #include <sys/zfs_dir.h>
 #include <sys/zfs_acl.h>
 #include <sys/zfs_ioctl.h>
 #include <sys/dbuf.h>
 #include <sys/zap.h>
 #include <sys/sa.h>
-#include <sys/dirent.h>
 #include <sys/policy.h>
 #include <sys/sunddi.h>
 #include <sys/sid.h>
 #include <sys/mode.h>
-#include "fs/fs_subr.h"
 #include <sys/zfs_ctldir.h>
 #include <sys/zfs_fuid.h>
 #include <sys/zfs_sa.h>
 #include <sys/zfs_vnops.h>
-#include <sys/dnlc.h>
 #include <sys/zfs_rlock.h>
-#include <sys/extdirent.h>
-#include <sys/kidmap.h>
 #include <sys/cred.h>
-#include <sys/attr.h>
 #include <sys/zpl.h>
+#include <sys/zil.h>
+#include <sys/sa_impl.h>
 
 /*
  * Programming rules.
  *
  *     If dmu_tx_assign() returns ERESTART and zfsvfs->z_assign is TXG_NOWAIT,
  *     then drop all locks, call dmu_tx_wait(), and try again.  On subsequent
- *     calls to dmu_tx_assign(), pass TXG_WAITED rather than TXG_NOWAIT,
+ *     calls to dmu_tx_assign(), pass TXG_NOTHROTTLE in addition to TXG_NOWAIT,
  *     to indicate that this operation has already called dmu_tx_wait().
  *     This will ensure that we don't retry forever, waiting a short bit
  *     each time.
  *     rw_enter(...);                  // grab any other locks you need
  *     tx = dmu_tx_create(...);        // get DMU tx
  *     dmu_tx_hold_*();                // hold each object you might modify
- *     error = dmu_tx_assign(tx, waited ? TXG_WAITED : TXG_NOWAIT);
+ *     error = dmu_tx_assign(tx, (waited ? TXG_NOTHROTTLE : 0) | TXG_NOWAIT);
  *     if (error) {
  *             rw_exit(...);           // drop locks
  *             zfs_dirent_unlock(dl);  // unlock directory entry
@@ -398,6 +389,7 @@ mappedread(struct inode *ip, int nbytes, uio_t *uio)
                pp = find_lock_page(mp, start >> PAGE_SHIFT);
                if (pp) {
                        ASSERT(PageUptodate(pp));
+                       unlock_page(pp);
 
                        pb = kmap(pp);
                        error = uiomove(pb + off, bytes, UIO_READ, uio);
@@ -407,7 +399,6 @@ mappedread(struct inode *ip, int nbytes, uio_t *uio)
                                flush_dcache_page(pp);
 
                        mark_page_accessed(pp);
-                       unlock_page(pp);
                        put_page(pp);
                } else {
                        error = dmu_read_uio_dbuf(sa_get_db(zp->z_sa_hdl),
@@ -447,15 +438,10 @@ unsigned long zfs_delete_blocks = DMU_MAX_DELETEBLKCNT;
 int
 zfs_read(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
 {
-       znode_t         *zp = ITOZ(ip);
-       zfsvfs_t        *zfsvfs = ITOZSB(ip);
-       ssize_t         n, nbytes;
-       int             error = 0;
-       rl_t            *rl;
-#ifdef HAVE_UIO_ZEROCOPY
-       xuio_t          *xuio = NULL;
-#endif /* HAVE_UIO_ZEROCOPY */
+       int error = 0;
 
+       znode_t *zp = ITOZ(ip);
+       zfsvfs_t *zfsvfs = ITOZSB(ip);
        ZFS_ENTER(zfsvfs);
        ZFS_VERIFY_ZP(zp);
 
@@ -491,8 +477,8 @@ zfs_read(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
        /*
         * Lock the range against changes.
         */
-       rl = zfs_range_lock(&zp->z_range_lock, uio->uio_loffset, uio->uio_resid,
-           RL_READER);
+       rl_t *rl = zfs_range_lock(&zp->z_range_lock,
+           uio->uio_loffset, uio->uio_resid, RL_READER);
 
        /*
         * If we are reading past end-of-file we can skip
@@ -504,9 +490,11 @@ zfs_read(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
        }
 
        ASSERT(uio->uio_loffset < zp->z_size);
-       n = MIN(uio->uio_resid, zp->z_size - uio->uio_loffset);
+       ssize_t n = MIN(uio->uio_resid, zp->z_size - uio->uio_loffset);
+       ssize_t start_resid = n;
 
 #ifdef HAVE_UIO_ZEROCOPY
+       xuio_t *xuio = NULL;
        if ((uio->uio_extflg == UIO_XUIO) &&
            (((xuio_t *)uio)->xu_type == UIOTYPE_ZEROCOPY)) {
                int nblk;
@@ -538,7 +526,7 @@ zfs_read(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
 #endif /* HAVE_UIO_ZEROCOPY */
 
        while (n > 0) {
-               nbytes = MIN(n, zfs_read_chunk_size -
+               ssize_t nbytes = MIN(n, zfs_read_chunk_size -
                    P2PHASE(uio->uio_loffset, zfs_read_chunk_size));
 
                if (zp->z_is_mapped && !(ioflag & O_DIRECT)) {
@@ -557,6 +545,10 @@ zfs_read(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
 
                n -= nbytes;
        }
+
+       int64_t nread = start_resid - n;
+       dataset_kstats_update_read_kstats(&zfsvfs->z_kstat, nread);
+       task_io_account_read(nread);
 out:
        zfs_range_unlock(rl);
 
@@ -587,46 +579,28 @@ out:
 int
 zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
 {
-       znode_t         *zp = ITOZ(ip);
-       rlim64_t        limit = uio->uio_limit;
-       ssize_t         start_resid = uio->uio_resid;
-       ssize_t         tx_bytes;
-       uint64_t        end_size;
-       dmu_tx_t        *tx;
-       zfsvfs_t        *zfsvfs = ZTOZSB(zp);
-       zilog_t         *zilog;
-       offset_t        woff;
-       ssize_t         n, nbytes;
-       rl_t            *rl;
-       int             max_blksz = zfsvfs->z_max_blksz;
-       int             error = 0;
-       arc_buf_t       *abuf;
-       const iovec_t   *aiov = NULL;
-       xuio_t          *xuio = NULL;
-       int             write_eof;
-       int             count = 0;
-       sa_bulk_attr_t  bulk[4];
-       uint64_t        mtime[2], ctime[2];
-       uint32_t        uid;
-#ifdef HAVE_UIO_ZEROCOPY
-       int             i_iov = 0;
-       const iovec_t   *iovp = uio->uio_iov;
-       ASSERTV(int     iovcnt = uio->uio_iovcnt);
-#endif
+       int error = 0;
+       ssize_t start_resid = uio->uio_resid;
 
        /*
         * Fasttrack empty write
         */
-       n = start_resid;
+       ssize_t n = start_resid;
        if (n == 0)
                return (0);
 
+       rlim64_t limit = uio->uio_limit;
        if (limit == RLIM64_INFINITY || limit > MAXOFFSET_T)
                limit = MAXOFFSET_T;
 
+       znode_t *zp = ITOZ(ip);
+       zfsvfs_t *zfsvfs = ZTOZSB(zp);
        ZFS_ENTER(zfsvfs);
        ZFS_VERIFY_ZP(zp);
 
+       sa_bulk_attr_t bulk[4];
+       int count = 0;
+       uint64_t mtime[2], ctime[2];
        SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_MTIME(zfsvfs), NULL, &mtime, 16);
        SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_CTIME(zfsvfs), NULL, &ctime, 16);
        SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_SIZE(zfsvfs), NULL,
@@ -653,17 +627,18 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
                return (SET_ERROR(EPERM));
        }
 
-       zilog = zfsvfs->z_log;
-
        /*
         * Validate file offset
         */
-       woff = ioflag & FAPPEND ? zp->z_size : uio->uio_loffset;
+       offset_t woff = ioflag & FAPPEND ? zp->z_size : uio->uio_loffset;
        if (woff < 0) {
                ZFS_EXIT(zfsvfs);
                return (SET_ERROR(EINVAL));
        }
 
+       int max_blksz = zfsvfs->z_max_blksz;
+       xuio_t *xuio = NULL;
+
        /*
         * Pre-fault the pages to ensure slow (eg NFS) pages
         * don't hold up txg.
@@ -677,6 +652,8 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
 #endif
                uio_prefaultpages(MIN(n, max_blksz), uio);
 
+       rl_t     *rl;
+
        /*
         * If in append mode, set the io offset pointer to eof.
         */
@@ -715,9 +692,16 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
                n = limit - woff;
 
        /* Will this write extend the file length? */
-       write_eof = (woff + n > zp->z_size);
+       int write_eof = (woff + n > zp->z_size);
+
+       uint64_t end_size = MAX(zp->z_size, woff + n);
+       zilog_t *zilog = zfsvfs->z_log;
+#ifdef HAVE_UIO_ZEROCOPY
+       int             i_iov = 0;
+       const iovec_t   *iovp = uio->uio_iov;
+       ASSERTV(int     iovcnt = uio->uio_iovcnt);
+#endif
 
-       end_size = MAX(zp->z_size, woff + n);
 
        /*
         * Write the file in reasonable size chunks.  Each chunk is written
@@ -725,17 +709,22 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
         * and allows us to do more fine-grained space accounting.
         */
        while (n > 0) {
-               abuf = NULL;
                woff = uio->uio_loffset;
-               if (zfs_owner_overquota(zfsvfs, zp, B_FALSE) ||
-                   zfs_owner_overquota(zfsvfs, zp, B_TRUE)) {
-                       if (abuf != NULL)
-                               dmu_return_arcbuf(abuf);
+
+               if (zfs_id_overblockquota(zfsvfs, DMU_USERUSED_OBJECT,
+                   KUID_TO_SUID(ip->i_uid)) ||
+                   zfs_id_overblockquota(zfsvfs, DMU_GROUPUSED_OBJECT,
+                   KGID_TO_SGID(ip->i_gid)) ||
+                   (zp->z_projid != ZFS_DEFAULT_PROJID &&
+                   zfs_id_overblockquota(zfsvfs, DMU_PROJECTUSED_OBJECT,
+                   zp->z_projid))) {
                        error = SET_ERROR(EDQUOT);
                        break;
                }
 
-               if (xuio && abuf == NULL) {
+               arc_buf_t *abuf = NULL;
+               const iovec_t *aiov = NULL;
+               if (xuio) {
 #ifdef HAVE_UIO_ZEROCOPY
                        ASSERT(i_iov < iovcnt);
                        ASSERT3U(uio->uio_segflg, !=, UIO_BVEC);
@@ -747,8 +736,7 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
                            aiov->iov_len == arc_buf_size(abuf)));
                        i_iov++;
 #endif
-               } else if (abuf == NULL && n >= max_blksz &&
-                   woff >= zp->z_size &&
+               } else if (n >= max_blksz && woff >= zp->z_size &&
                    P2PHASE(woff, max_blksz) == 0 &&
                    zp->z_blksz == max_blksz) {
                        /*
@@ -775,7 +763,7 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
                /*
                 * Start a transaction.
                 */
-               tx = dmu_tx_create(zfsvfs->z_os);
+               dmu_tx_t *tx = dmu_tx_create(zfsvfs->z_os);
                dmu_tx_hold_sa(tx, zp->z_sa_hdl, B_FALSE);
                dmu_tx_hold_write(tx, zp->z_id, woff, MIN(n, max_blksz));
                zfs_sa_upgrade_txholds(tx, zp);
@@ -816,8 +804,9 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
                 * XXX - should we really limit each write to z_max_blksz?
                 * Perhaps we should use SPA_MAXBLOCKSIZE chunks?
                 */
-               nbytes = MIN(n, max_blksz - P2PHASE(woff, max_blksz));
+               ssize_t nbytes = MIN(n, max_blksz - P2PHASE(woff, max_blksz));
 
+               ssize_t tx_bytes;
                if (abuf == NULL) {
                        tx_bytes = uio->uio_resid;
                        error = dmu_write_uio_dbuf(sa_get_db(zp->z_sa_hdl),
@@ -836,13 +825,14 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
                            aiov->iov_base != abuf->b_data)) {
                                ASSERT(xuio);
                                dmu_write(zfsvfs->z_os, zp->z_id, woff,
+                                   /* cppcheck-suppress nullPointer */
                                    aiov->iov_len, aiov->iov_base, tx);
                                dmu_return_arcbuf(abuf);
                                xuio_stat_wbuf_copied();
                        } else {
                                ASSERT(xuio || tx_bytes == max_blksz);
-                               dmu_assign_arcbuf(sa_get_db(zp->z_sa_hdl),
-                                   woff, abuf, tx);
+                               dmu_assign_arcbuf_by_dbuf(
+                                   sa_get_db(zp->z_sa_hdl), woff, abuf, tx);
                        }
                        ASSERT(tx_bytes <= uio->uio_resid);
                        uioskip(uio, tx_bytes);
@@ -876,7 +866,7 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
                 * user 0 is not an ephemeral uid.
                 */
                mutex_enter(&zp->z_acl_lock);
-               uid = KUID_TO_SUID(ip->i_uid);
+               uint32_t uid = KUID_TO_SUID(ip->i_uid);
                if ((zp->z_mode & (S_IXUSR | (S_IXUSR >> 3) |
                    (S_IXUSR >> 6))) != 0 &&
                    (zp->z_mode & (S_ISUID | S_ISGID)) != 0 &&
@@ -940,6 +930,10 @@ zfs_write(struct inode *ip, uio_t *uio, int ioflag, cred_t *cr)
            zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
                zil_commit(zilog, zp->z_id);
 
+       int64_t nwritten = start_resid - uio->uio_resid;
+       dataset_kstats_update_write_kstats(&zfsvfs->z_kstat, nwritten);
+       task_io_account_write(nwritten);
+
        ZFS_EXIT(zfsvfs);
        return (0);
 }
@@ -982,7 +976,7 @@ zfs_get_done(zgd_t *zgd, int error)
        zfs_iput_async(ZTOI(zp));
 
        if (error == 0 && zgd->zgd_bp)
-               zil_add_block(zgd->zgd_zilog, zgd->zgd_bp);
+               zil_lwb_add_block(zgd->zgd_lwb, zgd->zgd_bp);
 
        kmem_free(zgd, sizeof (zgd_t));
 }
@@ -995,7 +989,7 @@ static int zil_fault_io = 0;
  * Get data to generate a TX_WRITE intent log record.
  */
 int
-zfs_get_data(void *arg, lr_write_t *lr, char *buf, zio_t *zio)
+zfs_get_data(void *arg, lr_write_t *lr, char *buf, struct lwb *lwb, zio_t *zio)
 {
        zfsvfs_t *zfsvfs = arg;
        objset_t *os = zfsvfs->z_os;
@@ -1007,8 +1001,9 @@ zfs_get_data(void *arg, lr_write_t *lr, char *buf, zio_t *zio)
        zgd_t *zgd;
        int error = 0;
 
-       ASSERT(zio != NULL);
-       ASSERT(size != 0);
+       ASSERT3P(lwb, !=, NULL);
+       ASSERT3P(zio, !=, NULL);
+       ASSERT3U(size, !=, 0);
 
        /*
         * Nothing to do if the file has been removed
@@ -1025,7 +1020,7 @@ zfs_get_data(void *arg, lr_write_t *lr, char *buf, zio_t *zio)
        }
 
        zgd = (zgd_t *)kmem_zalloc(sizeof (zgd_t), KM_SLEEP);
-       zgd->zgd_zilog = zfsvfs->z_log;
+       zgd->zgd_lwb = lwb;
        zgd->zgd_private = zp;
 
        /*
@@ -1102,6 +1097,18 @@ zfs_get_data(void *arg, lr_write_t *lr, char *buf, zio_t *zio)
 
                        if (error == EALREADY) {
                                lr->lr_common.lrc_txtype = TX_WRITE2;
+                               /*
+                                * TX_WRITE2 relies on the data previously
+                                * written by the TX_WRITE that caused
+                                * EALREADY.  We zero out the BP because
+                                * it is the old, currently-on-disk BP,
+                                * so there's no need to zio_flush() its
+                                * vdevs (flushing would needlesly hurt
+                                * performance, and doesn't work on
+                                * indirect vdevs).
+                                */
+                               zgd->zgd_bp = NULL;
+                               BP_ZERO(bp);
                                error = 0;
                        }
                }
@@ -1377,6 +1384,7 @@ top:
 
        if (zp == NULL) {
                uint64_t txtype;
+               uint64_t projid = ZFS_DEFAULT_PROJID;
 
                /*
                 * Create a new file object and update the directory
@@ -1405,7 +1413,9 @@ top:
                        goto out;
                have_acl = B_TRUE;
 
-               if (zfs_acl_ids_overquota(zfsvfs, &acl_ids)) {
+               if (S_ISREG(vap->va_mode) || S_ISDIR(vap->va_mode))
+                       projid = zfs_inherit_projid(dzp);
+               if (zfs_acl_ids_overquota(zfsvfs, &acl_ids, projid)) {
                        zfs_acl_ids_free(&acl_ids);
                        error = SET_ERROR(EDQUOT);
                        goto out;
@@ -1426,7 +1436,9 @@ top:
                        dmu_tx_hold_write(tx, DMU_NEW_OBJECT,
                            0, acl_ids.z_aclp->z_acl_bytes);
                }
-               error = dmu_tx_assign(tx, waited ? TXG_WAITED : TXG_NOWAIT);
+
+               error = dmu_tx_assign(tx,
+                   (waited ? TXG_NOTHROTTLE : 0) | TXG_NOWAIT);
                if (error) {
                        zfs_dirent_unlock(dl);
                        if (error == ERESTART) {
@@ -1442,10 +1454,22 @@ top:
                }
                zfs_mknode(dzp, vap, tx, cr, 0, &zp, &acl_ids);
 
+               error = zfs_link_create(dl, zp, tx, ZNEW);
+               if (error != 0) {
+                       /*
+                        * Since, we failed to add the directory entry for it,
+                        * delete the newly created dnode.
+                        */
+                       zfs_znode_delete(zp, tx);
+                       remove_inode_hash(ZTOI(zp));
+                       zfs_acl_ids_free(&acl_ids);
+                       dmu_tx_commit(tx);
+                       goto out;
+               }
+
                if (fuid_dirtied)
                        zfs_fuid_sync(zfsvfs, tx);
 
-               (void) zfs_link_create(dl, zp, tx, ZNEW);
                txtype = zfs_log_create_txtype(Z_FILE, vsecp, vap);
                if (flag & FIGNORECASE)
                        txtype |= TX_CI;
@@ -1535,6 +1559,7 @@ zfs_tmpfile(struct inode *dip, vattr_t *vap, int excl,
        uid_t           uid;
        gid_t           gid;
        zfs_acl_ids_t   acl_ids;
+       uint64_t        projid = ZFS_DEFAULT_PROJID;
        boolean_t       fuid_dirtied;
        boolean_t       have_acl = B_FALSE;
        boolean_t       waited = B_FALSE;
@@ -1581,7 +1606,9 @@ top:
                goto out;
        have_acl = B_TRUE;
 
-       if (zfs_acl_ids_overquota(zfsvfs, &acl_ids)) {
+       if (S_ISREG(vap->va_mode) || S_ISDIR(vap->va_mode))
+               projid = zfs_inherit_projid(dzp);
+       if (zfs_acl_ids_overquota(zfsvfs, &acl_ids, projid)) {
                zfs_acl_ids_free(&acl_ids);
                error = SET_ERROR(EDQUOT);
                goto out;
@@ -1601,7 +1628,7 @@ top:
                dmu_tx_hold_write(tx, DMU_NEW_OBJECT,
                    0, acl_ids.z_aclp->z_acl_bytes);
        }
-       error = dmu_tx_assign(tx, waited ? TXG_WAITED : TXG_NOWAIT);
+       error = dmu_tx_assign(tx, (waited ? TXG_NOTHROTTLE : 0) | TXG_NOWAIT);
        if (error) {
                if (error == ERESTART) {
                        waited = B_TRUE;
@@ -1774,7 +1801,7 @@ top:
         */
        dmu_tx_mark_netfree(tx);
 
-       error = dmu_tx_assign(tx, waited ? TXG_WAITED : TXG_NOWAIT);
+       error = dmu_tx_assign(tx, (waited ? TXG_NOTHROTTLE : 0) | TXG_NOWAIT);
        if (error) {
                zfs_dirent_unlock(dl);
                if (error == ERESTART) {
@@ -1992,7 +2019,7 @@ top:
                return (error);
        }
 
-       if (zfs_acl_ids_overquota(zfsvfs, &acl_ids)) {
+       if (zfs_acl_ids_overquota(zfsvfs, &acl_ids, zfs_inherit_projid(dzp))) {
                zfs_acl_ids_free(&acl_ids);
                zfs_dirent_unlock(dl);
                ZFS_EXIT(zfsvfs);
@@ -2016,7 +2043,7 @@ top:
        dmu_tx_hold_sa_create(tx, acl_ids.z_aclp->z_acl_bytes +
            ZFS_SA_BASE_ATTR_SIZE);
 
-       error = dmu_tx_assign(tx, waited ? TXG_WAITED : TXG_NOWAIT);
+       error = dmu_tx_assign(tx, (waited ? TXG_NOTHROTTLE : 0) | TXG_NOWAIT);
        if (error) {
                zfs_dirent_unlock(dl);
                if (error == ERESTART) {
@@ -2036,13 +2063,18 @@ top:
         */
        zfs_mknode(dzp, vap, tx, cr, 0, &zp, &acl_ids);
 
-       if (fuid_dirtied)
-               zfs_fuid_sync(zfsvfs, tx);
-
        /*
         * Now put new name in parent dir.
         */
-       (void) zfs_link_create(dl, zp, tx, ZNEW);
+       error = zfs_link_create(dl, zp, tx, ZNEW);
+       if (error != 0) {
+               zfs_znode_delete(zp, tx);
+               remove_inode_hash(ZTOI(zp));
+               goto out;
+       }
+
+       if (fuid_dirtied)
+               zfs_fuid_sync(zfsvfs, tx);
 
        *ipp = ZTOI(zp);
 
@@ -2052,6 +2084,7 @@ top:
        zfs_log_create(zilog, tx, txtype, dzp, zp, dirname, vsecp,
            acl_ids.z_fuidp, vap);
 
+out:
        zfs_acl_ids_free(&acl_ids);
 
        dmu_tx_commit(tx);
@@ -2061,10 +2094,14 @@ top:
        if (zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
                zil_commit(zilog, 0);
 
-       zfs_inode_update(dzp);
-       zfs_inode_update(zp);
+       if (error != 0) {
+               iput(ZTOI(zp));
+       } else {
+               zfs_inode_update(dzp);
+               zfs_inode_update(zp);
+       }
        ZFS_EXIT(zfsvfs);
-       return (0);
+       return (error);
 }
 
 /*
@@ -2155,7 +2192,7 @@ top:
        zfs_sa_upgrade_txholds(tx, zp);
        zfs_sa_upgrade_txholds(tx, dzp);
        dmu_tx_mark_netfree(tx);
-       error = dmu_tx_assign(tx, waited ? TXG_WAITED : TXG_NOWAIT);
+       error = dmu_tx_assign(tx, (waited ? TXG_NOTHROTTLE : 0) | TXG_NOWAIT);
        if (error) {
                rw_exit(&zp->z_parent_lock);
                rw_exit(&zp->z_name_lock);
@@ -2222,7 +2259,7 @@ out:
  */
 /* ARGSUSED */
 int
-zfs_readdir(struct inode *ip, struct dir_context *ctx, cred_t *cr)
+zfs_readdir(struct inode *ip, zpl_dir_context_t *ctx, cred_t *cr)
 {
        znode_t         *zp = ITOZ(ip);
        zfsvfs_t        *zfsvfs = ITOZSB(ip);
@@ -2327,7 +2364,7 @@ zfs_readdir(struct inode *ip, struct dir_context *ctx, cred_t *cr)
                        type = ZFS_DIRENT_TYPE(zap.za_first_integer);
                }
 
-               done = !dir_emit(ctx, zap.za_name, strlen(zap.za_name),
+               done = !zpl_dir_emit(ctx, zap.za_name, strlen(zap.za_name),
                    objnum, type);
                if (done)
                        break;
@@ -2564,6 +2601,17 @@ zfs_getattr(struct inode *ip, vattr_t *vap, int flags, cred_t *cr)
                            ((zp->z_pflags & ZFS_SPARSE) != 0);
                        XVA_SET_RTN(xvap, XAT_SPARSE);
                }
+
+               if (XVA_ISSET_REQ(xvap, XAT_PROJINHERIT)) {
+                       xoap->xoa_projinherit =
+                           ((zp->z_pflags & ZFS_PROJINHERIT) != 0);
+                       XVA_SET_RTN(xvap, XAT_PROJINHERIT);
+               }
+
+               if (XVA_ISSET_REQ(xvap, XAT_PROJID)) {
+                       xoap->xoa_projid = zp->z_projid;
+                       XVA_SET_RTN(xvap, XAT_PROJID);
+               }
        }
 
        ZFS_TIME_DECODE(&vap->va_atime, atime);
@@ -2641,6 +2689,125 @@ zfs_getattr_fast(struct inode *ip, struct kstat *sp)
        return (0);
 }
 
+/*
+ * For the operation of changing file's user/group/project, we need to
+ * handle not only the main object that is assigned to the file directly,
+ * but also the ones that are used by the file via hidden xattr directory.
+ *
+ * Because the xattr directory may contains many EA entries, as to it may
+ * be impossible to change all of them via the transaction of changing the
+ * main object's user/group/project attributes. Then we have to change them
+ * via other multiple independent transactions one by one. It may be not good
+ * solution, but we have no better idea yet.
+ */
+static int
+zfs_setattr_dir(znode_t *dzp)
+{
+       struct inode    *dxip = ZTOI(dzp);
+       struct inode    *xip = NULL;
+       zfsvfs_t        *zfsvfs = ITOZSB(dxip);
+       objset_t        *os = zfsvfs->z_os;
+       zap_cursor_t    zc;
+       zap_attribute_t zap;
+       zfs_dirlock_t   *dl;
+       znode_t         *zp;
+       dmu_tx_t        *tx = NULL;
+       uint64_t        uid, gid;
+       sa_bulk_attr_t  bulk[4];
+       int             count = 0;
+       int             err;
+
+       zap_cursor_init(&zc, os, dzp->z_id);
+       while ((err = zap_cursor_retrieve(&zc, &zap)) == 0) {
+               if (zap.za_integer_length != 8 || zap.za_num_integers != 1) {
+                       err = ENXIO;
+                       break;
+               }
+
+               err = zfs_dirent_lock(&dl, dzp, (char *)zap.za_name, &zp,
+                   ZEXISTS, NULL, NULL);
+               if (err == ENOENT)
+                       goto next;
+               if (err)
+                       break;
+
+               xip = ZTOI(zp);
+               if (KUID_TO_SUID(xip->i_uid) == KUID_TO_SUID(dxip->i_uid) &&
+                   KGID_TO_SGID(xip->i_gid) == KGID_TO_SGID(dxip->i_gid) &&
+                   zp->z_projid == dzp->z_projid)
+                       goto next;
+
+               tx = dmu_tx_create(os);
+               if (!(zp->z_pflags & ZFS_PROJID))
+                       dmu_tx_hold_sa(tx, zp->z_sa_hdl, B_TRUE);
+               else
+                       dmu_tx_hold_sa(tx, zp->z_sa_hdl, B_FALSE);
+
+               err = dmu_tx_assign(tx, TXG_WAIT);
+               if (err)
+                       break;
+
+               mutex_enter(&dzp->z_lock);
+
+               if (KUID_TO_SUID(xip->i_uid) != KUID_TO_SUID(dxip->i_uid)) {
+                       xip->i_uid = dxip->i_uid;
+                       uid = zfs_uid_read(dxip);
+                       SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_UID(zfsvfs), NULL,
+                           &uid, sizeof (uid));
+               }
+
+               if (KGID_TO_SGID(xip->i_gid) != KGID_TO_SGID(dxip->i_gid)) {
+                       xip->i_gid = dxip->i_gid;
+                       gid = zfs_gid_read(dxip);
+                       SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_GID(zfsvfs), NULL,
+                           &gid, sizeof (gid));
+               }
+
+               if (zp->z_projid != dzp->z_projid) {
+                       if (!(zp->z_pflags & ZFS_PROJID)) {
+                               zp->z_pflags |= ZFS_PROJID;
+                               SA_ADD_BULK_ATTR(bulk, count,
+                                   SA_ZPL_FLAGS(zfsvfs), NULL, &zp->z_pflags,
+                                   sizeof (zp->z_pflags));
+                       }
+
+                       zp->z_projid = dzp->z_projid;
+                       SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_PROJID(zfsvfs),
+                           NULL, &zp->z_projid, sizeof (zp->z_projid));
+               }
+
+               mutex_exit(&dzp->z_lock);
+
+               if (likely(count > 0)) {
+                       err = sa_bulk_update(zp->z_sa_hdl, bulk, count, tx);
+                       dmu_tx_commit(tx);
+               } else {
+                       dmu_tx_abort(tx);
+               }
+               tx = NULL;
+               if (err != 0 && err != ENOENT)
+                       break;
+
+next:
+               if (xip) {
+                       iput(xip);
+                       xip = NULL;
+                       zfs_dirent_unlock(dl);
+               }
+               zap_cursor_advance(&zc);
+       }
+
+       if (tx)
+               dmu_tx_abort(tx);
+       if (xip) {
+               iput(xip);
+               zfs_dirent_unlock(dl);
+       }
+       zap_cursor_fini(&zc);
+
+       return (err == ENOENT ? 0 : err);
+}
+
 /*
  * Set the file attributes to the values contained in the
  * vattr structure.
@@ -2664,6 +2831,7 @@ zfs_setattr(struct inode *ip, vattr_t *vap, int flags, cred_t *cr)
 {
        znode_t         *zp = ITOZ(ip);
        zfsvfs_t        *zfsvfs = ITOZSB(ip);
+       objset_t        *os = zfsvfs->z_os;
        zilog_t         *zilog;
        dmu_tx_t        *tx;
        vattr_t         oldva;
@@ -2675,17 +2843,19 @@ zfs_setattr(struct inode *ip, vattr_t *vap, int flags, cred_t *cr)
        uint64_t        new_kuid = 0, new_kgid = 0, new_uid, new_gid;
        uint64_t        xattr_obj;
        uint64_t        mtime[2], ctime[2], atime[2];
+       uint64_t        projid = ZFS_INVALID_PROJID;
        znode_t         *attrzp;
        int             need_policy = FALSE;
-       int             err, err2;
+       int             err, err2 = 0;
        zfs_fuid_info_t *fuidp = NULL;
        xvattr_t *xvap = (xvattr_t *)vap;       /* vap may be an xvattr_t * */
        xoptattr_t      *xoap;
        zfs_acl_t       *aclp;
        boolean_t skipaclchk = (flags & ATTR_NOACLCHECK) ? B_TRUE : B_FALSE;
        boolean_t       fuid_dirtied = B_FALSE;
+       boolean_t       handle_eadir = B_FALSE;
        sa_bulk_attr_t  *bulk, *xattr_bulk;
-       int             count = 0, xattr_count = 0;
+       int             count = 0, xattr_count = 0, bulks = 8;
 
        if (mask == 0)
                return (0);
@@ -2693,6 +2863,41 @@ zfs_setattr(struct inode *ip, vattr_t *vap, int flags, cred_t *cr)
        ZFS_ENTER(zfsvfs);
        ZFS_VERIFY_ZP(zp);
 
+       /*
+        * If this is a xvattr_t, then get a pointer to the structure of
+        * optional attributes.  If this is NULL, then we have a vattr_t.
+        */
+       xoap = xva_getxoptattr(xvap);
+       if (xoap != NULL && (mask & ATTR_XVATTR)) {
+               if (XVA_ISSET_REQ(xvap, XAT_PROJID)) {
+                       if (!dmu_objset_projectquota_enabled(os) ||
+                           (!S_ISREG(ip->i_mode) && !S_ISDIR(ip->i_mode))) {
+                               ZFS_EXIT(zfsvfs);
+                               return (SET_ERROR(ENOTSUP));
+                       }
+
+                       projid = xoap->xoa_projid;
+                       if (unlikely(projid == ZFS_INVALID_PROJID)) {
+                               ZFS_EXIT(zfsvfs);
+                               return (SET_ERROR(EINVAL));
+                       }
+
+                       if (projid == zp->z_projid && zp->z_pflags & ZFS_PROJID)
+                               projid = ZFS_INVALID_PROJID;
+                       else
+                               need_policy = TRUE;
+               }
+
+               if (XVA_ISSET_REQ(xvap, XAT_PROJINHERIT) &&
+                   (xoap->xoa_projinherit !=
+                   ((zp->z_pflags & ZFS_PROJINHERIT) != 0)) &&
+                   (!dmu_objset_projectquota_enabled(os) ||
+                   (!S_ISREG(ip->i_mode) && !S_ISDIR(ip->i_mode)))) {
+                       ZFS_EXIT(zfsvfs);
+                       return (SET_ERROR(ENOTSUP));
+               }
+       }
+
        zilog = zfsvfs->z_log;
 
        /*
@@ -2718,17 +2923,11 @@ zfs_setattr(struct inode *ip, vattr_t *vap, int flags, cred_t *cr)
                return (SET_ERROR(EINVAL));
        }
 
-       /*
-        * If this is an xvattr_t, then get a pointer to the structure of
-        * optional attributes.  If this is NULL, then we have a vattr_t.
-        */
-       xoap = xva_getxoptattr(xvap);
-
        tmpxvattr = kmem_alloc(sizeof (xvattr_t), KM_SLEEP);
        xva_init(tmpxvattr);
 
-       bulk = kmem_alloc(sizeof (sa_bulk_attr_t) * 7, KM_SLEEP);
-       xattr_bulk = kmem_alloc(sizeof (sa_bulk_attr_t) * 7, KM_SLEEP);
+       bulk = kmem_alloc(sizeof (sa_bulk_attr_t) * bulks, KM_SLEEP);
+       xattr_bulk = kmem_alloc(sizeof (sa_bulk_attr_t) * bulks, KM_SLEEP);
 
        /*
         * Immutable files can only alter immutable bit and atime
@@ -2874,6 +3073,16 @@ top:
                        }
                }
 
+               if (XVA_ISSET_REQ(xvap, XAT_PROJINHERIT)) {
+                       if (xoap->xoa_projinherit !=
+                           ((zp->z_pflags & ZFS_PROJINHERIT) != 0)) {
+                               need_policy = TRUE;
+                       } else {
+                               XVA_CLR_REQ(xvap, XAT_PROJINHERIT);
+                               XVA_SET_REQ(tmpxvattr, XAT_PROJINHERIT);
+                       }
+               }
+
                if (XVA_ISSET_REQ(xvap, XAT_NOUNLINK)) {
                        if (xoap->xoa_nounlink !=
                            ((zp->z_pflags & ZFS_NOUNLINK) != 0)) {
@@ -2982,7 +3191,8 @@ top:
         */
        mask = vap->va_mask;
 
-       if ((mask & (ATTR_UID | ATTR_GID))) {
+       if ((mask & (ATTR_UID | ATTR_GID)) || projid != ZFS_INVALID_PROJID) {
+               handle_eadir = B_TRUE;
                err = sa_lookup(zp->z_sa_hdl, SA_ZPL_XATTR(zfsvfs),
                    &xattr_obj, sizeof (xattr_obj));
 
@@ -2995,7 +3205,8 @@ top:
                        new_kuid = zfs_fuid_create(zfsvfs,
                            (uint64_t)vap->va_uid, cr, ZFS_OWNER, &fuidp);
                        if (new_kuid != KUID_TO_SUID(ZTOI(zp)->i_uid) &&
-                           zfs_fuid_overquota(zfsvfs, B_FALSE, new_kuid)) {
+                           zfs_id_overquota(zfsvfs, DMU_USERUSED_OBJECT,
+                           new_kuid)) {
                                if (attrzp)
                                        iput(ZTOI(attrzp));
                                err = SET_ERROR(EDQUOT);
@@ -3007,15 +3218,24 @@ top:
                        new_kgid = zfs_fuid_create(zfsvfs,
                            (uint64_t)vap->va_gid, cr, ZFS_GROUP, &fuidp);
                        if (new_kgid != KGID_TO_SGID(ZTOI(zp)->i_gid) &&
-                           zfs_fuid_overquota(zfsvfs, B_TRUE, new_kgid)) {
+                           zfs_id_overquota(zfsvfs, DMU_GROUPUSED_OBJECT,
+                           new_kgid)) {
                                if (attrzp)
                                        iput(ZTOI(attrzp));
                                err = SET_ERROR(EDQUOT);
                                goto out2;
                        }
                }
+
+               if (projid != ZFS_INVALID_PROJID &&
+                   zfs_id_overquota(zfsvfs, DMU_PROJECTUSED_OBJECT, projid)) {
+                       if (attrzp)
+                               iput(ZTOI(attrzp));
+                       err = EDQUOT;
+                       goto out2;
+               }
        }
-       tx = dmu_tx_create(zfsvfs->z_os);
+       tx = dmu_tx_create(os);
 
        if (mask & ATTR_MODE) {
                uint64_t pmode = zp->z_mode;
@@ -3048,8 +3268,10 @@ top:
                mutex_exit(&zp->z_lock);
                dmu_tx_hold_sa(tx, zp->z_sa_hdl, B_TRUE);
        } else {
-               if ((mask & ATTR_XVATTR) &&
-                   XVA_ISSET_REQ(xvap, XAT_AV_SCANSTAMP))
+               if (((mask & ATTR_XVATTR) &&
+                   XVA_ISSET_REQ(xvap, XAT_AV_SCANSTAMP)) ||
+                   (projid != ZFS_INVALID_PROJID &&
+                   !(zp->z_pflags & ZFS_PROJID)))
                        dmu_tx_hold_sa(tx, zp->z_sa_hdl, B_TRUE);
                else
                        dmu_tx_hold_sa(tx, zp->z_sa_hdl, B_FALSE);
@@ -3078,6 +3300,26 @@ top:
         * updated as a side-effect of calling this function.
         */
 
+       if (projid != ZFS_INVALID_PROJID && !(zp->z_pflags & ZFS_PROJID)) {
+               /*
+                * For the existed object that is upgraded from old system,
+                * its on-disk layout has no slot for the project ID attribute.
+                * But quota accounting logic needs to access related slots by
+                * offset directly. So we need to adjust old objects' layout
+                * to make the project ID to some unified and fixed offset.
+                */
+               if (attrzp)
+                       err = sa_add_projid(attrzp->z_sa_hdl, tx, projid);
+               if (err == 0)
+                       err = sa_add_projid(zp->z_sa_hdl, tx, projid);
+
+               if (unlikely(err == EEXIST))
+                       err = 0;
+               else if (err != 0)
+                       goto out;
+               else
+                       projid = ZFS_INVALID_PROJID;
+       }
 
        if (mask & (ATTR_UID|ATTR_GID|ATTR_MODE))
                mutex_enter(&zp->z_acl_lock);
@@ -3093,6 +3335,12 @@ top:
                SA_ADD_BULK_ATTR(xattr_bulk, xattr_count,
                    SA_ZPL_FLAGS(zfsvfs), NULL, &attrzp->z_pflags,
                    sizeof (attrzp->z_pflags));
+               if (projid != ZFS_INVALID_PROJID) {
+                       attrzp->z_projid = projid;
+                       SA_ADD_BULK_ATTR(xattr_bulk, xattr_count,
+                           SA_ZPL_PROJID(zfsvfs), NULL, &attrzp->z_projid,
+                           sizeof (attrzp->z_projid));
+               }
        }
 
        if (mask & (ATTR_UID|ATTR_GID)) {
@@ -3155,23 +3403,30 @@ top:
                    &atime, sizeof (atime));
        }
 
-       if (mask & ATTR_MTIME) {
+       if (mask & (ATTR_MTIME | ATTR_SIZE)) {
                ZFS_TIME_ENCODE(&vap->va_mtime, mtime);
-               ZTOI(zp)->i_mtime = timespec_trunc(vap->va_mtime,
+               ZTOI(zp)->i_mtime = zpl_inode_timespec_trunc(vap->va_mtime,
                    ZTOI(zp)->i_sb->s_time_gran);
 
                SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_MTIME(zfsvfs), NULL,
                    mtime, sizeof (mtime));
        }
 
-       if (mask & ATTR_CTIME) {
+       if (mask & (ATTR_CTIME | ATTR_SIZE)) {
                ZFS_TIME_ENCODE(&vap->va_ctime, ctime);
-               ZTOI(zp)->i_ctime = timespec_trunc(vap->va_ctime,
+               ZTOI(zp)->i_ctime = zpl_inode_timespec_trunc(vap->va_ctime,
                    ZTOI(zp)->i_sb->s_time_gran);
                SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_CTIME(zfsvfs), NULL,
                    ctime, sizeof (ctime));
        }
 
+       if (projid != ZFS_INVALID_PROJID) {
+               zp->z_projid = projid;
+               SA_ADD_BULK_ATTR(bulk, count,
+                   SA_ZPL_PROJID(zfsvfs), NULL, &zp->z_projid,
+                   sizeof (zp->z_projid));
+       }
+
        if (attrzp && mask) {
                SA_ADD_BULK_ATTR(xattr_bulk, xattr_count,
                    SA_ZPL_CTIME(zfsvfs), NULL, &ctime,
@@ -3208,6 +3463,9 @@ top:
                if (XVA_ISSET_REQ(tmpxvattr, XAT_AV_QUARANTINED)) {
                        XVA_SET_REQ(xvap, XAT_AV_QUARANTINED);
                }
+               if (XVA_ISSET_REQ(tmpxvattr, XAT_PROJINHERIT)) {
+                       XVA_SET_REQ(xvap, XAT_PROJINHERIT);
+               }
 
                if (XVA_ISSET_REQ(xvap, XAT_AV_SCANSTAMP))
                        ASSERT(S_ISREG(ip->i_mode));
@@ -3231,7 +3489,7 @@ top:
                mutex_exit(&attrzp->z_lock);
        }
 out:
-       if (err == 0 && attrzp) {
+       if (err == 0 && xattr_count > 0) {
                err2 = sa_bulk_update(attrzp->z_sa_hdl, xattr_bulk,
                    xattr_count, tx);
                ASSERT(err2 == 0);
@@ -3252,20 +3510,24 @@ out:
                if (err == ERESTART)
                        goto top;
        } else {
-               err2 = sa_bulk_update(zp->z_sa_hdl, bulk, count, tx);
+               if (count > 0)
+                       err2 = sa_bulk_update(zp->z_sa_hdl, bulk, count, tx);
                dmu_tx_commit(tx);
-               if (attrzp)
+               if (attrzp) {
+                       if (err2 == 0 && handle_eadir)
+                               err2 = zfs_setattr_dir(attrzp);
                        iput(ZTOI(attrzp));
+               }
                zfs_inode_update(zp);
        }
 
 out2:
-       if (zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
+       if (os->os_sync == ZFS_SYNC_ALWAYS)
                zil_commit(zilog, 0);
 
 out3:
-       kmem_free(xattr_bulk, sizeof (sa_bulk_attr_t) * 7);
-       kmem_free(bulk, sizeof (sa_bulk_attr_t) * 7);
+       kmem_free(xattr_bulk, sizeof (sa_bulk_attr_t) * bulks);
+       kmem_free(bulk, sizeof (sa_bulk_attr_t) * bulks);
        kmem_free(tmpxvattr, sizeof (xvattr_t));
        ZFS_EXIT(zfsvfs);
        return (err);
@@ -3558,6 +3820,19 @@ top:
                return (terr);
        }
 
+       /*
+        * If we are using project inheritance, means if the directory has
+        * ZFS_PROJINHERIT set, then its descendant directories will inherit
+        * not only the project ID, but also the ZFS_PROJINHERIT flag. Under
+        * such case, we only allow renames into our tree when the project
+        * IDs are the same.
+        */
+       if (tdzp->z_pflags & ZFS_PROJINHERIT &&
+           tdzp->z_projid != szp->z_projid) {
+               error = SET_ERROR(EXDEV);
+               goto out;
+       }
+
        /*
         * Must have write access at the source to remove the old entry
         * and write access at the target to create the new entry.
@@ -3622,7 +3897,7 @@ top:
 
        zfs_sa_upgrade_txholds(tx, szp);
        dmu_tx_hold_zap(tx, zfsvfs->z_unlinkedobj, FALSE, NULL);
-       error = dmu_tx_assign(tx, waited ? TXG_WAITED : TXG_NOWAIT);
+       error = dmu_tx_assign(tx, (waited ? TXG_NOTHROTTLE : 0) | TXG_NOWAIT);
        if (error) {
                if (zl != NULL)
                        zfs_rename_unlock(&zl);
@@ -3656,6 +3931,8 @@ top:
                error = zfs_link_create(tdl, szp, tx, ZRENAMING);
                if (error == 0) {
                        szp->z_pflags |= ZFS_AV_MODIFIED;
+                       if (tdzp->z_pflags & ZFS_PROJINHERIT)
+                               szp->z_pflags |= ZFS_PROJINHERIT;
 
                        error = sa_update(szp->z_sa_hdl, SA_ZPL_FLAGS(zfsvfs),
                            (void *)&szp->z_pflags, sizeof (uint64_t), tx);
@@ -3682,6 +3959,13 @@ top:
                                VERIFY3U(zfs_link_destroy(tdl, szp, tx,
                                    ZRENAMING, NULL), ==, 0);
                        }
+               } else {
+                       /*
+                        * If we had removed the existing target, subsequent
+                        * call to zfs_link_create() to add back the same entry
+                        * but, the new dnode (szp) should not fail.
+                        */
+                       ASSERT(tzp == NULL);
                }
        }
 
@@ -3795,7 +4079,7 @@ top:
                return (error);
        }
 
-       if (zfs_acl_ids_overquota(zfsvfs, &acl_ids)) {
+       if (zfs_acl_ids_overquota(zfsvfs, &acl_ids, ZFS_DEFAULT_PROJID)) {
                zfs_acl_ids_free(&acl_ids);
                zfs_dirent_unlock(dl);
                ZFS_EXIT(zfsvfs);
@@ -3814,7 +4098,7 @@ top:
        }
        if (fuid_dirtied)
                zfs_fuid_txhold(zfsvfs, tx);
-       error = dmu_tx_assign(tx, waited ? TXG_WAITED : TXG_NOWAIT);
+       error = dmu_tx_assign(tx, (waited ? TXG_NOTHROTTLE : 0) | TXG_NOWAIT);
        if (error) {
                zfs_dirent_unlock(dl);
                if (error == ERESTART) {
@@ -3852,14 +4136,18 @@ top:
        /*
         * Insert the new object into the directory.
         */
-       (void) zfs_link_create(dl, zp, tx, ZNEW);
-
-       if (flags & FIGNORECASE)
-               txtype |= TX_CI;
-       zfs_log_symlink(zilog, tx, txtype, dzp, zp, name, link);
+       error = zfs_link_create(dl, zp, tx, ZNEW);
+       if (error != 0) {
+               zfs_znode_delete(zp, tx);
+               remove_inode_hash(ZTOI(zp));
+       } else {
+               if (flags & FIGNORECASE)
+                       txtype |= TX_CI;
+               zfs_log_symlink(zilog, tx, txtype, dzp, zp, name, link);
 
-       zfs_inode_update(dzp);
-       zfs_inode_update(zp);
+               zfs_inode_update(dzp);
+               zfs_inode_update(zp);
+       }
 
        zfs_acl_ids_free(&acl_ids);
 
@@ -3867,10 +4155,14 @@ top:
 
        zfs_dirent_unlock(dl);
 
-       *ipp = ZTOI(zp);
+       if (error == 0) {
+               *ipp = ZTOI(zp);
 
-       if (zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
-               zil_commit(zilog, 0);
+               if (zfsvfs->z_os->os_sync == ZFS_SYNC_ALWAYS)
+                       zil_commit(zilog, 0);
+       } else {
+               iput(ZTOI(zp));
+       }
 
        ZFS_EXIT(zfsvfs);
        return (error);
@@ -3970,6 +4262,18 @@ zfs_link(struct inode *tdip, struct inode *sip, char *name, cred_t *cr,
        szp = ITOZ(sip);
        ZFS_VERIFY_ZP(szp);
 
+       /*
+        * If we are using project inheritance, means if the directory has
+        * ZFS_PROJINHERIT set, then its descendant directories will inherit
+        * not only the project ID, but also the ZFS_PROJINHERIT flag. Under
+        * such case, we only allow hard link creation in our tree when the
+        * project IDs are the same.
+        */
+       if (dzp->z_pflags & ZFS_PROJINHERIT && dzp->z_projid != szp->z_projid) {
+               ZFS_EXIT(zfsvfs);
+               return (SET_ERROR(EXDEV));
+       }
+
        /*
         * We check i_sb because snapshots and the ctldir must have different
         * super blocks.
@@ -4040,7 +4344,7 @@ top:
 
        zfs_sa_upgrade_txholds(tx, szp);
        zfs_sa_upgrade_txholds(tx, dzp);
-       error = dmu_tx_assign(tx, waited ? TXG_WAITED : TXG_NOWAIT);
+       error = dmu_tx_assign(tx, (waited ? TXG_NOTHROTTLE : 0) | TXG_NOWAIT);
        if (error) {
                zfs_dirent_unlock(dl);
                if (error == ERESTART) {
@@ -4164,8 +4468,13 @@ zfs_putpage(struct inode *ip, struct page *pp, struct writeback_control *wbc)
         * is to register a page_mkwrite() handler to count the page
         * against its quota when it is about to be dirtied.
         */
-       if (zfs_owner_overquota(zfsvfs, zp, B_FALSE) ||
-           zfs_owner_overquota(zfsvfs, zp, B_TRUE)) {
+       if (zfs_id_overblockquota(zfsvfs, DMU_USERUSED_OBJECT,
+           KUID_TO_SUID(ip->i_uid)) ||
+           zfs_id_overblockquota(zfsvfs, DMU_GROUPUSED_OBJECT,
+           KGID_TO_SGID(ip->i_gid)) ||
+           (zp->z_projid != ZFS_DEFAULT_PROJID &&
+           zfs_id_overblockquota(zfsvfs, DMU_PROJECTUSED_OBJECT,
+           zp->z_projid))) {
                err = EDQUOT;
        }
 #endif
@@ -4911,7 +5220,7 @@ zfs_retzcbuf(struct inode *ip, xuio_t *xuio, cred_t *cr)
 }
 #endif /* HAVE_UIO_ZEROCOPY */
 
-#if defined(_KERNEL) && defined(HAVE_SPL)
+#if defined(_KERNEL)
 EXPORT_SYMBOL(zfs_open);
 EXPORT_SYMBOL(zfs_close);
 EXPORT_SYMBOL(zfs_read);