Whamcloud - gitweb
LU-10447 tests: replace $SET/$GETSTRIPE for misc tests
[fs/lustre-release.git] / lustre / tests / sanity-hsm.sh
index 86f91de..7a4d3c2 100755 (executable)
@@ -14,14 +14,21 @@ ONLY=${ONLY:-"$*"}
 # bug number for skipped test:
 ALWAYS_EXCEPT="$SANITY_HSM_EXCEPT"
 # UPDATE THE COMMENT ABOVE WITH BUG NUMBERS WHEN CHANGING ALWAYS_EXCEPT!
+if $SHARED_KEY; then
+# bug number for skipped tests:        LU-9795 LU-9795
+       ALWAYS_EXCEPT="         13      402b    $ALWAYS_EXCEPT"
+fi
 
 LUSTRE=${LUSTRE:-$(cd $(dirname $0)/..; echo $PWD)}
 
 . $LUSTRE/tests/test-framework.sh
 init_test_env $@
 . ${CONFIG:=$LUSTRE/tests/cfg/$NAME.sh}
+get_lustre_env
 init_logging
 
+[ -n "$FILESET" ] && skip "Not functional for FILESET set"
+
 MULTIOP=${MULTIOP:-multiop}
 OPENFILE=${OPENFILE:-openfile}
 MMAP_CAT=${MMAP_CAT:-mmap_cat}
@@ -34,8 +41,8 @@ FAIL_ON_ERROR=false
 
 check_and_setup_lustre
 
-if [[ $(lustre_version_code $SINGLEMDS) -lt $(version_code 2.4.53) ]]; then
-       skip_env "Need MDS version at least 2.4.53" && exit
+if [[ $MDS1_VERSION -lt $(version_code 2.4.53) ]]; then
+       skip_env "Need MDS version at least 2.4.53"
 fi
 
 # $RUNAS_ID may get set incorrectly somewhere else
@@ -109,16 +116,6 @@ init_agt_vars() {
        export HSMTOOL_UPDATE_INTERVAL=${HSMTOOL_UPDATE_INTERVAL:=""}
        export HSMTOOL_EVENT_FIFO=${HSMTOOL_EVENT_FIFO:=""}
        export HSMTOOL_TESTDIR
-       export HSMTOOL_BASE=$(basename "$HSMTOOL" | cut -f1 -d" ")
-       # $hsm_root/$HSMTMP Makes $hsm_root dir path less generic to ensure
-       # rm -rf $hsm_root/* is safe even if $hsm_root becomes unset to avoid
-       # deleting everything in filesystem, independent of any copytool.
-       export HSMTMP=${HSMTMP:-"shsm"}
-
-       HSM_ARCHIVE=$(copytool_device $SINGLEAGT)
-
-       [ -z "${HSM_ARCHIVE// /}" ] && error "HSM_ARCHIVE is empty!"
-       HSM_ARCHIVE=$HSM_ARCHIVE/$HSMTMP
 
        HSM_ARCHIVE_NUMBER=2
 
@@ -141,13 +138,6 @@ copytool_device() {
        echo -n ${!dev}
 }
 
-# Stop copytool and unregister an existing changelog user.
-cleanup() {
-       copytool_monitor_cleanup
-       copytool_cleanup
-       cdt_set_sanity_policy
-}
-
 get_mdt_devices() {
        local mdtno
        # get MDT device for each mdc
@@ -161,14 +151,15 @@ get_mdt_devices() {
 
 search_copytools() {
        local hosts=${1:-$(facet_active_host $SINGLEAGT)}
-       do_nodesv $hosts "pgrep -x $HSMTOOL_BASE"
+       do_nodesv $hosts "libtool execute pgrep -x $HSMTOOL"
 }
 
 kill_copytools() {
        local hosts=${1:-$(facet_active_host $SINGLEAGT)}
 
        echo "Killing existing copytools on $hosts"
-       do_nodesv $hosts "killall -q $HSMTOOL_BASE" || true
+       do_nodesv $hosts "libtool execute killall -q $HSMTOOL" || true
+       copytool_continue "$hosts"
 }
 
 wait_copytools() {
@@ -235,166 +226,15 @@ copytool_monitor_setup() {
        fi
 }
 
-copytool_monitor_cleanup() {
-       local facet=${1:-$SINGLEAGT}
-       local agent=$(facet_active_host $facet)
-
-       if [ -n "$HSMTOOL_MONITOR_DIR" ]; then
-               # Should die when the copytool dies, but just in case.
-               local cmd="kill \\\$(cat $HSMTOOL_MONITOR_DIR/monitor_pid)"
-               cmd+=" 2>/dev/null || true"
-               do_node $agent "$cmd"
-               do_node $agent "rm -fr $HSMTOOL_MONITOR_DIR"
-               export HSMTOOL_MONITOR_DIR=
-       fi
-
-       # The pdsh should die on its own when the monitor dies. Just
-       # in case, though, try to clean up to avoid any cruft.
-       if [ -n "$HSMTOOL_MONITOR_PDSH" ]; then
-               kill $HSMTOOL_MONITOR_PDSH 2>/dev/null || true
-               export HSMTOOL_MONITOR_PDSH=
-       fi
-}
-
-copytool_logfile()
+fid2archive()
 {
-       local host="$(facet_host "$1")"
-       local prefix=$TESTLOG_PREFIX
-       [ -n "$TESTNAME" ] && prefix+=.$TESTNAME
-
-       printf "${prefix}.copytool${archive_id}_log.${host}.log"
-}
-
-__lhsmtool_setup()
-{
-       local cmd="$HSMTOOL $HSMTOOL_VERBOSE --daemon --hsm-root \"$hsm_root\""
-       [ -n "$bandwidth" ] && cmd+=" --bandwidth $bandwidth"
-       [ -n "$archive_id" ] && cmd+=" --archive $archive_id"
-       [ ${#misc_options[@]} -gt 0 ] &&
-               cmd+=" $(IFS=" " echo "$@")"
-       cmd+=" \"$mountpoint\""
-
-       echo "Starting copytool $facet on $(facet_host $facet)"
-       stack_trap "do_facet $facet \"pkill -x $HSMTOOL_BASE\" || true" EXIT
-       do_facet $facet "$cmd < /dev/null > \"$(copytool_logfile $facet)\" 2>&1"
-}
-
-hsm_root() {
-       local facet="${1:-$SINGLEAGT}"
-
-       printf "$(copytool_device "$facet")/${TESTSUITE}.${TESTNAME}/"
-}
-
-copytool()
-{
-       local action=$1
-       shift
-
-       # Parse arguments
-       local fail_on_error=true
-       local -a misc_options
-       while [ $# -gt 0 ]; do
-               case "$1" in
-               -f|--facet)
-                       shift
-                       local facet="$1"
-                       ;;
-               -m|--mountpoint)
-                       shift
-                       local mountpoint="$1"
-                       ;;
-               -a|--archive-id)
-                       shift
-                       local archive_id="$1"
-                       ;;
-               -b|--bwlimit)
-                       shift
-                       local bandwidth="$1" # in MB/s
-                       ;;
-               -n|--no-fail)
-                       local fail_on_error=false
-                       ;;
-               *)
-                       # Uncommon(/copytool dependent) option
-                       misc_options+=("$1")
-                       ;;
-               esac
-               shift
-       done
-
-       # Use default values if needed
-       local facet=${facet:-$SINGLEAGT}
-       local mountpoint="${mountpoint:-${MOUNT2:-$MOUNT}}"
-       local hsm_root="$(hsm_root "$facet")"
-
-       stack_trap "do_facet $facet \"rm -rf \\\"$hsm_root\\\"\"" EXIT
-       do_facet $facet "mkdir -p \"$hsm_root\"" ||
-               error "mkdir \"$hsm_root\" failed"
+       local fid="$1"
 
        case "$HSMTOOL" in
        lhsmtool_posix)
-               local copytool=lhsmtool
+               printf "%s" "$(hsm_root)/*/*/*/*/*/*/$fid"
                ;;
        esac
-
-       __${copytool}_${action} "${misc_options[@]}"
-       if [ $? -ne 0 ]; then
-               local error_msg
-
-               case $action in
-               setup)
-                       local host="$(facet_host $facet)"
-                       error_msg="Failed to start copytool $facet on '$host'"
-                       ;;
-               esac
-
-               $fail_on_error && error "$error_msg" || echo "$error_msg"
-       fi
-}
-
-copytool_setup() {
-       local facet=${1:-$SINGLEAGT}
-       # Use MOUNT2 by default if defined
-       local lustre_mntpnt=${2:-${MOUNT2:-$MOUNT}}
-       local arc_id=$3
-       local hsm_root=${4:-$(copytool_device $facet)}
-
-       [ -z "${hsm_root// /}" ] && error "copytool_setup: hsm_root empty!"
-
-       local agent=$(facet_active_host $facet)
-
-       if $HSM_ARCHIVE_PURGE; then
-               echo "Purging archive on $agent"
-               do_facet $facet "rm -rf $hsm_root/$HSMTMP/*"
-       fi
-
-       echo "Starting copytool $facet on $agent"
-       do_facet $facet "mkdir -p $hsm_root/$HSMTMP/" ||
-                       error "mkdir '$hsm_root/$HSMTMP' failed"
-       # bandwidth is limited to 1MB/s so the copy time is known and
-       # independent of hardware
-       local cmd="$HSMTOOL $HSMTOOL_VERBOSE --daemon"
-       cmd+=" --hsm-root $hsm_root/$HSMTMP"
-       [[ -z "$arc_id" ]] || cmd+=" --archive $arc_id"
-       [[ -z "$HSMTOOL_UPDATE_INTERVAL" ]] ||
-               cmd+=" --update-interval $HSMTOOL_UPDATE_INTERVAL"
-       [[ -z "$HSMTOOL_EVENT_FIFO" ]] ||
-               cmd+=" --event-fifo $HSMTOOL_EVENT_FIFO"
-       cmd+=" --bandwidth 1 $lustre_mntpnt"
-
-       # Redirect the standard output and error to a log file which
-       # can be uploaded to Maloo.
-       local prefix=$TESTLOG_PREFIX
-       [[ -z "$TESTNAME" ]] || prefix=$prefix.$TESTNAME
-       local copytool_log=$prefix.copytool${arc_id}_log.$agent.log
-
-       stack_trap cleanup EXIT
-       do_facet $facet "$cmd < /dev/null > $copytool_log 2>&1"
-       if [[ $? !=  0 ]]; then
-               [[ $HSMTOOL_NOERROR == true ]] ||
-                       error "start copytool $facet on $agent failed"
-               echo "start copytool $facet on $agent failed"
-       fi
 }
 
 get_copytool_event_log() {
@@ -408,97 +248,29 @@ get_copytool_event_log() {
                error "Could not collect event log from $agent"
 }
 
-copytool_cleanup() {
-       trap - EXIT
-       local agt_facet=$SINGLEAGT
-       local agt_hosts=${1:-$(facet_active_host $agt_facet)}
-       local hsm_root=$(copytool_device $agt_facet)
-
-       [ -z "${hsm_root// /}" ] && error "copytool_cleanup: hsm_root empty!"
-
-       local i
-       local facet
-       local param
-       local -a state
-
-       kill_copytools $agt_hosts
-       wait_copytools $agt_hosts || error "copytools failed to stop"
-
-       # Clean all CDTs orphans requests from previous tests that
-       # would otherwise need to timeout to clear.
-       for ((i = 0; i < MDSCOUNT; i++)); do
-               facet=mds$((i + 1))
-               param=$(printf 'mdt.%s-MDT%04x.hsm_control' $FSNAME $i)
-               state[$i]=$(do_facet $facet "$LCTL get_param -n $param")
-
-               # Skip already stopping or stopped CDTs.
-               [[ "${state[$i]}" =~ ^stop ]] && continue
-
-               do_facet $facet "$LCTL set_param $param=shutdown"
-       done
-
-       for ((i = 0; i < MDSCOUNT; i++)); do
-               # Only check and restore CDTs that we stopped in the first loop.
-               [[ "${state[$i]}" =~ ^stop ]] && continue
-
-               facet=mds$((i + 1))
-               param=$(printf 'mdt.%s-MDT%04x.hsm_control' $FSNAME $i)
-
-               wait_result $facet "$LCTL get_param -n $param" stopped 20 ||
-                       error "$facet CDT state is not stopped"
-
-               # Restore old CDT state.
-               do_facet $facet "$LCTL set_param $param=${state[$i]}"
-       done
-
-       for ((i = 0; i < MDSCOUNT; i++)); do
-               # Only check CDTs that we stopped in the first loop.
-               [[ "${state[$i]}" =~ ^stop ]] && continue
-
-               facet=mds$((i + 1))
-               param=$(printf 'mdt.%s-MDT%04x.hsm_control' $FSNAME $i)
-
-               # Check that the old CDT state was restored.
-               wait_result $facet "$LCTL get_param -n $param" "${state[$i]}" \
-                       20 || error "$facet CDT state is not '${state[$i]}'"
-       done
-
-       if do_facet $agt_facet "df $hsm_root" >/dev/null 2>&1 ; then
-               do_facet $agt_facet "rm -rf $hsm_root/$HSMTMP/*"
-       fi
-}
-
 copytool_suspend() {
        local agents=${1:-$(facet_active_host $SINGLEAGT)}
 
-       do_nodesv $agents "pkill -STOP -x $HSMTOOL_BASE" || return 0
+       stack_trap \
+               "do_nodesv $agents libtool execute pkill -CONT -x '$HSMTOOL' || true" EXIT
+       do_nodesv $agents "libtool execute pkill -STOP -x $HSMTOOL" || return 0
        echo "Copytool is suspended on $agents"
 }
 
 copytool_continue() {
        local agents=${1:-$(facet_active_host $SINGLEAGT)}
 
-       do_nodesv $agents "pkill -CONT -x $HSMTOOL_BASE" || return 0
+       do_nodesv $agents "libtool execute pkill -CONT -x $HSMTOOL" || return 0
        echo "Copytool is continued on $agents"
 }
 
 copytool_remove_backend() {
        local fid=$1
-       local be=$(do_facet $SINGLEAGT find $HSM_ARCHIVE -name $fid)
+       local be=$(do_facet $SINGLEAGT find "$(hsm_root)" -name $fid)
        echo "Remove from backend: $fid = $be"
        do_facet $SINGLEAGT rm -f $be
 }
 
-import_file() {
-       mkdir -p "$(dirname "$2")" ||
-               error "cannot create directory '$(dirname "$2")'"
-
-       do_facet $SINGLEAGT \
-               "$HSMTOOL --archive $HSM_ARCHIVE_NUMBER --hsm-root $HSM_ARCHIVE\
-               --import $1 $2 $MOUNT" ||
-               error "import of $1 to $2 failed"
-}
-
 file_creation_failure() {
        local cmd=$1
        local file=$2
@@ -564,7 +336,7 @@ create_small_sync_file() {
 }
 
 create_archive_file() {
-       local file="$HSM_ARCHIVE/$1"
+       local file="$(hsm_root)/$1"
        local count=${2:-39}
        local source=/dev/urandom
 
@@ -577,44 +349,14 @@ create_archive_file() {
 }
 
 copy2archive() {
-       local file=$HSM_ARCHIVE/$2
-       do_facet $SINGLEAGT mkdir -p $(dirname $file)
-       do_facet $SINGLEAGT cp -p $1 $file || error "cannot copy $1 to $file"
-}
-
-mdts_set_param() {
-       local arg=$1
-       local key=$2
-       local value=$3
-       local mdtno
-       local rc=0
-       if [[ "$value" != "" ]]; then
-               value="=$value"
-       fi
-       for mdtno in $(seq 1 $MDSCOUNT); do
-               local idx=$(($mdtno - 1))
-               local facet=mds${mdtno}
-               # if $arg include -P option, run 1 set_param per MDT on the MGS
-               # else, run set_param on each MDT
-               [[ $arg = *"-P"* ]] && facet=mgs
-               do_facet $facet $LCTL set_param $arg mdt.${MDT[$idx]}.$key$value
-               [[ $? != 0 ]] && rc=1
-       done
-       return $rc
-}
+       local hsm_root="$(hsm_root)"
+       local file="$hsm_root/$2"
 
-mdts_check_param() {
-       local key="$1"
-       local target="$2"
-       local timeout="$3"
-       local mdtno
-       for mdtno in $(seq 1 $MDSCOUNT); do
-               local idx=$(($mdtno - 1))
-               wait_result mds${mdtno} \
-                       "$LCTL get_param -n $MDT_PREFIX${idx}.$key" "$target" \
-                       $timeout ||
-                       error "$key state is not '$target' on mds${mdtno}"
-       done
+       stack_trap "do_facet $SINGLEAGT rm -rf '$hsm_root'" EXIT
+       do_facet $SINGLEAGT mkdir -p "$(dirname "$file")" ||
+           error "mkdir '$(dirname "$file")' failed"
+       do_facet $SINGLEAGT cp -p "$1" "$file" ||
+               error "cannot copy '$1' to '$file'"
 }
 
 get_hsm_param() {
@@ -623,14 +365,6 @@ get_hsm_param() {
        echo $val
 }
 
-set_hsm_param() {
-       local param=$1
-       local value=$2
-       local opt=$3
-       mdts_set_param "$opt -n" "hsm.$param" "$value"
-       return $?
-}
-
 set_test_state() {
        local cmd=$1
        local target=$2
@@ -638,15 +372,6 @@ set_test_state() {
        mdts_check_param hsm_control "$target" 10
 }
 
-cdt_set_sanity_policy() {
-       if [[ "$CDT_POLICY_HAD_CHANGED" ]]
-       then
-               # clear all
-               mdts_set_param "" hsm.policy "+NRA"
-               mdts_set_param "" hsm.policy "-NBR"
-               CDT_POLICY_HAD_CHANGED=
-       fi
-}
 
 cdt_set_no_retry() {
        mdts_set_param "" hsm.policy "+NRA"
@@ -672,21 +397,6 @@ cdt_clear_mount_state() {
        mdts_set_param "-P -d" hsm_control ""
 }
 
-cdt_set_mount_state() {
-       mdts_set_param "-P" hsm_control "$1"
-       # set_param -P is asynchronous operation and could race with set_param.
-       # In such case configs could be retrieved and applied at mgc after
-       # set_param -P completion. Sleep here to avoid race with set_param.
-       # We need at least 20 seconds. 10 for mgc_requeue_thread to wake up
-       # MGC_TIMEOUT_MIN_SECONDS + MGC_TIMEOUT_RAND_CENTISEC(5 + 5)
-       # and 10 seconds to retrieve config from server.
-       sleep 20
-}
-
-cdt_check_state() {
-       mdts_check_param hsm_control "$1" 20
-}
-
 cdt_disable() {
        set_test_state disabled disabled
 }
@@ -709,37 +419,6 @@ cdt_restart() {
        cdt_set_sanity_policy
 }
 
-needclients() {
-       local client_count=$1
-       if [[ $CLIENTCOUNT -lt $client_count ]]; then
-               skip "Need $client_count or more clients, have $CLIENTCOUNT"
-               return 1
-       fi
-       return 0
-}
-
-path2fid() {
-       $LFS path2fid $1 | tr -d '[]'
-       return ${PIPESTATUS[0]}
-}
-
-get_hsm_flags() {
-       local f=$1
-       local u=$2
-       local st
-
-       if [[ $u == "user" ]]; then
-               st=$($RUNAS $LFS hsm_state $f)
-       else
-               u=root
-               st=$($LFS hsm_state $f)
-       fi
-
-       [[ $? == 0 ]] || error "$LFS hsm_state $f failed (run as $u)"
-
-       st=$(echo $st | cut -f 2 -d" " | tr -d "()," )
-       echo $st
-}
 
 get_hsm_archive_id() {
        local f=$1
@@ -751,14 +430,6 @@ get_hsm_archive_id() {
        echo $ar
 }
 
-check_hsm_flags() {
-       local f=$1
-       local fl=$2
-
-       local st=$(get_hsm_flags $f)
-       [[ $st == $fl ]] || error "hsm flags on $f are $st != $fl"
-}
-
 check_hsm_flags_user() {
        local f=$1
        local fl=$2
@@ -795,44 +466,6 @@ delete_large_files() {
        wait_delete_completed
 }
 
-make_custom_file_for_progress() {
-       local count=${2:-"39"}
-       local bs=$($LCTL get_param -n lov.*-clilov-*.stripesize | head -n1)
-       bs=${3:-$bs}
-
-       [[ $count -gt  0 ]] || error "Invalid file size"
-       [[ $bs -gt 0 ]] || error "Invalid stripe size"
-
-       if ! create_file "${1/$DIR/$DIR2}" $bs $count fsync; then
-               echo "The creation of '${1/$DIR/$DIR2}' failed" >&2
-               echo "It might be due to a lack of space in the filesystem" >&2
-               delete_large_files >&2
-               create_file "${1/$DIR/$DIR2}" $bs $count fsync ||
-                       file_creation_failure dd "${1/$DIR/$DIR2}" $?
-       fi
-}
-
-wait_result() {
-       local facet=$1
-       shift
-       wait_update --verbose $(facet_active_host $facet) "$@"
-}
-
-wait_request_state() {
-       local fid=$1
-       local request=$2
-       local state=$3
-       # 4th arg (mdt index) is optional
-       local mdtidx=${4:-0}
-       local mds=mds$(($mdtidx + 1))
-
-       local cmd="$LCTL get_param -n ${MDT_PREFIX}${mdtidx}.hsm.actions"
-       cmd+=" | awk '/'$fid'.*action='$request'/ {print \\\$13}' | cut -f2 -d="
-
-       wait_result $mds "$cmd" $state 200 ||
-               error "request on $fid is not $state on $mds"
-}
-
 get_request_state() {
        local fid=$1
        local request=$2
@@ -945,7 +578,7 @@ get_agent_uuid() {
 
        # Lustre mount-point is mandatory and last parameter on
        # copytool cmd-line.
-       local mntpnt=$(do_rpc_nodes $agent ps -C $HSMTOOL_BASE -o args= |
+       local mntpnt=$(do_rpc_nodes $agent libtool execute ps -C $HSMTOOL -o args= |
                       awk '{print $NF}')
        [ -n "$mntpnt" ] || error "Found no Agent or with no mount-point "\
                                  "parameter"
@@ -974,6 +607,8 @@ cdt_set_sanity_policy
 # finished requests are quickly removed from list
 set_hsm_param grace_delay 10
 
+CLIENT_NIDS=( $($LCTL list_nids all) )
+
 test_1A() { # was test_1
        mkdir -p $DIR/$tdir
        chmod 777 $DIR/$tdir
@@ -1027,11 +662,8 @@ test_1a() {
 }
 run_test 1a "mmap & cat a HSM released file"
 
-test_1b() {
-       mkdir -p $DIR/$tdir
-       $LFS setstripe -E 1M -S 1M -E 64M -c 2 -E -1 -c 4 $DIR/$tdir ||
-               error "failed to set default stripe"
-       local f=$DIR/$tdir/$tfile
+test_1bde_base() {
+       local f=$1
        rm -f $f
 
        dd if=/dev/urandom of=$f bs=1M count=1 conv=sync ||
@@ -1055,6 +687,15 @@ test_1b() {
        echo "verify restored state: "
        check_hsm_flags $f "0x00000009" && echo "pass"
 }
+
+test_1b() {
+       mkdir -p $DIR/$tdir
+       $LFS setstripe -E 1M -S 1M -E 64M -c 2 -E -1 -c 4 $DIR/$tdir ||
+               error "failed to set default stripe"
+       local f=$DIR/$tdir/$tfile
+
+       test_1bde_base $f
+}
 run_test 1b "Archive, Release and Restore composite file"
 
 test_1c() {
@@ -1083,10 +724,26 @@ test_1c() {
        [[ $st == $LOCAL_HSM_ARCHIVE_NUMBER ]] ||
                error "wrong archive number, $st != $LOCAL_HSM_ARCHIVE_NUMBER"
 
-       # Test whether setting archive number > 32 results in error.
-       $LFS hsm_set --exists --archive-id 33 $f &&
-               error "archive number is larger than 32"
-       check_hsm_flags_user $f "0x00000001"
+       LOCAL_HSM_ARCHIVE_NUMBER=33
+       if [ $(lustre_version_code client) -ge $(version_code 2.11.56) ] &&
+          [ $(lustre_version_code $SINGLEMDS) -ge $(version_code 2.11.56) ]; then
+               # lustre in the new version supports unlimited archiveID.
+               # Test whether setting archive number > 32 is supported
+               $LFS hsm_set --exists --archive-id $LOCAL_HSM_ARCHIVE_NUMBER $f ||
+                       error "archive ID $LOCAL_HSM_ARCHIVE_NUMBER too large?"
+               check_hsm_flags_user $f "0x00000001"
+
+               echo "verifying archive number is $LOCAL_HSM_ARCHIVE_NUMBER"
+               st=$(get_hsm_archive_id $f)
+               [[ $st == $LOCAL_HSM_ARCHIVE_NUMBER ]] ||
+                       error "wrong archive number, $st != $LOCAL_HSM_ARCHIVE_NUMBER"
+       else
+               # old client or old mds can only support at most 32 archiveID
+               # test whether setting archive number > 32 results in error.
+               $LFS hsm_set --exists --archive-id $LOCAL_HSM_ARCHIVE_NUMBER $f &&
+                       error "bitmap archive number is larger than 32"
+               check_hsm_flags_user $f "0x00000001"
+       fi
 
        # Test whether setting archive number 16 and archived flag.
        LOCAL_HSM_ARCHIVE_NUMBER=16
@@ -1102,34 +759,56 @@ test_1c() {
 run_test 1c "Check setting archive-id in lfs hsm_set"
 
 test_1d() {
+       [ $MDS1_VERSION -lt $(version_code 2.10.59) ] &&
+               skip "need MDS version at least 2.10.59"
+
        mkdir -p $DIR/$tdir
        $LFS setstripe -E 1M -L mdt -E -1 -c 2 $DIR/$tdir ||
                error "failed to set default stripe"
        local f=$DIR/$tdir/$tfile
-       rm -f $f
 
-       dd if=/dev/urandom of=$f bs=1M count=1 conv=sync ||
-               error "failed to create file"
-       local fid=$(path2fid $f)
+       test_1bde_base $f
+}
+run_test 1d "Archive, Release and Restore DoM file"
 
-       copytool setup
+test_1e() {
+       [ $(lustre_version_code $SINGLEMDS) -lt $(version_code $SEL_VER) ] &&
+               skip "skipped for lustre < $SEL_VER"
 
-       echo "archive $f"
-       $LFS hsm_archive $f || error "could not archive file"
-       wait_request_state $fid ARCHIVE SUCCEED
+       mkdir -p $DIR/$tdir
+       $LFS setstripe -E 1G -z 64M -E 10G -z 512M -E -1 -z 1G $DIR/$tdir ||
+               error "failed to set default stripe"
+       local comp_file=$DIR/$tdir/$tfile
 
-       echo "release $f"
-       $LFS hsm_release $f || error "could not release file"
-       echo "verify released state: "
-       check_hsm_flags $f "0x0000000d" && echo "pass"
+       test_1bde_base $comp_file
 
-       echo "restore $f"
-       $LFS hsm_restore $f || error "could not restore file"
-       wait_request_state $fid RESTORE SUCCEED
-       echo "verify restored state: "
-       check_hsm_flags $f "0x00000009" && echo "pass"
+       local flg_opts="--comp-start 0 -E 64M --comp-flags init"
+       local found=$($LFS find $flg_opts $comp_file | wc -l)
+       [ $found -eq 1 ] || error "1st component not found"
+
+       flg_opts="--comp-start 64M -E 1G --comp-flags extension"
+       found=$($LFS find $flg_opts $comp_file | wc -l)
+       [ $found -eq 1 ] || error "2nd component not found"
+
+       flg_opts="--comp-start 1G -E 1G --comp-flags ^init"
+       found=$($LFS find $flg_opts $comp_file | wc -l)
+       [ $found -eq 1 ] || error "3rd component not found"
+
+       flg_opts="--comp-start 1G -E 10G --comp-flags extension"
+       found=$($LFS find $flg_opts $comp_file | wc -l)
+       [ $found -eq 1 ] || error "4th component not found"
+
+       flg_opts="--comp-start 10G -E 10G --comp-flags ^init"
+       found=$($LFS find $flg_opts $comp_file | wc -l)
+       [ $found -eq 1 ] || error "5th component not found"
+
+       flg_opts="--comp-start 10G -E EOF --comp-flags extension"
+       found=$($LFS find $flg_opts $comp_file | wc -l)
+       [ $found -eq 1 ] || error "6th component not found"
+
+       sel_layout_sanity $comp_file 6
 }
-run_test 1d "Archive, Release and Restore DoM file"
+run_test 1e "Archive, Release and Restore SEL file"
 
 test_2() {
        local f=$DIR/$tdir/$tfile
@@ -1284,7 +963,7 @@ run_test 9a "Multiple remote agents"
 
 test_10a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir/d1
        local f=$DIR/$tdir/$tfile
@@ -1293,10 +972,13 @@ test_10a() {
                error "hsm_archive failed"
        wait_request_state $fid ARCHIVE SUCCEED
 
-       local AFILE=$(do_facet $SINGLEAGT ls $HSM_ARCHIVE'/*/*/*/*/*/*/'$fid) ||
-               error "fid $fid not in archive $HSM_ARCHIVE"
+       local hsm_root="$(copytool_device $SINGLEAGT)"
+       local archive="$(do_facet $SINGLEAGT \
+                        find "$hsm_root" -name "$fid" -print0)"
+       [ -n "$archive" ] || error "fid '$fid' not in archive '$hsm_root'"
+
        echo "Verifying content"
-       do_facet $SINGLEAGT diff $f $AFILE || error "archived file differs"
+       do_facet $SINGLEAGT diff $f $archive || error "archived file differs"
        echo "Verifying hsm state "
        check_hsm_flags $f "0x00000009"
 
@@ -1304,8 +986,6 @@ test_10a() {
        local st=$(get_hsm_archive_id $f)
        [[ $st == $HSM_ARCHIVE_NUMBER ]] ||
                error "Wrong archive number, $st != $HSM_ARCHIVE_NUMBER"
-
-       copytool_cleanup
 }
 run_test 10a "Archive a file"
 
@@ -1361,30 +1041,30 @@ test_11a() {
        copy2archive /etc/hosts $tdir/$tfile
        local f=$DIR/$tdir/$tfile
 
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        echo -n "Verifying released state: "
        check_hsm_flags $f "0x0000000d"
 
        local LSZ=$(stat -c "%s" $f)
-       local ASZ=$(do_facet $SINGLEAGT stat -c "%s" $HSM_ARCHIVE/$tdir/$tfile)
+       local ASZ=$(do_facet $SINGLEAGT stat -c "%s" "$(hsm_root)/$tdir/$tfile")
 
        echo "Verifying imported size $LSZ=$ASZ"
        [[ $LSZ -eq $ASZ ]] || error "Incorrect size $LSZ != $ASZ"
        echo -n "Verifying released pattern: "
-       local PTRN=$($GETSTRIPE -L $f)
+       local PTRN=$($LFS getstripe -L $f)
        echo $PTRN
        [[ $PTRN == released ]] || error "Is not released"
        local fid=$(path2fid $f)
        echo "Verifying new fid $fid in archive"
 
-       local AFILE=$(do_facet $SINGLEAGT ls $HSM_ARCHIVE'/*/*/*/*/*/*/'$fid) ||
-               error "fid $fid not in archive $HSM_ARCHIVE"
+       do_facet $SINGLEAGT "[ -f \"$(fid2archive "$fid")\" ]" ||
+               error "No archive for fid $fid"
 }
 run_test 11a "Import a file"
 
 test_11b() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        local f=$DIR/$tdir/$tfile
@@ -1396,25 +1076,23 @@ test_11b() {
        local FILE_HASH=$(md5sum $f)
        rm -f $f
 
-       import_file $fid $f
+       copytool import $fid $f
 
        echo "$FILE_HASH" | md5sum -c
 
        [[ $? -eq 0 ]] || error "Restored file differs"
-
-       copytool_cleanup
 }
 run_test 11b "Import a deleted file using its FID"
 
 test_12a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        copy2archive /etc/hosts $tdir/$tfile
 
        local f=$DIR/$tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        local f2=$DIR2/$tdir/$tfile
        echo "Verifying released state: "
        check_hsm_flags $f2 "0x0000000d"
@@ -1426,23 +1104,21 @@ test_12a() {
        echo "Verifying file state: "
        check_hsm_flags $f2 "0x00000009"
 
-       do_facet $SINGLEAGT diff -q $HSM_ARCHIVE/$tdir/$tfile $f
+       do_facet $SINGLEAGT diff -q $(hsm_root)/$tdir/$tfile $f
 
        [[ $? -eq 0 ]] || error "Restored file differs"
-
-       copytool_cleanup
 }
 run_test 12a "Restore an imported file explicitly"
 
 test_12b() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        copy2archive /etc/hosts $tdir/$tfile
 
        local f=$DIR/$tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        echo "Verifying released state: "
        check_hsm_flags $f "0x0000000d"
 
@@ -1451,11 +1127,9 @@ test_12b() {
        echo "Verifying file state after restore: "
        check_hsm_flags $f "0x00000009"
 
-       do_facet $SINGLEAGT diff -q $HSM_ARCHIVE/$tdir/$tfile $f
+       do_facet $SINGLEAGT diff -q $(hsm_root)/$tdir/$tfile $f
 
        [[ $? -eq 0 ]] || error "Restored file differs"
-
-       copytool_cleanup
 }
 run_test 12b "Restore an imported file implicitly"
 
@@ -1468,9 +1142,7 @@ test_12c() {
        local f=$DIR/$tdir/$tfile
        mkdir -p $DIR/$tdir
        $LFS setstripe -c 2 "$f"
-       local fid
-       fid=$(make_custom_file_for_progress $f 5)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_file "$f" 1M 5)
 
        local FILE_CRC=$(md5sum $f)
 
@@ -1512,7 +1184,7 @@ test_12e() {
        # test needs a running copytool
        copytool setup
 
-       mkdir -p $DIR/$tdir $HSM_ARCHIVE/$tdir
+       mkdir -p $DIR/$tdir
        local f=$DIR/$tdir/$tfile
        local fid=$(copy_file /etc/hosts $f)
        $LFS hsm_archive $f || error "archive request failed"
@@ -1617,20 +1289,18 @@ run_test 12m "Archive/release/implicit restore"
 
 test_12n() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        copy2archive /etc/hosts $tdir/$tfile
 
        local f=$DIR/$tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
 
        do_facet $SINGLEAGT cmp /etc/hosts $f ||
                error "Restored file differs"
 
        $LFS hsm_release $f || error "release of $f failed"
-
-       copytool_cleanup
 }
 run_test 12n "Import/implicit restore/release"
 
@@ -1701,13 +1371,9 @@ test_12p() {
 }
 run_test 12p "implicit restore of a file on copytool mount point"
 
-cleanup_test_12q() {
-               error "cannot umount $MOUNT3 on $SINGLEAGT"
-}
-
 test_12q() {
-       [ $(lustre_version_code $SINGLEMDS) -lt $(version_code 2.7.58) ] &&
-               skip "need MDS version at least 2.7.58" && return 0
+       [ $MDS1_VERSION -lt $(version_code 2.7.58) ] &&
+               skip "need MDS version at least 2.7.58"
 
        stack_trap "zconf_umount \"$(facet_host $SINGLEAGT)\" \"$MOUNT3\"" EXIT
        zconf_mount $(facet_host $SINGLEAGT) $MOUNT3 ||
@@ -1765,46 +1431,40 @@ test_12q() {
 run_test 12q "file attributes are refreshed after restore"
 
 test_13() {
-       # test needs a running copytool
-       copytool_setup
-
-       local ARC_SUBDIR="import.orig"
-       local d=""
-       local f=""
-
-       # populate directory to be imported
-       for d in $(seq 1 10); do
-               local CURR_DIR="$HSM_ARCHIVE/$ARC_SUBDIR/dir.$d"
-               do_facet $SINGLEAGT mkdir -p "$CURR_DIR"
-               for f in $(seq 1 10); do
-                       CURR_FILE="$CURR_DIR/$tfile.$f"
-                       # write file-specific data
-                       do_facet $SINGLEAGT \
-                               "echo d=$d, f=$f, dir=$CURR_DIR, "\
-                                       "file=$CURR_FILE > $CURR_FILE"
+       local -i i j k=0
+       for i in {1..10}; do
+               local archive_dir="$(hsm_root)"/subdir/dir.$i
+
+               do_facet $SINGLEAGT mkdir -p "$archive_dir"
+               for j in {1..10}; do
+                       local archive_file="$archive_dir"/file.$j
+
+                       do_facet $SINGLEAGT "echo $k > \"$archive_dir\"/file.$j"
+                       k+=1
                done
        done
+
        # import to Lustre
-       import_file "$ARC_SUBDIR" $DIR/$tdir
-       # diff lustre content and origin (triggers file restoration)
-       # there must be 10x10 identical files, and no difference
-       local cnt_ok=$(do_facet $SINGLEAGT diff -rs $HSM_ARCHIVE/$ARC_SUBDIR \
-                      $DIR/$tdir/$ARC_SUBDIR | grep identical | wc -l)
-       local cnt_diff=$(do_facet $SINGLEAGT diff -r $HSM_ARCHIVE/$ARC_SUBDIR \
-                        $DIR/$tdir/$ARC_SUBDIR | wc -l)
+       copytool import "subdir" "$DIR/$tdir"
 
-       [ $cnt_diff -eq 0 ] ||
-               error "$cnt_diff imported files differ from read data"
-       [ $cnt_ok -eq 100 ] ||
-               error "not enough identical files ($cnt_ok != 100)"
+       # To check the import, the test uses diff with the -r flag
+       # This is nice, but diff only checks files one by one, and triggering
+       # an implicit restore for one file at a time will consume as many
+       # seconds as there are files to compare. To speed this up, a restore
+       # operation is triggered manually first.
+       copytool setup
+       find "$DIR/$tdir"/subdir -type f -exec $LFS hsm_restore {} \;
 
-       copytool_cleanup
+       # Compare the imported data
+       do_facet $SINGLEAGT \
+               diff -r "$(hsm_root)"/subdir "$DIR/$tdir"/subdir ||
+               error "imported files differ from archived data"
 }
 run_test 13 "Recursively import and restore a directory"
 
 test_14() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        # archive a file
        local f=$DIR/$tdir/$tfile
@@ -1821,22 +1481,18 @@ test_14() {
        # rebind the archive to the newly created file
        echo "rebind $fid to $fid2"
 
-       do_facet $SINGLEAGT \
-               "$HSMTOOL --archive $HSM_ARCHIVE_NUMBER --hsm-root $HSM_ARCHIVE\
-                --rebind $fid $fid2 $DIR" || error "could not rebind file"
+       copytool rebind $fid $fid2
 
        # restore file and compare md5sum
        local sum2=$(md5sum $f | awk '{print $1}')
 
        [[ $sum == $sum2 ]] || error "md5sum mismatch after restore"
-
-       copytool_cleanup
 }
 run_test 14 "Rebind archived file to a new fid"
 
 test_15() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        # archive files
        local f=$DIR/$tdir/$tfile
@@ -1852,6 +1508,7 @@ test_15() {
        done
        wait_all_done $(($count*60))
 
+       stack_trap "rm -f $tmpfile" EXIT
        :>$tmpfile
        # delete the files
        for i in $(seq 1 $count); do
@@ -1868,9 +1525,7 @@ test_15() {
        [[ $nl == $count ]] || error "$nl files in list, $count expected"
 
        echo "rebind list of files"
-       do_facet $SINGLEAGT \
-               "$HSMTOOL --archive $HSM_ARCHIVE_NUMBER --hsm-root $HSM_ARCHIVE\
-                --rebind $tmpfile $DIR" || error "could not rebind file list"
+       copytool rebind "$tmpfile"
 
        # restore files and compare md5sum
        for i in $(seq 1 $count); do
@@ -1878,9 +1533,6 @@ test_15() {
                [[ $sum2 == ${sums[$i]} ]] ||
                    error "md5sum mismatch after restore ($sum2 != ${sums[$i]})"
        done
-
-       rm -f $tmpfile
-       copytool_cleanup
 }
 run_test 15 "Rebind a list of files"
 
@@ -2360,16 +2012,43 @@ test_24f() {
 }
 run_test 24f "root can archive, release, and restore tar files"
 
+test_24g() {
+       [ $MDS1_VERSION -lt $(version_code 2.11.56) ] &&
+               skip "need MDS version 2.11.56 or later"
+
+       local file=$DIR/$tdir/$tfile
+       local fid
+
+       echo "RUNAS = '$RUNAS'"
+
+       copytool setup
+
+       mkdir -p $DIR/$tdir
+       chmod ugo+rwx $DIR/$tdir
+
+       echo "Please listen carefully as our options have changed." | tee $file
+       fid=$(path2fid $file)
+       chmod ugo+rw $file
+
+       $LFS hsm_archive $file
+       wait_request_state $fid ARCHIVE SUCCEED
+       check_hsm_flags $file 0x00000009 # exists archived
+
+       echo "To be electrocuted by your telephone, press #." | $RUNAS tee $file
+       check_hsm_flags $file 0x0000000b # exists dirty archived
+}
+run_test 24g "write by non-owner still sets dirty" # LU-11369
+
 test_25a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        copy2archive /etc/hosts $tdir/$tfile
 
        local f=$DIR/$tdir/$tfile
 
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
 
        $LFS hsm_set --lost $f
 
@@ -2377,8 +2056,6 @@ test_25a() {
        local st=$?
 
        [[ $st == 1 ]] || error "lost file access should failed (returns $st)"
-
-       copytool_cleanup
 }
 run_test 25a "Restore lost file (HS_LOST flag) from import"\
             " (Operation not permitted)"
@@ -2410,9 +2087,7 @@ test_26A() { # was test_26
        copytool setup
 
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
@@ -2596,18 +2271,16 @@ run_test 26d "RAoLU when Client eviction"
 
 test_27a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        create_archive_file $tdir/$tfile
        local f=$DIR/$tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        local fid=$(path2fid $f)
 
        $LFS hsm_remove $f
 
        [[ $? != 0 ]] || error "Remove of a released file should fail"
-
-       copytool_cleanup
 }
 run_test 27a "Remove the archive of an imported file (Operation not permitted)"
 
@@ -2616,9 +2289,7 @@ test_27b() {
        copytool setup
 
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
@@ -2635,9 +2306,7 @@ test_28() {
        copytool setup
 
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
@@ -2785,29 +2454,25 @@ test_30a() {
        needclients 2 || return 0
 
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        copy2archive /bin/true $tdir/$tfile
 
        local f=$DIR/$tdir/true
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
 
        local fid=$(path2fid $f)
 
+       stack_trap "cdt_clear_no_retry" EXIT
        # set no retry action mode
        cdt_set_no_retry
        do_node $CLIENT2 $f
        local st=$?
 
-       # cleanup
-       # remove no try action mode
-       cdt_clear_no_retry
        $LFS hsm_state $f
 
        [[ $st == 0 ]] || error "Failed to exec a released file"
-
-       copytool_cleanup
 }
 run_test 30a "Restore at exec (import case)"
 
@@ -2914,20 +2579,18 @@ restore_and_check_size() {
 
 test_31a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        create_archive_file $tdir/$tfile
        local f=$DIR/$tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        local fid=$($LFS path2fid $f)
-       HSM_ARCHIVE_PURGE=false copytool_setup
+       copytool setup
 
        restore_and_check_size $f $fid
        local err=$?
 
        [[ $err -eq 0 ]] || error "File size changed during restore"
-
-       copytool_cleanup
 }
 run_test 31a "Import a large file and check size during restore"
 
@@ -2937,9 +2600,7 @@ test_31b() {
        copytool setup
 
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_file "$f" 1MB 39)
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
@@ -2957,9 +2618,7 @@ test_31c() {
        copytool setup
 
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 33 1048576)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_file "$f" 1M 39)
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
@@ -2973,63 +2632,30 @@ test_31c() {
 run_test 31c "Restore a large aligned file and check size during restore"
 
 test_33() {
-       # test needs a running copytool
-       copytool setup -b 1
-
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
+
+       copytool setup
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
        $LFS hsm_release $f
 
-       # to be sure wait_all_done will not be mislead by previous tests
-       # and ops.
-       cdt_purge
-       wait_for_grace_delay
-       # Also raise grace_delay significantly so the Canceled
-       # Restore action will stay enough long avail.
-       local old_grace=$(get_hsm_param grace_delay)
-       stack_trap "set_hsm_param grace_delay $old_grace" EXIT
-       set_hsm_param grace_delay 100
+       # Prevent restore from completing
+       copytool_suspend
 
+       # Implicit restore
        md5sum $f >/dev/null &
        local pid=$!
-       wait_request_state $fid RESTORE STARTED
 
+       wait_request_state $fid RESTORE STARTED
        kill -15 $pid
-       sleep 1
 
-       # Check restore trigger process was killed
-       local killed=$(ps -o pid,comm hp $pid >/dev/null)
+       copytool_continue
 
-       $LFS hsm_cancel $f
-
-       # instead of waiting+checking both Restore and Cancel ops
-       # sequentially, wait for both to be finished and then check
-       # each results.
-       wait_all_done 100 $fid
-       local rstate=$(get_request_state $fid RESTORE)
-       local cstate=$(get_request_state $fid CANCEL)
-
-       if [[ "$rstate" == "CANCELED" ]] ; then
-               [[ "$cstate" == "SUCCEED" ]] ||
-                       error "Restore state is CANCELED and Cancel state " \
-                              "is not SUCCEED but $cstate"
-               echo "Restore state is CANCELED, Cancel state is SUCCEED"
-       elif [[ "$rstate" == "SUCCEED" ]] ; then
-               [[ "$cstate" == "FAILED" ]] ||
-                       error "Restore state is SUCCEED and Cancel state " \
-                               "is not FAILED but $cstate"
-               echo "Restore state is SUCCEED, Cancel state is FAILED"
-       else
-               error "Restore state is $rstate and Cancel state is $cstate"
-       fi
-
-       [ -z $killed ] ||
-               error "Cannot kill process waiting for restore ($killed)"
+       # Check restore trigger process was killed
+       wait $pid
+       [ $? -eq 143 ] || error "md5sum was not 'Terminated'"
 }
 run_test 33 "Kill a restore waiting process"
 
@@ -3038,28 +2664,32 @@ test_34() {
        copytool setup -b 1
 
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
        $LFS hsm_release $f
 
+       # Prevent restore from completing
+       copytool_suspend
+
        md5sum $f >/dev/null &
        local pid=$!
+
        wait_request_state $fid RESTORE STARTED
 
-       rm $f || error "rm $f failed"
        # rm must not block during restore
-       wait_request_state $fid RESTORE STARTED
+       timeout --signal=KILL 1 rm "$f" || error "rm $f failed"
 
+       copytool_continue
        wait_request_state $fid RESTORE SUCCEED
-       # check md5sum pgm finished
-       local there=$(ps -o pid,comm hp $pid >/dev/null)
-       [[ -z $there ]] || error "Restore initiator does not exit"
 
+       # Check md5sum pgm finished
+       kill -0 $pid && error "Restore initiatior still running"
        wait $pid || error "Restore initiator failed with $?"
+
+       # Check the file was actually deleted
+       [ ! -f "$f" ] || error "$f was not deleted"
 }
 run_test 34 "Remove file during restore"
 
@@ -3069,31 +2699,32 @@ test_35() {
 
        local f=$DIR/$tdir/$tfile
        local f1=$DIR/$tdir/$tfile-1
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
-
+       local fid=$(create_empty_file "$f")
        local fid1=$(copy_file /etc/passwd $f1)
+
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
        $LFS hsm_release $f
 
+       # Prevent restore from completing
+       copytool_suspend
+
        md5sum $f >/dev/null &
        local pid=$!
+
        wait_request_state $fid RESTORE STARTED
 
-       mv $f1 $f || error "mv $f1 $f failed"
        # mv must not block during restore
-       wait_request_state $fid RESTORE STARTED
+       timeout --signal=KILL 1 mv "$f1" "$f" || error "mv $f1 $f failed"
 
+       copytool_continue
        wait_request_state $fid RESTORE SUCCEED
-       # check md5sum pgm finished
-       local there=$(ps -o pid,comm hp $pid >/dev/null)
-       [[ -z $there ]] || error "Restore initiator does not exit"
 
+       # Check md5sum pgm finished
+       kill -0 $pid && error "Restore initiatior still running"
        wait $pid || error "Restore initiator failed with $?"
 
-       fid2=$(path2fid $f)
+       local fid2=$(path2fid $f)
        [[ $fid2 == $fid1 ]] || error "Wrong fid after mv $fid2 != $fid1"
 }
 run_test 35 "Overwrite file during restore"
@@ -3103,28 +2734,29 @@ test_36() {
        copytool setup -b 1
 
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
        $LFS hsm_release $f
 
+       # Prevent restore from completing
+       copytool_suspend
+
        md5sum $f >/dev/null &
        local pid=$!
-       wait_request_state $fid RESTORE STARTED
 
-       mv $f $f.new
-       # rm must not block during restore
        wait_request_state $fid RESTORE STARTED
 
+       # mv must not block during restore
+       timeout --signal=KILL 10 mv "$f" "$f.new" ||
+               error "mv '$f' '$f.new' failed with rc=$?"
+
+       copytool_continue
        wait_request_state $fid RESTORE SUCCEED
-       # check md5sum pgm finished
-       local there=$(ps -o pid,comm hp $pid >/dev/null)
-       [[ -z $there ]] ||
-               error "Restore initiator does not exit"
 
+       # Check md5sum pgm finished
+       kill -0 $pid && error "Restore initiator is still running"
        wait $pid || error "Restore initiator failed with $?"
 }
 run_test 36 "Move file during restore"
@@ -3259,11 +2891,10 @@ test_53() {
 run_test 53 "Opened for read file on an evicted client should not be set dirty"
 
 test_54() {
-       # test needs a running copytool
-       copytool setup -b 1
-
        local f=$DIR/$tdir/$tfile
-       local fid=$(make_custom_file_for_progress $f 39 1000000)
+       local fid=$(create_file "$f" 1MB 39)
+
+       copytool setup -b 1
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f ||
                error "could not archive file"
@@ -3284,11 +2915,10 @@ test_54() {
 run_test 54 "Write during an archive cancels it"
 
 test_55() {
-       # test needs a running copytool
-       copytool setup -b 1
-
        local f=$DIR/$tdir/$tfile
-       local fid=$(make_custom_file_for_progress $f 39 1000000)
+       local fid=$(create_file "$f" 1MB 39)
+
+       copytool setup -b 1
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f ||
                error "could not archive file"
@@ -3309,13 +2939,10 @@ test_55() {
 run_test 55 "Truncate during an archive cancels it"
 
 test_56() {
-       # test needs a running copytool
-       copytool setup -b 1
-
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_file "$f" 1MB 39)
+
+       copytool setup -b 1
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f ||
                error "could not archive file"
@@ -3420,9 +3047,8 @@ run_test 58 "Truncate a released file will trigger restore"
 
 test_59() {
        local fid
-       local server_version=$(lustre_version_code $SINGLEMDS)
-       [[ $server_version -lt $(version_code 2.7.63) ]] &&
-               skip "Need MDS version at least 2.7.63" && return
+       [[ $MDS1_VERSION -lt $(version_code 2.7.63) ]] &&
+               skip "Need MDS version at least 2.7.63"
 
        copytool setup
        $MCREATE $DIR/$tfile || error "mcreate failed"
@@ -3438,16 +3064,13 @@ test_60() {
        # This test validates the fix for LU-4512. Ensure that the -u
        # option changes the progress reporting interval from the
        # default (30 seconds) to the user-specified interval.
+       local f=$DIR/$tdir/$tfile
+       local fid=$(create_file "$f" 1M 10)
+
        local interval=5
        local progress_timeout=$((interval * 4))
-
        copytool setup -b 1 --update-interval $interval
 
-       local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 10)
-       [ $? != 0 ] && skip "not enough free space" && return
-
        local mdtidx=0
        local mdt=${MDT_PREFIX}${mdtidx}
        local mds=mds$((mdtidx + 1))
@@ -3576,9 +3199,7 @@ test_71() {
        cdt_clear_no_retry
 
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_small_file "$f")
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f ||
                error "could not archive file"
@@ -3587,8 +3208,11 @@ test_71() {
        local expected_fields="event_time data_fid source_fid"
        expected_fields+=" total_bytes current_bytes"
 
-       local START_EVENT
-       local FINISH_EVENT
+       local -A events=(
+               [ARCHIVE_START]=false
+               [ARCHIVE_FINISH]=false
+               [ARCHIVE_RUNNING]=false
+               )
        while read event; do
                # Make sure we're not getting anything from previous events.
                for field in $expected_fields; do
@@ -3601,15 +3225,9 @@ test_71() {
                fi
                eval $parsed
 
-               if [ $event_type == "ARCHIVE_START" ]; then
-                       START_EVENT=$event
-                       continue
-               elif [ $event_type == "ARCHIVE_FINISH" ]; then
-                       FINISH_EVENT=$event
-                       continue
-               elif [ $event_type != "ARCHIVE_RUNNING" ]; then
-                       continue
-               fi
+               events["$event_type"]=true
+
+               [ "$event_type" != ARCHIVE_RUNNING ] && continue
 
                # Do some simple checking of the progress update events.
                for expected_field in $expected_fields; do
@@ -3618,24 +3236,18 @@ test_71() {
                        fi
                done
 
-               if [ $total_bytes -eq 0 ]; then
-                       error "Expected total_bytes to be > 0"
-               fi
+               [ $total_bytes -gt 0 ] || error "Expected total_bytes to be > 0"
 
-               # These should be identical throughout an archive
-               # operation.
-               if [ $source_fid != $data_fid ]; then
+               # These should be identical throughout an archive operation
+               [ $source_fid == $data_fid ] ||
                        error "Expected source_fid to equal data_fid"
-               fi
        done < <(echo $"$(get_copytool_event_log)")
 
-       if [ -z "$START_EVENT" ]; then
-               error "Copytool failed to send archive start event to FIFO"
-       fi
-
-       if [ -z "$FINISH_EVENT" ]; then
-               error "Copytool failed to send archive finish event to FIFO"
-       fi
+       # Check we received every type of events we were expecting
+       for event in "${!events[@]}"; do
+               ${events["$event"]} ||
+                       error "Copytool failed to send '$event' event to FIFO"
+       done
 
        echo "Archive events look OK."
 }
@@ -3646,10 +3258,10 @@ test_72() {
        local interval=5
 
        # test needs a new running copytool
-       copytool_cleanup
+       stack_trap copytool_monitor_cleanup EXIT
        copytool_monitor_setup
-       HSMTOOL_UPDATE_INTERVAL=$interval \
-       HSMTOOL_EVENT_FIFO=$HSMTOOL_MONITOR_DIR/fifo copytool_setup
+       copytool setup --update-interval $interval --event-fifo \
+               "$HSMTOOL_MONITOR_DIR/fifo"
        local test_file=$HSMTOOL_MONITOR_DIR/file
 
        local cmd="dd if=/dev/urandom of=$test_file count=16 bs=1000000 "
@@ -3660,7 +3272,7 @@ test_72() {
 
        mkdir -p $DIR/$tdir
        local f=$DIR/$tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        f=$DIR2/$tdir/$tfile
        echo "Verifying released state: "
        check_hsm_flags $f "0x0000000d"
@@ -3735,12 +3347,6 @@ test_72() {
        fi
 
        echo "Restore events look OK."
-
-       cdt_clear_no_retry
-       copytool_cleanup
-       copytool_monitor_cleanup
-
-       rm -rf $test_dir
 }
 run_test 72 "Copytool logs JSON restore events to FIFO"
 
@@ -3837,9 +3443,7 @@ DATA=CEA
 DATAHEX='[434541]'
 test_104() {
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER --data $DATA $f
        local data1=$(do_facet $SINGLEMDS "$LCTL get_param -n\
@@ -3849,10 +3453,7 @@ test_104() {
        [[ "$data1" == "$DATAHEX" ]] ||
                error "Data field in records is ($data1) and not ($DATAHEX)"
 
-       # archive the file
-       copytool setup
-
-       wait_request_state $fid ARCHIVE SUCCEED
+       cdt_purge
 }
 run_test 104 "Copy tool data field"
 
@@ -3966,14 +3567,14 @@ run_test 109 "Policy display/change"
 
 test_110a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
 
        copy2archive /etc/passwd $tdir/$tfile
 
        local f=$DIR/$tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        local fid=$(path2fid $f)
 
        cdt_set_non_blocking_restore
@@ -4019,14 +3620,14 @@ run_test 110b "Non blocking restore policy (release case)"
 
 test_111a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        copy2archive /etc/passwd $tdir/$tfile
 
        local f=$DIR/$tdir/$tfile
 
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        local fid=$(path2fid $f)
 
        cdt_set_no_retry
@@ -4042,19 +3643,18 @@ test_111a() {
 
        # Test result
        [[ $st == 0 ]] || error "Restore does not failed"
-
-       copytool_cleanup
 }
 run_test 111a "No retry policy (import case), restore will error"\
              " (No such file or directory)"
 
 test_111b() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        local f=$DIR/$tdir/$tfile
        local fid=$(copy_file /etc/passwd $f)
+       stack_trap cdt_clear_no_retry EXIT
        cdt_set_no_retry
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
@@ -4066,13 +3666,8 @@ test_111b() {
        wait_request_state $fid RESTORE FAILED
        local st=$?
 
-       # cleanup
-       cdt_clear_no_retry
-
        # Test result
        [[ $st == 0 ]] || error "Restore does not failed"
-
-       copytool_cleanup
 }
 run_test 111b "No retry policy (release case), restore will error"\
              " (No such file or directory)"
@@ -4098,34 +3693,72 @@ test_112() {
 }
 run_test 112 "State of recorded request"
 
+test_113() {
+       local file1=$DIR/$tdir/$tfile
+       local file2=$DIR2/$tdir/$tfile
+
+       local fid=$(create_small_sync_file $file1)
+
+       stack_trap "zconf_umount \"$(facet_host $SINGLEAGT)\" \"$MOUNT3\"" EXIT
+       zconf_mount "$(facet_host $SINGLEAGT)" "$MOUNT3" ||
+               error "cannot mount '$MOUNT3' on '$SINGLEAGT'"
+
+       copytool setup -m  "$MOUNT3"
+
+       do_nodes $(comma_list $(nodes_list)) $LCTL clear
+
+       $LFS hsm_archive $file1 || error "Fail to archive $file1"
+       wait_request_state $fid ARCHIVE SUCCEED
+
+       $LFS hsm_release $file1
+       echo "Verifying released state: "
+       check_hsm_flags $file1 "0x0000000d"
+
+       multiop_bg_pause $file1 oO_WRONLY:O_APPEND:_w4c || error "multiop failed"
+       MULTIPID=$!
+       stat $file2 &
+       kill -USR1 $MULTIPID
+
+       wait
+       sync
+
+       local size1=$(stat -c "%s" $file1)
+       local size2=$(stat -c "%s" $file2)
+
+       [ $size1 -eq $size2 ] || error "sizes are different $size1 $size2"
+}
+run_test 113 "wrong stat after restore"
+
 test_200() {
-       # test needs a running copytool
+       local f=$DIR/$tdir/$tfile
+       local fid=$(create_empty_file "$f")
+
        copytool setup
 
-       local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 103 1048576)
-       [ $? != 0 ] && skip "not enough free space" && return
+       # Prevent archive from completing
+       copytool_suspend
 
-       # test with cdt on is made in test_221
-       cdt_disable
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        # wait archive to register at CDT
-       wait_request_state $fid ARCHIVE WAITING
-       $LFS hsm_cancel $f
-       cdt_enable
+       wait_request_state $fid ARCHIVE STARTED
+
+       # Cancel the archive
+       $LFS hsm_cancel "$f"
+
        wait_request_state $fid ARCHIVE CANCELED
+
+       copytool_continue
        wait_request_state $fid CANCEL SUCCEED
 }
 run_test 200 "Register/Cancel archive"
 
 test_201() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        local f=$DIR/$tdir/$tfile
        create_archive_file $tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        local fid=$(path2fid $f)
 
        # test with cdt on is made in test_222
@@ -4137,29 +3770,25 @@ test_201() {
        cdt_enable
        wait_request_state $fid RESTORE CANCELED
        wait_request_state $fid CANCEL SUCCEED
-
-       copytool_cleanup
 }
 run_test 201 "Register/Cancel restore"
 
 test_202() {
+       local f=$DIR/$tdir/$tfile
+       local fid=$(create_empty_file "$f")
+
        # test needs a running copytool
        copytool setup
 
-       local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
-
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
 
-       cdt_disable
+       copytool_suspend
        $LFS hsm_remove $f
        # wait remove to register at CDT
-       wait_request_state $fid REMOVE WAITING
+       wait_request_state $fid REMOVE STARTED
        $LFS hsm_cancel $f
-       cdt_enable
+
        wait_request_state $fid REMOVE CANCELED
 }
 run_test 202 "Register/Cancel remove"
@@ -4216,20 +3845,21 @@ test_220a() {
 run_test 220a "Changelog for failed archive"
 
 test_221() {
-       # test needs a running copytool
-       copytool setup -b 1
-
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 103 1048576)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
+       copytool setup -b 1
        changelog_register
 
+       # Prevent archive from completing
+       copytool_suspend
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE STARTED
+
        $LFS hsm_cancel $f
        wait_request_state $fid ARCHIVE CANCELED
+
+       copytool_continue
        wait_request_state $fid CANCEL SUCCEED
 
        changelog_find -type HSM -target-fid $fid -flags 0x7d ||
@@ -4239,13 +3869,13 @@ run_test 221 "Changelog for archive canceled"
 
 test_222a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        copy2archive /etc/passwd $tdir/$tfile
 
        local f=$DIR/$tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        local fid=$(path2fid $f)
 
        changelog_register
@@ -4255,8 +3885,6 @@ test_222a() {
 
        changelog_find -type HSM -target-fid $fid -flags 0x80 ||
                error "The expected changelog was not emitted"
-
-       copytool_cleanup
 }
 run_test 222a "Changelog for explicit restore"
 
@@ -4284,13 +3912,13 @@ run_test 222b "Changelog for implicit restore"
 
 test_222c() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        copy2archive /etc/passwd $tdir/$tfile
 
        local f=$DIR/$tdir/$tfile
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        local fid=$(path2fid $f)
 
        changelog_register
@@ -4313,14 +3941,12 @@ test_222c() {
        # HE_RESTORE|ENOENT
        changelog_find -type HSM -target-fid $fid -flags 0x82 ||
                error "The expected changelog was not emitted"
-
-       copytool_cleanup
 }
 run_test 222c "Changelog for failed explicit restore"
 
 test_222d() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
        local f=$DIR/$tdir/$tfile
@@ -4339,21 +3965,19 @@ test_222d() {
        # HE_RESTORE|ENOENT
        changelog_find -type HSM -target-fid $fid -flags 0x82 ||
                error "The expected changelog was not emitted"
-
-       copytool_cleanup
 }
 run_test 222d "Changelog for failed implicit restore"
 
 test_223a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup -b 1
 
        local f=$DIR/$tdir/$tfile
        create_archive_file $tdir/$tfile
 
        changelog_register
 
-       import_file $tdir/$tfile $f
+       copytool import $tdir/$tfile $f
        local fid=$(path2fid $f)
 
        $LFS hsm_restore $f
@@ -4364,34 +3988,33 @@ test_223a() {
 
        changelog_find -type HSM -target-fid $fid -flags 0xfd ||
                error "The expected changelog was not emitted"
-
-       cleanup
 }
 run_test 223a "Changelog for restore canceled (import case)"
 
 test_223b() {
-       # test needs a running copytool
-       copytool setup -b 1
-
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
+       copytool setup -b 1
        changelog_register
+
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
        $LFS hsm_release $f
+
+       # Prevent restore from completing
+       copytool_suspend
        $LFS hsm_restore $f
        wait_request_state $fid RESTORE STARTED
+
        $LFS hsm_cancel $f
        wait_request_state $fid RESTORE CANCELED
+
+       copytool_continue
        wait_request_state $fid CANCEL SUCCEED
 
        changelog_find -type HSM -target-fid $fid -flags 0xfd ||
                error "The expected changelog was not emitted"
-
-       copytool_cleanup
 }
 run_test 223b "Changelog for restore canceled (release case)"
 
@@ -4418,7 +4041,7 @@ run_test 224A "Changelog for remove"
 
 test_224a() {
        # test needs a running copytool
-       copytool_setup
+       copytool setup
 
        mkdir -p $DIR/$tdir
 
@@ -4449,36 +4072,33 @@ test_224a() {
        # HE_REMOVE|ENOENT=0x202
        changelog_find -type HSM -target-fid $fid -flags 0x202 ||
                error "The expected changelog was not emitted"
-
-       cleanup
 }
 run_test 224a "Changelog for failed remove"
 
 test_225() {
-       # test needs a running copytool
-       copytool setup
-
        # test is not usable because remove request is too fast
        # so it is always finished before cancel can be done ...
        echo "Test disabled"
-       copytool_cleanup
        return 0
 
+       # test needs a running copytool
+       copytool setup
+
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
        changelog_register
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
 
-       # if cdt is on, it can serve too quickly the request
-       cdt_disable
+       # Prevent restore from completing
+       copytool_suspend
        $LFS hsm_remove $f
+
        $LFS hsm_cancel $f
-       cdt_enable
        wait_request_state $fid REMOVE CANCELED
+
+       copytool_continue
        wait_request_state $fid CANCEL SUCCEED
 
        changelog_find -type HSM -target-fid $fid -flags 0x27d
@@ -4598,67 +4218,88 @@ test_228() {
 run_test 228 "On released file, return extend to FIEMAP. For [cp,tar] --sparse"
 
 test_250() {
-       # test needs a running copytool
-       copytool setup
+       local file="$DIR/$tdir/$tfile"
 
-       mkdir -p $DIR/$tdir
-       local maxrequest=$(get_hsm_param max_requests)
-       local rqcnt=$(($maxrequest * 3))
-       local i=""
+       # set max_requests to allow one request of each type to be started (3)
+       stack_trap \
+               "set_hsm_param max_requests $(get_hsm_param max_requests)" EXIT
+       set_hsm_param max_requests 3
+       # speed up test
+       stack_trap \
+               "set_hsm_param loop_period $(get_hsm_param loop_period)" EXIT
+       set_hsm_param loop_period 1
 
-       cdt_disable
-       for i in $(seq -w 1 $rqcnt); do
-               rm -f $DIR/$tdir/$i
-               dd if=/dev/urandom of=$DIR/$tdir/$i bs=1M count=10 conv=fsync
+       # send 1 requests of each kind twice
+       copytool setup
+       # setup the files
+       for action in archive restore remove; do
+               local filepath="$file"-to-$action
+               local fid=$(create_empty_file "$filepath")
+               local fid2=$(create_empty_file "$filepath".bis)
+
+               if [ "$action" != archive ]; then
+                       "$LFS" hsm_archive "$filepath"
+                       wait_request_state $fid ARCHIVE SUCCEED
+                       "$LFS" hsm_archive "$filepath".bis
+                       wait_request_state $fid2 ARCHIVE SUCCEED
+               fi
+               if [ "$action" == restore ]; then
+                       "$LFS" hsm_release "$filepath"
+                       "$LFS" hsm_release "$filepath".bis
+               fi
        done
-       # we do it in 2 steps, so all requests arrive at the same time
-       for i in $(seq -w 1 $rqcnt); do
-               $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $DIR/$tdir/$i
+
+       # suspend the copytool to prevent requests from completing
+       stack_trap "copytool_continue" EXIT
+       copytool_suspend
+
+       # send `max_requests' requests (one of each kind)
+       for action in archive restore remove; do
+               filepath="$file"-to-$action
+               "$LFS" hsm_${action} "$filepath"
+               wait_request_state $(path2fid "$filepath") "${action^^}" STARTED
        done
-       cdt_enable
-       local cnt=$rqcnt
-       local wt=$rqcnt
-       while [[ $cnt != 0 || $wt != 0 ]]; do
-               sleep 1
-               cnt=$(do_facet $SINGLEMDS "$LCTL get_param -n\
-                       $HSM_PARAM.actions |\
-                       grep STARTED | grep -v CANCEL | wc -l")
-               [[ $cnt -le $maxrequest ]] ||
-                       error "$cnt > $maxrequest too many started requests"
-               wt=$(do_facet $SINGLEMDS "$LCTL get_param\
-                       $HSM_PARAM.actions |\
-                       grep WAITING | wc -l")
-               echo "max=$maxrequest started=$cnt waiting=$wt"
+
+       # send another batch of requests
+       for action in archive restore remove; do
+               "$LFS" hsm_${action} "$file-to-$action".bis
        done
+       # wait for `loop_period' seconds to make sure the coordinator has time
+       # to register those, even though it should not
+       sleep 1
+
+       # only the first batch of request should be started
+       local -i count
+       count=$(do_facet $SINGLEMDS "$LCTL" get_param -n $HSM_PARAM.actions |
+               grep -c STARTED)
+
+       ((count == 3)) ||
+               error "expected 3 STARTED requests, found $count"
 }
 run_test 250 "Coordinator max request"
 
 test_251() {
-       # test needs a running copytool
-       copytool setup -b 1
-
        local f=$DIR/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 103 1048576)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
 
        cdt_disable
        # to have a short test
        local old_to=$(get_hsm_param active_request_timeout)
-       set_hsm_param active_request_timeout 4
+       set_hsm_param active_request_timeout 1
        # to be sure the cdt will wake up frequently so
        # it will be able to cancel the "old" request
        local old_loop=$(get_hsm_param loop_period)
-       set_hsm_param loop_period 2
+       set_hsm_param loop_period 1
        cdt_enable
 
-       # clear locks to avoid extra delay caused by flush/cancel
-       # and thus prevent early copytool death to timeout.
-       cancel_lru_locks osc
+       copytool setup
 
+       # Prevent archive from completing
+       copytool_suspend
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE STARTED
-       sleep 5
+
+       # Let the request timeout
        wait_request_state $fid ARCHIVE CANCELED
 
        set_hsm_param active_request_timeout $old_to
@@ -4667,7 +4308,6 @@ test_251() {
 run_test 251 "Coordinator request timeout"
 
 test_252() {
-       mkdir -p $DIR/$tdir
        local f=$DIR/$tdir/$tfile
        local fid=$(create_empty_file "$f")
 
@@ -4675,12 +4315,13 @@ test_252() {
        stack_trap "set_hsm_param loop_period $(get_hsm_param loop_period)" EXIT
        set_hsm_param loop_period 1
 
-       copytool_setup
-       copytool_suspend
+       copytool setup
 
+       # Prevent archive from completing
+       copytool_suspend
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE STARTED
-       rm -f $f
+       rm -f "$f"
 
        stack_trap "set_hsm_param active_request_timeout \
                    $(get_hsm_param active_request_timeout)" EXIT
@@ -4727,8 +4368,8 @@ run_test 253 "Check for wrong file size after release"
 
 test_254a()
 {
-       [ $(lustre_version_code $SINGLEMDS) -lt $(version_code 2.10.56) ] &&
-               skip "need MDS version at least 2.10.56" && return
+       [ $MDS1_VERSION -lt $(version_code 2.10.56) ] &&
+               skip "need MDS version at least 2.10.56"
 
        # Check that the counters are initialized to 0
        local count
@@ -4745,15 +4386,15 @@ run_test 254a "Request counters are initialized to zero"
 
 test_254b()
 {
-       [ $(lustre_version_code $SINGLEMDS) -lt $(version_code 2.10.56) ] &&
-               skip "need MDS version at least 2.10.56" && return
+       [ $MDS1_VERSION -lt $(version_code 2.10.56) ] &&
+               skip "need MDS version at least 2.10.56"
 
        # The number of request to launch (at least 32)
        local request_count=$((RANDOM % 32 + 32))
        printf "Will launch %i requests of each type\n" "$request_count"
 
        # Launch a copytool to process requests
-       copytool_setup
+       copytool setup
 
        # Set hsm.max_requests to allow starting all requests at the same time
        stack_trap \
@@ -4815,16 +4456,281 @@ test_254b()
 }
 run_test 254b "Request counters are correctly incremented and decremented"
 
-test_300() {
-       [ "$CLIENTONLY" ] && skip "CLIENTONLY mode" && return
+test_255()
+{
+       [ $MDS1_VERSION -lt $(version_code 2.12.0) ] &&
+               skip "Need MDS version at least 2.12.0"
 
-       # the only way to test ondisk conf is to restart MDS ...
-       echo "Stop coordinator and remove coordinator state at mount"
-       # stop coordinator
-       cdt_shutdown
-       # clean on disk conf set by default
-       cdt_clear_mount_state
-       cdt_check_state stopped
+       local file="$DIR/$tdir/$tfile"
+       local fid=$(create_empty_file "$file")
+
+       # How do you make sure the coordinator has consumed any outstanding
+       # event, without triggering an event yourself?
+       #
+       # You wait for a request to disappear from the coordinator's llog.
+
+       # Warning: the setup represents 90% of this test
+
+       # Create and process an HSM request
+       copytool setup
+       "$LFS" hsm_archive "$file"
+       wait_request_state $fid ARCHIVE SUCCEED
+
+       kill_copytools
+       wait_copytools || error "failed to stop copytools"
+
+       # Launch a new HSM request
+       rm "$file"
+       create_empty_file "$file"
+       "$LFS" hsm_archive "$file"
+
+       cdt_shutdown
+
+       # Have the completed request be removed as soon as the cdt wakes up
+       stack_trap "set_hsm_param grace_delay $(get_hsm_param grace_delay)" EXIT
+       set_hsm_param grace_delay 1
+       # (Hopefully, time on the MDS will behave nicely)
+       do_facet $SINGLEMDS sleep 2 &
+
+       # Increase `loop_period' as a mean to prevent the coordinator from
+       # waking itself up to do some housekeeping.
+       stack_trap "set_hsm_param loop_period $(get_hsm_param loop_period)" EXIT
+       set_hsm_param loop_period 1000
+
+       wait $! || error "waiting failed"
+       cdt_enable
+       wait_request_state $fid ARCHIVE ""
+       # The coordinator will not wake up on its own for ~`loop_period' secs...
+
+       # ... Unless a copytool registers. Now the real test begins
+       copytool setup
+       wait_request_state $(path2fid "$file") ARCHIVE SUCCEED
+}
+run_test 255 "Copytool registration wakes the coordinator up"
+
+# tests 260[a-c] rely on the parsing of the copytool's log file, they might
+# break in the future because of that.
+test_260a()
+{
+       [ $MDS1_VERSION -lt $(version_code 2.11.56) ] &&
+               skip "need MDS version 2.11.56 or later"
+
+       local -a files=("$DIR/$tdir/$tfile".{0..15})
+       local file
+
+       for file in "${files[@]}"; do
+               create_small_file "$file"
+       done
+
+       # Set a few hsm parameters
+       stack_trap \
+               "set_hsm_param loop_period $(get_hsm_param loop_period)" EXIT
+       set_hsm_param loop_period 1
+       stack_trap \
+               "set_hsm_param max_requests $(get_hsm_param max_requests)" EXIT
+       set_hsm_param max_requests 3
+
+       # Release one file
+       copytool setup
+       "$LFS" hsm_archive "${files[0]}"
+       wait_request_state "$(path2fid "${files[0]}")" ARCHIVE SUCCEED
+       "$LFS" hsm_release "${files[0]}"
+
+       # Stop the copytool
+       kill_copytools
+       wait_copytools || error "copytools failed to stop"
+
+       # Send several archive requests
+       for file in "${files[@]:1}"; do
+               "$LFS" hsm_archive "$file"
+       done
+
+       # Send one restore request
+       "$LFS" hsm_restore "${files[0]}"
+
+       # Launch a copytool
+       copytool setup
+
+       # Wait for all the requests to complete
+       wait_request_state "$(path2fid "${files[0]}")" RESTORE SUCCEED
+       for file in "${files[@]:1}"; do
+               wait_request_state "$(path2fid "$file")" ARCHIVE SUCCEED
+       done
+
+       # Collect the actions in the order in which the copytool processed them
+       local -a actions=(
+               $(do_facet "$SINGLEAGT" grep -o '\"RESTORE\\|ARCHIVE\"' \
+                       "$(copytool_logfile "$SINGLEAGT")")
+               )
+
+       printf '%s\n' "${actions[@]}"
+
+       local action
+       for action in "${actions[@]:0:3}"; do
+               [ "$action" == RESTORE ] && return
+       done
+
+       error "Too many ARCHIVE requests were run before the RESTORE request"
+}
+run_test 260a "Restore request have priority over other requests"
+
+# This test is very much tied to the implementation of the current priorisation
+# mechanism in the coordinator. It might not make sense to keep it in the future
+test_260b()
+{
+       [ $MDS1_VERSION -lt $(version_code 2.11.56) ] &&
+               skip "need MDS version 2.11.56 or later"
+
+       local -a files=("$DIR/$tdir/$tfile".{0..15})
+       local file
+
+       for file in "${files[@]}"; do
+               create_small_file "$file"
+       done
+
+       # Set a few hsm parameters
+       stack_trap \
+               "set_hsm_param loop_period $(get_hsm_param loop_period)" EXIT
+       set_hsm_param loop_period 1
+       stack_trap \
+               "set_hsm_param max_requests $(get_hsm_param max_requests)" EXIT
+       set_hsm_param max_requests 3
+
+       # Release one file
+       copytool setup --archive-id 2
+       "$LFS" hsm_archive --archive 2 "${files[0]}"
+       wait_request_state "$(path2fid "${files[0]}")" ARCHIVE SUCCEED
+       "$LFS" hsm_release "${files[0]}"
+
+       # Stop the copytool
+       kill_copytools
+       wait_copytools || error "copytools failed to stop"
+
+       # Send several archive requests
+       for file in "${files[@]:1}"; do
+               "$LFS" hsm_archive "$file"
+       done
+
+       # Send one restore request
+       "$LFS" hsm_restore "${files[0]}"
+
+       # Launch a copytool
+       copytool setup
+       copytool setup --archive-id 2
+
+       # Wait for all the requests to complete
+       wait_request_state "$(path2fid "${files[0]}")" RESTORE SUCCEED
+       for file in "${files[@]:1}"; do
+               wait_request_state "$(path2fid "$file")" ARCHIVE SUCCEED
+       done
+
+       # Collect the actions in the order in which the copytool processed them
+       local -a actions=(
+               $(do_facet "$SINGLEAGT" grep -o '\"RESTORE\\|ARCHIVE\"' \
+                       "$(copytool_logfile "$SINGLEAGT")")
+               )
+
+       printf '%s\n' "${actions[@]}"
+
+       local action
+       for action in "${actions[@]:0:3}"; do
+               [ "$action" == RESTORE ] && return
+       done
+
+       error "Too many ARCHIVE requests were run before the RESTORE request"
+}
+run_test 260b "Restore request have priority over other requests"
+
+# This test is very much tied to the implementation of the current priorisation
+# mechanism in the coordinator. It might not make sense to keep it in the future
+test_260c()
+{
+       [ $MDS1_VERSION -lt $(version_code 2.12.0) ] &&
+               skip "Need MDS version at least 2.12.0"
+
+       local -a files=("$DIR/$tdir/$tfile".{0..15})
+       local file
+
+       for file in "${files[@]}"; do
+               create_small_file "$file"
+       done
+
+       # Set a few hsm parameters
+       stack_trap \
+               "set_hsm_param loop_period $(get_hsm_param loop_period)" EXIT
+       set_hsm_param loop_period 1000
+       stack_trap \
+               "set_hsm_param max_requests $(get_hsm_param max_requests)" EXIT
+       set_hsm_param max_requests 3
+
+       # Release one file
+       copytool setup --archive-id 2
+       "$LFS" hsm_archive --archive 2 "${files[0]}"
+       wait_request_state "$(path2fid "${files[0]}")" ARCHIVE SUCCEED
+       "$LFS" hsm_release "${files[0]}"
+
+       # Stop the copytool
+       kill_copytools
+       wait_copytools || error "copytools failed to stop"
+
+       # Force the next coordinator run to do housekeeping
+       cdt_shutdown
+       cdt_enable
+
+       "$LFS" hsm_archive "${files[1]}"
+
+       # Launch a copytool
+       copytool setup
+       copytool setup --archive-id 2
+
+       wait_request_state "$(path2fid "${files[1]}")" ARCHIVE SUCCEED
+       # The coordinator just did a housekeeping run it won't do another one
+       # for around `loop_period' seconds => requests will not be reordered
+       # if it costs too much (ie. when the coordinator has to discard a whole
+       # hal)
+
+       # Send several archive requests
+       for file in "${files[@]:2}"; do
+               "$LFS" hsm_archive "$file"
+       done
+
+       # Send one restore request
+       "$LFS" hsm_restore "${files[0]}"
+
+       # Wait for all the requests to complete
+       wait_request_state "$(path2fid "${files[0]}")" RESTORE SUCCEED
+       for file in "${files[@]:2}"; do
+               wait_request_state "$(path2fid "$file")" ARCHIVE SUCCEED
+       done
+
+       # Collect the actions in the order in which the copytool processed them
+       local -a actions=(
+               $(do_facet "$SINGLEAGT" grep -o '\"RESTORE\\|ARCHIVE\"' \
+                       "$(copytool_logfile "$SINGLEAGT")")
+               )
+
+       printf '%s\n' "${actions[@]}"
+
+       local action
+       for action in "${actions[@]:0:3}"; do
+               [ "$action" == RESTORE ] &&
+                       error "Restore requests should not be prioritised" \
+                             "unless the coordinator is doing housekeeping"
+       done
+       return 0
+}
+run_test 260c "Requests are not reordered on the 'hot' path of the coordinator"
+
+test_300() {
+       [ "$CLIENTONLY" ] && skip "CLIENTONLY mode" && return
+
+       # the only way to test ondisk conf is to restart MDS ...
+       echo "Stop coordinator and remove coordinator state at mount"
+       # stop coordinator
+       cdt_shutdown
+       # clean on disk conf set by default
+       cdt_clear_mount_state
+       cdt_check_state stopped
 
        # check cdt still off after umount/remount
        fail $SINGLEMDS
@@ -4900,9 +4806,9 @@ test_400() {
        local dir_mdt1=$DIR/$tdir/mdt1
 
        # create 1 dir per MDT
-       stack_trap "rm -rf $dir_mdt0"
+       stack_trap "rm -rf $dir_mdt0" EXIT
        $LFS mkdir -i 0 $dir_mdt0 || error "lfs mkdir"
-       stack_trap "rm -rf $dir_mdt1"
+       stack_trap "rm -rf $dir_mdt1" EXIT
        $LFS mkdir -i 1 $dir_mdt1 || error "lfs mkdir"
 
        # create 1 file in each MDT
@@ -5112,8 +5018,8 @@ run_test 405 "archive and release under striped directory"
 test_406() {
        [ $MDSCOUNT -lt 2 ] && skip "needs >= 2 MDTs" && return 0
 
-       [ $(lustre_version_code $SINGLEMDS) -lt $(version_code 2.7.64) ] &&
-               skip "need MDS version at least 2.7.64" && return 0
+       [ $MDS1_VERSION -lt $(version_code 2.7.64) ] &&
+               skip "need MDS version at least 2.7.64"
 
        local fid
        local mdt_index
@@ -5168,17 +5074,11 @@ test_406() {
 run_test 406 "attempting to migrate HSM archived files is safe"
 
 test_407() {
-       needclients 2 || return 0
-       # test needs a running copytool
-       copytool setup
-
-       mkdir -p $DIR/$tdir
-
        local f=$DIR/$tdir/$tfile
        local f2=$DIR2/$tdir/$tfile
-       local fid
-       fid=$(make_custom_file_for_progress $f 39 1000000)
-       [ $? != 0 ] && skip "not enough free space" && return
+       local fid=$(create_empty_file "$f")
+
+       copytool setup
 
        $LFS hsm_archive --archive $HSM_ARCHIVE_NUMBER $f
        wait_request_state $fid ARCHIVE SUCCEED
@@ -5187,6 +5087,9 @@ test_407() {
 #define OBD_FAIL_MDS_HSM_CDT_DELAY      0x164
        do_facet $SINGLEMDS $LCTL set_param fail_val=5 fail_loc=0x164
 
+       # Prevent restore from completing
+       copytool_suspend
+
        md5sum $f &
        # 1st request holds layout lock while appropriate
        # RESTORE record is still not added to llog
@@ -5201,21 +5104,374 @@ test_407() {
                do_facet $SINGLEMDS "$LCTL get_param $HSM_PARAM.actions"&
        fail $SINGLEMDS
 
+       copytool_continue
        wait_request_state $fid RESTORE SUCCEED
 }
 run_test 407 "Check for double RESTORE records in llog"
 
 test_500()
 {
-       [ $(lustre_version_code $SINGLEMDS) -lt $(version_code 2.6.92) ] &&
-               skip "HSM migrate is not supported" && return
+       [ $MDS1_VERSION -lt $(version_code 2.6.92) ] &&
+               skip "HSM migrate is not supported"
 
        test_mkdir -p $DIR/$tdir
-       llapi_hsm_test -d $DIR/$tdir || error "One llapi HSM test failed"
+
+       if [ $(lustre_version_code client) -lt $(version_code 2.11.56) ] ||
+            [ $(lustre_version_code $SINGLEMDS) -lt $(version_code 2.11.56) ];
+       then
+               llapi_hsm_test -d $DIR/$tdir -b ||
+                       error "One llapi HSM test failed"
+       else
+               llapi_hsm_test -d $DIR/$tdir ||
+                       error "One llapi HSM test failed"
+       fi
 }
 run_test 500 "various LLAPI HSM tests"
 
-copytool_cleanup
+test_600() {
+       [ $MDS1_VERSION -lt $(version_code 2.10.58) ] &&
+               skip "need MDS version at least 2.10.58"
+
+       mkdir -p $DIR/$tdir
+
+       local f=$DIR/$tdir/$tfile
+
+       changelog_register
+       # set changelog_mask to ALL
+       changelog_chmask "ALL"
+
+       chmod 777 $DIR/$tdir
+       $RUNAS touch $f || error "touch $f failed as $RUNAS_ID"
+       local fid=$(path2fid $f)
+
+       local entry
+       entry=$(changelog_find -type CREAT -target-fid $fid -uid "$RUNAS_ID" \
+                              -gid "$RUNAS_GID") ||
+               error "No matching CREAT entry"
+
+       # Parse the changelog
+       eval local -A changelog=$(changelog2array $entry)
+       local nid="${changelog[nid]}"
+
+       # Check its NID
+       echo "Got NID '$nid'"
+       [ -n "$nid" ] && [[ "${CLIENT_NIDS[*]}" =~ $nid ]] ||
+               error "nid '$nid' does not match any client NID:" \
+                     "${CLIENT_NIDS[@]}"
+}
+run_test 600 "Changelog fields 'u=' and 'nid='"
+
+test_601() {
+       [ $MDS1_VERSION -lt $(version_code 2.10.58) ] &&
+               skip "need MDS version at least 2.10.58"
+
+       mkdir -p $DIR/$tdir
+
+       local f=$DIR/$tdir/$tfile
+
+       changelog_register
+       # set changelog_mask to ALL
+       changelog_chmask "ALL"
+
+       touch $f || error "touch $f failed"
+       local fid=$(path2fid $f)
+
+       changelog_clear
+       cat $f || error "cat $f failed"
+
+       changelog_find -type OPEN -target-fid $fid -mode "r--" ||
+               error "No matching OPEN entry"
+}
+run_test 601 "OPEN Changelog entry"
+
+test_602() {
+       [ $MDS1_VERSION -lt $(version_code 2.10.58) ] &&
+               skip "need MDS version at least 2.10.58"
+
+       mkdir -p $DIR/$tdir
+
+       local f=$DIR/$tdir/$tfile
+
+       changelog_register
+       # set changelog_mask to ALL
+       changelog_chmask "ALL"
+
+       touch $f || error "touch $f failed"
+       local fid=$(path2fid $f)
+
+       changelog_clear
+       cat $f || error "cat $f failed"
+
+       changelog_find -type CLOSE -target-fid $fid || error "No CLOSE entry"
+
+       changelog_clear
+       changelog_dump
+       echo f > $f || error "write $f failed"
+       changelog_dump
+
+       changelog_find -type CLOSE -target-fid $fid || error "No CLOSE entry"
+
+       # remove OPEN from changelog_mask
+       changelog_chmask "-OPEN"
+
+       changelog_clear
+       changelog_dump
+       cat $f || error "cat $f failed"
+       changelog_dump
+
+       changelog_find -type CLOSE -target-fid $fid &&
+               error "There should be no CLOSE entry"
+
+       changelog_clear
+       changelog_dump
+       echo f > $f || error "write $f failed"
+       changelog_dump
+
+       changelog_find -type CLOSE -target-fid $fid || error "No CLOSE entry"
+}
+run_test 602 "Changelog record CLOSE only if open+write or OPEN recorded"
+
+test_603() {
+       [ $MDS1_VERSION -lt $(version_code 2.10.58) ] &&
+               skip "need MDS version at least 2.10.58"
+
+       mkdir -p $DIR/$tdir
+
+       local f=$DIR/$tdir/$tfile
+
+       changelog_register
+       # set changelog_mask to ALL
+       changelog_chmask "ALL"
+
+       touch $f || error "touch $f failed"
+       local fid=$(path2fid $f)
+
+       setfattr -n user.xattr1 -v "value1" $f || error "setfattr $f failed"
+
+       changelog_clear
+       getfattr -n user.xattr1 $f || error "getfattr $f failed"
+
+       changelog_find -type GXATR -target-fid $fid -xattr "user.xattr1" ||
+               error "No matching GXATR entry"
+}
+run_test 603 "GETXATTR Changelog entry"
+
+test_604() {
+       [ $MDS1_VERSION -lt $(version_code 2.10.58) ] &&
+               skip "need MDS version at least 2.10.58"
+
+       mkdir -p $DIR/$tdir
+
+       local f=$DIR/$tdir/$tfile
+       local f2=$DIR2/$tdir/$tfile
+       local procname="mdd.$FSNAME-MDT0000.changelog_deniednext"
+       local timeout
+       timeout="$(do_facet mds1 "$LCTL" get_param -n "$procname")"
+       stack_trap "do_facet mds1 '$LCTL' set_param '$procname=$timeout'" EXIT
+       do_facet mds1 lctl set_param "$procname=20"
+
+
+       changelog_register
+       # set changelog_mask to ALL
+       changelog_chmask "ALL"
+
+       touch $f || error "touch $f failed"
+       local fid=$(path2fid $f)
+
+       chmod 600 $f
+
+       changelog_clear
+       changelog_dump
+       $RUNAS cat $f2 && error "cat $f2 by user $RUNAS_ID should have failed"
+       changelog_dump
+
+       local entry
+       entry=$(changelog_find -type NOPEN -target-fid $fid -uid "$RUNAS_ID" \
+                              -gid "$RUNAS_GID" -mode "r--") ||
+               error "No matching NOPEN entry"
+
+       # Parse the changelog
+       eval local -A changelog=$(changelog2array $entry)
+       local nid="${changelog[nid]}"
+
+       # Check its NID
+       echo "Got NID '$nid'"
+       [ -n "$nid" ] && [[ "${CLIENT_NIDS[*]}" =~ $nid ]] ||
+               error "nid '$nid' does not match any client NID:" \
+                     "${CLIENT_NIDS[@]}"
+
+       changelog_clear
+       changelog_dump
+       $RUNAS cat $f2 && error "cat $f2 by user $RUNAS_ID should have failed"
+       changelog_dump
+
+       changelog_find -type NOPEN -target-fid $fid &&
+               error "There should be no NOPEN entry"
+
+       # Sleep for `changelog_deniednext` seconds
+       sleep 20
+
+       changelog_clear
+       changelog_dump
+       $RUNAS cat $f2 && error "cat $f by user $RUNAS_ID should have failed"
+       changelog_dump
+
+       entry=$(changelog_find -type NOPEN -target-fid $fid -uid "$RUNAS_ID" \
+                              -gid "$RUNAS_GID" -mode "r--") ||
+               error "No matching NOPEN entry"
+
+       # Parse the changelog
+       eval local -A changelog=$(changelog2array $entry)
+       local nid="${changelog[nid]}"
+
+       # Check the NID
+       echo "Got NID '$nid'"
+       [ -n "$nid" ] && [[ "${CLIENT_NIDS[*]}" =~ $nid ]] ||
+               error "nid '$nid' does not match any client NID:" \
+                     "${CLIENT_NIDS[@]}"
+}
+run_test 604 "NOPEN Changelog entry"
+
+test_605() {
+       [ $MDS1_VERSION -lt $(version_code 2.10.58) ] &&
+               skip "need MDS version at least 2.10.58"
+
+       mkdir -p $DIR/$tdir
+
+       local f=$DIR/$tdir/$tfile
+       local f2=$DIR2/$tdir/$tfile
+
+       changelog_register
+       # set changelog_mask to ALL
+       changelog_chmask "ALL"
+
+       touch $f || error "touch $f failed"
+       local fid=$(path2fid $f)
+
+       changelog_clear
+       changelog_dump
+       exec 3<> $f || error "open $f failed"
+       changelog_dump
+
+       local entry
+       changelog_find -type OPEN -target-fid $fid || error "No OPEN entry"
+
+       changelog_clear
+       changelog_dump
+       exec 4<> $f || error "open $f failed"
+       changelog_dump
+
+       changelog_find -type OPEN -target-fid $fid &&
+               error "There should be no OPEN entry"
+
+       exec 4>&- || error "close $f failed"
+       changelog_dump
+
+       changelog_find -type CLOSE -target-fid $fid &&
+               error "There should be no CLOSE entry"
+
+       changelog_clear
+       changelog_dump
+       # access in rw, so different access mode should generate entries
+       cat $f || error "cat $f failed"
+       changelog_dump
+
+       changelog_find -type OPEN -target-fid $fid || error "No OPEN entry"
+
+       changelog_find -type CLOSE -target-fid $fid || error "No CLOSE entry"
+
+       changelog_clear
+       changelog_dump
+       # same access as first one, should not generate new entries
+       exec 4<> $f || error "open $f failed"
+       changelog_dump
+
+       changelog_find -type OPEN -target-fid $fid &&
+               error "There should be no OPEN entry"
+
+       exec 4>&- || error "close $f failed"
+       changelog_dump
+
+       changelog_find -type CLOSE -target-fid $fid &&
+               error "There should be no CLOSE entry"
+
+       changelog_clear
+       changelog_dump
+       # access by different user should generate new entries
+       $RUNAS cat $f || error "cat $f by user $RUNAS_ID failed"
+       changelog_dump
+
+       changelog_find -type OPEN -target-fid $fid || error "No OPEN entry"
+
+       changelog_find -type CLOSE -target-fid $fid || error "No CLOSE entry"
+
+       changelog_clear
+       changelog_dump
+       exec 3>&- || error "close $f failed"
+       changelog_dump
+
+       changelog_find -type CLOSE -target-fid $fid || error "No CLOSE entry"
+}
+run_test 605 "Test OPEN and CLOSE rate limit in Changelogs"
+
+test_606() {
+       [ $MDS1_VERSION -lt $(version_code 2.10.58) ] &&
+               skip "need MDS version at least 2.10.58"
+
+       local llog_reader=$(do_facet mgs "which llog_reader 2> /dev/null")
+       llog_reader=${llog_reader:-$LUSTRE/utils/llog_reader}
+       [ -z $(do_facet mgs ls -d $llog_reader 2> /dev/null) ] &&
+                       skip_env "missing llog_reader" && return
+       local fstype=$(facet_fstype mds1)
+
+       mkdir -p $DIR/$tdir
+
+       local f=$DIR/$tdir/$tfile
+
+       changelog_register
+       # set changelog_mask to ALL
+       changelog_chmask "ALL"
+
+       chmod 777 $DIR/$tdir
+       $RUNAS touch $f || error "touch $f failed as $RUNAS_ID"
+       local fid=$(path2fid $f)
+       rm $f || error "rm $f failed"
+
+       local mntpt=$(facet_mntpt mds1)
+       local pass=true
+       local entry
+
+       #remount mds1 as ldiskfs or zfs type
+       stack_trap "stop mds1; start mds1 $(mdsdevname 1) $MDS_MOUNT_OPTS" EXIT
+       stop mds1 || error "stop mds1 failed"
+       mount_fstype mds1 || error "remount mds1 failed"
+
+       for ((i = 0; i < 1; i++)); do
+               do_facet mds1 $llog_reader $mntpt/changelog_catalog
+               local cat_file=$(do_facet mds1 $llog_reader \
+                               $mntpt/changelog_catalog | awk \
+                               '{match($0,"path=([^ ]+)",a)}END{print a[1]}')
+               [ -n "$cat_file" ] || error "no catalog file"
+
+               entry=$(do_facet mds1 $llog_reader $mntpt/$cat_file |
+                       awk "/CREAT/ && /target:\[$fid\]/ {print}")
+               [ -n "$entry" ] || error "no CREAT entry"
+       done
+
+       local uidgid=$(echo $entry |
+               sed 's+.*\ user:\([0-9][0-9]*:[0-9][0-9]*\)\ .*+\1+')
+       [ -n "$uidgid" ] || error "uidgid is empty"
+       echo "Got UID/GID $uidgid"
+       [ "$uidgid" = "$RUNAS_ID:$RUNAS_GID" ] ||
+               error "uidgid '$uidgid' != '$RUNAS_ID:$RUNAS_GID'"
+       local nid=$(echo $entry |
+               sed 's+.*\ nid:\(\S\S*@\S\S*\)\ .*+\1+')
+       [ -n "$nid" ] || error "nid is empty"
+       echo "Got NID $nid"
+       [ -n "$nid" ] && [[ "${CLIENT_NIDS[*]}" =~ $nid ]] ||
+               error "nid '$nid' does not match any NID ${CLIENT_NIDS[@]}"
+}
+run_test 606 "llog_reader groks changelog fields"
 
 complete $SECONDS
 check_and_cleanup_lustre