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.

Sign up for GitHub

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

Jump to bottom

[XDP] First cut implementation of AIE Debug feature for all devices #8701

Merged
merged 63 commits into from
Jan 24, 2025
Merged

[XDP] First cut implementation of AIE Debug feature for all devices #8701

merged 63 commits into from
Jan 24, 2025

Conversation

pgschuey
Copy link
Collaborator

Problem solved by the commit

  • Observability is needed to see the configurations of AI Engines

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

  • None

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

  • Expanded capabilities of AIE Debug plugin and supporting writer

Risks (if any) associated the changes in the commit

  • Lots of file changes

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

  • Compiled on client and VE2
  • Tested on vck190 and vek280

Documentation impact (if any)

  • Internal feature for now

Paul Schumacher and others added 30 commits October 15, 2024 15:48
initial debug plugin changes
84e8de8 
Signed-off-by: Paul Schumacher <[email protected]>
header fixes
f4efa88 
Signed-off-by: Paul Schumacher <[email protected]>
fixed compilation issues
7ea81b3 
Signed-off-by: Paul Schumacher <[email protected]>
@predutta
AIE Debug Plugin Edge implementation
47f393c 
Signed-off-by: Prerona Dutta <[email protected]>
@predutta
AIE Debug Plugin Edge implementation with necesssary hooks
c6153a5 
Signed-off-by: Prerona Dutta <[email protected]>
@predutta
Resolving hw_emu related linking issues
97cc763 
Signed-off-by: Prerona Dutta <[email protected]>
@predutta
Fixing Seg fault and populating register issues
df3afdc 
Signed-off-by: Prerona Dutta <[email protected]>
@predutta
Adding capability to specify register names
ca49669 
Signed-off-by: Prerona Dutta <[email protected]>
@predutta
Adding python script to populate register name to value map for writer
7eb5310 
Signed-off-by: Prerona Dutta <[email protected]>
@predutta
adding writer
b5422ad 
Signed-off-by: Prerona Dutta <[email protected]>
@predutta
Adding code for writer and some refactoring
ee3df7a 
Signed-off-by: Prerona Dutta <[email protected]>
@predutta
Removing AIE4 and adding TODO comments
3ae4300 
Signed-off-by: Prerona Dutta <[email protected]>
initial improvements
bd9aaaa 
Signed-off-by: Schumacher <[email protected]>
merge of master
459fa94 
Signed-off-by: Schumacher <[email protected]>
cleaned up generations subdir
f7e801c 
Signed-off-by: Schumacher <[email protected]>
compile fixes
5a3e559 
Signed-off-by: Schumacher <[email protected]>
more compile corrections
50ab298 
Signed-off-by: Schumacher <[email protected]>
Fixing mismatched data type in edge/poll()
198dcff
removed debug messages and default is all
a8e947b 
Signed-off-by: Paul Schumacher <[email protected]>
streamlined code
720f17b 
Signed-off-by: Paul Schumacher <[email protected]>
temp debug messages
000e76e 
Signed-off-by: Paul Schumacher <[email protected]>
fixed bug
1febfe0 
Signed-off-by: Paul Schumacher <[email protected]>
debugging
e80308b 
Signed-off-by: Paul Schumacher <[email protected]>
fixed bug
a62d531 
Signed-off-by: Paul Schumacher <[email protected]>
moved code around
1d3fb94 
Signed-off-by: Paul Schumacher <[email protected]>
fixes with Prerona
5491a41 
Signed-off-by: Paul Schumacher <[email protected]>
bug fix
20a170e 
Signed-off-by: Paul Schumacher <[email protected]>
const char now string
cc2096e 
Signed-off-by: Paul Schumacher <[email protected]>
bug fixes in database & writer
b9de87c 
Signed-off-by: Paul Schumacher <[email protected]>
fixed order bug
29a5f61 
Signed-off-by: Paul Schumacher <[email protected]>
predutta and others added 11 commits January 14, 2025 14:52
@predutta
fixing bug
ca88286 
Signed-off-by: Prerona Dutta <[email protected]>
merge with master
b1a1f26 
Signed-off-by: Paul Schumacher <[email protected]>
updated headers and register sets
4b6630c 
Signed-off-by: Paul Schumacher <[email protected]>
Minor bug fixes
68bf579
removed a lot of uuuuunnecessary regs
e89bd1f 
Signed-off-by: Paul Schumacher <[email protected]>
Fixing bugs in aie_debug/writer
a06d419
Resolve conflicts
a2aae7d
client improvements
f9ee9c9 
Signed-off-by: Paul Schumacher <[email protected]>
@vipangul
Successful client build
6aa7c46 
Signed-off-by: Vinod Pangul <[email protected]>
updated writer
bd255c3 
Signed-off-by: Paul Schumacher <[email protected]>
writer and bit width updates
46f5004 
Signed-off-by: Paul Schumacher <[email protected]>
@pgschuey pgschuey requested a review from predutta January 17, 2025 21:39
Paul Schumacher and others added 10 commits January 17, 2025 22:05
removed driver include and native x86 builds for debug plugin
d20ffa5 
Signed-off-by: Paul Schumacher <[email protected]>
@vipangul
PASS: Client build
b4f1ca9 
Signed-off-by: Vinod Pangul <[email protected]>
@vipangul
Merge branch 'brAieDebugPlugin' of https://github.com/pgschuey/XRT in…
6699fd8 
…to brAieDebugPlugin
added back driver include
d72319d 
Signed-off-by: Paul Schumacher <[email protected]>
updated comments in cmake files
37d4872 
Signed-off-by: Paul Schumacher <[email protected]>
@vipangul
Merge branch 'brAieDebugPlugin' of https://github.com/pgschuey/XRT in…
3ef6c75 
…to brAieDebugPlugin
cleanup of branch
8cab2ab 
Signed-off-by: Paul Schumacher <[email protected]>
DCO Remediation Commit for Snigdha Gupta <[email protected]…
2ec3902 
…x.xilinx.com>

I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: 198dcff
I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: 1d797c6
I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: 85017b1
I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: 7e44a7b
I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: 4d58363
I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: f5ecbfc
I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: 4a0ae01
I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: f9e83bf
I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: 68bf579
I, Snigdha Gupta <[email protected]>, hereby add my Signed-off-by to this commit: a06d419

Signed-off-by: Snigdha Gupta <[email protected]>
build AIE debug for VE2
f44d5fb 
Signed-off-by: Paul Schumacher <[email protected]>
Merge branch 'master' of https://github.com/Xilinx/XRT into brAieDebu…
bdd2a3e 
…gPlugin
jvillarre
jvillarre approved these changes Jan 24, 2025
View reviewed changes
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.

Known issues discussed internally to be addressed in future pull requests.

@jvillarre jvillarre merged commit 81ca1b7 into Xilinx:master Jan 24, 2025
20 checks passed
rchane pushed a commit to rchane/XRT that referenced this pull request Jan 24, 2025
@pgschuey @rchane
[XDP] First cut implementation of AIE Debug feature for all devices (X…
b667ded 
…ilinx#8701)
@parthash0804 parthash0804 mentioned this pull request Jan 27, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jvillarre jvillarre jvillarre approved these changes

@predutta predutta Awaiting requested review from predutta

@chvamshi-xilinx chvamshi-xilinx Awaiting requested review from chvamshi-xilinx chvamshi-xilinx is a code owner

@stsoe stsoe Awaiting requested review from stsoe stsoe is a code owner

@rozumx rozumx Awaiting requested review from rozumx rozumx is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pgschuey @jvillarre @predutta @vipangul