]> git.proxmox.com Git - mirror_zfs.git/blobdiff - module/zfs/dmu.c
cstyle: Resolve C style issues
[mirror_zfs.git] / module / zfs / dmu.c
index 4ec9cb46ad328b3dbf41ac545777f24a0d6fe91c..9e99558a36ad7430fd64ef7fdd881d640193bc1a 100644 (file)
@@ -20,7 +20,7 @@
  */
 /*
  * Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
- * Copyright (c) 2012 by Delphix. All rights reserved.
+ * Copyright (c) 2013 by Delphix. All rights reserved.
  * Copyright (c) 2013 by Saso Kiselkov. All rights reserved.
  */
 
 #include <sys/zfs_ioctl.h>
 #include <sys/zap.h>
 #include <sys/zio_checksum.h>
+#include <sys/zio_compress.h>
 #include <sys/sa.h>
 #ifdef _KERNEL
 #include <sys/vmsystm.h>
 #include <sys/zfs_znode.h>
 #endif
 
+/*
+ * Enable/disable nopwrite feature.
+ */
+int zfs_nopwrite_enabled = 1;
+
 const dmu_object_type_info_t dmu_ot[DMU_OT_NUMTYPES] = {
        {       DMU_BSWAP_UINT8,        TRUE,   "unallocated"           },
        {       DMU_BSWAP_ZAP,          TRUE,   "object directory"      },
@@ -138,7 +144,7 @@ dmu_buf_hold(objset_t *os, uint64_t object, uint64_t offset,
        db = dbuf_hold(dn, blkid, tag);
        rw_exit(&dn->dn_struct_rwlock);
        if (db == NULL) {
-               err = EIO;
+               err = SET_ERROR(EIO);
        } else {
                err = dbuf_read(db, NULL, db_flags);
                if (err) {
@@ -169,9 +175,9 @@ dmu_set_bonus(dmu_buf_t *db_fake, int newsize, dmu_tx_t *tx)
        dn = DB_DNODE(db);
 
        if (dn->dn_bonus != db) {
-               error = EINVAL;
+               error = SET_ERROR(EINVAL);
        } else if (newsize < 0 || newsize > db_fake->db_size) {
-               error = EINVAL;
+               error = SET_ERROR(EINVAL);
        } else {
                dnode_setbonuslen(dn, newsize, tx);
                error = 0;
@@ -192,9 +198,9 @@ dmu_set_bonustype(dmu_buf_t *db_fake, dmu_object_type_t type, dmu_tx_t *tx)
        dn = DB_DNODE(db);
 
        if (!DMU_OT_IS_VALID(type)) {
-               error = EINVAL;
+               error = SET_ERROR(EINVAL);
        } else if (dn->dn_bonus != db) {
-               error = EINVAL;
+               error = SET_ERROR(EINVAL);
        } else {
                dnode_setbonus_type(dn, type, tx);
                error = 0;
@@ -321,12 +327,12 @@ dmu_spill_hold_existing(dmu_buf_t *bonus, void *tag, dmu_buf_t **dbp)
        dn = DB_DNODE(db);
 
        if (spa_version(dn->dn_objset->os_spa) < SPA_VERSION_SA) {
-               err = EINVAL;
+               err = SET_ERROR(EINVAL);
        } else {
                rw_enter(&dn->dn_struct_rwlock, RW_READER);
 
                if (!dn->dn_have_spill) {
-                       err = ENOENT;
+                       err = SET_ERROR(ENOENT);
                } else {
                        err = dmu_spill_hold_by_dnode(dn,
                            DB_RF_HAVESTRUCT | DB_RF_CANFAIL, tag, dbp);
@@ -364,13 +370,11 @@ static int
 dmu_buf_hold_array_by_dnode(dnode_t *dn, uint64_t offset, uint64_t length,
     int read, void *tag, int *numbufsp, dmu_buf_t ***dbpp, uint32_t flags)
 {
-       dsl_pool_t *dp = NULL;
        dmu_buf_t **dbp;
        uint64_t blkid, nblks, i;
        uint32_t dbuf_flags;
        int err;
        zio_t *zio;
-       hrtime_t start = 0;
 
        ASSERT(length <= DMU_MAX_ACCESS);
 
@@ -392,16 +396,13 @@ dmu_buf_hold_array_by_dnode(dnode_t *dn, uint64_t offset, uint64_t length,
                            (longlong_t)dn->dn_object, dn->dn_datablksz,
                            (longlong_t)offset, (longlong_t)length);
                        rw_exit(&dn->dn_struct_rwlock);
-                       return (EIO);
+                       return (SET_ERROR(EIO));
                }
                nblks = 1;
        }
-       dbp = kmem_zalloc(sizeof (dmu_buf_t *) * nblks, KM_PUSHPAGE | KM_NODEBUG);
+       dbp = kmem_zalloc(sizeof (dmu_buf_t *) * nblks,
+           KM_PUSHPAGE | KM_NODEBUG);
 
-       if (dn->dn_objset->os_dsl_dataset)
-               dp = dn->dn_objset->os_dsl_dataset->ds_dir->dd_pool;
-       if (dp && dsl_pool_sync_context(dp))
-               start = gethrtime();
        zio = zio_root(dn->dn_objset->os_spa, NULL, NULL, ZIO_FLAG_CANFAIL);
        blkid = dbuf_whichblock(dn, offset);
        for (i = 0; i < nblks; i++) {
@@ -410,7 +411,7 @@ dmu_buf_hold_array_by_dnode(dnode_t *dn, uint64_t offset, uint64_t length,
                        rw_exit(&dn->dn_struct_rwlock);
                        dmu_buf_rele_array(dbp, nblks, tag);
                        zio_nowait(zio);
-                       return (EIO);
+                       return (SET_ERROR(EIO));
                }
                /* initiate async i/o */
                if (read) {
@@ -422,9 +423,6 @@ dmu_buf_hold_array_by_dnode(dnode_t *dn, uint64_t offset, uint64_t length,
 
        /* wait for async i/o */
        err = zio_wait(zio);
-       /* track read overhead when we are in sync context */
-       if (dp && dsl_pool_sync_context(dp))
-               dp->dp_read_overhead += gethrtime() - start;
        if (err) {
                dmu_buf_rele_array(dbp, nblks, tag);
                return (err);
@@ -439,7 +437,7 @@ dmu_buf_hold_array_by_dnode(dnode_t *dn, uint64_t offset, uint64_t length,
                            db->db_state == DB_FILL)
                                cv_wait(&db->db_changed, &db->db_mtx);
                        if (db->db_state == DB_UNCACHED)
-                               err = EIO;
+                               err = SET_ERROR(EIO);
                        mutex_exit(&db->db_mtx);
                        if (err) {
                                dmu_buf_rele_array(dbp, nblks, tag);
@@ -506,12 +504,22 @@ dmu_buf_rele_array(dmu_buf_t **dbp_fake, int numbufs, void *tag)
        kmem_free(dbp, sizeof (dmu_buf_t *) * numbufs);
 }
 
+/*
+ * Issue prefetch i/os for the given blocks.
+ *
+ * Note: The assumption is that we *know* these blocks will be needed
+ * almost immediately.  Therefore, the prefetch i/os will be issued at
+ * ZIO_PRIORITY_SYNC_READ
+ *
+ * Note: indirect blocks and other metadata will be read synchronously,
+ * causing this function to block if they are not already cached.
+ */
 void
 dmu_prefetch(objset_t *os, uint64_t object, uint64_t offset, uint64_t len)
 {
        dnode_t *dn;
        uint64_t blkid;
-       int nblks, i, err;
+       int nblks, err;
 
        if (zfs_prefetch_disable)
                return;
@@ -524,7 +532,7 @@ dmu_prefetch(objset_t *os, uint64_t object, uint64_t offset, uint64_t len)
 
                rw_enter(&dn->dn_struct_rwlock, RW_READER);
                blkid = dbuf_whichblock(dn, object * sizeof (dnode_phys_t));
-               dbuf_prefetch(dn, blkid);
+               dbuf_prefetch(dn, blkid, ZIO_PRIORITY_SYNC_READ);
                rw_exit(&dn->dn_struct_rwlock);
                return;
        }
@@ -541,16 +549,18 @@ dmu_prefetch(objset_t *os, uint64_t object, uint64_t offset, uint64_t len)
        rw_enter(&dn->dn_struct_rwlock, RW_READER);
        if (dn->dn_datablkshift) {
                int blkshift = dn->dn_datablkshift;
-               nblks = (P2ROUNDUP(offset+len, 1<<blkshift) -
-                   P2ALIGN(offset, 1<<blkshift)) >> blkshift;
+               nblks = (P2ROUNDUP(offset + len, 1 << blkshift) -
+                   P2ALIGN(offset, 1 << blkshift)) >> blkshift;
        } else {
                nblks = (offset < dn->dn_datablksz);
        }
 
        if (nblks != 0) {
+               int i;
+
                blkid = dbuf_whichblock(dn, offset);
                for (i = 0; i < nblks; i++)
-                       dbuf_prefetch(dn, blkid+i);
+                       dbuf_prefetch(dn, blkid + i, ZIO_PRIORITY_SYNC_READ);
        }
 
        rw_exit(&dn->dn_struct_rwlock);
@@ -563,98 +573,95 @@ dmu_prefetch(objset_t *os, uint64_t object, uint64_t offset, uint64_t len)
  * the end so that the file gets shorter over time (if we crashes in the
  * middle, this will leave us in a better state).  We find allocated file
  * data by simply searching the allocated level 1 indirects.
+ *
+ * On input, *start should be the first offset that does not need to be
+ * freed (e.g. "offset + length").  On return, *start will be the first
+ * offset that should be freed.
  */
 static int
-get_next_chunk(dnode_t *dn, uint64_t *start, uint64_t limit)
+get_next_chunk(dnode_t *dn, uint64_t *start, uint64_t minimum)
 {
-       uint64_t len = *start - limit;
-       uint64_t blkcnt = 0;
-       uint64_t maxblks = DMU_MAX_ACCESS / (1ULL << (dn->dn_indblkshift + 1));
+       uint64_t maxblks = DMU_MAX_ACCESS >> (dn->dn_indblkshift + 1);
+       /* bytes of data covered by a level-1 indirect block */
        uint64_t iblkrange =
            dn->dn_datablksz * EPB(dn->dn_indblkshift, SPA_BLKPTRSHIFT);
+       uint64_t blks;
 
-       ASSERT(limit <= *start);
+       ASSERT3U(minimum, <=, *start);
 
-       if (len <= iblkrange * maxblks) {
-               *start = limit;
+       if (*start - minimum <= iblkrange * maxblks) {
+               *start = minimum;
                return (0);
        }
        ASSERT(ISP2(iblkrange));
 
-       while (*start > limit && blkcnt < maxblks) {
+       for (blks = 0; *start > minimum && blks < maxblks; blks++) {
                int err;
 
-               /* find next allocated L1 indirect */
+               /*
+                * dnode_next_offset(BACKWARDS) will find an allocated L1
+                * indirect block at or before the input offset.  We must
+                * decrement *start so that it is at the end of the region
+                * to search.
+                */
+               (*start)--;
                err = dnode_next_offset(dn,
                    DNODE_FIND_BACKWARDS, start, 2, 1, 0);
 
-               /* if there are no more, then we are done */
+               /* if there are no indirect blocks before start, we are done */
                if (err == ESRCH) {
-                       *start = limit;
-                       return (0);
-               } else if (err) {
+                       *start = minimum;
+                       break;
+               } else if (err != 0) {
                        return (err);
                }
-               blkcnt += 1;
 
-               /* reset offset to end of "next" block back */
+               /* set start to the beginning of this L1 indirect */
                *start = P2ALIGN(*start, iblkrange);
-               if (*start <= limit)
-                       *start = limit;
-               else
-                       *start -= 1;
        }
+       if (*start < minimum)
+               *start = minimum;
        return (0);
 }
 
 static int
 dmu_free_long_range_impl(objset_t *os, dnode_t *dn, uint64_t offset,
-    uint64_t length, boolean_t free_dnode)
+    uint64_t length)
 {
-       dmu_tx_t *tx;
-       uint64_t object_size, start, end, len;
-       boolean_t trunc = (length == DMU_OBJECT_END);
-       int align, err;
-
-       align = 1 << dn->dn_datablkshift;
-       ASSERT(align > 0);
-       object_size = align == 1 ? dn->dn_datablksz :
-           (dn->dn_maxblkid + 1) << dn->dn_datablkshift;
-
-       end = offset + length;
-       if (trunc || end > object_size)
-               end = object_size;
-       if (end <= offset)
+       uint64_t object_size = (dn->dn_maxblkid + 1) * dn->dn_datablksz;
+       int err;
+
+       if (offset >= object_size)
                return (0);
-       length = end - offset;
 
-       while (length) {
-               start = end;
-               /* assert(offset <= start) */
-               err = get_next_chunk(dn, &start, offset);
+       if (length == DMU_OBJECT_END || offset + length > object_size)
+               length = object_size - offset;
+
+       while (length != 0) {
+               uint64_t chunk_end, chunk_begin;
+               dmu_tx_t *tx;
+
+               chunk_end = chunk_begin = offset + length;
+
+               /* move chunk_begin backwards to the beginning of this chunk */
+               err = get_next_chunk(dn, &chunk_begin, offset);
                if (err)
                        return (err);
-               len = trunc ? DMU_OBJECT_END : end - start;
+               ASSERT3U(chunk_begin, >=, offset);
+               ASSERT3U(chunk_begin, <=, chunk_end);
 
                tx = dmu_tx_create(os);
-               dmu_tx_hold_free(tx, dn->dn_object, start, len);
+               dmu_tx_hold_free(tx, dn->dn_object,
+                   chunk_begin, chunk_end - chunk_begin);
                err = dmu_tx_assign(tx, TXG_WAIT);
                if (err) {
                        dmu_tx_abort(tx);
                        return (err);
                }
-
-               dnode_free_range(dn, start, trunc ? -1 : len, tx);
-
-               if (start == 0 && free_dnode) {
-                       ASSERT(trunc);
-                       dnode_free(dn, tx);
-               }
-
-               length -= end - start;
-
+               dnode_free_range(dn, chunk_begin, chunk_end - chunk_begin, tx);
                dmu_tx_commit(tx);
-               end = start;
+
+               length -= chunk_end - chunk_begin;
        }
        return (0);
 }
@@ -669,38 +676,42 @@ dmu_free_long_range(objset_t *os, uint64_t object,
        err = dnode_hold(os, object, FTAG, &dn);
        if (err != 0)
                return (err);
-       err = dmu_free_long_range_impl(os, dn, offset, length, FALSE);
+       err = dmu_free_long_range_impl(os, dn, offset, length);
+
+       /*
+        * It is important to zero out the maxblkid when freeing the entire
+        * file, so that (a) subsequent calls to dmu_free_long_range_impl()
+        * will take the fast path, and (b) dnode_reallocate() can verify
+        * that the entire file has been freed.
+        */
+       if (offset == 0 && length == DMU_OBJECT_END)
+               dn->dn_maxblkid = 0;
+
        dnode_rele(dn, FTAG);
        return (err);
 }
 
 int
-dmu_free_object(objset_t *os, uint64_t object)
+dmu_free_long_object(objset_t *os, uint64_t object)
 {
-       dnode_t *dn;
        dmu_tx_t *tx;
        int err;
 
-       err = dnode_hold_impl(os, object, DNODE_MUST_BE_ALLOCATED,
-           FTAG, &dn);
+       err = dmu_free_long_range(os, object, 0, DMU_OBJECT_END);
        if (err != 0)
                return (err);
-       if (dn->dn_nlevels == 1) {
-               tx = dmu_tx_create(os);
-               dmu_tx_hold_bonus(tx, object);
-               dmu_tx_hold_free(tx, dn->dn_object, 0, DMU_OBJECT_END);
-               err = dmu_tx_assign(tx, TXG_WAIT);
-               if (err == 0) {
-                       dnode_free_range(dn, 0, DMU_OBJECT_END, tx);
-                       dnode_free(dn, tx);
-                       dmu_tx_commit(tx);
-               } else {
-                       dmu_tx_abort(tx);
-               }
+
+       tx = dmu_tx_create(os);
+       dmu_tx_hold_bonus(tx, object);
+       dmu_tx_hold_free(tx, object, 0, DMU_OBJECT_END);
+       err = dmu_tx_assign(tx, TXG_WAIT);
+       if (err == 0) {
+               err = dmu_object_free(os, object, tx);
+               dmu_tx_commit(tx);
        } else {
-               err = dmu_free_long_range_impl(os, dn, 0, DMU_OBJECT_END, TRUE);
+               dmu_tx_abort(tx);
        }
-       dnode_rele(dn, FTAG);
+
        return (err);
 }
 
@@ -867,9 +878,9 @@ static xuio_stats_t xuio_stats = {
        { "write_buf_nocopy",   KSTAT_DATA_UINT64 }
 };
 
-#define XUIOSTAT_INCR(stat, val)        \
-        atomic_add_64(&xuio_stats.stat.value.ui64, (val))
-#define XUIOSTAT_BUMP(stat)     XUIOSTAT_INCR(stat, 1)
+#define        XUIOSTAT_INCR(stat, val)        \
+       atomic_add_64(&xuio_stats.stat.value.ui64, (val))
+#define        XUIOSTAT_BUMP(stat)     XUIOSTAT_INCR(stat, 1)
 
 int
 dmu_xuio_init(xuio_t *xuio, int nblk)
@@ -1034,7 +1045,7 @@ dmu_req_copy(void *arg_buf, int size, int *offset, struct request *req)
                bv->bv_len -= tocpy;
        }
 
-       return 0;
+       return (0);
 }
 
 static void
@@ -1057,13 +1068,13 @@ dmu_bio_clone(struct bio *bio, struct bio **bio_copy)
        struct bio *bio_new;
 
        if (bio == NULL)
-               return EINVAL;
+               return (EINVAL);
 
        while (bio) {
                bio_new = bio_clone(bio, GFP_NOIO);
                if (bio_new == NULL) {
                        dmu_bio_put(bio_root);
-                       return ENOMEM;
+                       return (ENOMEM);
                }
 
                if (bio_last) {
@@ -1079,7 +1090,7 @@ dmu_bio_clone(struct bio *bio, struct bio **bio_copy)
 
        *bio_copy = bio_root;
 
-       return 0;
+       return (0);
 }
 
 int
@@ -1096,7 +1107,7 @@ dmu_read_req(objset_t *os, uint64_t object, struct request *req)
         * to be reading in parallel.
         */
        err = dmu_buf_hold_array(os, object, offset, size, TRUE, FTAG,
-                                &numbufs, &dbp);
+           &numbufs, &dbp);
        if (err)
                return (err);
 
@@ -1158,7 +1169,7 @@ dmu_write_req(objset_t *os, uint64_t object, struct request *req, dmu_tx_t *tx)
                return (0);
 
        err = dmu_buf_hold_array(os, object, offset, size, FALSE, FTAG,
-                                &numbufs, &dbp);
+           &numbufs, &dbp);
        if (err)
                return (err);
 
@@ -1474,6 +1485,16 @@ dmu_sync_done(zio_t *zio, arc_buf_t *buf, void *varg)
        mutex_enter(&db->db_mtx);
        ASSERT(dr->dt.dl.dr_override_state == DR_IN_DMU_SYNC);
        if (zio->io_error == 0) {
+               dr->dt.dl.dr_nopwrite = !!(zio->io_flags & ZIO_FLAG_NOPWRITE);
+               if (dr->dt.dl.dr_nopwrite) {
+                       ASSERTV(blkptr_t *bp = zio->io_bp);
+                       ASSERTV(blkptr_t *bp_orig = &zio->io_bp_orig);
+                       ASSERTV(uint8_t chksum = BP_GET_CHECKSUM(bp_orig));
+
+                       ASSERT(BP_EQUAL(bp, bp_orig));
+                       ASSERT(zio->io_prop.zp_compress != ZIO_COMPRESS_OFF);
+                       ASSERT(zio_checksum_table[chksum].ci_dedup);
+               }
                dr->dt.dl.dr_overridden_by = *zio->io_bp;
                dr->dt.dl.dr_override_state = DR_OVERRIDDEN;
                dr->dt.dl.dr_copies = zio->io_prop.zp_copies;
@@ -1495,11 +1516,22 @@ dmu_sync_late_arrival_done(zio_t *zio)
 {
        blkptr_t *bp = zio->io_bp;
        dmu_sync_arg_t *dsa = zio->io_private;
+       ASSERTV(blkptr_t *bp_orig = &zio->io_bp_orig);
 
        if (zio->io_error == 0 && !BP_IS_HOLE(bp)) {
-               ASSERT(zio->io_bp->blk_birth == zio->io_txg);
-               ASSERT(zio->io_txg > spa_syncing_txg(zio->io_spa));
-               zio_free(zio->io_spa, zio->io_txg, zio->io_bp);
+               /*
+                * If we didn't allocate a new block (i.e. ZIO_FLAG_NOPWRITE)
+                * then there is nothing to do here. Otherwise, free the
+                * newly allocated block in this txg.
+                */
+               if (zio->io_flags & ZIO_FLAG_NOPWRITE) {
+                       ASSERT(BP_EQUAL(bp, bp_orig));
+               } else {
+                       ASSERT(BP_IS_HOLE(bp_orig) || !BP_EQUAL(bp, bp_orig));
+                       ASSERT(zio->io_bp->blk_birth == zio->io_txg);
+                       ASSERT(zio->io_txg > spa_syncing_txg(zio->io_spa));
+                       zio_free(zio->io_spa, zio->io_txg, zio->io_bp);
+               }
        }
 
        dmu_tx_commit(dsa->dsa_tx);
@@ -1520,7 +1552,8 @@ dmu_sync_late_arrival(zio_t *pio, objset_t *os, dmu_sync_cb_t *done, zgd_t *zgd,
        dmu_tx_hold_space(tx, zgd->zgd_db->db_size);
        if (dmu_tx_assign(tx, TXG_WAIT) != 0) {
                dmu_tx_abort(tx);
-               return (EIO);   /* Make zl_get_data do txg_waited_synced() */
+               /* Make zl_get_data do txg_waited_synced() */
+               return (SET_ERROR(EIO));
        }
 
        dsa = kmem_alloc(sizeof (dmu_sync_arg_t), KM_PUSHPAGE);
@@ -1531,8 +1564,8 @@ dmu_sync_late_arrival(zio_t *pio, objset_t *os, dmu_sync_cb_t *done, zgd_t *zgd,
 
        zio_nowait(zio_write(pio, os->os_spa, dmu_tx_get_txg(tx), zgd->zgd_bp,
            zgd->zgd_db->db_data, zgd->zgd_db->db_size, zp,
-           dmu_sync_late_arrival_ready, dmu_sync_late_arrival_done, dsa,
-           ZIO_PRIORITY_SYNC_WRITE, ZIO_FLAG_CANFAIL | ZIO_FLAG_FASTWRITE, zb));
+           dmu_sync_late_arrival_ready, NULL, dmu_sync_late_arrival_done, dsa,
+           ZIO_PRIORITY_SYNC_WRITE, ZIO_FLAG_CANFAIL|ZIO_FLAG_FASTWRITE, zb));
 
        return (0);
 }
@@ -1544,7 +1577,7 @@ dmu_sync_late_arrival(zio_t *pio, objset_t *os, dmu_sync_cb_t *done, zgd_t *zgd,
  *
  * Return values:
  *
- *     EEXIST: this txg has already been synced, so there's nothing to to.
+ *     EEXIST: this txg has already been synced, so there's nothing to do.
  *             The caller should not log the write.
  *
  *     ENOENT: the block was dbuf_free_range()'d, so there's nothing to do.
@@ -1576,7 +1609,6 @@ dmu_sync(zio_t *pio, uint64_t txg, dmu_sync_cb_t *done, zgd_t *zgd)
        dnode_t *dn;
 
        ASSERT(pio != NULL);
-       ASSERT(BP_IS_HOLE(bp));
        ASSERT(txg != 0);
 
        SET_BOOKMARK(&zb, ds->ds_object,
@@ -1606,7 +1638,7 @@ dmu_sync(zio_t *pio, uint64_t txg, dmu_sync_cb_t *done, zgd_t *zgd)
                 * This txg has already synced.  There's nothing to do.
                 */
                mutex_exit(&db->db_mtx);
-               return (EEXIST);
+               return (SET_ERROR(EEXIST));
        }
 
        if (txg <= spa_syncing_txg(os->os_spa)) {
@@ -1628,9 +1660,26 @@ dmu_sync(zio_t *pio, uint64_t txg, dmu_sync_cb_t *done, zgd_t *zgd)
                 * There's no need to log writes to freed blocks, so we're done.
                 */
                mutex_exit(&db->db_mtx);
-               return (ENOENT);
+               return (SET_ERROR(ENOENT));
        }
 
+       ASSERT(dr->dr_next == NULL || dr->dr_next->dr_txg < txg);
+
+       /*
+        * Assume the on-disk data is X, the current syncing data is Y,
+        * and the current in-memory data is Z (currently in dmu_sync).
+        * X and Z are identical but Y is has been modified. Normally,
+        * when X and Z are the same we will perform a nopwrite but if Y
+        * is different we must disable nopwrite since the resulting write
+        * of Y to disk can free the block containing X. If we allowed a
+        * nopwrite to occur the block pointing to Z would reference a freed
+        * block. Since this is a rare case we simplify this by disabling
+        * nopwrite if the current dmu_sync-ing dbuf has been modified in
+        * a previous transaction.
+        */
+       if (dr->dr_next)
+               zp.zp_nopwrite = B_FALSE;
+
        ASSERT(dr->dr_txg == txg);
        if (dr->dt.dl.dr_override_state == DR_IN_DMU_SYNC ||
            dr->dt.dl.dr_override_state == DR_OVERRIDDEN) {
@@ -1640,7 +1689,7 @@ dmu_sync(zio_t *pio, uint64_t txg, dmu_sync_cb_t *done, zgd_t *zgd)
                 * have been dirtied since, or we would have cleared the state.
                 */
                mutex_exit(&db->db_mtx);
-               return (EALREADY);
+               return (SET_ERROR(EALREADY));
        }
 
        ASSERT(dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN);
@@ -1655,8 +1704,9 @@ dmu_sync(zio_t *pio, uint64_t txg, dmu_sync_cb_t *done, zgd_t *zgd)
 
        zio_nowait(arc_write(pio, os->os_spa, txg,
            bp, dr->dt.dl.dr_data, DBUF_IS_L2CACHEABLE(db),
-           DBUF_IS_L2COMPRESSIBLE(db), &zp, dmu_sync_ready, dmu_sync_done,
-           dsa, ZIO_PRIORITY_SYNC_WRITE, ZIO_FLAG_CANFAIL | ZIO_FLAG_FASTWRITE, &zb));
+           DBUF_IS_L2COMPRESSIBLE(db), &zp, dmu_sync_ready,
+           NULL, dmu_sync_done, dsa, ZIO_PRIORITY_SYNC_WRITE,
+           ZIO_FLAG_CANFAIL, &zb));
 
        return (0);
 }
@@ -1715,14 +1765,26 @@ dmu_write_policy(objset_t *os, dnode_t *dn, int level, int wp, zio_prop_t *zp)
        enum zio_checksum checksum = os->os_checksum;
        enum zio_compress compress = os->os_compress;
        enum zio_checksum dedup_checksum = os->os_dedup_checksum;
-       boolean_t dedup;
+       boolean_t dedup = B_FALSE;
+       boolean_t nopwrite = B_FALSE;
        boolean_t dedup_verify = os->os_dedup_verify;
        int copies = os->os_copies;
 
        /*
-        * Determine checksum setting.
+        * We maintain different write policies for each of the following
+        * types of data:
+        *       1. metadata
+        *       2. preallocated blocks (i.e. level-0 blocks of a dump device)
+        *       3. all other level 0 blocks
         */
        if (ismd) {
+               /*
+                * XXX -- we should design a compression algorithm
+                * that specializes in arrays of bps.
+                */
+               compress = zfs_mdcomp_disable ? ZIO_COMPRESS_EMPTY :
+                   ZIO_COMPRESS_LZJB;
+
                /*
                 * Metadata always gets checksummed.  If the data
                 * checksum is multi-bit correctable, and it's not a
@@ -1733,45 +1795,47 @@ dmu_write_policy(objset_t *os, dnode_t *dn, int level, int wp, zio_prop_t *zp)
                if (zio_checksum_table[checksum].ci_correctable < 1 ||
                    zio_checksum_table[checksum].ci_eck)
                        checksum = ZIO_CHECKSUM_FLETCHER_4;
-       } else {
-               checksum = zio_checksum_select(dn->dn_checksum, checksum);
-       }
+       } else if (wp & WP_NOFILL) {
+               ASSERT(level == 0);
 
-       /*
-        * Determine compression setting.
-        */
-       if (ismd) {
                /*
-                * XXX -- we should design a compression algorithm
-                * that specializes in arrays of bps.
+                * If we're writing preallocated blocks, we aren't actually
+                * writing them so don't set any policy properties.  These
+                * blocks are currently only used by an external subsystem
+                * outside of zfs (i.e. dump) and not written by the zio
+                * pipeline.
                 */
-               compress = zfs_mdcomp_disable ? ZIO_COMPRESS_EMPTY :
-                   ZIO_COMPRESS_LZJB;
+               compress = ZIO_COMPRESS_OFF;
+               checksum = ZIO_CHECKSUM_OFF;
        } else {
                compress = zio_compress_select(dn->dn_compress, compress);
-       }
 
-       /*
-        * Determine dedup setting.  If we are in dmu_sync(), we won't
-        * actually dedup now because that's all done in syncing context;
-        * but we do want to use the dedup checkum.  If the checksum is not
-        * strong enough to ensure unique signatures, force dedup_verify.
-        */
-       dedup = (!ismd && dedup_checksum != ZIO_CHECKSUM_OFF);
-       if (dedup) {
-               checksum = dedup_checksum;
-               if (!zio_checksum_table[checksum].ci_dedup)
-                       dedup_verify = 1;
-       }
+               checksum = (dedup_checksum == ZIO_CHECKSUM_OFF) ?
+                   zio_checksum_select(dn->dn_checksum, checksum) :
+                   dedup_checksum;
 
-       if (wp & WP_DMU_SYNC)
-               dedup = 0;
+               /*
+                * Determine dedup setting.  If we are in dmu_sync(),
+                * we won't actually dedup now because that's all
+                * done in syncing context; but we do want to use the
+                * dedup checkum.  If the checksum is not strong
+                * enough to ensure unique signatures, force
+                * dedup_verify.
+                */
+               if (dedup_checksum != ZIO_CHECKSUM_OFF) {
+                       dedup = (wp & WP_DMU_SYNC) ? B_FALSE : B_TRUE;
+                       if (!zio_checksum_table[checksum].ci_dedup)
+                               dedup_verify = B_TRUE;
+               }
 
-       if (wp & WP_NOFILL) {
-               ASSERT(!ismd && level == 0);
-               checksum = ZIO_CHECKSUM_OFF;
-               compress = ZIO_COMPRESS_OFF;
-               dedup = B_FALSE;
+               /*
+                * Enable nopwrite if we have a cryptographically secure
+                * checksum that has no known collisions (i.e. SHA-256)
+                * and compression is enabled.  We don't enable nopwrite if
+                * dedup is enabled as the two features are mutually exclusive.
+                */
+               nopwrite = (!dedup && zio_checksum_table[checksum].ci_dedup &&
+                   compress != ZIO_COMPRESS_OFF && zfs_nopwrite_enabled);
        }
 
        zp->zp_checksum = checksum;
@@ -1781,6 +1845,7 @@ dmu_write_policy(objset_t *os, dnode_t *dn, int level, int wp, zio_prop_t *zp)
        zp->zp_copies = MIN(copies + ismd, spa_max_replication(os->os_spa));
        zp->zp_dedup = dedup;
        zp->zp_dedup_verify = dedup && dedup_verify;
+       zp->zp_nopwrite = nopwrite;
 }
 
 int
@@ -1830,7 +1895,7 @@ __dmu_object_info_from_dnode(dnode_t *dn, dmu_object_info_t *doi)
        doi->doi_checksum = dn->dn_checksum;
        doi->doi_compress = dn->dn_compress;
        doi->doi_physical_blocks_512 = (DN_USED_BYTES(dnp) + 256) >> 9;
-       doi->doi_max_offset = (dnp->dn_maxblkid + 1) * dn->dn_datablksz;
+       doi->doi_max_offset = (dn->dn_maxblkid + 1) * dn->dn_datablksz;
        doi->doi_fill_count = 0;
        for (i = 0; i < dnp->dn_nblkptr; i++)
                doi->doi_fill_count += dnp->dn_blkptr[i].blk_fill;
@@ -1965,7 +2030,7 @@ dmu_init(void)
 void
 dmu_fini(void)
 {
-       arc_fini();
+       arc_fini(); /* arc depends on l2arc, so arc must go first */
        l2arc_fini();
        dmu_tx_fini();
        zfetch_fini();
@@ -1984,7 +2049,7 @@ EXPORT_SYMBOL(dmu_buf_rele_array);
 EXPORT_SYMBOL(dmu_prefetch);
 EXPORT_SYMBOL(dmu_free_range);
 EXPORT_SYMBOL(dmu_free_long_range);
-EXPORT_SYMBOL(dmu_free_object);
+EXPORT_SYMBOL(dmu_free_long_object);
 EXPORT_SYMBOL(dmu_read);
 EXPORT_SYMBOL(dmu_write);
 EXPORT_SYMBOL(dmu_prealloc);
@@ -2005,4 +2070,8 @@ EXPORT_SYMBOL(dmu_ot);
 
 module_param(zfs_mdcomp_disable, int, 0644);
 MODULE_PARM_DESC(zfs_mdcomp_disable, "Disable meta data compression");
+
+module_param(zfs_nopwrite_enabled, int, 0644);
+MODULE_PARM_DESC(zfs_nopwrite_enabled, "Enable NOP writes");
+
 #endif