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

Fixes for CR-1167717, CR-1173167, and CR-1173061 #7681

Merged
merged 5 commits into from
Aug 29, 2023

Conversation

pgschuey
Copy link
Collaborator

Problem solved by the commit

  • Don't poll core errors in status on AIE1 devices as it's unreliable
  • Cleaned up AIE status to use local utilities
  • Issue warning when user requests graph iterator-based trace start but design was not compiled to support that
  • Streamlined and fixed bugs in stream switch port requests

Risks (if any) associated the changes in the commit

  • AIE1 users no longer get runtime analysis of errors (but it wasn't reliable anyway)
  • Old xclbins (without graph iterator metadata) will not support graph iterator-based trace start

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

  • Tested on vck190 and vek280

Documentation impact (if any)

  • AIE1 caveat on status?

Paul Schumacher added 4 commits August 22, 2023 13:03
Signed-off-by: Paul Schumacher <[email protected]>
Signed-off-by: Paul Schumacher <[email protected]>
Signed-off-by: Paul Schumacher <[email protected]>
@gbuildx
Copy link
Collaborator

gbuildx commented Aug 29, 2023

Build Passed!

@jvillarre jvillarre merged commit 86dce5a into Xilinx:master Aug 29, 2023
2 checks passed
@pgschuey pgschuey deleted the aie_changes7 branch November 10, 2023 22: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.

3 participants