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

[XDP] Add changes to support new profile format and update #3

Closed
wants to merge 11 commits into from

Conversation

vipangul
Copy link
Owner

@vipangul vipangul commented Jul 9, 2024

Problem solved by the commit

  • This is an enhancement where user set metric sets are also reported in the profile report. are expanded and converted to valid metric sets at runtime.
    • Update the AIe Profile offsets to uniquely and easily distinguish between different AIE modules.

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

An enhancement in report format.

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

An user provided metric sets for aie profile gets expanded into valid metrics sets and mapped to corresponding tiles.
This forms the valid AIE Profile configuration which is being used to report in the new aie profile report.
Further vp_analyzer recognizes this new information based on file version and packs it in protobuf message for Rigel to visualize.

Risks (if any) associated the changes in the commit

Minimal, backward compatibility needs to be re-verified.

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

Verified on edge vck190

Documentation impact (if any)

@vipangul vipangul closed this Jul 9, 2024
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.

1 participant