forked from openzfs/zfs
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
zvol: allow rename of in use ZVOL dataset
While ZFS allow renaming of in use ZVOLs at the DSL level without issues the ZVOL layer does not correctly update the renamed dataset if the device node is open (zv->zv_open_count > 0): trying to access the stale dataset name, for instance during a zfs receive, will cause the following failure: VERIFY3(zv->zv_objset->os_dsl_dataset->ds_owner == zv) failed ((null) == ffff8800dbb6fc00) PANIC at zvol.c:1255:zvol_resume() Showing stack for process 1390 CPU: 0 PID: 1390 Comm: zfs Tainted: P O 3.16.0-4-amd64 #1 Debian 3.16.51-3 Hardware name: Bochs Bochs, BIOS Bochs 01/01/2011 0000000000000000 ffffffff8151ea00 ffffffffa0758a80 ffff88028aefba30 ffffffffa0417219 ffff880037179220 ffffffff00000030 ffff88028aefba40 ffff88028aefb9e0 2833594649524556 6f5f767a3e2d767a 6f3e2d7465736a62 Call Trace: [<0>] ? dump_stack+0x5d/0x78 [<0>] ? spl_panic+0xc9/0x110 [spl] [<0>] ? mutex_lock+0xe/0x2a [<0>] ? zfs_refcount_remove_many+0x1ad/0x250 [zfs] [<0>] ? rrw_exit+0xc8/0x2e0 [zfs] [<0>] ? mutex_lock+0xe/0x2a [<0>] ? dmu_objset_from_ds+0x9a/0x250 [zfs] [<0>] ? dmu_objset_hold_flags+0x71/0xc0 [zfs] [<0>] ? zvol_resume+0x178/0x280 [zfs] [<0>] ? zfs_ioc_recv_impl+0x88b/0xf80 [zfs] [<0>] ? zfs_refcount_remove_many+0x1ad/0x250 [zfs] [<0>] ? zfs_ioc_recv+0x1c2/0x2a0 [zfs] [<0>] ? dmu_buf_get_user+0x13/0x20 [zfs] [<0>] ? __alloc_pages_nodemask+0x166/0xb50 [<0>] ? zfsdev_ioctl+0x896/0x9c0 [zfs] [<0>] ? handle_mm_fault+0x464/0x1140 [<0>] ? do_vfs_ioctl+0x2cf/0x4b0 [<0>] ? __do_page_fault+0x177/0x410 [<0>] ? SyS_ioctl+0x81/0xa0 [<0>] ? async_page_fault+0x28/0x30 [<0>] ? system_call_fast_compare_end+0x10/0x15 Reviewed by: Tom Caputi <[email protected]> Reviewed-by: Brian Behlendorf <[email protected]> Signed-off-by: loli10K <[email protected]> Closes openzfs#6263 Closes openzfs#8371
- Loading branch information
1 parent
0c637f3
commit c44a3ec
Showing
7 changed files
with
78 additions
and
10 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -36,4 +36,6 @@ verify_runnable "global" | |
|
||
default_zvol_cleanup | ||
|
||
[[ -d $TESTDIR ]] && rm -rf $TESTDIR | ||
|
||
log_pass |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -36,4 +36,6 @@ verify_runnable "global" | |
|
||
default_zvol_setup $DISK $VOLSIZE | ||
|
||
log_must mkdir $TESTDIR | ||
|
||
log_pass |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
71 changes: 71 additions & 0 deletions
71
tests/zfs-tests/tests/functional/zvol/zvol_misc/zvol_misc_rename_inuse.ksh
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,71 @@ | ||
#!/bin/ksh -p | ||
# | ||
# This file and its contents are supplied under the terms of the | ||
# Common Development and Distribution License ("CDDL"), version 1.0. | ||
# You may only use this file in accordance with the terms of version | ||
# 1.0 of the CDDL. | ||
# | ||
# A full copy of the text of the CDDL should have accompanied this | ||
# source. A copy of the CDDL is also available via the Internet at | ||
# http://www.illumos.org/license/CDDL. | ||
# | ||
|
||
# | ||
# Copyright 2019, loli10K <[email protected]>. All rights reserved. | ||
# | ||
|
||
. $STF_SUITE/include/libtest.shlib | ||
. $STF_SUITE/tests/functional/zvol/zvol_common.shlib | ||
. $STF_SUITE/tests/functional/zvol/zvol_misc/zvol_misc_common.kshlib | ||
|
||
# | ||
# DESCRIPTION: | ||
# Verify 'zfs rename' works on a ZVOL already in use as block device | ||
# | ||
# STRATEGY: | ||
# 1. Create a ZVOL | ||
# 2. Create a filesystem on the ZVOL device and mount it | ||
# 3. Rename the ZVOL dataset | ||
# 4. Receive a send stream with the same name as the old ZVOL dataset and verify | ||
# we don't trigger any issue like the one reported in #6263 | ||
# | ||
|
||
verify_runnable "global" | ||
|
||
function cleanup | ||
{ | ||
log_must umount "$MNTPFS" | ||
log_must rmdir "$MNTPFS" | ||
for ds in "$SENDFS" "$ZVOL" "$ZVOL-renamed"; do | ||
destroy_dataset "$ds" '-rf' | ||
done | ||
udev_wait | ||
} | ||
|
||
log_assert "Verify 'zfs rename' works on a ZVOL already in use as block device" | ||
log_onexit cleanup | ||
|
||
ZVOL="$TESTPOOL/vol.$$" | ||
ZDEV="$ZVOL_DEVDIR/$ZVOL" | ||
MNTPFS="$TESTDIR/zvol_inuse_rename" | ||
SENDFS="$TESTPOOL/sendfs.$$" | ||
|
||
# 1. Create a ZVOL | ||
log_must zfs create -V $VOLSIZE "$ZVOL" | ||
|
||
# 2. Create a filesystem on the ZVOL device and mount it | ||
udev_wait | ||
log_must eval "echo y | newfs $ZDEV >/dev/null 2>&1" | ||
log_must mkdir "$MNTPFS" | ||
log_must mount "$ZDEV" "$MNTPFS" | ||
|
||
# 3. Rename the ZVOL dataset | ||
log_must zfs rename "$ZVOL" "$ZVOL-renamed" | ||
|
||
# 4. Receive a send stream with the same name as the old ZVOL dataset and verify | ||
# we don't trigger any issue like the one reported in #6263 | ||
log_must zfs create "$SENDFS" | ||
log_must zfs snap "$SENDFS@snap" | ||
log_must eval "zfs send $SENDFS@snap | zfs recv $ZVOL" | ||
|
||
log_pass "Renaming in use ZVOL works successfully" |