Skip to content

[XDP] Fix to query buffers for memory tiles for aie trace#9535

Closed
vipangul wants to merge 1 commit intoXilinx:masterfrom
vipangul:CR-1250957.memory_tile_graph_settings
Closed

[XDP] Fix to query buffers for memory tiles for aie trace#9535
vipangul wants to merge 1 commit intoXilinx:masterfrom
vipangul:CR-1250957.memory_tile_graph_settings

Conversation

@vipangul
Copy link
Copy Markdown
Collaborator

@vipangul vipangul commented Jan 8, 2026

Problem solved by the commit

CR-1250957 - Memory tiles were not getting configured for aie_trace graph based settings with specific buffers.

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

It has mainly two issues -

  1. buffers were not collected for memory tiles to compared against user provided settings ( instead kernels were used )
  2. In case of memory tiles, Warnings were displaying valid kernels instead valid buffers.

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

Fixed behavior 1 & 2.

Risks (if any) associated the changes in the commit

Low

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

verified original test to get valid aie trace data.

Documentation impact (if any)

Signed-off-by: Vinod Pangul <146476973+vipangul@users.noreply.github.com>
@vipangul vipangul requested a review from pgschuey January 8, 2026 23:08
@vipangul vipangul requested a review from jvillarre as a code owner January 8, 2026 23:08
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 8, 2026

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

@vipangul
Copy link
Copy Markdown
Collaborator Author

vipangul commented Jan 9, 2026

This change will be committed to xdp submodule directly instead.

@vipangul vipangul closed this Jan 9, 2026
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