Add die to CPU topology data structures.

Drawing dies on the topology chart is a minimal effort for the time being.
An improved version will be added later on.
This commit is contained in:
Bartosz Taudul 2024-08-22 01:33:22 +02:00
parent e947aeb72d
commit 357b557684
No known key found for this signature in database
GPG Key ID: B7FE2008B7575DF3
4 changed files with 126 additions and 68 deletions

View File

@ -704,6 +704,7 @@ void View::DrawInfo()
std::vector<int> maxthreads( topology.size() );
float ptsz = 0;
float dtsz = 0;
float ctsz = 0;
float ttsz = 0;
for( auto& package : topology )
@ -715,7 +716,13 @@ void View::DrawInfo()
ImGui::PopFont();
size_t mt = 0;
for( auto& core : package.second )
for( auto& die : package.second )
{
sprintf( buf, ICON_FA_DICE_D6 " Die %" PRIu32, die.first );
const auto dsz = ImGui::CalcTextSize( buf ).x;
if( dsz > dtsz ) dtsz = dsz;
for( auto& core : die.second )
{
sprintf( buf, ICON_FA_MICROCHIP "%" PRIu32, core.first );
const auto csz = ImGui::CalcTextSize( buf ).x;
@ -731,6 +738,7 @@ void View::DrawInfo()
if( tsz > ttsz ) ttsz = tsz;
}
}
}
maxthreads[package.first] = (int)mt;
}
@ -752,21 +760,32 @@ void View::DrawInfo()
draw->AddText( dpos, 0xFFFFFFFF, buf );
dpos.y += ty;
std::vector<decltype(package->second.begin())> dsort;
dsort.reserve( package->second.size() );
for( auto it = package->second.begin(); it != package->second.end(); ++it ) dsort.emplace_back( it );
std::sort( dsort.begin(), dsort.end(), [] ( const auto& l, const auto& r ) { return l->first < r->first; } );
for( auto& die : dsort )
{
dpos.y += small;
sprintf( buf, ICON_FA_DICE_D6 " Die %" PRIu32, die->first );
draw->AddText( dpos, 0xFFFFFFFF, buf );
dpos.y += ty;
const auto inCoreWidth = ( ttsz + margin ) * maxthreads[package->first];
const auto coreWidth = inCoreWidth + 2 * margin;
const auto inCoreHeight = margin + 2 * small + ty;
const auto coreHeight = inCoreHeight + ty;
const auto cpl = std::max( 1, (int)floor( ( remainingWidth - 2 * margin ) / coreWidth ) );
const auto cl = ( package->second.size() + cpl - 1 ) / cpl;
const auto cl = ( die->second.size() + cpl - 1 ) / cpl;
const auto pw = cpl * coreWidth + 2 * margin;
const auto ph = margin + cl * coreHeight;
if( pw > width ) width = pw;
draw->AddRect( dpos, dpos + ImVec2( margin + coreWidth * std::min<size_t>( cpl, package->second.size() ), ph ), 0xFFFFFFFF );
draw->AddRect( dpos, dpos + ImVec2( margin + coreWidth * std::min<size_t>( cpl, die->second.size() ), ph ), 0xFFFFFFFF );
std::vector<decltype(package->second.begin())> csort;
csort.reserve( package->second.size() );
for( auto it = package->second.begin(); it != package->second.end(); ++it ) csort.emplace_back( it );
std::vector<decltype(die->second.begin())> csort;
csort.reserve( die->second.size() );
for( auto it = die->second.begin(); it != die->second.end(); ++it ) csort.emplace_back( it );
std::sort( csort.begin(), csort.end(), [] ( const auto& l, const auto& r ) { return l->first < r->first; } );
auto cpos = dpos + ImVec2( margin, margin );
int ll = cpl;
@ -795,6 +814,7 @@ void View::DrawInfo()
}
dpos.y += ph;
}
}
ImGui::ItemSize( ImVec2( width, dpos.y - origy ) );
ImGui::TreePop();
}

View File

@ -7,7 +7,7 @@ namespace Version
{
enum { Major = 0 };
enum { Minor = 11 };
enum { Patch = 1 };
enum { Patch = 2 };
}
}

View File

@ -643,14 +643,41 @@ Worker::Worker( FileRead& f, EventType::Type eventMask, bool bgTasks, bool allow
uint32_t packageId;
uint64_t psz;
f.Read2( packageId, psz );
auto& package = *m_data.cpuTopology.emplace( packageId, unordered_flat_map<uint32_t, std::vector<uint32_t>> {} ).first;
auto& package = *m_data.cpuTopology.emplace( packageId, unordered_flat_map<uint32_t, unordered_flat_map<uint32_t, std::vector<uint32_t>>> {} ).first;
package.second.reserve( psz );
for( uint64_t j=0; j<psz; j++ )
{
if( fileVer >= FileVersion( 0, 11, 2 ) )
{
uint32_t dieId;
uint64_t dsz;
f.Read2( dieId, dsz );
auto& die = *package.second.emplace( dieId, unordered_flat_map<uint32_t, std::vector<uint32_t>> {} ).first;
die.second.reserve( dsz );
for( uint64_t k=0; k<dsz; k++ )
{
uint32_t coreId;
uint64_t csz;
f.Read2( coreId, csz );
auto& core = *package.second.emplace( coreId, std::vector<uint32_t> {} ).first;
auto& core = *die.second.emplace( coreId, std::vector<uint32_t> {} ).first;
core.second.reserve( csz );
for( uint64_t l=0; l<csz; l++ )
{
uint32_t thread;
f.Read( thread );
core.second.emplace_back( thread );
m_data.cpuTopologyMap.emplace( thread, CpuThreadTopology { packageId, coreId } );
}
}
}
else
{
auto& die = *package.second.emplace( 0, unordered_flat_map<uint32_t, std::vector<uint32_t>> {} ).first;
uint32_t coreId;
uint64_t csz;
f.Read2( coreId, csz );
auto& core = *die.second.emplace( coreId, std::vector<uint32_t> {} ).first;
core.second.reserve( csz );
for( uint64_t k=0; k<csz; k++ )
{
@ -662,6 +689,7 @@ Worker::Worker( FileRead& f, EventType::Type eventMask, bool bgTasks, bool allow
}
}
}
}
f.Read( &m_data.crashEvent, sizeof( m_data.crashEvent ) );
@ -6859,9 +6887,13 @@ void Worker::ProcessSourceCodeNotAvailable( const QueueSourceCodeNotAvailable& e
void Worker::ProcessCpuTopology( const QueueCpuTopology& ev )
{
auto package = m_data.cpuTopology.find( ev.package );
if( package == m_data.cpuTopology.end() ) package = m_data.cpuTopology.emplace( ev.package, unordered_flat_map<uint32_t, std::vector<uint32_t>> {} ).first;
auto core = package->second.find( ev.core );
if( core == package->second.end() ) core = package->second.emplace( ev.core, std::vector<uint32_t> {} ).first;
if( package == m_data.cpuTopology.end() ) package = m_data.cpuTopology.emplace( ev.package, unordered_flat_map<uint32_t, unordered_flat_map<uint32_t, std::vector<uint32_t>>> {} ).first;
auto die = package->second.find( ev.die );
if( die == package->second.end() ) die = package->second.emplace( ev.die, unordered_flat_map<uint32_t, std::vector<uint32_t>> {} ).first;
auto core = die->second.find( ev.core );
if( core == die->second.end() ) core = die->second.emplace( ev.core, std::vector<uint32_t> {} ).first;
core->second.emplace_back( ev.thread );
assert( m_data.cpuTopologyMap.find( ev.thread ) == m_data.cpuTopologyMap.end() );
@ -7699,7 +7731,12 @@ void Worker::Write( FileWrite& f, bool fiDict )
sz = package.second.size();
f.Write( &package.first, sizeof( package.first ) );
f.Write( &sz, sizeof( sz ) );
for( auto& core : package.second )
for( auto& die : package.second )
{
sz = die.second.size();
f.Write( &die.first, sizeof( die.first ) );
f.Write( &sz, sizeof( sz ) );
for( auto& core : die.second )
{
sz = core.second.size();
f.Write( &core.first, sizeof( core.first ) );
@ -7710,6 +7747,7 @@ void Worker::Write( FileWrite& f, bool fiDict )
}
}
}
}
f.Write( &m_data.crashEvent, sizeof( m_data.crashEvent ) );

View File

@ -392,7 +392,7 @@ private:
bool ctxUsageReady = false;
#endif
unordered_flat_map<uint32_t, unordered_flat_map<uint32_t, std::vector<uint32_t>>> cpuTopology;
unordered_flat_map<uint32_t, unordered_flat_map<uint32_t, unordered_flat_map<uint32_t, std::vector<uint32_t>>>> cpuTopology;
unordered_flat_map<uint32_t, CpuThreadTopology> cpuTopologyMap;
unordered_flat_map<uint64_t, MemoryBlock> symbolCode;