diff options
| author | Alan Somers <[email protected]> | 2021-01-12 14:59:56 -0700 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2021-01-26 19:36:02 -0800 |
| commit | 67874d5487f74c11503cb4f195c7a658f628b168 (patch) | |
| tree | f00662022ac7ee19892b51a4d81fe692a8b76f0f /module | |
| parent | a0e01997ec013fa24a570a36689235ee6c21d9e1 (diff) | |
Read all disk labels concurrently in vdev_label_read_config
This is similar to what we already do in vdev_geom_read_config.
Sponsored by: Axcient
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Alan Somers <[email protected]>
Closes #11470
Diffstat (limited to 'module')
| -rw-r--r-- | module/zfs/vdev_label.c | 35 |
1 files changed, 21 insertions, 14 deletions
diff --git a/module/zfs/vdev_label.c b/module/zfs/vdev_label.c index fbd117d2d..d5966e1bd 100644 --- a/module/zfs/vdev_label.c +++ b/module/zfs/vdev_label.c @@ -754,9 +754,9 @@ 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; + vdev_phys_t *vp[VDEV_LABELS]; + abd_t *vp_abd[VDEV_LABELS]; + zio_t *zio[VDEV_LABELS]; uint64_t best_txg = 0; uint64_t label_txg = 0; int error = 0; @@ -776,21 +776,24 @@ vdev_label_read_config(vdev_t *vd, uint64_t txg) if (vd->vdev_ops == &vdev_draid_spare_ops) return (vdev_draid_read_config_spare(vd)); - vp_abd = abd_alloc_linear(sizeof (vdev_phys_t), B_TRUE); - vp = abd_to_buf(vp_abd); + for (int l = 0; l < VDEV_LABELS; l++) { + vp_abd[l] = abd_alloc_linear(sizeof (vdev_phys_t), B_TRUE); + vp[l] = abd_to_buf(vp_abd[l]); + } retry: for (int l = 0; l < VDEV_LABELS; l++) { - nvlist_t *label = NULL; - - zio = zio_root(spa, NULL, NULL, flags); + zio[l] = zio_root(spa, NULL, NULL, flags); - vdev_label_read(zio, vd, l, vp_abd, - offsetof(vdev_label_t, vl_vdev_phys), - sizeof (vdev_phys_t), NULL, NULL, flags); + vdev_label_read(zio[l], vd, l, vp_abd[l], + offsetof(vdev_label_t, vl_vdev_phys), sizeof (vdev_phys_t), + NULL, NULL, flags); + } + for (int l = 0; l < VDEV_LABELS; l++) { + nvlist_t *label = NULL; - if (zio_wait(zio) == 0 && - nvlist_unpack(vp->vp_nvlist, sizeof (vp->vp_nvlist), + if (zio_wait(zio[l]) == 0 && + nvlist_unpack(vp[l]->vp_nvlist, sizeof (vp[l]->vp_nvlist), &label, 0) == 0) { /* * Auxiliary vdevs won't have txg values in their @@ -803,6 +806,8 @@ retry: ZPOOL_CONFIG_POOL_TXG, &label_txg); if ((error || label_txg == 0) && !config) { config = label; + for (l++; l < VDEV_LABELS; l++) + zio_wait(zio[l]); break; } else if (label_txg <= txg && label_txg > best_txg) { best_txg = label_txg; @@ -831,7 +836,9 @@ retry: (u_longlong_t)txg); } - abd_free(vp_abd); + for (int l = 0; l < VDEV_LABELS; l++) { + abd_free(vp_abd[l]); + } return (config); } |