mirror of
https://git.proxmox.com/git/mirror_zfs.git
synced 2024-12-25 18:59:33 +03:00
Remove zvol device node
The 'zfs destroy' changes in 330d06f
disrupted how zvol devices
get removed on ZoL. However, it basically boils down to the
fact that we are no longer reliably calling zvol_remove_minor()
via zfs_ioc_destroy_snaps().
Therefore we add the missing call and handle things similarly
to the existing zfs_unmount_snap() case. Ideally we would check
if this is of type DMU_OST_ZFS or DMU_OST_ZVOL and just do the
right thing as in zfs_ioc_destroy(). However, it looks like
it would be fairly expensive to get the type, and it's harmless
to simply attempt the umount and minor removal.
This is also an issue in the latest FreeBSD and Illumos code.
It was being tracked under the following issue, and we may want
to refresh our code when they settle on what they want to do
about it upstream.
https://www.illumos.org/issues/3170
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Issue #903
This commit is contained in:
parent
1ecc6d1265
commit
4ca9a43644
@ -3163,6 +3163,7 @@ zfs_ioc_destroy_snaps_nvl(zfs_cmd_t *zc)
|
||||
}
|
||||
|
||||
(void) zfs_unmount_snap(name, NULL);
|
||||
(void) zvol_remove_minor(name);
|
||||
}
|
||||
|
||||
err = dmu_snapshots_destroy_nvl(nvl, zc->zc_defer_destroy,
|
||||
|
Loading…
Reference in New Issue
Block a user