intel/gpu_dump: fix argument passing
authorLionel Landwerlin <lionel.g.landwerlin@intel.com>
Sat, 8 Jun 2019 20:48:02 +0000 (23:48 +0300)
committerLionel Landwerlin <lionel.g.landwerlin@intel.com>
Sun, 9 Jun 2019 19:45:13 +0000 (19:45 +0000)
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 <lionel.g.landwerlin@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Eric Engestrom <eric@engestrom.ch>
src/intel/tools/intel_dump_gpu.in
src/intel/tools/intel_sanitize_gpu.in

index a4ceaba7090dc35f2ab7b31fc3d2aed47ef03879..f9fed66c1e3413a8b4eca00baeb2669d3861e89f 100755 (executable)
@@ -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=$?
index a5b032abc6d350a56adf9877b639be7648fcd649..912387ae6564343f2fe11c5d94277455e78022f3 100755 (executable)
@@ -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