[CRIU] [PATCH] zdtm.sh: set correct path for dump files (v2)

Andrew Vagin avagin at parallels.com
Wed Dec 24 00:57:51 PST 2014


On Mon, Dec 22, 2014 at 01:19:10PM -0500, Christopher Covington wrote:
> On 12/22/2014 04:29 AM, Andrey Vagin wrote:
> > Don't use an uninitialized variable to create a path pattern.
> > 
> > v2: fix the comment
> > 
> > Cc: Christopher Covington <cov at codeaurora.org>
> > Reported-by: Christopher Covington <cov at codeaurora.org>
> > Signed-off-by: Andrey Vagin <avagin at openvz.org>
> 
> Reviewed-by: Christopher Covington <cov at codeaurora.org>

Pls, ignore this patch. I have sent a new version without readlink -m.

> 
> >  test/zdtm.sh | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/test/zdtm.sh b/test/zdtm.sh
> > index 965563c..ecc95cd 100755
> > --- a/test/zdtm.sh
> > +++ b/test/zdtm.sh
> > @@ -656,7 +656,8 @@ EOF
> >  		gen_args="$gen_args --force-irmap"
> >  	fi
> >  
> > -	ddump=`readlink -fm dump/$test_name/$PID/$i`
> > +	# X will be substituted with an iteration number
> > +	ddump=`readlink -fm dump/$test_name/$PID/X`
> >  
> >  	[ -n "$PERF" ] && gen_args="$gen_args --action-script $SCRIPTDIR/perf-script.sh"
> >  
> > 
> 
> 
> -- 
> Qualcomm Innovation Center, Inc.
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project


More information about the CRIU mailing list