add new function in xrt-smi to query the load on AIE array. by zhangchiming · Pull Request #9983 · Xilinx/XRT

zhangchiming · 2026-08-17T23:37:03Z

Problem solved by the commit

Adds a new aie-load report to xrt-smi so users can query AIE4 array hardware utilization (as a 0-100% load figure, analogous to Windows Task Manager CPU usage) directly from the command line:
xrt-smi examine -r aie-load [--duration ]
The load percentage value is included in the existing xrt-smi examine -r platform report
The AIE load percentage is calculated as hw activities counters delta (between two sampling point) divided by max counters increment possible (determined by number of AIE rows/colums, AIE array clocks and time between two sampling point) .

How problem was solved, alternative solutions (if any) and why they were rejected

Added query::aie_load (with a sample_duration_ms arg) to query_requests.h and a new ReportAieLoad report class that calls device_queryquery::aie_load() directly

What has been tested and how, request additional testing if necessary

Manually verified xrt-smi examine -r aie-load and xrt-smi examine -r aie-load --duration against the modified driver, Manually verified xrt-smi examine -r platform includes the new AIE Load line (text output)

Documentation impact (if any)
xrt-smi examine --help / --list-opcodes-equivalent output now includes aie-load and --duration; user-facing XRT documentation (if any lists examine report names) should be updated to include aie-load.

Documentation impact (if any)

NA

Signed-off-by: Chiming <chimingz@amd.com>

xrt-pr-bot · 2026-08-17T23:37:24Z

⚠️ Authorization Failed

@zhangchiming is not a repository collaborator.

To proceed:

XRT Admins: Add the build label to authorize this PR build
OR Add @zhangchiming as a repository collaborator

github-actions

clang-tidy made some suggestions

github-actions · 2026-08-18T16:56:17Z

+  struct result_type {
+    uint32_t load_percent;          // 0-100, or UINT32_MAX if unavailable
+    uint64_t timestamp_ms;          // FW uptime when second snapshot was taken
+    uint64_t activity_counters[8];  // raw counter values from second snapshot


warning: 8 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers]

uint64_t activity_counters[8]; // raw counter values from second snapshot ^

github-actions · 2026-08-18T16:56:17Z

+  struct result_type {
+    uint32_t load_percent;          // 0-100, or UINT32_MAX if unavailable
+    uint64_t timestamp_ms;          // FW uptime when second snapshot was taken
+    uint64_t activity_counters[8];  // raw counter values from second snapshot


warning: do not declare C-style arrays, use std::array<> instead [cppcoreguidelines-avoid-c-arrays]

uint64_t activity_counters[8]; // raw counter values from second snapshot ^

github-actions · 2026-08-18T16:56:17Z

+  static const key_type key = key_type::aie_load;
+
+  virtual std::any
+  get(const device*) const override = 0;


warning: 'virtual' is redundant since the function is already declared 'override' [cppcoreguidelines-explicit-virtual-functions]

src/runtime_src/core/common/query_requests.h:4337:

- virtual std::any + std::any

github-actions · 2026-08-18T16:56:17Z

+  get(const device*) const override = 0;
+
+  virtual std::any
+  get(const device*, const std::any&) const override = 0;


warning: 'virtual' is redundant since the function is already declared 'override' [cppcoreguidelines-explicit-virtual-functions]

src/runtime_src/core/common/query_requests.h:4340:

- virtual std::any + std::any

github-actions · 2026-08-18T16:56:17Z

+    node.put("operations_per_second", data.operations_per_second);
+
+    boost::property_tree::ptree pt_counters;
+    for (size_t i = 0; i < 8; i++) {


warning: 8 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers]

for (size_t i = 0; i < 8; i++) { ^

github-actions · 2026-08-18T16:56:17Z

+  void setDuration(uint32_t duration_ms) { m_duration_ms = duration_ms; }
+
+ public:
+  virtual void getPropertyTreeInternal(const xrt_core::device* dev, boost::property_tree::ptree& pt) const;


warning: prefer using 'override' or (rarely) 'final' instead of 'virtual' [cppcoreguidelines-explicit-virtual-functions]

Suggested change

virtual void getPropertyTreeInternal(const xrt_core::device* dev, boost::property_tree::ptree& pt) const;

void getPropertyTreeInternal(const xrt_core::device* dev, boost::property_tree::ptree& pt) const override;

github-actions · 2026-08-18T16:56:17Z

+
+ public:
+  virtual void getPropertyTreeInternal(const xrt_core::device* dev, boost::property_tree::ptree& pt) const;
+  virtual void getPropertyTree20202(const xrt_core::device* dev, boost::property_tree::ptree& pt) const;


warning: prefer using 'override' or (rarely) 'final' instead of 'virtual' [cppcoreguidelines-explicit-virtual-functions]

Suggested change

virtual void getPropertyTree20202(const xrt_core::device* dev, boost::property_tree::ptree& pt) const;

void getPropertyTree20202(const xrt_core::device* dev, boost::property_tree::ptree& pt) const override;

github-actions · 2026-08-18T16:56:17Z

+  virtual void getPropertyTreeInternal(const xrt_core::device* dev, boost::property_tree::ptree& pt) const;
+  virtual void getPropertyTree20202(const xrt_core::device* dev, boost::property_tree::ptree& pt) const;
+  virtual void writeReport(const xrt_core::device* dev, const boost::property_tree::ptree& pt,
+                           const std::vector<std::string>& elementsFilter, std::ostream& output) const;


warning: prefer using 'override' or (rarely) 'final' instead of 'virtual' [cppcoreguidelines-explicit-virtual-functions]

Suggested change

const std::vector<std::string>& elementsFilter, std::ostream& output) const;

void writeReport(const xrt_core::device* dev, const boost::property_tree::ptree& pt,

const std::vector<std::string>& elementsFilter, std::ostream& output) const override;

github-actions · 2026-08-18T16:56:17Z

          [&](const xrt_core::device*) {
            std::ostringstream console;
-            XBU::produce_reports(device, reportsToProcess, schema_version, schema_label, {}, console, oSchemaOutput);
+            XBU::produce_reports(device, reportsToProcess, schemaVersion, {}, console, oSchemaOutput);


warning: too few arguments to function call, expected 7, have 6 [clang-diagnostic-error]

XBU::produce_reports(device, reportsToProcess, schemaVersion, {}, console, oSchemaOutput); ^

Additional context

src/runtime_src/core/tools/common/XBHelpMenus.h:40: 'produce_reports' declared here

produce_reports( const std::shared_ptr<xrt_core::device>& device, ^

github-actions · 2026-08-18T16:56:17Z

          *options.m_watchIntervalSec);
    } else {
-      XBU::produce_reports(device, reportsToProcess, schema_version, schema_label, {}, std::cout, oSchemaOutput);
+      XBU::produce_reports(device, reportsToProcess, schemaVersion, {}, std::cout, oSchemaOutput);


warning: too few arguments to function call, expected 7, have 6 [clang-diagnostic-error]

XBU::produce_reports(device, reportsToProcess, schemaVersion, {}, std::cout, oSchemaOutput); ^

Additional context

src/runtime_src/core/tools/common/XBHelpMenus.h:40: 'produce_reports' declared here

produce_reports( const std::shared_ptr<xrt_core::device>& device, ^

Signed-off-by: Chiming <chimingz@amd.com>

aktondak · 2026-08-18T20:26:32Z

This should be added as part of the platform report and not a standalone report.

Signed-off-by: Chiming <chimingz@amd.com>

github-actions · 2026-08-18T21:27:45Z

clang-tidy review says "All clean, LGTM! 👍"

add new function in xrt-smi to query the load on AIE array.

58b94ee

Signed-off-by: Chiming <chimingz@amd.com>

zhangchiming requested review from AShivangi, chvamshi-xilinx, rbramand-xilinx and stsoe as code owners August 17, 2026 23:37

zhangchiming closed this Aug 18, 2026

zhangchiming reopened this Aug 18, 2026

github-actions Bot reviewed Aug 18, 2026

View reviewed changes

zhangchiming closed this Aug 18, 2026

clean up the code.

46c6136

Signed-off-by: Chiming <chimingz@amd.com>

zhangchiming reopened this Aug 18, 2026

remove aie-load standalone option from xrt-smi.

6aa0ff6

Signed-off-by: Chiming <chimingz@amd.com>

	virtual void getPropertyTreeInternal(const xrt_core::device* dev, boost::property_tree::ptree& pt) const;
	void getPropertyTreeInternal(const xrt_core::device* dev, boost::property_tree::ptree& pt) const override;

	virtual void getPropertyTree20202(const xrt_core::device* dev, boost::property_tree::ptree& pt) const;
	void getPropertyTree20202(const xrt_core::device* dev, boost::property_tree::ptree& pt) const override;

	const std::vector<std::string>& elementsFilter, std::ostream& output) const;
	void writeReport(const xrt_core::device* dev, const boost::property_tree::ptree& pt,
	const std::vector<std::string>& elementsFilter, std::ostream& output) const override;

Uh oh!

Conversation

zhangchiming commented Aug 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

xrt-pr-bot Bot commented Aug 17, 2026

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

aktondak commented Aug 18, 2026

Uh oh!

github-actions Bot commented Aug 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

zhangchiming commented Aug 17, 2026 •

edited

Loading