]> git.proxmox.com Git - mirror_zfs.git/blobdiff - module/zfs/vdev_disk.c
OpenZFS 7614, 9064 - zfs device evacuation/removal
[mirror_zfs.git] / module / zfs / vdev_disk.c
index e7e2b3b93f407a0eb2cb2e766081c91dcf1a21e1..056381c9d83069521e58e9529e1553010be9f47a 100644 (file)
  * Produced at Lawrence Livermore National Laboratory (cf, DISCLAIMER).
  * Rewritten for Linux by Brian Behlendorf <behlendorf1@llnl.gov>.
  * LLNL-CODE-403049.
- * Copyright (c) 2012, 2014 by Delphix. All rights reserved.
+ * Copyright (c) 2012, 2015 by Delphix. All rights reserved.
  */
 
 #include <sys/zfs_context.h>
-#include <sys/spa.h>
+#include <sys/spa_impl.h>
 #include <sys/vdev_disk.h>
 #include <sys/vdev_impl.h>
+#include <sys/abd.h>
 #include <sys/fs/zfs.h>
 #include <sys/zio.h>
 #include <sys/sunldi.h>
+#include <linux/mod_compat.h>
 
 char *zfs_vdev_scheduler = VDEV_SCHEDULER;
 static void *zfs_vdev_holder = VDEV_HOLDER;
@@ -41,10 +43,8 @@ static void *zfs_vdev_holder = VDEV_HOLDER;
  * Virtual device vector for disks.
  */
 typedef struct dio_request {
-       struct completion       dr_comp;        /* Completion for sync IO */
-       atomic_t                dr_ref;         /* References */
        zio_t                   *dr_zio;        /* Parent ZIO */
-       int                     dr_rw;          /* Read/Write */
+       atomic_t                dr_ref;         /* References */
        int                     dr_error;       /* Bio error */
        int                     dr_bio_count;   /* Count of bio's */
        struct bio              *dr_bio[0];     /* Attached bio's */
@@ -99,10 +99,10 @@ static void
 vdev_disk_error(zio_t *zio)
 {
 #ifdef ZFS_DEBUG
-       printk("ZFS: zio error=%d type=%d offset=%llu size=%llu "
-           "flags=%x delay=%llu\n", zio->io_error, zio->io_type,
+       printk(KERN_WARNING "ZFS: zio error=%d type=%d offset=%llu size=%llu "
+           "flags=%x\n", zio->io_error, zio->io_type,
            (u_longlong_t)zio->io_offset, (u_longlong_t)zio->io_size,
-           zio->io_flags, (u_longlong_t)zio->io_delay);
+           zio->io_flags);
 #endif
 }
 
@@ -114,15 +114,23 @@ vdev_disk_error(zio_t *zio)
  * physical device.  This yields the largest possible requests for
  * the device with the lowest total overhead.
  */
-static int
+static void
 vdev_elevator_switch(vdev_t *v, char *elevator)
 {
        vdev_disk_t *vd = v->vdev_tsd;
-       struct block_device *bdev = vd->vd_bdev;
-       struct request_queue *q = bdev_get_queue(bdev);
-       char *device = bdev->bd_disk->disk_name;
+       struct request_queue *q;
+       char *device;
        int error;
 
+       for (int c = 0; c < v->vdev_children; c++)
+               vdev_elevator_switch(v->vdev_child[c], elevator);
+
+       if (!v->vdev_ops->vdev_op_leaf || vd->vd_bdev == NULL)
+               return;
+
+       q = bdev_get_queue(vd->vd_bdev);
+       device = vd->vd_bdev->bd_disk->disk_name;
+
        /*
         * Skip devices which are not whole disks (partitions).
         * Device-mapper devices are excepted since they may be whole
@@ -132,15 +140,15 @@ vdev_elevator_switch(vdev_t *v, char *elevator)
         * "Skip devices without schedulers" check below will fail.
         */
        if (!v->vdev_wholedisk && strncmp(device, "dm-", 3) != 0)
-               return (0);
+               return;
 
        /* Skip devices without schedulers (loop, ram, dm, etc) */
        if (!q->elevator || !blk_queue_stackable(q))
-               return (0);
+               return;
 
        /* Leave existing scheduler when set to "none" */
-       if (strncmp(elevator, "none", 4) && (strlen(elevator) == 4) == 0)
-               return (0);
+       if ((strncmp(elevator, "none", 4) == 0) && (strlen(elevator) == 4))
+               return;
 
 #ifdef HAVE_ELEVATOR_CHANGE
        error = elevator_change(q, elevator);
@@ -157,20 +165,17 @@ vdev_elevator_switch(vdev_t *v, char *elevator)
        "     2>/dev/null; " \
        "echo %s"
 
-       {
-               char *argv[] = { "/bin/sh", "-c", NULL, NULL };
-               char *envp[] = { NULL };
+       char *argv[] = { "/bin/sh", "-c", NULL, NULL };
+       char *envp[] = { NULL };
 
-               argv[2] = kmem_asprintf(SET_SCHEDULER_CMD, device, elevator);
-               error = call_usermodehelper(argv[0], argv, envp, UMH_WAIT_PROC);
-               strfree(argv[2]);
-       }
+       argv[2] = kmem_asprintf(SET_SCHEDULER_CMD, device, elevator);
+       error = call_usermodehelper(argv[0], argv, envp, UMH_WAIT_PROC);
+       strfree(argv[2]);
 #endif /* HAVE_ELEVATOR_CHANGE */
        if (error)
-               printk("ZFS: Unable to set \"%s\" scheduler for %s (%s): %d\n",
-                   elevator, v->vdev_path, device, error);
-
-       return (error);
+               printk(KERN_NOTICE "ZFS: Unable to set \"%s\" scheduler"
+                   " for %s (%s): %d\n", elevator, v->vdev_path, device,
+                   error);
 }
 
 /*
@@ -244,12 +249,12 @@ vdev_disk_open(vdev_t *v, uint64_t *psize, uint64_t *max_psize,
 {
        struct block_device *bdev = ERR_PTR(-ENXIO);
        vdev_disk_t *vd;
-       int mode, block_size;
+       int count = 0, mode, block_size;
 
        /* Must have a pathname and it must be absolute. */
        if (v->vdev_path == NULL || v->vdev_path[0] != '/') {
                v->vdev_stat.vs_aux = VDEV_AUX_BAD_LABEL;
-               return (EINVAL);
+               return (SET_ERROR(EINVAL));
        }
 
        /*
@@ -264,7 +269,7 @@ vdev_disk_open(vdev_t *v, uint64_t *psize, uint64_t *max_psize,
 
        vd = kmem_zalloc(sizeof (vdev_disk_t), KM_SLEEP);
        if (vd == NULL)
-               return (ENOMEM);
+               return (SET_ERROR(ENOMEM));
 
        /*
         * Devices are always opened by the path provided at configuration
@@ -279,16 +284,35 @@ vdev_disk_open(vdev_t *v, uint64_t *psize, uint64_t *max_psize,
         * /dev/[hd]d devices which may be reordered due to probing order.
         * Devices in the wrong locations will be detected by the higher
         * level vdev validation.
+        *
+        * The specified paths may be briefly removed and recreated in
+        * response to udev events.  This should be exceptionally unlikely
+        * because the zpool command makes every effort to verify these paths
+        * have already settled prior to reaching this point.  Therefore,
+        * a ENOENT failure at this point is highly likely to be transient
+        * and it is reasonable to sleep and retry before giving up.  In
+        * practice delays have been observed to be on the order of 100ms.
         */
        mode = spa_mode(v->vdev_spa);
        if (v->vdev_wholedisk && v->vdev_expanding)
                bdev = vdev_disk_rrpart(v->vdev_path, mode, vd);
-       if (IS_ERR(bdev))
+
+       while (IS_ERR(bdev) && count < 50) {
                bdev = vdev_bdev_open(v->vdev_path,
                    vdev_bdev_mode(mode), zfs_vdev_holder);
+               if (unlikely(PTR_ERR(bdev) == -ENOENT)) {
+                       msleep(10);
+                       count++;
+               } else if (IS_ERR(bdev)) {
+                       break;
+               }
+       }
+
        if (IS_ERR(bdev)) {
+               dprintf("failed open v->vdev_path=%s, error=%d count=%d\n",
+                   v->vdev_path, -PTR_ERR(bdev), count);
                kmem_free(vd, sizeof (vdev_disk_t));
-               return (-PTR_ERR(bdev));
+               return (SET_ERROR(-PTR_ERR(bdev)));
        }
 
        v->vdev_tsd = vd;
@@ -344,7 +368,6 @@ vdev_disk_dio_alloc(int bio_count)
        dr = kmem_zalloc(sizeof (dio_request_t) +
            sizeof (struct bio *) * bio_count, KM_SLEEP);
        if (dr) {
-               init_completion(&dr->dr_comp);
                atomic_set(&dr->dr_ref, 0);
                dr->dr_bio_count = bio_count;
                dr->dr_error = 0;
@@ -369,27 +392,6 @@ vdev_disk_dio_free(dio_request_t *dr)
            sizeof (struct bio *) * dr->dr_bio_count);
 }
 
-static int
-vdev_disk_dio_is_sync(dio_request_t *dr)
-{
-#ifdef HAVE_BIO_RW_SYNC
-       /* BIO_RW_SYNC preferred interface from 2.6.12-2.6.29 */
-       return (dr->dr_rw & (1 << BIO_RW_SYNC));
-#else
-#ifdef HAVE_BIO_RW_SYNCIO
-       /* BIO_RW_SYNCIO preferred interface from 2.6.30-2.6.35 */
-       return (dr->dr_rw & (1 << BIO_RW_SYNCIO));
-#else
-#ifdef HAVE_REQ_SYNC
-       /* REQ_SYNC preferred interface from 2.6.36-2.6.xx */
-       return (dr->dr_rw & REQ_SYNC);
-#else
-#error "Unable to determine bio sync flag"
-#endif /* HAVE_REQ_SYNC */
-#endif /* HAVE_BIO_RW_SYNC */
-#endif /* HAVE_BIO_RW_SYNCIO */
-}
-
 static void
 vdev_disk_dio_get(dio_request_t *dr)
 {
@@ -412,49 +414,36 @@ vdev_disk_dio_put(dio_request_t *dr)
                vdev_disk_dio_free(dr);
 
                if (zio) {
-                       zio->io_delay = jiffies_64 - zio->io_delay;
                        zio->io_error = error;
                        ASSERT3S(zio->io_error, >=, 0);
                        if (zio->io_error)
                                vdev_disk_error(zio);
-                       zio_interrupt(zio);
+
+                       zio_delay_interrupt(zio);
                }
        }
 
        return (rc);
 }
 
-BIO_END_IO_PROTO(vdev_disk_physio_completion, bio, size, error)
+BIO_END_IO_PROTO(vdev_disk_physio_completion, bio, error)
 {
        dio_request_t *dr = bio->bi_private;
        int rc;
 
-#ifndef HAVE_2ARGS_BIO_END_IO_T
-       if (BIO_BI_SIZE(bio))
-               return (1);
-#endif /* HAVE_2ARGS_BIO_END_IO_T */
-
-       if (error == 0 && !test_bit(BIO_UPTODATE, &bio->bi_flags))
-               error = (-EIO);
-
-       if (dr->dr_error == 0)
-               dr->dr_error = -error;
+       if (dr->dr_error == 0) {
+#ifdef HAVE_1ARG_BIO_END_IO_T
+               dr->dr_error = BIO_END_IO_ERROR(bio);
+#else
+               if (error)
+                       dr->dr_error = -(error);
+               else if (!test_bit(BIO_UPTODATE, &bio->bi_flags))
+                       dr->dr_error = EIO;
+#endif
+       }
 
-       /* Drop reference aquired by __vdev_disk_physio */
+       /* Drop reference acquired by __vdev_disk_physio */
        rc = vdev_disk_dio_put(dr);
-
-       /* Wake up synchronous waiter this is the last outstanding bio */
-       if ((rc == 1) && vdev_disk_dio_is_sync(dr))
-               complete(&dr->dr_comp);
-
-       BIO_END_IO_RETURN(0);
-}
-
-static inline unsigned long
-bio_nr_pages(void *bio_ptr, unsigned int bio_size)
-{
-       return ((((unsigned long)bio_ptr + bio_size + PAGE_SIZE - 1) >>
-           PAGE_SHIFT) - ((unsigned long)bio_ptr >> PAGE_SHIFT));
 }
 
 static unsigned int
@@ -496,44 +485,74 @@ bio_map(struct bio *bio, void *bio_ptr, unsigned int bio_size)
        return (bio_size);
 }
 
+static unsigned int
+bio_map_abd_off(struct bio *bio, abd_t *abd, unsigned int size, size_t off)
+{
+       if (abd_is_linear(abd))
+               return (bio_map(bio, ((char *)abd_to_buf(abd)) + off, size));
+
+       return (abd_scatter_bio_map_off(bio, abd, size, off));
+}
+
 static inline void
-vdev_submit_bio(int rw, struct bio *bio)
+vdev_submit_bio_impl(struct bio *bio)
+{
+#ifdef HAVE_1ARG_SUBMIT_BIO
+       submit_bio(bio);
+#else
+       submit_bio(0, bio);
+#endif
+}
+
+#ifndef HAVE_BIO_SET_DEV
+static inline void
+bio_set_dev(struct bio *bio, struct block_device *bdev)
+{
+       bio->bi_bdev = bdev;
+}
+#endif /* !HAVE_BIO_SET_DEV */
+
+static inline void
+vdev_submit_bio(struct bio *bio)
 {
 #ifdef HAVE_CURRENT_BIO_TAIL
        struct bio **bio_tail = current->bio_tail;
        current->bio_tail = NULL;
-       submit_bio(rw, bio);
+       vdev_submit_bio_impl(bio);
        current->bio_tail = bio_tail;
 #else
        struct bio_list *bio_list = current->bio_list;
        current->bio_list = NULL;
-       submit_bio(rw, bio);
+       vdev_submit_bio_impl(bio);
        current->bio_list = bio_list;
 #endif
 }
 
 static int
-__vdev_disk_physio(struct block_device *bdev, zio_t *zio, caddr_t kbuf_ptr,
-    size_t kbuf_size, uint64_t kbuf_offset, int flags)
+__vdev_disk_physio(struct block_device *bdev, zio_t *zio,
+    size_t io_size, uint64_t io_offset, int rw, int flags)
 {
        dio_request_t *dr;
-       caddr_t bio_ptr;
+       uint64_t abd_offset;
        uint64_t bio_offset;
        int bio_size, bio_count = 16;
        int i = 0, error = 0;
+#if defined(HAVE_BLK_QUEUE_HAVE_BLK_PLUG)
+       struct blk_plug plug;
+#endif
 
-       ASSERT3U(kbuf_offset + kbuf_size, <=, bdev->bd_inode->i_size);
+       ASSERT(zio != NULL);
+       ASSERT3U(io_offset + io_size, <=, bdev->bd_inode->i_size);
 
 retry:
        dr = vdev_disk_dio_alloc(bio_count);
        if (dr == NULL)
-               return (ENOMEM);
+               return (SET_ERROR(ENOMEM));
 
        if (zio && !(zio->io_flags & (ZIO_FLAG_IO_RETRY | ZIO_FLAG_TRYHARD)))
                bio_set_flags_failfast(bdev, &flags);
 
        dr->dr_zio = zio;
-       dr->dr_rw = flags;
 
        /*
         * When the IO size exceeds the maximum bio size for the request
@@ -542,9 +561,10 @@ retry:
         * their volume block size to match the maximum request size and
         * the common case will be one bio per vdev IO request.
         */
-       bio_ptr    = kbuf_ptr;
-       bio_offset = kbuf_offset;
-       bio_size   = kbuf_size;
+
+       abd_offset = 0;
+       bio_offset = io_offset;
+       bio_size   = io_size;
        for (i = 0; i <= dr->dr_bio_count; i++) {
 
                /* Finished constructing bio's for given buffer */
@@ -564,73 +584,64 @@ retry:
 
                /* bio_alloc() with __GFP_WAIT never returns NULL */
                dr->dr_bio[i] = bio_alloc(GFP_NOIO,
-                   MIN(bio_nr_pages(bio_ptr, bio_size), BIO_MAX_PAGES));
+                   MIN(abd_nr_pages_off(zio->io_abd, bio_size, abd_offset),
+                   BIO_MAX_PAGES));
                if (unlikely(dr->dr_bio[i] == NULL)) {
                        vdev_disk_dio_free(dr);
-                       return (ENOMEM);
+                       return (SET_ERROR(ENOMEM));
                }
 
                /* Matching put called by vdev_disk_physio_completion */
                vdev_disk_dio_get(dr);
 
-               dr->dr_bio[i]->bi_bdev = bdev;
+               bio_set_dev(dr->dr_bio[i], bdev);
                BIO_BI_SECTOR(dr->dr_bio[i]) = bio_offset >> 9;
-               dr->dr_bio[i]->bi_rw = dr->dr_rw;
                dr->dr_bio[i]->bi_end_io = vdev_disk_physio_completion;
                dr->dr_bio[i]->bi_private = dr;
+               bio_set_op_attrs(dr->dr_bio[i], rw, flags);
 
                /* Remaining size is returned to become the new size */
-               bio_size = bio_map(dr->dr_bio[i], bio_ptr, bio_size);
+               bio_size = bio_map_abd_off(dr->dr_bio[i], zio->io_abd,
+                   bio_size, abd_offset);
 
                /* Advance in buffer and construct another bio if needed */
-               bio_ptr    += BIO_BI_SIZE(dr->dr_bio[i]);
+               abd_offset += BIO_BI_SIZE(dr->dr_bio[i]);
                bio_offset += BIO_BI_SIZE(dr->dr_bio[i]);
        }
 
        /* Extra reference to protect dio_request during vdev_submit_bio */
        vdev_disk_dio_get(dr);
-       if (zio)
-               zio->io_delay = jiffies_64;
+
+#if defined(HAVE_BLK_QUEUE_HAVE_BLK_PLUG)
+       if (dr->dr_bio_count > 1)
+               blk_start_plug(&plug);
+#endif
 
        /* Submit all bio's associated with this dio */
        for (i = 0; i < dr->dr_bio_count; i++)
                if (dr->dr_bio[i])
-                       vdev_submit_bio(dr->dr_rw, dr->dr_bio[i]);
+                       vdev_submit_bio(dr->dr_bio[i]);
 
-       /*
-        * On synchronous blocking requests we wait for all bio the completion
-        * callbacks to run.  We will be woken when the last callback runs
-        * for this dio.  We are responsible for putting the last dio_request
-        * reference will in turn put back the last bio references.  The
-        * only synchronous consumer is vdev_disk_read_rootlabel() all other
-        * IO originating from vdev_disk_io_start() is asynchronous.
-        */
-       if (vdev_disk_dio_is_sync(dr)) {
-               wait_for_completion(&dr->dr_comp);
-               error = dr->dr_error;
-               ASSERT3S(atomic_read(&dr->dr_ref), ==, 1);
-       }
+#if defined(HAVE_BLK_QUEUE_HAVE_BLK_PLUG)
+       if (dr->dr_bio_count > 1)
+               blk_finish_plug(&plug);
+#endif
 
        (void) vdev_disk_dio_put(dr);
 
        return (error);
 }
 
-int
-vdev_disk_physio(struct block_device *bdev, caddr_t kbuf,
-    size_t size, uint64_t offset, int flags)
-{
-       bio_set_flags_failfast(bdev, &flags);
-       return (__vdev_disk_physio(bdev, NULL, kbuf, size, offset, flags));
-}
-
-BIO_END_IO_PROTO(vdev_disk_io_flush_completion, bio, size, rc)
+BIO_END_IO_PROTO(vdev_disk_io_flush_completion, bio, error)
 {
        zio_t *zio = bio->bi_private;
+#ifdef HAVE_1ARG_BIO_END_IO_T
+       zio->io_error = BIO_END_IO_ERROR(bio);
+#else
+       zio->io_error = -error;
+#endif
 
-       zio->io_delay = jiffies_64 - zio->io_delay;
-       zio->io_error = -rc;
-       if (rc && (rc == -EOPNOTSUPP))
+       if (zio->io_error && (zio->io_error == EOPNOTSUPP))
                zio->io_vd->vdev_nowritecache = B_TRUE;
 
        bio_put(bio);
@@ -638,8 +649,6 @@ BIO_END_IO_PROTO(vdev_disk_io_flush_completion, bio, size, rc)
        if (zio->io_error)
                vdev_disk_error(zio);
        zio_interrupt(zio);
-
-       BIO_END_IO_RETURN(0);
 }
 
 static int
@@ -650,18 +659,18 @@ vdev_disk_io_flush(struct block_device *bdev, zio_t *zio)
 
        q = bdev_get_queue(bdev);
        if (!q)
-               return (ENXIO);
+               return (SET_ERROR(ENXIO));
 
        bio = bio_alloc(GFP_NOIO, 0);
        /* bio_alloc() with __GFP_WAIT never returns NULL */
        if (unlikely(bio == NULL))
-               return (ENOMEM);
+               return (SET_ERROR(ENOMEM));
 
        bio->bi_end_io = vdev_disk_io_flush_completion;
        bio->bi_private = zio;
-       bio->bi_bdev = bdev;
-       zio->io_delay = jiffies_64;
-       vdev_submit_bio(VDEV_WRITE_FLUSH_FUA, bio);
+       bio_set_dev(bio, bdev);
+       bio_set_flush(bio);
+       vdev_submit_bio(bio);
        invalidate_bdev(bdev);
 
        return (0);
@@ -672,7 +681,7 @@ vdev_disk_io_start(zio_t *zio)
 {
        vdev_t *v = zio->io_vd;
        vdev_disk_t *vd = v->vdev_tsd;
-       int flags, error;
+       int rw, flags, error;
 
        switch (zio->io_type) {
        case ZIO_TYPE_IOCTL:
@@ -699,8 +708,6 @@ vdev_disk_io_start(zio_t *zio)
                                return;
 
                        zio->io_error = error;
-                       if (error == ENOTSUP)
-                               v->vdev_nowritecache = B_TRUE;
 
                        break;
 
@@ -711,17 +718,25 @@ vdev_disk_io_start(zio_t *zio)
                zio_execute(zio);
                return;
        case ZIO_TYPE_WRITE:
-               if (zio->io_priority == ZIO_PRIORITY_SYNC_WRITE)
-                       flags = WRITE_SYNC;
-               else
-                       flags = WRITE;
+               rw = WRITE;
+#if defined(HAVE_BLK_QUEUE_HAVE_BIO_RW_UNPLUG)
+               flags = (1 << BIO_RW_UNPLUG);
+#elif defined(REQ_UNPLUG)
+               flags = REQ_UNPLUG;
+#else
+               flags = 0;
+#endif
                break;
 
        case ZIO_TYPE_READ:
-               if (zio->io_priority == ZIO_PRIORITY_SYNC_READ)
-                       flags = READ_SYNC;
-               else
-                       flags = READ;
+               rw = READ;
+#if defined(HAVE_BLK_QUEUE_HAVE_BIO_RW_UNPLUG)
+               flags = (1 << BIO_RW_UNPLUG);
+#elif defined(REQ_UNPLUG)
+               flags = REQ_UNPLUG;
+#else
+               flags = 0;
+#endif
                break;
 
        default:
@@ -730,8 +745,9 @@ vdev_disk_io_start(zio_t *zio)
                return;
        }
 
-       error = __vdev_disk_physio(vd->vd_bdev, zio, zio->io_data,
-           zio->io_size, zio->io_offset, flags);
+       zio->io_target_timestamp = zio_handle_io_delay(zio);
+       error = __vdev_disk_physio(vd->vd_bdev, zio,
+           zio->io_size, zio->io_offset, rw, flags);
        if (error) {
                zio->io_error = error;
                zio_interrupt(zio);
@@ -788,6 +804,35 @@ vdev_disk_rele(vdev_t *vd)
        /* XXX: Implement me as a vnode rele for the device */
 }
 
+static int
+param_set_vdev_scheduler(const char *val, zfs_kernel_param_t *kp)
+{
+       spa_t *spa = NULL;
+       char *p;
+
+       if (val == NULL)
+               return (SET_ERROR(-EINVAL));
+
+       if ((p = strchr(val, '\n')) != NULL)
+               *p = '\0';
+
+       mutex_enter(&spa_namespace_lock);
+       while ((spa = spa_next(spa)) != NULL) {
+               if (spa_state(spa) != POOL_STATE_ACTIVE ||
+                   !spa_writeable(spa) || spa_suspended(spa))
+                       continue;
+
+               spa_open_ref(spa, FTAG);
+               mutex_exit(&spa_namespace_lock);
+               vdev_elevator_switch(spa->spa_root_vdev, (char *)val);
+               mutex_enter(&spa_namespace_lock);
+               spa_close(spa, FTAG);
+       }
+       mutex_exit(&spa_namespace_lock);
+
+       return (param_set_charp(val, kp));
+}
+
 vdev_ops_t vdev_disk_ops = {
        vdev_disk_open,
        vdev_disk_close,
@@ -795,74 +840,14 @@ vdev_ops_t vdev_disk_ops = {
        vdev_disk_io_start,
        vdev_disk_io_done,
        NULL,
+       NULL,
        vdev_disk_hold,
        vdev_disk_rele,
+       NULL,
        VDEV_TYPE_DISK,         /* name of this vdev type */
        B_TRUE                  /* leaf vdev */
 };
 
-/*
- * Given the root disk device devid or pathname, read the label from
- * the device, and construct a configuration nvlist.
- */
-int
-vdev_disk_read_rootlabel(char *devpath, char *devid, nvlist_t **config)
-{
-       struct block_device *bdev;
-       vdev_label_t *label;
-       uint64_t s, size;
-       int i;
-
-       bdev = vdev_bdev_open(devpath, vdev_bdev_mode(FREAD), zfs_vdev_holder);
-       if (IS_ERR(bdev))
-               return (-PTR_ERR(bdev));
-
-       s = bdev_capacity(bdev);
-       if (s == 0) {
-               vdev_bdev_close(bdev, vdev_bdev_mode(FREAD));
-               return (EIO);
-       }
-
-       size = P2ALIGN_TYPED(s, sizeof (vdev_label_t), uint64_t);
-       label = vmem_alloc(sizeof (vdev_label_t), KM_SLEEP);
-
-       for (i = 0; i < VDEV_LABELS; i++) {
-               uint64_t offset, state, txg = 0;
-
-               /* read vdev label */
-               offset = vdev_label_offset(size, i, 0);
-               if (vdev_disk_physio(bdev, (caddr_t)label,
-                   VDEV_SKIP_SIZE + VDEV_PHYS_SIZE, offset, READ_SYNC) != 0)
-                       continue;
-
-               if (nvlist_unpack(label->vl_vdev_phys.vp_nvlist,
-                   sizeof (label->vl_vdev_phys.vp_nvlist), config, 0) != 0) {
-                       *config = NULL;
-                       continue;
-               }
-
-               if (nvlist_lookup_uint64(*config, ZPOOL_CONFIG_POOL_STATE,
-                   &state) != 0 || state >= POOL_STATE_DESTROYED) {
-                       nvlist_free(*config);
-                       *config = NULL;
-                       continue;
-               }
-
-               if (nvlist_lookup_uint64(*config, ZPOOL_CONFIG_POOL_TXG,
-                   &txg) != 0 || txg == 0) {
-                       nvlist_free(*config);
-                       *config = NULL;
-                       continue;
-               }
-
-               break;
-       }
-
-       vmem_free(label, sizeof (vdev_label_t));
-       vdev_bdev_close(bdev, vdev_bdev_mode(FREAD));
-
-       return (0);
-}
-
-module_param(zfs_vdev_scheduler, charp, 0644);
+module_param_call(zfs_vdev_scheduler, param_set_vdev_scheduler,
+    param_get_charp, &zfs_vdev_scheduler, 0644);
 MODULE_PARM_DESC(zfs_vdev_scheduler, "I/O scheduler");