]> 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 1e16b1eb05784675b5f62c3179696f99e3e40227..9e99558a36ad7430fd64ef7fdd881d640193bc1a 100644 (file)
@@ -370,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);
 
@@ -402,11 +400,9 @@ dmu_buf_hold_array_by_dnode(dnode_t *dn, uint64_t offset, uint64_t length,
                }
                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;
-       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++) {
@@ -427,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);
@@ -511,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;
@@ -529,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;
        }
@@ -546,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);
@@ -568,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);
 }
@@ -674,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);
 }
 
@@ -872,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)
@@ -1039,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
@@ -1062,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) {
@@ -1084,7 +1090,7 @@ dmu_bio_clone(struct bio *bio, struct bio **bio_copy)
 
        *bio_copy = bio_root;
 
-       return 0;
+       return (0);
 }
 
 int
@@ -1101,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);
 
@@ -1163,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);
 
@@ -1558,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);
 }
@@ -1698,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);
 }
@@ -2042,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);