diff options
author | Finn Williams <Finn.Williams@arm.com> | 2020-04-23 17:55:18 +0100 |
---|---|---|
committer | Jim Flynn <jim.flynn@arm.com> | 2020-04-27 09:06:24 +0000 |
commit | d7fcafaaaa917db458be362060e1b7855cad8083 (patch) | |
tree | 534e2898f2a7dd39a22bca8a073b085d8a7b7f80 /src/profiling | |
parent | 8e84398b8eb4b2a2a558803f4c6359f6b255d49f (diff) | |
download | armnn-d7fcafaaaa917db458be362060e1b7855cad8083.tar.gz |
IVGCVSW-4722 Add missing m_TimelineEnabled bool to ConnectionAcknowledgedCommandHandler
* Added timeline bool to ConnectionAcknowledgedCommandHandler
* Added option to enable timeline profiling in ExecuteNetwork
* Added CommandHandler stub to allow gatordMock to ignore packets
Signed-off-by: Finn Williams <Finn.Williams@arm.com>
Change-Id: I314f9411e0079cba8f103d3b8a89f2bf38bb21ab
Diffstat (limited to 'src/profiling')
-rw-r--r-- | src/profiling/ConnectionAcknowledgedCommandHandler.cpp | 8 | ||||
-rw-r--r-- | src/profiling/ConnectionAcknowledgedCommandHandler.hpp | 6 | ||||
-rw-r--r-- | src/profiling/ProfilingService.cpp | 1 |
3 files changed, 13 insertions, 2 deletions
diff --git a/src/profiling/ConnectionAcknowledgedCommandHandler.cpp b/src/profiling/ConnectionAcknowledgedCommandHandler.cpp index 995562fb3f..7690573ccf 100644 --- a/src/profiling/ConnectionAcknowledgedCommandHandler.cpp +++ b/src/profiling/ConnectionAcknowledgedCommandHandler.cpp @@ -40,8 +40,12 @@ void ConnectionAcknowledgedCommandHandler::operator()(const Packet& packet) m_StateMachine.TransitionToState(ProfilingState::Active); // Send the counter directory packet. m_SendCounterPacket.SendCounterDirectoryPacket(m_CounterDirectory); - m_SendTimelinePacket.SendTimelineMessageDirectoryPackage(); - TimelineUtilityMethods::SendWellKnownLabelsAndEventClasses(m_SendTimelinePacket); + + if (m_TimelineEnabled) + { + m_SendTimelinePacket.SendTimelineMessageDirectoryPackage(); + TimelineUtilityMethods::SendWellKnownLabelsAndEventClasses(m_SendTimelinePacket); + } if(m_BackendProfilingContext.has_value()) { diff --git a/src/profiling/ConnectionAcknowledgedCommandHandler.hpp b/src/profiling/ConnectionAcknowledgedCommandHandler.hpp index e2bdff8e96..053d3c32fc 100644 --- a/src/profiling/ConnectionAcknowledgedCommandHandler.hpp +++ b/src/profiling/ConnectionAcknowledgedCommandHandler.hpp @@ -44,12 +44,18 @@ public: void operator()(const Packet& packet) override; + void setTimelineEnabled(bool timelineEnabled) + { + m_TimelineEnabled = timelineEnabled; + } + private: const ICounterDirectory& m_CounterDirectory; ISendCounterPacket& m_SendCounterPacket; ISendTimelinePacket& m_SendTimelinePacket; ProfilingStateMachine& m_StateMachine; Optional<BackendProfilingContexts> m_BackendProfilingContext; + bool m_TimelineEnabled = false; }; } // namespace profiling diff --git a/src/profiling/ProfilingService.cpp b/src/profiling/ProfilingService.cpp index 4d7241e7db..d87ed0bf67 100644 --- a/src/profiling/ProfilingService.cpp +++ b/src/profiling/ProfilingService.cpp @@ -35,6 +35,7 @@ void ProfilingService::ResetExternalProfilingOptions(const ExternalProfilingOpti // Update the profiling options m_Options = options; m_TimelineReporting = options.m_TimelineEnabled; + m_ConnectionAcknowledgedCommandHandler.setTimelineEnabled(options.m_TimelineEnabled); // Check if the profiling service needs to be reset if (resetProfilingService) |