commit 8355ec982eb3ff51a6a8042fe760138638ca550b
author Jim Flynn <jim.flynn@arm.com> Tue Sep 17 12:29:50 2019 +0100
committer Matteo Martincigh <matteo.martincigh@arm.com> Wed Sep 18 15:13:43 2019 +0000
tree 23150be7ae10712db7af3d6e18f1a381d945344a
parent 0bd586ceb2a1e3f8132d009cf48dc46c76ae09e4

IVGCVSW-3432 Fix a multithread store conflict

* Unit test was using the same CaptureData object across 50 threads

Change-Id: I0249b5a8e0bb05e3d3efdd855f5b34b1d5ef3dc9
Signed-off-by: Jim Flynn <jim.flynn@arm.com>

src/profiling/Holder.hpp

diff --git a/src/profiling/Holder.hpp b/src/profiling/Holder.hpp
index c22c72a..d8d1f5b 100644
--- a/src/profiling/Holder.hpp
+++ b/src/profiling/Holder.hpp
@@ -26,7 +26,7 @@
     CaptureData& operator= (const CaptureData& captureData);
 
     void SetCapturePeriod(uint32_t capturePeriod);
-    void SetCounterIds(std::vector<uint16_t>& counterIds);
+    void SetCounterIds(const std::vector<uint16_t>& counterIds);
     uint32_t GetCapturePeriod() const;
     std::vector<uint16_t> GetCounterIds() const;
 
@@ -41,7 +41,7 @@
     Holder()
         : m_CaptureData() {};
     CaptureData GetCaptureData() const;
-    void SetCaptureData(uint32_t capturePeriod, std::vector<uint16_t>& counterIds);
+    void SetCaptureData(uint32_t capturePeriod, const std::vector<uint16_t>& counterIds);
 
 private:
     mutable std::mutex m_CaptureThreadMutex;