Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VITIS-11093 Upgrade ReportAiePartitions #7883

Merged
merged 1 commit into from
Jan 16, 2024
Merged

Conversation

dbenusov
Copy link
Contributor

Problem solved by the commit

The aie partitions report displays information that is no longer relevant or misleading.

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

Drivers were changed, software should reflect what the driver is doing.

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

Rename usage metrics column to command submissions. Remove device BO metrics column.

Risks (if any) associated the changes in the commit

None

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

Windows MCDM Virtual Machine Build 26016

Z:\XRT-MCDM\build\WDebug\xilinx\xrt>xbutil examine -d -r aie-partitions
------------------------------------------------------------
                        EARLY ACCESS
        This release of xbutil contains early access
         experimental features which may have bugs.
------------------------------------------------------------

---------------------------------
[0000:18:00.1] : RyzenAI-Phoenix
---------------------------------
AIE Partitions
  Partition Index: 0
    Columns: [1]
    HW Contexts:
      Slot ID  Xclbin UUID                             Command Submissions  Migration Count
      ---------------------------------------------------------------------------------------
      4        {c186a256-edec-c7a3-9203-572944446cab}  0                    0

Documentation impact (if any)

None.

…e device BO metrics column.

Signed-off-by: Daniel Benusovich <[email protected]>
@gbuildx
Copy link
Collaborator

gbuildx commented Jan 12, 2024

Build Passed!

@maxzhen
Copy link
Collaborator

maxzhen commented Jan 16, 2024

Merging it per request by @AShivangi

@maxzhen maxzhen merged commit f0b9740 into Xilinx:master Jan 16, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants