Skip to content

drivers: unable to take a task with utubettl #244

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

Merged
merged 2 commits into from
May 26, 2025

Conversation

oleg-jukovec
Copy link
Contributor

Previously, the function incorrectly retrieved the lowest priority task instead of the actual task in the ready buffer. This caused a logical error when the current task had the lowest priority.

The implementation error caused:

  1. Incorrect selection of the lowest priority task for a take.
  2. ready buffer appearing empty when tasks were actually in ready state.

@oleg-jukovec oleg-jukovec force-pushed the oleg-jukovec/fix-utubettl-put-ready branch 2 times, most recently from fcedb37 to ceb10ae Compare May 23, 2025 15:34
@oleg-jukovec oleg-jukovec requested review from palage4a and RunsFor May 23, 2025 15:42
@oleg-jukovec oleg-jukovec force-pushed the oleg-jukovec/fix-utubettl-put-ready branch from ceb10ae to ba6a307 Compare May 23, 2025 15:56
Previously, the function incorrectly retrieved the lowest priority
task instead of the actual task in the ready buffer. This caused a
logical error when the current task had the lowest priority.

The implementation error caused:

1. Incorrect selection of the lowest priority task for a take.
2. ready buffer appearing empty when tasks were actually in ready
   state.
@oleg-jukovec oleg-jukovec force-pushed the oleg-jukovec/fix-utubettl-put-ready branch from ba6a307 to 34a289d Compare May 26, 2025 09:26
@oleg-jukovec oleg-jukovec merged commit 11488e6 into master May 26, 2025
5 checks passed
@oleg-jukovec oleg-jukovec deleted the oleg-jukovec/fix-utubettl-put-ready branch May 26, 2025 10:59
@oleg-jukovec oleg-jukovec mentioned this pull request May 26, 2025
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.

2 participants