-
Notifications
You must be signed in to change notification settings - Fork 506
Support multi ctrlpkt sections in ELF #9306
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
Conversation
clang-tidy review says "All clean, LGTM! 👍" |
what patching scheme are we using for AIE4 control packet? |
May be @Himanshu-xilinx can answer this. |
patching schema for controlpacket
@rbramand-xilinx |
Thanks Himanshu, will add these changes in XRT |
Signed-off-by: Rahul Bramandlapalli <rbramand@amd.com>
08015e2
to
dc7d43e
Compare
clang-tidy review says "All clean, LGTM! 👍" |
Signed-off-by: Rahul Bramandlapalli <rbramand@amd.com>
clang-tidy review says "All clean, LGTM! 👍" |
@larry9523 , @HimanshuChoudhary-Xilinx added ctrlpkt patching in aie4, please review |
Problem solved by the commit
New spec changes in ELF for aie2ps/aie4 introduces ctrlpkt data as separate sections. Spec for same is captured at - https://amd.atlassian.net/wiki/spaces/AIE/pages/1099675323/Multi+uc+control+packet+design
Added changes in XRT to support these new changes in ELF.
Changes are backward compatible (tests with older ELFs will also work)
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
Separate xrt::bo objects were created for each ctrlpkt section, and these were populated with the corresponding section data. Added logic to patch these ctrlpkt bo's address in control code.
The changes in XRT are designed to be data-driven. Both existing ELF files (with ctrlpkt data embedded in the .pad section) and proposed ELF files (with ctrlpkt data in a dedicated section) are supported.
Risks (if any) associated the changes in the commit
Low
What has been tested and how, request additional testing if necessary
Tested eff_net_ctrlpkt test case on Telluride board with existing ELF and new ELF and both work as expected.
Documentation impact (if any)
NA