diff options
author | Jim Flynn <jim.flynn@arm.com> | 2019-09-17 12:29:50 +0100 |
---|---|---|
committer | Matteo Martincigh <matteo.martincigh@arm.com> | 2019-09-18 15:13:43 +0000 |
commit | 8355ec982eb3ff51a6a8042fe760138638ca550b (patch) | |
tree | 23150be7ae10712db7af3d6e18f1a381d945344a /src/profiling/Holder.cpp | |
parent | 0bd586ceb2a1e3f8132d009cf48dc46c76ae09e4 (diff) | |
download | armnn-8355ec982eb3ff51a6a8042fe760138638ca550b.tar.gz |
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>
Diffstat (limited to 'src/profiling/Holder.cpp')
-rw-r--r-- | src/profiling/Holder.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/profiling/Holder.cpp b/src/profiling/Holder.cpp index 9def49d22e..5916017eb6 100644 --- a/src/profiling/Holder.cpp +++ b/src/profiling/Holder.cpp @@ -24,7 +24,7 @@ void CaptureData::SetCapturePeriod(uint32_t capturePeriod) m_CapturePeriod = capturePeriod; } -void CaptureData::SetCounterIds(std::vector<uint16_t>& counterIds) +void CaptureData::SetCounterIds(const std::vector<uint16_t>& counterIds) { m_CounterIds = counterIds; } @@ -45,7 +45,7 @@ CaptureData Holder::GetCaptureData() const return m_CaptureData; } -void Holder::SetCaptureData(uint32_t capturePeriod, std::vector<uint16_t>& counterIds) +void Holder::SetCaptureData(uint32_t capturePeriod, const std::vector<uint16_t>& counterIds) { std::lock_guard<std::mutex> lockGuard(m_CaptureThreadMutex); m_CaptureData.SetCapturePeriod(capturePeriod); @@ -54,4 +54,4 @@ void Holder::SetCaptureData(uint32_t capturePeriod, std::vector<uint16_t>& count } // namespace profiling -} // namespace armnn
\ No newline at end of file +} // namespace armnn |