]> granicus.if.org Git - zfs/blobdiff - module/zfs/vdev_label.c
OpenZFS 9166 - zfs storage pool checkpoint
[zfs] / module / zfs / vdev_label.c
index 85d133a5ac71c94bcbfd33e0e1c3166b61c0af1b..29d7d651bf0d651c50b52b1e382f6c54a55f2e00 100644 (file)
@@ -21,7 +21,7 @@
 
 /*
  * Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
- * Copyright (c) 2012, 2016 by Delphix. All rights reserved.
+ * Copyright (c) 2012, 2018 by Delphix. All rights reserved.
  */
 
 /*
  *     1. Uniquely identify this device as part of a ZFS pool and confirm its
  *        identity within the pool.
  *
- *     2. Verify that all the devices given in a configuration are present
+ *     2. Verify that all the devices given in a configuration are present
  *         within the pool.
  *
- *     3. Determine the uberblock for the pool.
+ *     3. Determine the uberblock for the pool.
  *
- *     4. In case of an import operation, determine the configuration of the
+ *     4. In case of an import operation, determine the configuration of the
  *         toplevel vdev of which it is a part.
  *
- *     5. If an import operation cannot find all the devices in the pool,
+ *     5. If an import operation cannot find all the devices in the pool,
  *         provide enough information to the administrator to determine which
  *         devices are missing.
  *
@@ -77,9 +77,9 @@
  * In order to identify which labels are valid, the labels are written in the
  * following manner:
  *
- *     1. For each vdev, update 'L1' to the new label
- *     2. Update the uberblock
- *     3. For each vdev, update 'L2' to the new label
+ *     1. For each vdev, update 'L1' to the new label
+ *     2. Update the uberblock
+ *     3. For each vdev, update 'L2' to the new label
  *
  * Given arbitrary failure, we can determine the correct label to use based on
  * the transaction group.  If we fail after updating L1 but before updating the
  *
  * The nvlist describing the pool and vdev contains the following elements:
  *
- *     version         ZFS on-disk version
- *     name            Pool name
- *     state           Pool state
- *     txg             Transaction group in which this label was written
- *     pool_guid       Unique identifier for this pool
- *     vdev_tree       An nvlist describing vdev tree.
+ *     version         ZFS on-disk version
+ *     name            Pool name
+ *     state           Pool state
+ *     txg             Transaction group in which this label was written
+ *     pool_guid       Unique identifier for this pool
+ *     vdev_tree       An nvlist describing vdev tree.
  *     features_for_read
  *                     An nvlist of the features necessary for reading the MOS.
  *
  * Each leaf device label also contains the following:
  *
- *     top_guid        Unique ID for top-level vdev in which this is contained
- *     guid            Unique ID for the leaf vdev
+ *     top_guid        Unique ID for top-level vdev in which this is contained
+ *     guid            Unique ID for the leaf vdev
  *
  * The 'vs' configuration follows the format described in 'spa_config.c'.
  */
@@ -352,6 +352,37 @@ vdev_config_generate_stats(vdev_t *vd, nvlist_t *nv)
        kmem_free(vsx, sizeof (*vsx));
 }
 
+static void
+root_vdev_actions_getprogress(vdev_t *vd, nvlist_t *nvl)
+{
+       spa_t *spa = vd->vdev_spa;
+
+       if (vd != spa->spa_root_vdev)
+               return;
+
+       /* provide either current or previous scan information */
+       pool_scan_stat_t ps;
+       if (spa_scan_get_stats(spa, &ps) == 0) {
+               fnvlist_add_uint64_array(nvl,
+                   ZPOOL_CONFIG_SCAN_STATS, (uint64_t *)&ps,
+                   sizeof (pool_scan_stat_t) / sizeof (uint64_t));
+       }
+
+       pool_removal_stat_t prs;
+       if (spa_removal_get_stats(spa, &prs) == 0) {
+               fnvlist_add_uint64_array(nvl,
+                   ZPOOL_CONFIG_REMOVAL_STATS, (uint64_t *)&prs,
+                   sizeof (prs) / sizeof (uint64_t));
+       }
+
+       pool_checkpoint_stat_t pcs;
+       if (spa_checkpoint_get_stats(spa, &pcs) == 0) {
+               fnvlist_add_uint64_array(nvl,
+                   ZPOOL_CONFIG_CHECKPOINT_STATS, (uint64_t *)&pcs,
+                   sizeof (pcs) / sizeof (uint64_t));
+       }
+}
+
 /*
  * Generate the nvlist representing this vdev's config.
  */
@@ -474,20 +505,7 @@ vdev_config_generate(spa_t *spa, vdev_t *vd, boolean_t getstats,
        if (getstats) {
                vdev_config_generate_stats(vd, nv);
 
-               /* provide either current or previous scan information */
-               pool_scan_stat_t ps;
-               if (spa_scan_get_stats(spa, &ps) == 0) {
-                       fnvlist_add_uint64_array(nv,
-                           ZPOOL_CONFIG_SCAN_STATS, (uint64_t *)&ps,
-                           sizeof (pool_scan_stat_t) / sizeof (uint64_t));
-               }
-
-               pool_removal_stat_t prs;
-               if (spa_removal_get_stats(spa, &prs) == 0) {
-                       fnvlist_add_uint64_array(nv,
-                           ZPOOL_CONFIG_REMOVAL_STATS, (uint64_t *)&prs,
-                           sizeof (prs) / sizeof (uint64_t));
-               }
+               root_vdev_actions_getprogress(vd, nv);
 
                /*
                 * Note: this can be called from open context
@@ -515,22 +533,33 @@ vdev_config_generate(spa_t *spa, vdev_t *vd, boolean_t getstats,
                         * histograms.
                         */
                        uint64_t seg_count = 0;
+                       uint64_t to_alloc = vd->vdev_stat.vs_alloc;
 
                        /*
                         * There are the same number of allocated segments
                         * as free segments, so we will have at least one
-                        * entry per free segment.
+                        * entry per free segment.  However, small free
+                        * segments (smaller than vdev_removal_max_span)
+                        * will be combined with adjacent allocated segments
+                        * as a single mapping.
                         */
                        for (int i = 0; i < RANGE_TREE_HISTOGRAM_SIZE; i++) {
-                               seg_count += vd->vdev_mg->mg_histogram[i];
+                               if (1ULL << (i + 1) < vdev_removal_max_span) {
+                                       to_alloc +=
+                                           vd->vdev_mg->mg_histogram[i] <<
+                                           (i + 1);
+                               } else {
+                                       seg_count +=
+                                           vd->vdev_mg->mg_histogram[i];
+                               }
                        }
 
                        /*
-                        * The maximum length of a mapping is SPA_MAXBLOCKSIZE,
-                        * so we need at least one entry per SPA_MAXBLOCKSIZE
-                        * of allocated data.
+                        * The maximum length of a mapping is
+                        * zfs_remove_max_segment, so we need at least one entry
+                        * per zfs_remove_max_segment of allocated data.
                         */
-                       seg_count += vd->vdev_stat.vs_alloc / SPA_MAXBLOCKSIZE;
+                       seg_count += to_alloc / zfs_remove_max_segment;
 
                        fnvlist_add_uint64(nv, ZPOOL_CONFIG_INDIRECT_SIZE,
                            seg_count *
@@ -673,6 +702,7 @@ vdev_label_read_config(vdev_t *vd, uint64_t txg)
        abd_t *vp_abd;
        zio_t *zio;
        uint64_t best_txg = 0;
+       uint64_t label_txg = 0;
        int error = 0;
        int flags = ZIO_FLAG_CONFIG_WRITER | ZIO_FLAG_CANFAIL |
            ZIO_FLAG_SPECULATIVE;
@@ -698,8 +728,6 @@ retry:
                if (zio_wait(zio) == 0 &&
                    nvlist_unpack(vp->vp_nvlist, sizeof (vp->vp_nvlist),
                    &label, 0) == 0) {
-                       uint64_t label_txg = 0;
-
                        /*
                         * Auxiliary vdevs won't have txg values in their
                         * labels and newly added vdevs may not have been
@@ -730,6 +758,15 @@ retry:
                goto retry;
        }
 
+       /*
+        * We found a valid label but it didn't pass txg restrictions.
+        */
+       if (config == NULL && label_txg != 0) {
+               vdev_dbgmsg(vd, "label discarded as txg is too large "
+                   "(%llu > %llu)", (u_longlong_t)label_txg,
+                   (u_longlong_t)txg);
+       }
+
        abd_free(vp_abd);
 
        return (config);
@@ -1506,11 +1543,10 @@ vdev_config_sync(vdev_t **svd, int svdcount, uint64_t txg)
 {
        spa_t *spa = svd[0]->vdev_spa;
        uberblock_t *ub = &spa->spa_uberblock;
-       vdev_t *vd;
-       zio_t *zio;
        int error = 0;
        int flags = ZIO_FLAG_CONFIG_WRITER | ZIO_FLAG_CANFAIL;
 
+       ASSERT(svdcount != 0);
 retry:
        /*
         * Normally, we don't want to try too hard to write every label and
@@ -1552,9 +1588,10 @@ retry:
         * written in this txg will be committed to stable storage
         * before any uberblock that references them.
         */
-       zio = zio_root(spa, NULL, NULL, flags);
+       zio_t *zio = zio_root(spa, NULL, NULL, flags);
 
-       for (vd = txg_list_head(&spa->spa_vdev_txg_list, TXG_CLEAN(txg)); vd;
+       for (vdev_t *vd =
+           txg_list_head(&spa->spa_vdev_txg_list, TXG_CLEAN(txg)); vd != NULL;
            vd = txg_list_next(&spa->spa_vdev_txg_list, vd, TXG_CLEAN(txg)))
                zio_flush(zio, vd);
 
@@ -1569,8 +1606,14 @@ retry:
         * the new labels to disk to ensure that all even-label updates
         * are committed to stable storage before the uberblock update.
         */
-       if ((error = vdev_label_sync_list(spa, 0, txg, flags)) != 0)
+       if ((error = vdev_label_sync_list(spa, 0, txg, flags)) != 0) {
+               if ((flags & ZIO_FLAG_TRYHARD) != 0) {
+                       zfs_dbgmsg("vdev_label_sync_list() returned error %d "
+                           "for pool '%s' when syncing out the even labels "
+                           "of dirty vdevs", error, spa_name(spa));
+               }
                goto retry;
+       }
 
        /*
         * Sync the uberblocks to all vdevs in svd[].
@@ -1587,8 +1630,13 @@ retry:
         *      been successfully committed) will be valid with respect
         *      to the new uberblocks.
         */
-       if ((error = vdev_uberblock_sync_list(svd, svdcount, ub, flags)) != 0)
+       if ((error = vdev_uberblock_sync_list(svd, svdcount, ub, flags)) != 0) {
+               if ((flags & ZIO_FLAG_TRYHARD) != 0) {
+                       zfs_dbgmsg("vdev_uberblock_sync_list() returned error "
+                           "%d for pool '%s'", error, spa_name(spa));
+               }
                goto retry;
+       }
 
        if (spa_multihost(spa))
                mmp_update_uberblock(spa, ub);
@@ -1603,8 +1651,14 @@ retry:
         * to disk to ensure that all odd-label updates are committed to
         * stable storage before the next transaction group begins.
         */
-       if ((error = vdev_label_sync_list(spa, 1, txg, flags)) != 0)
+       if ((error = vdev_label_sync_list(spa, 1, txg, flags)) != 0) {
+               if ((flags & ZIO_FLAG_TRYHARD) != 0) {
+                       zfs_dbgmsg("vdev_label_sync_list() returned error %d "
+                           "for pool '%s' when syncing out the odd labels of "
+                           "dirty vdevs", error, spa_name(spa));
+               }
                goto retry;
+       }
 
        return (0);
 }