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 AIE Halt Plugin to set Debug Halt on AIE Cores for debugging on Client #8344

Merged
merged 6 commits into from
Aug 12, 2024

Conversation

IshitaGhosh
Copy link
Collaborator

Problem solved by the commit

Debugging tools for client require an initial setup before txn starts and so AIE cores need to be paused. This XDP AIE Halt plugin provides user-space solution for the setup requirement of debugging tool.
It sets debug halts on AIE cores of current design via txn.

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

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

Earlier, design control code needed to be recompiled with halts so that it can be debugging. This recompilation step is not needed now and saves time.

Risks (if any) associated the changes in the commit

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

Unit tests. This has been used successfully with Debugger too.

Documentation impact (if any)

Ishita Ghosh added 6 commits August 10, 2024 16:34
… using txn

Signed-off-by: Ishita Ghosh <ishita.ghosh@amd.com>
Signed-off-by: Ishita Ghosh <ishita.ghosh@amd.com>
Signed-off-by: Ishita Ghosh <ishita.ghosh@amd.com>
Signed-off-by: Ishita Ghosh <ishita.ghosh@amd.com>
Signed-off-by: Ishita Ghosh <ishita.ghosh@amd.com>
…text, using transaction

Signed-off-by: Ishita Ghosh <ishita.ghosh@amd.com>
Copy link
Collaborator

@jvillarre jvillarre left a comment

Choose a reason for hiding this comment

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

Approved on the understanding that this should only be used on debugging cases where more there is only one hardware context created.

@jvillarre jvillarre merged commit ae4a667 into Xilinx:master Aug 12, 2024
17 checks passed
@IshitaGhosh IshitaGhosh deleted the haltPlugIn branch September 14, 2024 17:12
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