aboutsummaryrefslogtreecommitdiff
path: root/src/profiling
diff options
context:
space:
mode:
authorFinn Williams <Finn.Williams@arm.com>2020-04-23 17:55:18 +0100
committerJim Flynn <jim.flynn@arm.com>2020-04-27 09:06:24 +0000
commitd7fcafaaaa917db458be362060e1b7855cad8083 (patch)
tree534e2898f2a7dd39a22bca8a073b085d8a7b7f80 /src/profiling
parent8e84398b8eb4b2a2a558803f4c6359f6b255d49f (diff)
downloadarmnn-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.cpp8
-rw-r--r--src/profiling/ConnectionAcknowledgedCommandHandler.hpp6
-rw-r--r--src/profiling/ProfilingService.cpp1
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)