Skip to content

[XDP] Support for zocl multi partition for AIE Trace Plugin #9122

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

parthash0804
Copy link
Collaborator

@parthash0804 parthash0804 commented Aug 4, 2025

Problem solved by the commit

Added support for zocl multipartition for AIE Trace Plugin.

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

NA

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

  1. Using unique device ID for each partition.
  2. Storing XAieDev Instance corresponding to each partition in static DB.
  3. Generating seperate trace reports for each parition.

Risks (if any) associated the changes in the commit

Needs more thorough testing for various flow.

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

Tested ZOCL Multipartition design on VCK190
Tested single parition design on vck190 with register xclbin and load xclbin flow.
Tested independent compilation design for both PLIO and GMIO offload on VCK190.
Tested ML Design on VE2.
Tested Single parition design on Strix

Documentation impact (if any)

NA

@parthash0804 parthash0804 requested a review from jvillarre as a code owner August 4, 2025 18:39
@parthash0804 parthash0804 added the do not merge hold off on merging label Aug 4, 2025
Copy link

github-actions bot commented Aug 4, 2025

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

@parthash0804 parthash0804 force-pushed the zocl_multi_partition branch from 8830e2d to 5a2fa02 Compare August 5, 2025 07:31
Copy link

github-actions bot commented Aug 5, 2025

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

Copy link
Collaborator

@IshitaGhosh IshitaGhosh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good. Please remove the CR number.

Signed-off-by: parthash0804 <[email protected]>
Copy link

github-actions bot commented Aug 7, 2025

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

@parthash0804 parthash0804 removed the do not merge hold off on merging label Aug 7, 2025
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.

2 participants