-
Notifications
You must be signed in to change notification settings - Fork 2k
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
doc: a note about creating threads w/ same prio #5292
Conversation
honestly, I think this added text sounds a little vague. AFAIK the behavior for multiple threads with the same prio is well defined an deterministic (round robin if equal prio), right? Also, why should the number of threads be 'signifiacantly' lower than the available prio levels, isn't 'num_of_threads <= prio levels' the more precise statement here? |
Could you elaborate on why it is not a good idea? Does it break priority handling? Loss of interrupts? Deadlocks? |
I'm not so sure about the "well defined". AFAIK it is neither well documented nor completely deterministic, in a sense that a thread with the same priority may run as long as it wants and other threads on the same priority gets only scheduled if the first thread yields or another thread with higher priority gets scheduled. Of course, this is somewhat deterministic, too - as basically any computer program is. ;-)
This was not a condition, but a statement.
What's the difference? |
Updated the text and tried to address your comments. |
better :-) ACK when squashed |
0319307
to
57e9a25
Compare
May I ask, why this is added as note? Notes are added to the documentation at the very bottom, but the content seems to me important enough to be in the details section. |
Why should the note go to the bottom? (At least with my local doxygen version that doesn't happen.) |
Oh, you're right. |
backport needed? |
I don't think so, because http://doc.riot-os.org is built from master. |
Something we might want to consider to change, as soon as the documentation is somewhat matured enough. Documentation that refers to newer versions of an API while it is recommended to use a stable version seems a little bit undesirable to me. |
No description provided.