Skip to content

[TEST ONLY][DO NOT REVIEW] ll_schedule: don't re-enable the domain if no tasks are active - #4871

Closed
lyakh wants to merge 1 commit into
thesofproject:mainfrom
lyakh:20211008-misc_4
Closed

lyakh wants to merge 1 commit into
thesofproject:mainfrom
lyakh:20211008-misc_4

Conversation

@lyakh

@lyakh lyakh commented Oct 13, 2021

Copy link
Copy Markdown
Collaborator

If the last task completes in a domain, don't re-enable it. It will be re-enabled when a new task is submitted.

If the last task completes in a domain, don't re-enable it. It will
be re-enabled when a new task is submitted.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>

/* tasks on current core finished, re-enable domain on it */
domain_enable(domain, core);
if (atomic_read(&sch->num_tasks))

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

the comment above doesnt match the change?

@keyonjie

Copy link
Copy Markdown
Contributor

Looks good to me, thanks.

@lyakh

lyakh commented Oct 14, 2021

Copy link
Copy Markdown
Collaborator Author

sorry, I should've explained - this is a draft PR because it's just a "bisect" of #4856 where 3 patches cause a single QB failure. This is one of those 3 commits, that I think is the most likely culprit, although I still don't see why it would be wrong. So, this PR is testing-only, not for review.

@lyakh

lyakh commented Oct 14, 2021

Copy link
Copy Markdown
Collaborator Author

SOFCI TEST

@lyakh lyakh changed the title ll_schedule: don't re-enable the domain if no tasks are active [TEST ONLY][DO NOT REVIEW] ll_schedule: don't re-enable the domain if no tasks are active Oct 14, 2021
@lyakh lyakh closed this Oct 15, 2021
@lyakh
lyakh deleted the 20211008-misc_4 branch October 15, 2021 12:43
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