svn commit: r253806 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

Alexander Motin mav at
Tue Jul 30 14:50:45 UTC 2013

Author: mav
Date: Tue Jul 30 14:50:44 2013
New Revision: 253806

  Allow three IOCTLs to be used on suspended pool, restoring state that
  existed before IOCTL code refactoring merged change 4445fffb from illumos
  at r248571.
  This change allows `zpool clear` to be used again to recover suspended pool.
  It seems the only was supposed by the code to restore pool operation after
  reconnecting lost disks that were required for data completeness.  There
  are still cases where `zpool clear` command can just safely stuck due to
  deadlocks inside ZFS kernel part, but probably that is better then having
  no chances to recover at all.


Modified: head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_ioctl.c
--- head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_ioctl.c	Tue Jul 30 13:07:55 2013	(r253805)
+++ head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_ioctl.c	Tue Jul 30 14:50:44 2013	(r253806)
@@ -5495,10 +5495,10 @@ zfs_ioctl_init(void)
 	    zfs_secpolicy_read, B_FALSE, POOL_CHECK_NONE);
 	zfs_ioctl_register_pool(ZFS_IOC_ERROR_LOG, zfs_ioc_error_log,
-	    zfs_secpolicy_inject, B_FALSE, POOL_CHECK_SUSPENDED);
+	    zfs_secpolicy_inject, B_FALSE, POOL_CHECK_NONE);
-	    zfs_secpolicy_diff, B_FALSE, POOL_CHECK_SUSPENDED);
+	    zfs_secpolicy_diff, B_FALSE, POOL_CHECK_NONE);
 	    zfs_secpolicy_config, B_FALSE, POOL_CHECK_SUSPENDED);
@@ -5507,7 +5507,7 @@ zfs_ioctl_init(void)
 	    zfs_secpolicy_config, B_TRUE, POOL_CHECK_NONE);
 	zfs_ioctl_register_pool(ZFS_IOC_CLEAR, zfs_ioc_clear,
-	    zfs_secpolicy_config, B_TRUE, POOL_CHECK_SUSPENDED);
+	    zfs_secpolicy_config, B_TRUE, POOL_CHECK_NONE);
 	zfs_ioctl_register_pool(ZFS_IOC_POOL_REOPEN, zfs_ioc_pool_reopen,
 	    zfs_secpolicy_config, B_TRUE, POOL_CHECK_SUSPENDED);

More information about the svn-src-head mailing list