generic/403: don't spew '$GETFATTR_PROG: Killed' messages

Use a runfile presence check to control the background getfattr loop
instead of using kill -9.  This helps us to avoid the problem that
the controlling bash will print a process killed message, which
wrecks the golden output.

Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Eryu Guan <eguan@redhat.com>
Signed-off-by: Eryu Guan <eguan@redhat.com>
This commit is contained in:
Darrick J. Wong
2018-01-24 15:53:35 -08:00
committed by Eryu Guan
parent a6b5c98da8
commit 9385d3a901
+4 -2
View File
@@ -61,7 +61,9 @@ touch $SCRATCH_MNT/file
$SETFATTR_PROG -n trusted.small -v a $SCRATCH_MNT/file
# start a background getxattr loop for the existing xattr
while [ true ]; do
runfile="$tmp.getfattr"
touch $runfile
while [ -e $runfile ]; do
$GETFATTR_PROG --absolute-names -n trusted.small $SCRATCH_MNT/file \
> /dev/null || break
done &
@@ -75,7 +77,7 @@ for i in $(seq 0 99); do
$SETFATTR_PROG -x trusted.big $SCRATCH_MNT/file
done
kill -9 $getfattr_pid > /dev/null 2>&1
rm -f $runfile
wait > /dev/null 2>&1
echo Silence is golden