diff options
author | Bartosz Taudul <wolf.pld@gmail.com> | 2019-08-17 18:05:29 +0300 |
---|---|---|
committer | Bartosz Taudul <wolf.pld@gmail.com> | 2019-08-17 18:05:29 +0300 |
commit | 414f903cc5b3f5380e6df1c51d88c41723c59224 (patch) | |
tree | 424026b09c4a7682572555519a509055d0f2376c /server | |
parent | f957f64ce12a06848363c8d2ecac4ba6123501cd (diff) |
Collect thread wakeup data.
Diffstat (limited to 'server')
-rw-r--r-- | server/TracyEvent.hpp | 2 | ||||
-rw-r--r-- | server/TracyVersion.hpp | 2 | ||||
-rw-r--r-- | server/TracyView.cpp | 77 | ||||
-rw-r--r-- | server/TracyWorker.cpp | 61 | ||||
-rw-r--r-- | server/TracyWorker.hpp | 1 |
5 files changed, 105 insertions, 38 deletions
diff --git a/server/TracyEvent.hpp b/server/TracyEvent.hpp index ac2c0f03..1e5cb831 100644 --- a/server/TracyEvent.hpp +++ b/server/TracyEvent.hpp @@ -235,6 +235,7 @@ enum { CrashEventSize = sizeof( CrashEvent ) }; struct ContextSwitchData { enum : int8_t { NoState = 100 }; + enum : int8_t { Wakeup = -2 }; int64_t Start() const { return int64_t( _start_cpu ) >> 8; } void SetStart( int64_t start ) { assert( start < ( 1ll << 47 ) ); _start_cpu = ( _start_cpu & 0xFF ) | uint64_t( start << 8 ); } @@ -249,6 +250,7 @@ struct ContextSwitchData uint64_t _start_cpu; uint64_t _end_reason_state; + int64_t wakeup; }; enum { ContextSwitchDataSize = sizeof( ContextSwitchData ) }; diff --git a/server/TracyVersion.hpp b/server/TracyVersion.hpp index b8671501..25b75cb8 100644 --- a/server/TracyVersion.hpp +++ b/server/TracyVersion.hpp @@ -7,7 +7,7 @@ namespace Version { enum { Major = 0 }; enum { Minor = 5 }; -enum { Patch = 3 }; +enum { Patch = 4 }; } } diff --git a/server/TracyView.cpp b/server/TracyView.cpp index aa72ba4b..1ac3ae92 100644 --- a/server/TracyView.cpp +++ b/server/TracyView.cpp @@ -2372,43 +2372,62 @@ void View::DrawContextSwitches( const ContextSwitch* ctx, bool hover, double pxn { const bool migration = pit->Cpu() != ev.Cpu(); const auto px0 = std::max( { ( pit->End() - m_zvStart ) * pxns, -10.0, minpx } ); + const auto pxw = std::max( ( ev.wakeup - m_zvStart ) * pxns, -10.0 ); const auto px1 = std::min( ( ev.Start() - m_zvStart ) * pxns, w + 10.0 ); const auto color = migration ? 0xFFEE7711 : 0xFF2222AA; - draw->AddLine( wpos + ImVec2( px0, round( offset + ty * 0.5 ) - 0.5 ), wpos + ImVec2( px1, round( offset + ty * 0.5 ) - 0.5 ), color, 2 ); + draw->AddLine( wpos + ImVec2( px0, round( offset + ty * 0.5 ) - 0.5 ), wpos + ImVec2( pxw, round( offset + ty * 0.5 ) - 0.5 ), color, 2 ); + if( ev.wakeup != ev.Start() ) + { + draw->AddLine( wpos + ImVec2( pxw, round( offset + ty * 0.5 ) - 0.5 ), wpos + ImVec2( px1, round( offset + ty * 0.5 ) - 0.5 ), 0xFF2280A0, 2 ); + } - if( hover && ImGui::IsMouseHoveringRect( wpos + ImVec2( px0, offset ), wpos + ImVec2( px1, offset + ty ) ) ) + if( hover ) { - ImGui::BeginTooltip(); - TextFocused( "Thread is", migration ? "migrating CPUs" : "waiting" ); - TextFocused( "Waiting time:", TimeToString( ev.Start() - pit->End() ) ); - if( migration ) - { - TextFocused( "CPU:", RealToString( pit->Cpu(), true ) ); - ImGui::SameLine(); -#ifdef TRACY_EXTENDED_FONT - TextFocused( ICON_FA_LONG_ARROW_ALT_RIGHT, RealToString( ev.Cpu(), true ) ); -#else - TextFocused( "->", RealToString( ev.Cpu(), true ) ); -#endif - } - else - { - TextFocused( "CPU:", RealToString( ev.Cpu(), true ) ); - } - if( pit->Reason() != 100 ) + if( ImGui::IsMouseHoveringRect( wpos + ImVec2( px0, offset ), wpos + ImVec2( pxw, offset + ty ) ) ) { - TextFocused( "Wait reason:", DecodeContextSwitchReasonCode( pit->Reason() ) ); + ImGui::BeginTooltip(); + TextFocused( "Thread is", migration ? "migrating CPUs" : "waiting" ); + TextFocused( "Waiting time:", TimeToString( ev.wakeup - pit->End() ) ); + if( migration ) + { + TextFocused( "CPU:", RealToString( pit->Cpu(), true ) ); + ImGui::SameLine(); + #ifdef TRACY_EXTENDED_FONT + TextFocused( ICON_FA_LONG_ARROW_ALT_RIGHT, RealToString( ev.Cpu(), true ) ); + #else + TextFocused( "->", RealToString( ev.Cpu(), true ) ); + #endif + } + else + { + TextFocused( "CPU:", RealToString( ev.Cpu(), true ) ); + } + if( pit->Reason() != 100 ) + { + TextFocused( "Wait reason:", DecodeContextSwitchReasonCode( pit->Reason() ) ); + ImGui::SameLine(); + TextDisabledUnformatted( DecodeContextSwitchReason( pit->Reason() ) ); + } + TextFocused( "Wait state:", DecodeContextSwitchStateCode( pit->State() ) ); ImGui::SameLine(); - TextDisabledUnformatted( DecodeContextSwitchReason( pit->Reason() ) ); - } - TextFocused( "Wait state:", DecodeContextSwitchStateCode( pit->State() ) ); - ImGui::SameLine(); - TextDisabledUnformatted( DecodeContextSwitchState( pit->State() ) ); - ImGui::EndTooltip(); + TextDisabledUnformatted( DecodeContextSwitchState( pit->State() ) ); + ImGui::EndTooltip(); - if( ImGui::IsMouseClicked( 2 ) ) + if( ImGui::IsMouseClicked( 2 ) ) + { + ZoomToRange( pit->End(), ev.wakeup ); + } + } + else if( ev.wakeup != ev.Start() && ImGui::IsMouseHoveringRect( wpos + ImVec2( pxw, offset ), wpos + ImVec2( px1, offset + ty ) ) ) { - ZoomToRange( pit->End(), ev.Start() ); + ImGui::BeginTooltip(); + TextFocused( "Thread is", "waking up" ); + TextFocused( "Scheduling delay:", TimeToString( ev.Start() - ev.wakeup ) ); + if( ImGui::IsMouseClicked( 2 ) ) + { + ZoomToRange( pit->End(), ev.wakeup ); + } + ImGui::EndTooltip(); } } } diff --git a/server/TracyWorker.cpp b/server/TracyWorker.cpp index 21719191..a8761478 100644 --- a/server/TracyWorker.cpp +++ b/server/TracyWorker.cpp @@ -1415,7 +1415,17 @@ Worker::Worker( FileRead& f, EventType::Type eventMask ) auto ptr = data->v.data(); for( uint64_t j=0; j<csz; j++ ) { - ptr->SetStart( ReadTimeOffset( f, refTime ) ); + if( fileVer >= FileVersion( 0, 5, 4 ) ) + { + ptr->wakeup = ReadTimeOffset( f, refTime ); + ptr->SetStart( ReadTimeOffset( f, refTime ) ); + } + else + { + int64_t start = ReadTimeOffset( f, refTime ); + ptr->wakeup = start; + ptr->SetStart( start ); + } int64_t diff; f.Read( diff ); if( diff > 0 ) runningTime += diff; @@ -3062,6 +3072,9 @@ bool Worker::Process( const QueueItem& ev ) case QueueType::ContextSwitch: ProcessContextSwitch( ev.contextSwitch ); break; + case QueueType::ThreadWakeup: + ProcessThreadWakeup( ev.threadWakeup ); + break; default: assert( false ); break; @@ -4119,13 +4132,22 @@ void Worker::ProcessContextSwitch( const QueueContextSwitch& ev ) it = m_data.ctxSwitch.emplace( ev.newThread, ctx ).first; } auto& data = it->second->v; - assert( data.empty() || (uint64_t)data.back().End() <= time ); - auto& item = data.push_next(); - item.SetStart( time ); - item.SetEnd( -1 ); - item.SetCpu( ev.cpu ); - item.SetReason( -1 ); - item.SetState( -1 ); + ContextSwitchData* item = nullptr; + if( !data.empty() && data.back().Reason() == ContextSwitchData::Wakeup ) + { + item = &data.back(); + } + else + { + assert( data.empty() || (uint64_t)data.back().End() <= time ); + item = &data.push_next(); + item->wakeup = time; + } + item->SetStart( time ); + item->SetEnd( -1 ); + item->SetCpu( ev.cpu ); + item->SetReason( -1 ); + item->SetState( -1 ); auto& cx = cs.push_next(); cx.SetStart( time ); @@ -4140,6 +4162,28 @@ void Worker::ProcessContextSwitch( const QueueContextSwitch& ev ) } } +void Worker::ProcessThreadWakeup( const QueueThreadWakeup& ev ) +{ + const auto time = TscTime( ev.time - m_data.baseTime ); + m_data.lastTime = std::max( m_data.lastTime, time ); + + auto it = m_data.ctxSwitch.find( ev.thread ); + if( it == m_data.ctxSwitch.end() ) + { + auto ctx = m_slab.AllocInit<ContextSwitch>(); + it = m_data.ctxSwitch.emplace( ev.thread, ctx ).first; + } + auto& data = it->second->v; + if( !data.empty() && data.back().End() < 0 ) return; // wakeup of a running thread + auto& item = data.push_next(); + item.wakeup = time; + item.SetStart( time ); + item.SetEnd( -1 ); + item.SetCpu( 0 ); + item.SetReason( ContextSwitchData::Wakeup ); + item.SetState( -1 ); +} + void Worker::MemAllocChanged( int64_t time ) { const auto val = (double)m_data.memory.usage; @@ -4948,6 +4992,7 @@ void Worker::Write( FileWrite& f ) int64_t refTime = 0; for( auto& cs : ctx->second->v ) { + WriteTimeOffset( f, refTime, cs.wakeup ); WriteTimeOffset( f, refTime, cs.Start() ); WriteTimeOffset( f, refTime, cs.End() ); uint8_t cpu = cs.Cpu(); diff --git a/server/TracyWorker.hpp b/server/TracyWorker.hpp index 606f5b9d..67be1353 100644 --- a/server/TracyWorker.hpp +++ b/server/TracyWorker.hpp @@ -440,6 +440,7 @@ private: tracy_force_inline void ProcessCrashReport( const QueueCrashReport& ev ); tracy_force_inline void ProcessSysTime( const QueueSysTime& ev ); tracy_force_inline void ProcessContextSwitch( const QueueContextSwitch& ev ); + tracy_force_inline void ProcessThreadWakeup( const QueueThreadWakeup& ev ); tracy_force_inline void ProcessZoneBeginImpl( ZoneEvent* zone, const QueueZoneBegin& ev ); tracy_force_inline void ProcessZoneBeginAllocSrcLocImpl( ZoneEvent* zone, const QueueZoneBegin& ev ); |