diff --git a/tests/zfs-tests/tests/functional/cli_root/zpool/zpool_002_pos.ksh b/tests/zfs-tests/tests/functional/cli_root/zpool/zpool_002_pos.ksh index 9440e807f206..caf8a9a2d0ee 100755 --- a/tests/zfs-tests/tests/functional/cli_root/zpool/zpool_002_pos.ksh +++ b/tests/zfs-tests/tests/functional/cli_root/zpool/zpool_002_pos.ksh @@ -1,112 +1,111 @@ #!/bin/ksh -p # # CDDL HEADER START # # The contents of this file are subject to the terms of the # Common Development and Distribution License (the "License"). # You may not use this file except in compliance with the License. # # You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE # or http://www.opensolaris.org/os/licensing. # See the License for the specific language governing permissions # and limitations under the License. # # When distributing Covered Code, include this CDDL HEADER in each # file and include the License file at usr/src/OPENSOLARIS.LICENSE. # If applicable, add the following below this CDDL HEADER, with the # fields enclosed by brackets "[]" replaced with your own identifying # information: Portions Copyright [yyyy] [name of copyright owner] # # CDDL HEADER END # # # Copyright 2009 Sun Microsystems, Inc. All rights reserved. # Use is subject to license terms. # # # Copyright (c) 2012, 2016 by Delphix. All rights reserved. # . $STF_SUITE/include/libtest.shlib # # DESCRIPTION: # With ZFS_ABORT set, all zpool commands should be able to abort and generate a core file. # # STRATEGY: # 1. Create an array of zpool command # 2. Execute each command in the array # 3. Verify the command aborts and generate a core file # verify_runnable "global" function cleanup { unset ZFS_ABORT - if [[ -d $corepath ]]; then - rm -rf $corepath + if is_freebsd && [ -n "$old_corefile" ]; then + sysctl kern.corefile=$old_corefile fi - if poolexists $pool; then - log_must zpool destroy -f $pool - fi + # Clean up the pool created if we failed to abort. + poolexists $pool && destroy_pool $pool + + rm -rf $corepath $vdev1 $vdev2 $vdev3 } log_assert "With ZFS_ABORT set, all zpool commands can abort and generate a core file." log_onexit cleanup -#preparation work for testing corepath=$TESTDIR/core +corefile=$corepath/zpool.core if [[ -d $corepath ]]; then - rm -rf $corepath + log_must rm -rf $corepath fi -mkdir $corepath +log_must mkdir $corepath pool=pool.$$ vdev1=$TESTDIR/file1 vdev2=$TESTDIR/file2 vdev3=$TESTDIR/file3 for vdev in $vdev1 $vdev2 $vdev3; do - mkfile $MINVDEVSIZE $vdev + log_must mkfile $MINVDEVSIZE $vdev done set -A cmds "create $pool mirror $vdev1 $vdev2" "list $pool" "iostat $pool" \ "status $pool" "upgrade $pool" "get delegation $pool" "set delegation=off $pool" \ "export $pool" "import -d $TESTDIR $pool" "offline $pool $vdev1" \ "online $pool $vdev1" "clear $pool" "detach $pool $vdev2" \ "attach $pool $vdev1 $vdev2" "replace $pool $vdev2 $vdev3" \ "scrub $pool" "destroy -f $pool" set -A badparams "" "create" "destroy" "add" "remove" "list *" "iostat" "status" \ "online" "offline" "clear" "attach" "detach" "replace" "scrub" \ "import" "export" "upgrade" "history -?" "get" "set" if is_linux; then - ulimit -c unlimited - echo "$corepath/core.zpool" >/proc/sys/kernel/core_pattern + echo $corefile >/proc/sys/kernel/core_pattern echo 0 >/proc/sys/kernel/core_uses_pid - export ASAN_OPTIONS="abort_on_error=1:disable_coredump=0" elif is_freebsd; then - ulimit -c unlimited - log_must sysctl kern.corefile=$corepath/core.zpool - export ASAN_OPTIONS="abort_on_error=1:disable_coredump=0" -else - coreadm -p ${corepath}/core.%f + old_corefile=$(sysctl -n kern.corefile) + log_must sysctl kern.corefile=$corefile fi +ulimit -c unlimited +export ASAN_OPTIONS="abort_on_error=1:disable_coredump=0" export ZFS_ABORT=yes for subcmd in "${cmds[@]}" "${badparams[@]}"; do - corefile=${corepath}/core.zpool zpool $subcmd >/dev/null 2>&1 if [[ ! -e $corefile ]]; then - log_fail "zpool $subcmd cannot generate core file with ZFS_ABORT set." + log_fail "zpool $subcmd cannot generate core file with ZFS_ABORT set." fi rm -f $corefile done +unset ZFS_ABORT + log_pass "With ZFS_ABORT set, zpool command can abort and generate core file as expected." diff --git a/tests/zfs-tests/tests/functional/cli_root/zpool/zpool_003_pos.ksh b/tests/zfs-tests/tests/functional/cli_root/zpool/zpool_003_pos.ksh index d51633745d76..71d73c0f80e5 100755 --- a/tests/zfs-tests/tests/functional/cli_root/zpool/zpool_003_pos.ksh +++ b/tests/zfs-tests/tests/functional/cli_root/zpool/zpool_003_pos.ksh @@ -1,90 +1,101 @@ #!/bin/ksh -p # # CDDL HEADER START # # The contents of this file are subject to the terms of the # Common Development and Distribution License (the "License"). # You may not use this file except in compliance with the License. # # You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE # or http://www.opensolaris.org/os/licensing. # See the License for the specific language governing permissions # and limitations under the License. # # When distributing Covered Code, include this CDDL HEADER in each # file and include the License file at usr/src/OPENSOLARIS.LICENSE. # If applicable, add the following below this CDDL HEADER, with the # fields enclosed by brackets "[]" replaced with your own identifying # information: Portions Copyright [yyyy] [name of copyright owner] # # CDDL HEADER END # # # Copyright 2007 Sun Microsystems, Inc. All rights reserved. # Use is subject to license terms. # # # Copyright (c) 2012, 2016 by Delphix. All rights reserved. # . $STF_SUITE/include/libtest.shlib # # DESCRIPTION: # Verify debugging features of zpool such as ABORT and freeze/unfreeze # should run successfully. # # STRATEGY: # 1. Create an array containing each zpool options. # 2. For each element, execute the zpool command. # 3. Verify it run successfully. # function cleanup { + unset ZFS_ABORT + if is_freebsd && [ -n "$old_corefile" ]; then sysctl kern.corefile=$old_corefile fi + + rm -rf $corepath + + # Don't leave the pool frozen. + destroy_pool $TESTPOOL + default_mirror_setup $DISKS } verify_runnable "both" log_assert "Debugging features of zpool should succeed." log_onexit cleanup -log_must zpool -? > /dev/null 2>&1 +corepath=$TESTDIR/core +corefile=$corepath/zpool.core +if [[ -d $corepath ]]; then + log_must rm -rf $corepath +fi +log_must mkdir $corepath + +log_must eval "zpool -? >/dev/null 2>&1" if is_global_zone ; then log_must zpool freeze $TESTPOOL else log_mustnot zpool freeze $TESTPOOL log_mustnot zpool freeze ${TESTPOOL%%/*} fi log_mustnot zpool freeze fakepool -# Remove corefile possibly left by previous failing run of this test. -[[ -f core ]] && log_must rm -f core - if is_linux; then - ulimit -c unlimited - echo "core" >/proc/sys/kernel/core_pattern + echo $corefile >/proc/sys/kernel/core_pattern echo 0 >/proc/sys/kernel/core_uses_pid - export ASAN_OPTIONS="abort_on_error=1:disable_coredump=0" elif is_freebsd; then - ulimit -c unlimited old_corefile=$(sysctl -n kern.corefile) - log_must sysctl kern.corefile=core - export ASAN_OPTIONS="abort_on_error=1:disable_coredump=0" + log_must sysctl kern.corefile=$corefile fi +ulimit -c unlimited + +export ASAN_OPTIONS="abort_on_error=1:disable_coredump=0" +export ZFS_ABORT=yes + +zpool >/dev/null 2>&1 -ZFS_ABORT=1; export ZFS_ABORT -zpool > /dev/null 2>&1 unset ZFS_ABORT -[[ -f core ]] || log_fail "zpool did not dump core by request." -[[ -f core ]] && log_must rm -f core +[[ -f $corefile ]] || log_fail "zpool did not dump core by request." log_pass "Debugging features of zpool succeed."