Skip to content
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion profile/plugin/aie_profile/aie_profile_metadata.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -921,7 +921,7 @@ namespace xdp {
}
catch (...) {
std::stringstream msg;
msg << "Channel specifications in graph_based_interface_metrics "
msg << "Channel specifications in graph_based_interface_tile_metrics "
<< "are not valid and hence ignored.";
xrt_core::message::send(severity_level::warning, "XRT", msg.str());
}
Expand Down Expand Up @@ -1083,6 +1083,9 @@ namespace xdp {
}
catch (std::invalid_argument const&) {
// maxColumn is not an integer i.e either 1st style or wrong format, skip for now
xrt_core::message::send(severity_level::warning, "XRT",
"tile_based_interface_tile_metrics: invalid range line. Ignored: "
+ metricsSettings[i]);
continue;
Comment thread
predutta marked this conversation as resolved.
}

Expand Down Expand Up @@ -1149,6 +1152,10 @@ namespace xdp {

try {
col = aie::convertStringToUint8(metrics[i][1]);
xrt_core::message::send(severity_level::warning, "XRT",
"tile_based_interface_tile_metrics: invalid format. Ignored: "
+ metricsSettings[i]);
continue;
Comment thread
predutta marked this conversation as resolved.
}
catch (std::invalid_argument const&) {
// max column is not a number, so the expected single column specification. Handle this here
Expand Down
5 changes: 4 additions & 1 deletion profile/plugin/aie_profile/edge/aie_profile.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,10 @@ namespace xdp {
if (!runtimeCounters) {
// No runtime counters means there were no valid metrics configured for profiling this design. There is nothing to profile, so return early.
xrt_core::message::send(severity_level::warning, "XRT",
"No valid metric setting found for this design. Please specify valid tile_based_[aie|aie_memory|interface_tile]_metrics under \"AIE_profile_settings\" section in your xrt.ini if you want to configure this design.");
"No valid metric setting found for this design. Please specify valid "
"graph_based_[aie|aie_memory|memory_tile|interface_tile]_metrics and/or "
"tile_based_[aie|aie_memory|memory_tile|interface_tile|microcontroller]_metrics "
"under \"AIE_profile_settings\" section in your xrt.ini if you want to configure this design.");
return;
}
}
Expand Down
5 changes: 4 additions & 1 deletion profile/plugin/aie_profile/ve2/aie_profile.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,10 @@ namespace xdp {

if (counters.empty()) {
xrt_core::message::send(severity_level::warning, "XRT",
"AIE Profile Counters were not found for this design. Please specify tile_based_[aie|aie_memory|interface_tile]_metrics under \"AIE_profile_settings\" section in your xrt.ini.");
"AIE Profile Counters were not found for this design. Please specify "
"graph_based_[aie|aie_memory|memory_tile|interface_tile]_metrics and/or "
"tile_based_[aie|aie_memory|memory_tile|interface_tile|microcontroller]_metrics "
"under \"AIE_profile_settings\" section in your xrt.ini.");
(db->getStaticInfo()).setIsAIECounterRead(deviceID,true);
return;
}
Expand Down
13 changes: 10 additions & 3 deletions profile/plugin/aie_trace/aie_trace_metadata.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -417,7 +417,7 @@ namespace xdp {
&& (std::find(allValidEntries.begin(), allValidEntries.end(), graphMetrics[i][1]) == allValidEntries.end())) {
std::stringstream msg;
msg << "Could not find " << entryType << " " << graphMetrics[i][1]
<< " as specified in graph_based_" << tileName << "_metrics setting."
<< " as specified in graph_based_" << tileName << "_tile_metrics setting."
<< " The following " << entryType << "s are valid : ";
if (!allValidEntries.empty()) {
msg << allValidEntries[0];
Expand Down Expand Up @@ -460,7 +460,7 @@ namespace xdp {
&& (std::find(allValidEntries.begin(), allValidEntries.end(), graphMetrics[i][1]) == allValidEntries.end())) {
std::stringstream msg;
msg << "Could not find " << entryType << " " << graphMetrics[i][1]
<< " as specified in graph_based_" << tileName << "_metrics setting."
<< " as specified in graph_based_" << tileName << "_tile_metrics setting."
<< " The following " << entryType << "s are valid : ";
if (!allValidEntries.empty()) {
msg << allValidEntries[0];
Expand Down Expand Up @@ -788,7 +788,7 @@ namespace xdp {
}
} catch (...) {
std::stringstream msg;
msg << "Channel specifications in graph_based_interface_metrics "
msg << "Channel specifications in graph_based_interface_tile_metrics "
<< "are not valid and hence ignored.";
xrt_core::message::send(severity_level::warning, "XRT", msg.str());
}
Expand Down Expand Up @@ -894,6 +894,9 @@ namespace xdp {
}
catch (std::invalid_argument const&) {
// Max column is not an integer, so either first style or wrong format. Skip for now.
xrt_core::message::send(severity_level::warning, "XRT",
"tile_based_interface_tile_metrics: invalid range line. Ignored: "
+ metricsSettings[i]);
continue;
Comment thread
predutta marked this conversation as resolved.
}

Expand Down Expand Up @@ -952,6 +955,10 @@ namespace xdp {
uint8_t col = 0;
try {
col = aie::convertStringToUint8(metrics[i][1]);
xrt_core::message::send(severity_level::warning, "XRT",
"tile_based_interface_tile_metrics: invalid format. Ignored: "
+ metricsSettings[i]);
continue;
}
catch (std::invalid_argument const&) {
// Max column is not an integer, so expected single column specification. Handle this here.
Expand Down
Loading