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

Mark xcu status done in CU XGQ check #8697

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

karthdmg-xilinx
Copy link
Collaborator

Problem solved by the commit

Mark xcu status done in CU XGQ check.
This change is made to mark the done status of xcu xgq command to avoid calling down_timeout() on semaphore twice.

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

The entry in xgq completion list may not have been added by the time the cu interrupt thread polls and calls down_timout()
on semaphore. If the command done status is not set, the down_timeout() is called twice resulting in CU_TIMER timeout and causing unnecessary delay.

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

Mark command status done so that the down_timeout() is called only once.

Risks (if any) associated the changes in the commit

None

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

Ran the filter2d application multiple times and verified the no CU_TIMER delay is encountered.

Documentation impact (if any)

None

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