Fix for access beyond end of device error
[zfs.git] / module / zfs / vdev_disk.c
1 /*
2  * CDDL HEADER START
3  *
4  * The contents of this file are subject to the terms of the
5  * Common Development and Distribution License (the "License").
6  * You may not use this file except in compliance with the License.
7  *
8  * You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
9  * or http://www.opensolaris.org/os/licensing.
10  * See the License for the specific language governing permissions
11  * and limitations under the License.
12  *
13  * When distributing Covered Code, include this CDDL HEADER in each
14  * file and include the License file at usr/src/OPENSOLARIS.LICENSE.
15  * If applicable, add the following below this CDDL HEADER, with the
16  * fields enclosed by brackets "[]" replaced with your own identifying
17  * information: Portions Copyright [yyyy] [name of copyright owner]
18  *
19  * CDDL HEADER END
20  */
21 /*
22  * Copyright (C) 2008-2010 Lawrence Livermore National Security, LLC.
23  * Produced at Lawrence Livermore National Laboratory (cf, DISCLAIMER).
24  * Rewritten for Linux by Brian Behlendorf <behlendorf1@llnl.gov>.
25  * LLNL-CODE-403049.
26  */
27
28 #include <sys/zfs_context.h>
29 #include <sys/spa.h>
30 #include <sys/vdev_disk.h>
31 #include <sys/vdev_impl.h>
32 #include <sys/fs/zfs.h>
33 #include <sys/zio.h>
34 #include <sys/sunldi.h>
35
36 /*
37  * Virtual device vector for disks.
38  */
39 typedef struct dio_request {
40         struct completion       dr_comp;        /* Completion for sync IO */
41         atomic_t                dr_ref;         /* References */
42         zio_t                   *dr_zio;        /* Parent ZIO */
43         int                     dr_rw;          /* Read/Write */
44         int                     dr_error;       /* Bio error */
45         int                     dr_bio_count;   /* Count of bio's */
46         struct bio              *dr_bio[0];     /* Attached bio's */
47 } dio_request_t;
48
49
50 #ifdef HAVE_OPEN_BDEV_EXCLUSIVE
51 static fmode_t
52 vdev_bdev_mode(int smode)
53 {
54         fmode_t mode = 0;
55
56         ASSERT3S(smode & (FREAD | FWRITE), !=, 0);
57
58         if (smode & FREAD)
59                 mode |= FMODE_READ;
60
61         if (smode & FWRITE)
62                 mode |= FMODE_WRITE;
63
64         return mode;
65 }
66 #else
67 static int
68 vdev_bdev_mode(int smode)
69 {
70         int mode = 0;
71
72         ASSERT3S(smode & (FREAD | FWRITE), !=, 0);
73
74         if ((smode & FREAD) && !(smode & FWRITE))
75                 mode = MS_RDONLY;
76
77         return mode;
78 }
79 #endif /* HAVE_OPEN_BDEV_EXCLUSIVE */
80
81 static uint64_t
82 bdev_capacity(struct block_device *bdev)
83 {
84         struct hd_struct *part = bdev->bd_part;
85
86         /* The partition capacity referenced by the block device */
87         if (part)
88                return part->nr_sects;
89
90         /* Otherwise assume the full device capacity */
91         return get_capacity(bdev->bd_disk);
92 }
93
94 static void
95 vdev_disk_error(zio_t *zio)
96 {
97 #ifdef ZFS_DEBUG
98         printk("ZFS: zio error=%d type=%d offset=%llu size=%llu "
99             "flags=%x delay=%llu\n", zio->io_error, zio->io_type,
100             (u_longlong_t)zio->io_offset, (u_longlong_t)zio->io_size,
101             zio->io_flags, (u_longlong_t)zio->io_delay);
102 #endif
103 }
104
105 static int
106 vdev_disk_open(vdev_t *v, uint64_t *psize, uint64_t *ashift)
107 {
108         struct block_device *bdev;
109         vdev_disk_t *vd;
110         int mode, block_size;
111
112         /* Must have a pathname and it must be absolute. */
113         if (v->vdev_path == NULL || v->vdev_path[0] != '/') {
114                 v->vdev_stat.vs_aux = VDEV_AUX_BAD_LABEL;
115                 return EINVAL;
116         }
117
118         vd = kmem_zalloc(sizeof(vdev_disk_t), KM_SLEEP);
119         if (vd == NULL)
120                 return ENOMEM;
121
122         /*
123          * Devices are always opened by the path provided at configuration
124          * time.  This means that if the provided path is a udev by-id path
125          * then drives may be recabled without an issue.  If the provided
126          * path is a udev by-path path then the physical location information
127          * will be preserved.  This can be critical for more complicated
128          * configurations where drives are located in specific physical
129          * locations to maximize the systems tolerence to component failure.
130          * Alternately you can provide your own udev rule to flexibly map
131          * the drives as you see fit.  It is not advised that you use the
132          * /dev/[hd]d devices which may be reorder due to probing order.
133          * Devices in the wrong locations will be detected by the higher
134          * level vdev validation.
135          */
136         mode = spa_mode(v->vdev_spa);
137         bdev = vdev_bdev_open(v->vdev_path, vdev_bdev_mode(mode), vd);
138         if (IS_ERR(bdev)) {
139                 kmem_free(vd, sizeof(vdev_disk_t));
140                 return -PTR_ERR(bdev);
141         }
142
143         v->vdev_tsd = vd;
144         vd->vd_bdev = bdev;
145         block_size =  vdev_bdev_block_size(bdev);
146
147         /* We think the wholedisk property should always be set when this
148          * function is called.  ASSERT here so if any legitimate cases exist
149          * where it's not set, we'll find them during debugging.  If we never
150          * hit the ASSERT, this and the following conditional statement can be
151          * removed. */
152         ASSERT3S(v->vdev_wholedisk, !=, -1ULL);
153
154         /* The wholedisk property was initialized to -1 in vdev_alloc() if it
155          * was unspecified.  In that case, check if this is a whole device.
156          * When bdev->bd_contains == bdev we have a whole device and not simply
157          * a partition. */
158         if (v->vdev_wholedisk == -1ULL)
159                 v->vdev_wholedisk = (bdev->bd_contains == bdev);
160
161         /* Clear the nowritecache bit, causes vdev_reopen() to try again. */
162         v->vdev_nowritecache = B_FALSE;
163
164         /* Physical volume size in bytes */
165         *psize = bdev_capacity(bdev) * block_size;
166
167         /* Based on the minimum sector size set the block size */
168         *ashift = highbit(MAX(block_size, SPA_MINBLOCKSIZE)) - 1;
169
170         return 0;
171 }
172
173 static void
174 vdev_disk_close(vdev_t *v)
175 {
176         vdev_disk_t *vd = v->vdev_tsd;
177
178         if (vd == NULL)
179                 return;
180
181         if (vd->vd_bdev != NULL)
182                 vdev_bdev_close(vd->vd_bdev,
183                                 vdev_bdev_mode(spa_mode(v->vdev_spa)));
184
185         kmem_free(vd, sizeof(vdev_disk_t));
186         v->vdev_tsd = NULL;
187 }
188
189 static dio_request_t *
190 vdev_disk_dio_alloc(int bio_count)
191 {
192         dio_request_t *dr;
193         int i;
194
195         dr = kmem_zalloc(sizeof(dio_request_t) +
196                          sizeof(struct bio *) * bio_count, KM_SLEEP);
197         if (dr) {
198                 init_completion(&dr->dr_comp);
199                 atomic_set(&dr->dr_ref, 0);
200                 dr->dr_bio_count = bio_count;
201                 dr->dr_error = 0;
202
203                 for (i = 0; i < dr->dr_bio_count; i++)
204                         dr->dr_bio[i] = NULL;
205         }
206
207         return dr;
208 }
209
210 static void
211 vdev_disk_dio_free(dio_request_t *dr)
212 {
213         int i;
214
215         for (i = 0; i < dr->dr_bio_count; i++)
216                 if (dr->dr_bio[i])
217                         bio_put(dr->dr_bio[i]);
218
219         kmem_free(dr, sizeof(dio_request_t) +
220                   sizeof(struct bio *) * dr->dr_bio_count);
221 }
222
223 static int
224 vdev_disk_dio_is_sync(dio_request_t *dr)
225 {
226 #ifdef HAVE_BIO_RW_SYNC
227         /* BIO_RW_SYNC preferred interface from 2.6.12-2.6.29 */
228         return (dr->dr_rw & (1 << BIO_RW_SYNC));
229 #else
230 # ifdef HAVE_BIO_RW_SYNCIO
231         /* BIO_RW_SYNCIO preferred interface from 2.6.30-2.6.35 */
232         return (dr->dr_rw & (1 << BIO_RW_SYNCIO));
233 # else
234 #  ifdef HAVE_REQ_SYNC
235         /* REQ_SYNC preferred interface from 2.6.36-2.6.xx */
236         return (dr->dr_rw & REQ_SYNC);
237 #  else
238 #   error "Unable to determine bio sync flag"
239 #  endif /* HAVE_REQ_SYNC */
240 # endif /* HAVE_BIO_RW_SYNC */
241 #endif /* HAVE_BIO_RW_SYNCIO */
242 }
243
244 static void
245 vdev_disk_dio_get(dio_request_t *dr)
246 {
247         atomic_inc(&dr->dr_ref);
248 }
249
250 static int
251 vdev_disk_dio_put(dio_request_t *dr)
252 {
253         int rc = atomic_dec_return(&dr->dr_ref);
254
255         /*
256          * Free the dio_request when the last reference is dropped and
257          * ensure zio_interpret is called only once with the correct zio
258          */
259         if (rc == 0) {
260                 zio_t *zio = dr->dr_zio;
261                 int error = dr->dr_error;
262
263                 vdev_disk_dio_free(dr);
264
265                 if (zio) {
266                         zio->io_delay = jiffies_to_msecs(
267                             jiffies_64 - zio->io_delay);
268                         zio->io_error = error;
269                         ASSERT3S(zio->io_error, >=, 0);
270                         if (zio->io_error)
271                                 vdev_disk_error(zio);
272                         zio_interrupt(zio);
273                 }
274         }
275
276         return rc;
277 }
278
279 BIO_END_IO_PROTO(vdev_disk_physio_completion, bio, size, error)
280 {
281         dio_request_t *dr = bio->bi_private;
282         int rc;
283
284         /* Fatal error but print some useful debugging before asserting */
285         if (dr == NULL)
286                 PANIC("dr == NULL, bio->bi_private == NULL\n"
287                     "bi_next: %p, bi_flags: %lx, bi_rw: %lu, bi_vcnt: %d\n"
288                     "bi_idx: %d, bi_size: %d, bi_end_io: %p, bi_cnt: %d\n",
289                     bio->bi_next, bio->bi_flags, bio->bi_rw, bio->bi_vcnt,
290                     bio->bi_idx, bio->bi_size, bio->bi_end_io,
291                     atomic_read(&bio->bi_cnt));
292
293 #ifndef HAVE_2ARGS_BIO_END_IO_T
294         if (bio->bi_size)
295                 return 1;
296 #endif /* HAVE_2ARGS_BIO_END_IO_T */
297
298         if (error == 0 && !test_bit(BIO_UPTODATE, &bio->bi_flags))
299                 error = -EIO;
300
301         if (dr->dr_error == 0)
302                 dr->dr_error = -error;
303
304         /* Drop reference aquired by __vdev_disk_physio */
305         rc = vdev_disk_dio_put(dr);
306
307         /* Wake up synchronous waiter this is the last outstanding bio */
308         if ((rc == 1) && vdev_disk_dio_is_sync(dr))
309                 complete(&dr->dr_comp);
310
311         BIO_END_IO_RETURN(0);
312 }
313
314 static inline unsigned long
315 bio_nr_pages(void *bio_ptr, unsigned int bio_size)
316 {
317         return ((((unsigned long)bio_ptr + bio_size + PAGE_SIZE - 1) >>
318                 PAGE_SHIFT) - ((unsigned long)bio_ptr >> PAGE_SHIFT));
319 }
320
321 static unsigned int
322 bio_map(struct bio *bio, void *bio_ptr, unsigned int bio_size)
323 {
324         unsigned int offset, size, i;
325         struct page *page;
326
327         offset = offset_in_page(bio_ptr);
328         for (i = 0; i < bio->bi_max_vecs; i++) {
329                 size = PAGE_SIZE - offset;
330
331                 if (bio_size <= 0)
332                         break;
333
334                 if (size > bio_size)
335                         size = bio_size;
336
337                 if (kmem_virt(bio_ptr))
338                         page = vmalloc_to_page(bio_ptr);
339                 else
340                         page = virt_to_page(bio_ptr);
341
342                 if (bio_add_page(bio, page, size, offset) != size)
343                         break;
344
345                 bio_ptr  += size;
346                 bio_size -= size;
347                 offset = 0;
348         }
349
350         return bio_size;
351 }
352
353 static int
354 __vdev_disk_physio(struct block_device *bdev, zio_t *zio, caddr_t kbuf_ptr,
355                    size_t kbuf_size, uint64_t kbuf_offset, int flags)
356 {
357         dio_request_t *dr;
358         caddr_t bio_ptr;
359         uint64_t bio_offset;
360         int bio_size, bio_count = 16;
361         int i = 0, error = 0, block_size;
362
363         ASSERT3U(kbuf_offset + kbuf_size, <=, bdev->bd_inode->i_size);
364
365 retry:
366         dr = vdev_disk_dio_alloc(bio_count);
367         if (dr == NULL)
368                 return ENOMEM;
369
370         if (zio && !(zio->io_flags & (ZIO_FLAG_IO_RETRY | ZIO_FLAG_TRYHARD)))
371                         bio_set_flags_failfast(bdev, &flags);
372
373         dr->dr_zio = zio;
374         dr->dr_rw = flags;
375         block_size = vdev_bdev_block_size(bdev);
376
377         /*
378          * When the IO size exceeds the maximum bio size for the request
379          * queue we are forced to break the IO in multiple bio's and wait
380          * for them all to complete.  Ideally, all pool users will set
381          * their volume block size to match the maximum request size and
382          * the common case will be one bio per vdev IO request.
383          */
384         bio_ptr    = kbuf_ptr;
385         bio_offset = kbuf_offset;
386         bio_size   = kbuf_size;
387         for (i = 0; i <= dr->dr_bio_count; i++) {
388
389                 /* Finished constructing bio's for given buffer */
390                 if (bio_size <= 0)
391                         break;
392
393                 /*
394                  * By default only 'bio_count' bio's per dio are allowed.
395                  * However, if we find ourselves in a situation where more
396                  * are needed we allocate a larger dio and warn the user.
397                  */
398                 if (dr->dr_bio_count == i) {
399                         vdev_disk_dio_free(dr);
400                         bio_count *= 2;
401                         printk("WARNING: Resized bio's/dio to %d\n",bio_count);
402                         goto retry;
403                 }
404
405                 dr->dr_bio[i] = bio_alloc(GFP_NOIO,
406                                           bio_nr_pages(bio_ptr, bio_size));
407                 if (dr->dr_bio[i] == NULL) {
408                         vdev_disk_dio_free(dr);
409                         return ENOMEM;
410                 }
411
412                 /* Matching put called by vdev_disk_physio_completion */
413                 vdev_disk_dio_get(dr);
414
415                 dr->dr_bio[i]->bi_bdev = bdev;
416                 dr->dr_bio[i]->bi_sector = bio_offset / block_size;
417                 dr->dr_bio[i]->bi_rw = dr->dr_rw;
418                 dr->dr_bio[i]->bi_end_io = vdev_disk_physio_completion;
419                 dr->dr_bio[i]->bi_private = dr;
420
421                 /* Remaining size is returned to become the new size */
422                 bio_size = bio_map(dr->dr_bio[i], bio_ptr, bio_size);
423
424                 /* Advance in buffer and construct another bio if needed */
425                 bio_ptr    += dr->dr_bio[i]->bi_size;
426                 bio_offset += dr->dr_bio[i]->bi_size;
427         }
428
429         /* Extra reference to protect dio_request during submit_bio */
430         vdev_disk_dio_get(dr);
431         if (zio)
432                 zio->io_delay = jiffies_64;
433
434         /* Submit all bio's associated with this dio */
435         for (i = 0; i < dr->dr_bio_count; i++)
436                 if (dr->dr_bio[i])
437                         submit_bio(dr->dr_rw, dr->dr_bio[i]);
438
439         /*
440          * On synchronous blocking requests we wait for all bio the completion
441          * callbacks to run.  We will be woken when the last callback runs
442          * for this dio.  We are responsible for putting the last dio_request
443          * reference will in turn put back the last bio references.  The
444          * only synchronous consumer is vdev_disk_read_rootlabel() all other
445          * IO originating from vdev_disk_io_start() is asynchronous.
446          */
447         if (vdev_disk_dio_is_sync(dr)) {
448                 wait_for_completion(&dr->dr_comp);
449                 error = dr->dr_error;
450                 ASSERT3S(atomic_read(&dr->dr_ref), ==, 1);
451         }
452
453         (void)vdev_disk_dio_put(dr);
454
455         return error;
456 }
457
458 int
459 vdev_disk_physio(struct block_device *bdev, caddr_t kbuf,
460                  size_t size, uint64_t offset, int flags)
461 {
462         bio_set_flags_failfast(bdev, &flags);
463         return __vdev_disk_physio(bdev, NULL, kbuf, size, offset, flags);
464 }
465
466 /* 2.6.24 API change */
467 #ifdef HAVE_BIO_EMPTY_BARRIER
468 BIO_END_IO_PROTO(vdev_disk_io_flush_completion, bio, size, rc)
469 {
470         zio_t *zio = bio->bi_private;
471
472         zio->io_delay = jiffies_to_msecs(jiffies_64 - zio->io_delay);
473         zio->io_error = -rc;
474         if (rc && (rc == -EOPNOTSUPP))
475                 zio->io_vd->vdev_nowritecache = B_TRUE;
476
477         bio_put(bio);
478         ASSERT3S(zio->io_error, >=, 0);
479         if (zio->io_error)
480                 vdev_disk_error(zio);
481         zio_interrupt(zio);
482
483         BIO_END_IO_RETURN(0);
484 }
485
486 static int
487 vdev_disk_io_flush(struct block_device *bdev, zio_t *zio)
488 {
489         struct request_queue *q;
490         struct bio *bio;
491
492         q = bdev_get_queue(bdev);
493         if (!q)
494                 return ENXIO;
495
496         bio = bio_alloc(GFP_KERNEL, 0);
497         if (!bio)
498                 return ENOMEM;
499
500         bio->bi_end_io = vdev_disk_io_flush_completion;
501         bio->bi_private = zio;
502         bio->bi_bdev = bdev;
503         zio->io_delay = jiffies_64;
504         submit_bio(WRITE_BARRIER, bio);
505
506         return 0;
507 }
508 #else
509 static int
510 vdev_disk_io_flush(struct block_device *bdev, zio_t *zio)
511 {
512         return ENOTSUP;
513 }
514 #endif /* HAVE_BIO_EMPTY_BARRIER */
515
516 static int
517 vdev_disk_io_start(zio_t *zio)
518 {
519         vdev_t *v = zio->io_vd;
520         vdev_disk_t *vd = v->vdev_tsd;
521         int flags, error;
522
523         switch (zio->io_type) {
524         case ZIO_TYPE_IOCTL:
525
526                 if (!vdev_readable(v)) {
527                         zio->io_error = ENXIO;
528                         return ZIO_PIPELINE_CONTINUE;
529                 }
530
531                 switch (zio->io_cmd) {
532                 case DKIOCFLUSHWRITECACHE:
533
534                         if (zfs_nocacheflush)
535                                 break;
536
537                         if (v->vdev_nowritecache) {
538                                 zio->io_error = ENOTSUP;
539                                 break;
540                         }
541
542                         error = vdev_disk_io_flush(vd->vd_bdev, zio);
543                         if (error == 0)
544                                 return ZIO_PIPELINE_STOP;
545
546                         zio->io_error = error;
547                         if (error == ENOTSUP)
548                                 v->vdev_nowritecache = B_TRUE;
549
550                         break;
551
552                 default:
553                         zio->io_error = ENOTSUP;
554                 }
555
556                 return ZIO_PIPELINE_CONTINUE;
557
558         case ZIO_TYPE_WRITE:
559                 flags = WRITE;
560                 break;
561
562         case ZIO_TYPE_READ:
563                 flags = READ;
564                 break;
565
566         default:
567                 zio->io_error = ENOTSUP;
568                 return ZIO_PIPELINE_CONTINUE;
569         }
570
571         error = __vdev_disk_physio(vd->vd_bdev, zio, zio->io_data,
572                                    zio->io_size, zio->io_offset, flags);
573         if (error) {
574                 zio->io_error = error;
575                 return ZIO_PIPELINE_CONTINUE;
576         }
577
578         return ZIO_PIPELINE_STOP;
579 }
580
581 static void
582 vdev_disk_io_done(zio_t *zio)
583 {
584         /*
585          * If the device returned EIO, we revalidate the media.  If it is
586          * determined the media has changed this triggers the asynchronous
587          * removal of the device from the configuration.
588          */
589         if (zio->io_error == EIO) {
590                 vdev_t *v = zio->io_vd;
591                 vdev_disk_t *vd = v->vdev_tsd;
592
593                 if (check_disk_change(vd->vd_bdev)) {
594                         vdev_bdev_invalidate(vd->vd_bdev);
595                         v->vdev_remove_wanted = B_TRUE;
596                         spa_async_request(zio->io_spa, SPA_ASYNC_REMOVE);
597                 }
598         }
599 }
600
601 static void
602 vdev_disk_hold(vdev_t *vd)
603 {
604         ASSERT(spa_config_held(vd->vdev_spa, SCL_STATE, RW_WRITER));
605
606         /* We must have a pathname, and it must be absolute. */
607         if (vd->vdev_path == NULL || vd->vdev_path[0] != '/')
608                 return;
609
610         /*
611          * Only prefetch path and devid info if the device has
612          * never been opened.
613          */
614         if (vd->vdev_tsd != NULL)
615                 return;
616
617         /* XXX: Implement me as a vnode lookup for the device */
618         vd->vdev_name_vp = NULL;
619         vd->vdev_devid_vp = NULL;
620 }
621
622 static void
623 vdev_disk_rele(vdev_t *vd)
624 {
625         ASSERT(spa_config_held(vd->vdev_spa, SCL_STATE, RW_WRITER));
626
627         /* XXX: Implement me as a vnode rele for the device */
628 }
629
630 vdev_ops_t vdev_disk_ops = {
631         vdev_disk_open,
632         vdev_disk_close,
633         vdev_default_asize,
634         vdev_disk_io_start,
635         vdev_disk_io_done,
636         NULL,
637         vdev_disk_hold,
638         vdev_disk_rele,
639         VDEV_TYPE_DISK,         /* name of this vdev type */
640         B_TRUE                  /* leaf vdev */
641 };
642
643 /*
644  * Given the root disk device devid or pathname, read the label from
645  * the device, and construct a configuration nvlist.
646  */
647 int
648 vdev_disk_read_rootlabel(char *devpath, char *devid, nvlist_t **config)
649 {
650         struct block_device *bdev;
651         vdev_label_t *label;
652         uint64_t s, size;
653         int i;
654
655         bdev = vdev_bdev_open(devpath, vdev_bdev_mode(FREAD), NULL);
656         if (IS_ERR(bdev))
657                 return -PTR_ERR(bdev);
658
659         s = bdev_capacity(bdev) * vdev_bdev_block_size(bdev);
660         if (s == 0) {
661                 vdev_bdev_close(bdev, vdev_bdev_mode(FREAD));
662                 return EIO;
663         }
664
665         size = P2ALIGN_TYPED(s, sizeof(vdev_label_t), uint64_t);
666         label = vmem_alloc(sizeof(vdev_label_t), KM_SLEEP);
667
668         for (i = 0; i < VDEV_LABELS; i++) {
669                 uint64_t offset, state, txg = 0;
670
671                 /* read vdev label */
672                 offset = vdev_label_offset(size, i, 0);
673                 if (vdev_disk_physio(bdev, (caddr_t)label,
674                     VDEV_SKIP_SIZE + VDEV_PHYS_SIZE, offset, READ_SYNC) != 0)
675                         continue;
676
677                 if (nvlist_unpack(label->vl_vdev_phys.vp_nvlist,
678                     sizeof (label->vl_vdev_phys.vp_nvlist), config, 0) != 0) {
679                         *config = NULL;
680                         continue;
681                 }
682
683                 if (nvlist_lookup_uint64(*config, ZPOOL_CONFIG_POOL_STATE,
684                     &state) != 0 || state >= POOL_STATE_DESTROYED) {
685                         nvlist_free(*config);
686                         *config = NULL;
687                         continue;
688                 }
689
690                 if (nvlist_lookup_uint64(*config, ZPOOL_CONFIG_POOL_TXG,
691                     &txg) != 0 || txg == 0) {
692                         nvlist_free(*config);
693                         *config = NULL;
694                         continue;
695                 }
696
697                 break;
698         }
699
700         vmem_free(label, sizeof(vdev_label_t));
701         vdev_bdev_close(bdev, vdev_bdev_mode(FREAD));
702
703         return 0;
704 }