]> git.proxmox.com Git - mirror_zfs.git/blobdiff - module/zfs/vdev_label.c
Undo c89 workarounds to match with upstream
[mirror_zfs.git] / module / zfs / vdev_label.c
index 4edbfa41e6677a54fbc72e5f62330ee98536bede..26fc936459b2f8745873347e621d6bf68529bc14 100644 (file)
@@ -21,7 +21,7 @@
 
 /*
  * Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
- * Copyright (c) 2013 by Delphix. All rights reserved.
+ * Copyright (c) 2012, 2015 by Delphix. All rights reserved.
  */
 
 /*
 #include <sys/metaslab.h>
 #include <sys/zio.h>
 #include <sys/dsl_scan.h>
+#include <sys/abd.h>
 #include <sys/fs/zfs.h>
 
 /*
@@ -178,11 +179,12 @@ vdev_label_number(uint64_t psize, uint64_t offset)
 }
 
 static void
-vdev_label_read(zio_t *zio, vdev_t *vd, int l, void *buf, uint64_t offset,
-       uint64_t size, zio_done_func_t *done, void *private, int flags)
+vdev_label_read(zio_t *zio, vdev_t *vd, int l, abd_t *buf, uint64_t offset,
+    uint64_t size, zio_done_func_t *done, void *private, int flags)
 {
-       ASSERT(spa_config_held(zio->io_spa, SCL_STATE_ALL, RW_WRITER) ==
-           SCL_STATE_ALL);
+       ASSERT(
+           spa_config_held(zio->io_spa, SCL_STATE, RW_READER) == SCL_STATE ||
+           spa_config_held(zio->io_spa, SCL_STATE, RW_WRITER) == SCL_STATE);
        ASSERT(flags & ZIO_FLAG_CONFIG_WRITER);
 
        zio_nowait(zio_read_phys(zio, vd,
@@ -191,14 +193,13 @@ vdev_label_read(zio_t *zio, vdev_t *vd, int l, void *buf, uint64_t offset,
            ZIO_PRIORITY_SYNC_READ, flags, B_TRUE));
 }
 
-static void
-vdev_label_write(zio_t *zio, vdev_t *vd, int l, void *buf, uint64_t offset,
-       uint64_t size, zio_done_func_t *done, void *private, int flags)
+void
+vdev_label_write(zio_t *zio, vdev_t *vd, int l, abd_t *buf, uint64_t offset,
+    uint64_t size, zio_done_func_t *done, void *private, int flags)
 {
-       ASSERT(spa_config_held(zio->io_spa, SCL_ALL, RW_WRITER) == SCL_ALL ||
-           (spa_config_held(zio->io_spa, SCL_CONFIG | SCL_STATE, RW_READER) ==
-           (SCL_CONFIG | SCL_STATE) &&
-           dsl_pool_sync_context(spa_get_dsl(zio->io_spa))));
+       ASSERT(
+           spa_config_held(zio->io_spa, SCL_STATE, RW_READER) == SCL_STATE ||
+           spa_config_held(zio->io_spa, SCL_STATE, RW_WRITER) == SCL_STATE);
        ASSERT(flags & ZIO_FLAG_CONFIG_WRITER);
 
        zio_nowait(zio_write_phys(zio, vd,
@@ -518,6 +519,7 @@ vdev_config_generate(spa_t *spa, vdev_t *vd, boolean_t getstats,
                if (vd->vdev_ishole)
                        fnvlist_add_uint64(nv, ZPOOL_CONFIG_IS_HOLE, B_TRUE);
 
+               /* Set the reason why we're FAULTED/DEGRADED. */
                switch (vd->vdev_stat.vs_aux) {
                case VDEV_AUX_ERR_EXCEEDED:
                        aux = "err_exceeded";
@@ -528,8 +530,15 @@ vdev_config_generate(spa_t *spa, vdev_t *vd, boolean_t getstats,
                        break;
                }
 
-               if (aux != NULL)
+               if (aux != NULL && !vd->vdev_tmpoffline) {
                        fnvlist_add_string(nv, ZPOOL_CONFIG_AUX_STATE, aux);
+               } else {
+                       /*
+                        * We're healthy - clear any previous AUX_STATE values.
+                        */
+                       if (nvlist_exists(nv, ZPOOL_CONFIG_AUX_STATE))
+                               nvlist_remove_all(nv, ZPOOL_CONFIG_AUX_STATE);
+               }
 
                if (vd->vdev_splitting && vd->vdev_orig_guid != 0LL) {
                        fnvlist_add_uint64(nv, ZPOOL_CONFIG_ORIG_GUID,
@@ -587,27 +596,28 @@ vdev_label_read_config(vdev_t *vd, uint64_t txg)
        spa_t *spa = vd->vdev_spa;
        nvlist_t *config = NULL;
        vdev_phys_t *vp;
+       abd_t *vp_abd;
        zio_t *zio;
        uint64_t best_txg = 0;
        int error = 0;
        int flags = ZIO_FLAG_CONFIG_WRITER | ZIO_FLAG_CANFAIL |
            ZIO_FLAG_SPECULATIVE;
-       int l;
 
        ASSERT(spa_config_held(spa, SCL_STATE_ALL, RW_WRITER) == SCL_STATE_ALL);
 
        if (!vdev_readable(vd))
                return (NULL);
 
-       vp = zio_buf_alloc(sizeof (vdev_phys_t));
+       vp_abd = abd_alloc_linear(sizeof (vdev_phys_t), B_TRUE);
+       vp = abd_to_buf(vp_abd);
 
 retry:
-       for (l = 0; l < VDEV_LABELS; l++) {
+       for (int l = 0; l < VDEV_LABELS; l++) {
                nvlist_t *label = NULL;
 
                zio = zio_root(spa, NULL, NULL, flags);
 
-               vdev_label_read(zio, vd, l, vp,
+               vdev_label_read(zio, vd, l, vp_abd,
                    offsetof(vdev_label_t, vl_vdev_phys),
                    sizeof (vdev_phys_t), NULL, NULL, flags);
 
@@ -646,7 +656,7 @@ retry:
                goto retry;
        }
 
-       zio_buf_free(vp, sizeof (vdev_phys_t));
+       abd_free(vp_abd);
 
        return (config);
 }
@@ -782,20 +792,20 @@ vdev_label_init(vdev_t *vd, uint64_t crtxg, vdev_labeltype_t reason)
        spa_t *spa = vd->vdev_spa;
        nvlist_t *label;
        vdev_phys_t *vp;
-       char *pad2;
+       abd_t *vp_abd;
+       abd_t *pad2;
        uberblock_t *ub;
+       abd_t *ub_abd;
        zio_t *zio;
        char *buf;
        size_t buflen;
        int error;
        uint64_t spare_guid = 0, l2cache_guid = 0;
        int flags = ZIO_FLAG_CONFIG_WRITER | ZIO_FLAG_CANFAIL;
-       int c, l;
-       vdev_t *pvd;
 
        ASSERT(spa_config_held(spa, SCL_ALL, RW_WRITER) == SCL_ALL);
 
-       for (c = 0; c < vd->vdev_children; c++)
+       for (int c = 0; c < vd->vdev_children; c++)
                if ((error = vdev_label_init(vd->vdev_child[c],
                    crtxg, reason)) != 0)
                        return (error);
@@ -831,7 +841,7 @@ vdev_label_init(vdev_t *vd, uint64_t crtxg, vdev_labeltype_t reason)
 
                vd->vdev_guid += guid_delta;
 
-               for (pvd = vd; pvd != NULL; pvd = pvd->vdev_parent)
+               for (vdev_t *pvd = vd; pvd != NULL; pvd = pvd->vdev_parent)
                        pvd->vdev_guid_sum += guid_delta;
 
                /*
@@ -851,7 +861,7 @@ vdev_label_init(vdev_t *vd, uint64_t crtxg, vdev_labeltype_t reason)
 
                vd->vdev_guid += guid_delta;
 
-               for (pvd = vd; pvd != NULL; pvd = pvd->vdev_parent)
+               for (vdev_t *pvd = vd; pvd != NULL; pvd = pvd->vdev_parent)
                        pvd->vdev_guid_sum += guid_delta;
 
                /*
@@ -867,8 +877,9 @@ vdev_label_init(vdev_t *vd, uint64_t crtxg, vdev_labeltype_t reason)
        /*
         * Initialize its label.
         */
-       vp = zio_buf_alloc(sizeof (vdev_phys_t));
-       bzero(vp, sizeof (vdev_phys_t));
+       vp_abd = abd_alloc_linear(sizeof (vdev_phys_t), B_TRUE);
+       abd_zero(vp_abd, sizeof (vdev_phys_t));
+       vp = abd_to_buf(vp_abd);
 
        /*
         * Generate a label describing the pool and our top-level vdev.
@@ -928,22 +939,23 @@ vdev_label_init(vdev_t *vd, uint64_t crtxg, vdev_labeltype_t reason)
        error = nvlist_pack(label, &buf, &buflen, NV_ENCODE_XDR, KM_SLEEP);
        if (error != 0) {
                nvlist_free(label);
-               zio_buf_free(vp, sizeof (vdev_phys_t));
+               abd_free(vp_abd);
                /* EFAULT means nvlist_pack ran out of room */
-               return (error == EFAULT ? ENAMETOOLONG : EINVAL);
+               return (SET_ERROR(error == EFAULT ? ENAMETOOLONG : EINVAL));
        }
 
        /*
         * Initialize uberblock template.
         */
-       ub = zio_buf_alloc(VDEV_UBERBLOCK_RING);
-       bzero(ub, VDEV_UBERBLOCK_RING);
-       *ub = spa->spa_uberblock;
+       ub_abd = abd_alloc_linear(VDEV_UBERBLOCK_RING, B_TRUE);
+       abd_zero(ub_abd, VDEV_UBERBLOCK_RING);
+       abd_copy_from_buf(ub_abd, &spa->spa_uberblock, sizeof (uberblock_t));
+       ub = abd_to_buf(ub_abd);
        ub->ub_txg = 0;
 
        /* Initialize the 2nd padding area. */
-       pad2 = zio_buf_alloc(VDEV_PAD_SIZE);
-       bzero(pad2, VDEV_PAD_SIZE);
+       pad2 = abd_alloc_for_io(VDEV_PAD_SIZE, B_TRUE);
+       abd_zero(pad2, VDEV_PAD_SIZE);
 
        /*
         * Write everything in parallel.
@@ -951,9 +963,9 @@ vdev_label_init(vdev_t *vd, uint64_t crtxg, vdev_labeltype_t reason)
 retry:
        zio = zio_root(spa, NULL, NULL, flags);
 
-       for (l = 0; l < VDEV_LABELS; l++) {
+       for (int l = 0; l < VDEV_LABELS; l++) {
 
-               vdev_label_write(zio, vd, l, vp,
+               vdev_label_write(zio, vd, l, vp_abd,
                    offsetof(vdev_label_t, vl_vdev_phys),
                    sizeof (vdev_phys_t), NULL, NULL, flags);
 
@@ -966,7 +978,7 @@ retry:
                    offsetof(vdev_label_t, vl_pad2),
                    VDEV_PAD_SIZE, NULL, NULL, flags);
 
-               vdev_label_write(zio, vd, l, ub,
+               vdev_label_write(zio, vd, l, ub_abd,
                    offsetof(vdev_label_t, vl_uberblock),
                    VDEV_UBERBLOCK_RING, NULL, NULL, flags);
        }
@@ -979,9 +991,9 @@ retry:
        }
 
        nvlist_free(label);
-       zio_buf_free(pad2, VDEV_PAD_SIZE);
-       zio_buf_free(ub, VDEV_UBERBLOCK_RING);
-       zio_buf_free(vp, sizeof (vdev_phys_t));
+       abd_free(pad2);
+       abd_free(ub_abd);
+       abd_free(vp_abd);
 
        /*
         * If this vdev hasn't been previously identified as a spare, then we
@@ -1039,7 +1051,7 @@ vdev_uberblock_load_done(zio_t *zio)
        vdev_t *vd = zio->io_vd;
        spa_t *spa = zio->io_spa;
        zio_t *rio = zio->io_private;
-       uberblock_t *ub = zio->io_data;
+       uberblock_t *ub = abd_to_buf(zio->io_abd);
        struct ubl_cbdata *cbp = rio->io_private;
 
        ASSERT3U(zio->io_size, ==, VDEV_UBERBLOCK_SIZE(vd));
@@ -1060,24 +1072,22 @@ vdev_uberblock_load_done(zio_t *zio)
                mutex_exit(&rio->io_lock);
        }
 
-       zio_buf_free(zio->io_data, zio->io_size);
+       abd_free(zio->io_abd);
 }
 
 static void
 vdev_uberblock_load_impl(zio_t *zio, vdev_t *vd, int flags,
     struct ubl_cbdata *cbp)
 {
-       int c, l, n;
-
-       for (c = 0; c < vd->vdev_children; c++)
+       for (int c = 0; c < vd->vdev_children; c++)
                vdev_uberblock_load_impl(zio, vd->vdev_child[c], flags, cbp);
 
        if (vd->vdev_ops->vdev_op_leaf && vdev_readable(vd)) {
-               for (l = 0; l < VDEV_LABELS; l++) {
-                       for (n = 0; n < VDEV_UBERBLOCK_COUNT(vd); n++) {
+               for (int l = 0; l < VDEV_LABELS; l++) {
+                       for (int n = 0; n < VDEV_UBERBLOCK_COUNT(vd); n++) {
                                vdev_label_read(zio, vd, l,
-                                   zio_buf_alloc(VDEV_UBERBLOCK_SIZE(vd)),
-                                   VDEV_UBERBLOCK_OFFSET(vd, n),
+                                   abd_alloc_linear(VDEV_UBERBLOCK_SIZE(vd),
+                                   B_TRUE), VDEV_UBERBLOCK_OFFSET(vd, n),
                                    VDEV_UBERBLOCK_SIZE(vd),
                                    vdev_uberblock_load_done, zio, flags);
                        }
@@ -1125,6 +1135,60 @@ vdev_uberblock_load(vdev_t *rvd, uberblock_t *ub, nvlist_t **config)
        spa_config_exit(spa, SCL_ALL, FTAG);
 }
 
+/*
+ * For use when a leaf vdev is expanded.
+ * The location of labels 2 and 3 changed, and at the new location the
+ * uberblock rings are either empty or contain garbage.  The sync will write
+ * new configs there because the vdev is dirty, but expansion also needs the
+ * uberblock rings copied.  Read them from label 0 which did not move.
+ *
+ * Since the point is to populate labels {2,3} with valid uberblocks,
+ * we zero uberblocks we fail to read or which are not valid.
+ */
+
+static void
+vdev_copy_uberblocks(vdev_t *vd)
+{
+       abd_t *ub_abd;
+       zio_t *write_zio;
+       int locks = (SCL_L2ARC | SCL_ZIO);
+       int flags = ZIO_FLAG_CONFIG_WRITER | ZIO_FLAG_CANFAIL |
+           ZIO_FLAG_SPECULATIVE;
+
+       ASSERT(spa_config_held(vd->vdev_spa, SCL_STATE, RW_READER) ==
+           SCL_STATE);
+       ASSERT(vd->vdev_ops->vdev_op_leaf);
+
+       spa_config_enter(vd->vdev_spa, locks, FTAG, RW_READER);
+
+       ub_abd = abd_alloc_linear(VDEV_UBERBLOCK_SIZE(vd), B_TRUE);
+
+       write_zio = zio_root(vd->vdev_spa, NULL, NULL, flags);
+       for (int n = 0; n < VDEV_UBERBLOCK_COUNT(vd); n++) {
+               const int src_label = 0;
+               zio_t *zio;
+
+               zio = zio_root(vd->vdev_spa, NULL, NULL, flags);
+               vdev_label_read(zio, vd, src_label, ub_abd,
+                   VDEV_UBERBLOCK_OFFSET(vd, n), VDEV_UBERBLOCK_SIZE(vd),
+                   NULL, NULL, flags);
+
+               if (zio_wait(zio) || uberblock_verify(abd_to_buf(ub_abd)))
+                       abd_zero(ub_abd, VDEV_UBERBLOCK_SIZE(vd));
+
+               for (int l = 2; l < VDEV_LABELS; l++)
+                       vdev_label_write(write_zio, vd, l, ub_abd,
+                           VDEV_UBERBLOCK_OFFSET(vd, n),
+                           VDEV_UBERBLOCK_SIZE(vd), NULL, NULL,
+                           flags | ZIO_FLAG_DONT_PROPAGATE);
+       }
+       (void) zio_wait(write_zio);
+
+       spa_config_exit(vd->vdev_spa, locks, FTAG);
+
+       abd_free(ub_abd);
+}
+
 /*
  * On success, increment root zio's count of good writes.
  * We only get credit for writes to known-visible vdevs; see spa_vdev_add().
@@ -1144,10 +1208,7 @@ vdev_uberblock_sync_done(zio_t *zio)
 static void
 vdev_uberblock_sync(zio_t *zio, uberblock_t *ub, vdev_t *vd, int flags)
 {
-       uberblock_t *ubbuf;
-       int c, l, n;
-
-       for (c = 0; c < vd->vdev_children; c++)
+       for (int c = 0; c < vd->vdev_children; c++)
                vdev_uberblock_sync(zio, ub, vd->vdev_child[c], flags);
 
        if (!vd->vdev_ops->vdev_op_leaf)
@@ -1156,19 +1217,28 @@ vdev_uberblock_sync(zio_t *zio, uberblock_t *ub, vdev_t *vd, int flags)
        if (!vdev_writeable(vd))
                return;
 
-       n = ub->ub_txg & (VDEV_UBERBLOCK_COUNT(vd) - 1);
+       /* If the vdev was expanded, need to copy uberblock rings. */
+       if (vd->vdev_state == VDEV_STATE_HEALTHY &&
+           vd->vdev_copy_uberblocks == B_TRUE) {
+               vdev_copy_uberblocks(vd);
+               vd->vdev_copy_uberblocks = B_FALSE;
+       }
+
+       int m = spa_multihost(vd->vdev_spa) ? MMP_BLOCKS_PER_LABEL : 0;
+       int n = ub->ub_txg % (VDEV_UBERBLOCK_COUNT(vd) - m);
 
-       ubbuf = zio_buf_alloc(VDEV_UBERBLOCK_SIZE(vd));
-       bzero(ubbuf, VDEV_UBERBLOCK_SIZE(vd));
-       *ubbuf = *ub;
+       /* Copy the uberblock_t into the ABD */
+       abd_t *ub_abd = abd_alloc_for_io(VDEV_UBERBLOCK_SIZE(vd), B_TRUE);
+       abd_zero(ub_abd, VDEV_UBERBLOCK_SIZE(vd));
+       abd_copy_from_buf(ub_abd, ub, sizeof (uberblock_t));
 
-       for (l = 0; l < VDEV_LABELS; l++)
-               vdev_label_write(zio, vd, l, ubbuf,
+       for (int l = 0; l < VDEV_LABELS; l++)
+               vdev_label_write(zio, vd, l, ub_abd,
                    VDEV_UBERBLOCK_OFFSET(vd, n), VDEV_UBERBLOCK_SIZE(vd),
                    vdev_uberblock_sync_done, zio->io_private,
                    flags | ZIO_FLAG_DONT_PROPAGATE);
 
-       zio_buf_free(ubbuf, VDEV_UBERBLOCK_SIZE(vd));
+       abd_free(ub_abd);
 }
 
 /* Sync the uberblocks to all vdevs in svd[] */
@@ -1178,11 +1248,10 @@ vdev_uberblock_sync_list(vdev_t **svd, int svdcount, uberblock_t *ub, int flags)
        spa_t *spa = svd[0]->vdev_spa;
        zio_t *zio;
        uint64_t good_writes = 0;
-       int v;
 
        zio = zio_root(spa, NULL, &good_writes, flags);
 
-       for (v = 0; v < svdcount; v++)
+       for (int v = 0; v < svdcount; v++)
                vdev_uberblock_sync(zio, ub, svd[v], flags);
 
        (void) zio_wait(zio);
@@ -1194,7 +1263,7 @@ vdev_uberblock_sync_list(vdev_t **svd, int svdcount, uberblock_t *ub, int flags)
         */
        zio = zio_root(spa, NULL, NULL, flags);
 
-       for (v = 0; v < svdcount; v++)
+       for (int v = 0; v < svdcount; v++)
                zio_flush(zio, svd[v]);
 
        (void) zio_wait(zio);
@@ -1245,11 +1314,11 @@ vdev_label_sync(zio_t *zio, vdev_t *vd, int l, uint64_t txg, int flags)
 {
        nvlist_t *label;
        vdev_phys_t *vp;
+       abd_t *vp_abd;
        char *buf;
        size_t buflen;
-       int c;
 
-       for (c = 0; c < vd->vdev_children; c++)
+       for (int c = 0; c < vd->vdev_children; c++)
                vdev_label_sync(zio, vd->vdev_child[c], l, txg, flags);
 
        if (!vd->vdev_ops->vdev_op_leaf)
@@ -1263,15 +1332,16 @@ vdev_label_sync(zio_t *zio, vdev_t *vd, int l, uint64_t txg, int flags)
         */
        label = spa_config_generate(vd->vdev_spa, vd, txg, B_FALSE);
 
-       vp = zio_buf_alloc(sizeof (vdev_phys_t));
-       bzero(vp, sizeof (vdev_phys_t));
+       vp_abd = abd_alloc_linear(sizeof (vdev_phys_t), B_TRUE);
+       abd_zero(vp_abd, sizeof (vdev_phys_t));
+       vp = abd_to_buf(vp_abd);
 
        buf = vp->vp_nvlist;
        buflen = sizeof (vp->vp_nvlist);
 
        if (!nvlist_pack(label, &buf, &buflen, NV_ENCODE_XDR, KM_SLEEP)) {
                for (; l < VDEV_LABELS; l += 2) {
-                       vdev_label_write(zio, vd, l, vp,
+                       vdev_label_write(zio, vd, l, vp_abd,
                            offsetof(vdev_label_t, vl_vdev_phys),
                            sizeof (vdev_phys_t),
                            vdev_label_sync_done, zio->io_private,
@@ -1279,7 +1349,7 @@ vdev_label_sync(zio_t *zio, vdev_t *vd, int l, uint64_t txg, int flags)
                }
        }
 
-       zio_buf_free(vp, sizeof (vdev_phys_t));
+       abd_free(vp_abd);
        nvlist_free(label);
 }
 
@@ -1298,12 +1368,11 @@ vdev_label_sync_list(spa_t *spa, int l, uint64_t txg, int flags)
 
        for (vd = list_head(dl); vd != NULL; vd = list_next(dl, vd)) {
                uint64_t *good_writes;
-               zio_t *vio;
 
                ASSERT(!vd->vdev_ishole);
 
                good_writes = kmem_zalloc(sizeof (uint64_t), KM_SLEEP);
-               vio = zio_null(zio, spa, NULL,
+               zio_t *vio = zio_null(zio, spa, NULL,
                    (vd->vdev_islog || vd->vdev_aux != NULL) ?
                    vdev_label_sync_ignore_done : vdev_label_sync_top_done,
                    good_writes, flags);
@@ -1369,10 +1438,13 @@ retry:
         * and the vdev configuration hasn't changed,
         * then there's nothing to do.
         */
-       if (ub->ub_txg < txg &&
-           uberblock_update(ub, spa->spa_root_vdev, txg) == B_FALSE &&
-           list_is_empty(&spa->spa_config_dirty_list))
-               return (0);
+       if (ub->ub_txg < txg) {
+               boolean_t changed = uberblock_update(ub, spa->spa_root_vdev,
+                   txg, spa->spa_mmp.mmp_delay);
+
+               if (!changed && list_is_empty(&spa->spa_config_dirty_list))
+                       return (0);
+       }
 
        if (txg > spa_freeze_txg(spa))
                return (0);
@@ -1423,6 +1495,10 @@ retry:
        if ((error = vdev_uberblock_sync_list(svd, svdcount, ub, flags)) != 0)
                goto retry;
 
+
+       if (spa_multihost(spa))
+               mmp_update_uberblock(spa, ub);
+
        /*
         * Sync out odd labels for every dirty vdev.  If the system dies
         * in the middle of this process, the even labels and the new