Bug #12085


zvol_dumpio would be cleaner with vdev_op_dumpio

Added by Mike Gerdts about 4 years ago. Updated about 4 years ago.

zfs - Zettabyte File System
Start date:
Due date:
% Done:


Estimated time:
Gerrit CR:
External Bug:


While working on the fix for #4454, I found that zvol_dumpio has logic that is more properly placed in op callbacks for mirror, raidz, and disk vdevs. Fixing this ahead of #4454 will make it so that there's no need to have yet more leakage of vdev_disk internals to zvol.c.

Related issues

Related to illumos gate - Bug #14115: dumping to zvol on raidz will corrupt the poolClosed

Actions #1

Updated by Mike Gerdts about 4 years ago

  • % Done changed from 0 to 90
Actions #2

Updated by Jason King about 4 years ago

  • Assignee changed from Mike Gerdts to Jason King

To test this, running a BE with the change, I created zpools with the following configurations:

single disk (used the existing rpool in the VM)
2 unmirrored disks (zpool create yolopool disk1 disk2)
2 mirrored disks (zpool create testpool disk1 disk2)
3 disk raidz (zpool create testpool raidz c3t0d0 c4t0d0 c5t0d0)
3 disk raidz3 (zpool create testpool raidz2 c3t0d0 c4t0d0 c5t0d0)

For each pool, I created a dump zvol (zfs create -V 8G .../dump), then set the dump device to that zvol (dumpadm -d /dev/zvol/dsk/..../dump). Then I used the age-old method of echo rootdir/Z 0 | mdb -kw to force a panic and dump.

During each panic, I made sure the dump was successful (as reported on the console), then after the system was running (post-panic), I uncompressed the resulting core file, loaded it in mdb, and ran '::status' to ensure mdb no errors were reported while processing/reading the resulting dump.

Actions #3

Updated by Electric Monk about 4 years ago

  • Status changed from New to Closed
  • % Done changed from 90 to 100

git commit ac04831df9010f533275633d3bb9829b86c171d3

commit  ac04831df9010f533275633d3bb9829b86c171d3
Author: Mike Gerdts <>
Date:   2020-01-11T00:01:41.000Z

    12085 zvol_dumpio would be cleaner with vdev_op_dumpio
    Reviewed by: Dan McDonald <>
    Reviewed by: Toomas Soome <>
    Approved by: Robert Mustacchi <>

Actions #4

Updated by Joshua M. Clulow over 2 years ago

  • Related to Bug #14115: dumping to zvol on raidz will corrupt the pool added

Also available in: Atom PDF