From 17898a9b7e0cd3bfc95c186586e5cafa1e3a2c9d Mon Sep 17 00:00:00 2001 From: Lionel Landwerlin Date: Sat, 8 Jun 2019 23:48:02 +0300 Subject: [PATCH] intel/gpu_dump: fix argument passing We were dropping "/' around arguments grouped together. This was triggering failures with : $ ./framemetrics -g "Memory Writes Distribution Gen9" -o /tmp/output.csv -f ./my.trace 10 11 Signed-off-by: Lionel Landwerlin Reviewed-by: Jordan Justen Reviewed-by: Eric Engestrom --- src/intel/tools/intel_dump_gpu.in | 4 ++-- src/intel/tools/intel_sanitize_gpu.in | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/intel/tools/intel_dump_gpu.in b/src/intel/tools/intel_dump_gpu.in index a4ceaba7..f9fed66 100755 --- a/src/intel/tools/intel_dump_gpu.in +++ b/src/intel/tools/intel_dump_gpu.in @@ -115,9 +115,9 @@ tmp_file=`mktemp` echo -e $args > $tmp_file if [ -z $gdb ]; then - LD_PRELOAD="$ld_preload" INTEL_DUMP_GPU_CONFIG=$tmp_file $@ + LD_PRELOAD="$ld_preload" INTEL_DUMP_GPU_CONFIG=$tmp_file "$@" else - gdb -iex "set exec-wrapper env LD_PRELOAD=$ld_preload INTEL_DUMP_GPU_CONFIG=$tmp_file" --args $@ + gdb -iex "set exec-wrapper env LD_PRELOAD=$ld_preload INTEL_DUMP_GPU_CONFIG=$tmp_file" --args "$@" fi ret=$? diff --git a/src/intel/tools/intel_sanitize_gpu.in b/src/intel/tools/intel_sanitize_gpu.in index a5b032a..912387a 100755 --- a/src/intel/tools/intel_sanitize_gpu.in +++ b/src/intel/tools/intel_sanitize_gpu.in @@ -53,5 +53,5 @@ ld_preload="@install_libexecdir@/libintel_sanitize_gpu.so${LD_PRELOAD:+:$LD_PREL if [ -z $gdb ]; then LD_PRELOAD=$ld_preload exec "$@" else - gdb -iex "set exec-wrapper env LD_PRELOAD=$ld_preload" --args $@ + gdb -iex "set exec-wrapper env LD_PRELOAD=$ld_preload" --args "$@" fi -- 2.7.4