commit 5d9d019b2c7ca3dc59bfbb44b3169ee5cd71dc79
author Anthony Barbier <anthony.barbier@arm.com> Fri Jan 26 16:38:07 2018 +0000
committer Anthony Barbier <anthony.barbier@arm.com> Fri Nov 02 16:45:00 2018 +0000
tree c6ca631efb8589384bf84c7be5caf0b67392580d
parent e6630e4063fc3aa4312a2c8d094318b09ad2c3f5

COMPMID-863: Only output (end-start) for OpenCL timers

Currently we output an array of timestamps: queued, submitted, start, end
This patch instead only output end-start (i.e the time it took to execute the kernel on the GPU)

Change-Id: Ic3c2b68128f6acd6bb018b7b3ead0b69dd5aca59
Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/117865
Reviewed-by: Pablo Tello <pablo.tello@arm.com>
Tested-by: Jenkins <bsgcomp@arm.com>
Reviewed-by: Kevin Petit <kevin.petit@arm.com>

tests/framework/instruments/OpenCLTimer.cpp

diff --git a/tests/framework/instruments/OpenCLTimer.cpp b/tests/framework/instruments/OpenCLTimer.cpp
index 3de953f..9743015 100644
--- a/tests/framework/instruments/OpenCLTimer.cpp
+++ b/tests/framework/instruments/OpenCLTimer.cpp
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2017 ARM Limited.
+ * Copyright (c) 2017-2018 ARM Limited.
  *
  * SPDX-License-Identifier: MIT
  *
@@ -142,20 +142,10 @@
     unsigned int    kernel_number = 0;
     for(auto kernel : kernels)
     {
-        //cl_int status = kernel.event.getInfo(<CL_EVENT_COMMAND_EXECUTION_STATUS>();
-        cl_ulong queued = kernel.event.getProfilingInfo<CL_PROFILING_COMMAND_QUEUED>();
-        cl_ulong submit = kernel.event.getProfilingInfo<CL_PROFILING_COMMAND_SUBMIT>();
-        cl_ulong start  = kernel.event.getProfilingInfo<CL_PROFILING_COMMAND_START>();
-        cl_ulong end    = kernel.event.getProfilingInfo<CL_PROFILING_COMMAND_END>();
+        cl_ulong start = kernel.event.getProfilingInfo<CL_PROFILING_COMMAND_START>();
+        cl_ulong end   = kernel.event.getProfilingInfo<CL_PROFILING_COMMAND_END>();
 
-        std::list<std::string> raw_data =
-        {
-            "queued", support::cpp11::to_string(queued),
-            "submit", support::cpp11::to_string(submit),
-            "start", support::cpp11::to_string(start),
-            "end", support::cpp11::to_string(end),
-        };
-        measurements.emplace(kernel.name + " #" + support::cpp11::to_string(kernel_number++), Measurement((end - start) / _scale_factor, _unit, raw_data));
+        measurements.emplace(kernel.name + " #" + support::cpp11::to_string(kernel_number++), Measurement((end - start) / _scale_factor, _unit));
     }
 
     return measurements;