Remove wholedisk check from vdev_disk_open()
authorBrian Behlendorf <behlendorf1@llnl.gov>
Tue, 26 Feb 2013 19:28:14 +0000 (11:28 -0800)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Thu, 28 Feb 2013 20:02:59 +0000 (12:02 -0800)
As described by the comment and enforced the by assertion the
v->vdev_wholedisk will never be -1.  The wholedisk handling
is performed by the user space utilities.  To prevent confusion
this dead code is being removed.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
module/zfs/vdev_disk.c

index 4e38c78..f93b7bc 100644 (file)
@@ -294,20 +294,6 @@ skip_open:
        /*  Determine the physical block size */
        block_size = vdev_bdev_block_size(vd->vd_bdev);
 
-       /* We think the wholedisk property should always be set when this
-        * function is called.  ASSERT here so if any legitimate cases exist
-        * where it's not set, we'll find them during debugging.  If we never
-        * hit the ASSERT, this and the following conditional statement can be
-        * removed. */
-       ASSERT3S(v->vdev_wholedisk, !=, -1ULL);
-
-       /* The wholedisk property was initialized to -1 in vdev_alloc() if it
-        * was unspecified.  In that case, check if this is a whole device.
-        * When bdev->bd_contains == bdev we have a whole device and not simply
-        * a partition. */
-       if (v->vdev_wholedisk == -1ULL)
-               v->vdev_wholedisk = (bdev->bd_contains == bdev);
-
        /* Clear the nowritecache bit, causes vdev_reopen() to try again. */
        v->vdev_nowritecache = B_FALSE;