Skip to content

remove _current NULL check in ARM's _is_thread_user() #4082

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.

Sign up for GitHub

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

Jump to bottom
Closed
andrewboie opened this issue Sep 26, 2017 · 3 comments
Closed

remove _current NULL check in ARM's _is_thread_user() #4082

andrewboie opened this issue Sep 26, 2017 · 3 comments
Assignees
@andrewboie
Labels
area: Kernel bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@andrewboie
Copy link
Contributor

andrewboie commented Sep 26, 2017 •
edited by nashif
Loading

See #4080 . the call to k_object_access_grant() in setup_new_thread() makes this necessary. revise the code so we don't need the extra NULL check in this very hot code path.
@andrewboie andrewboie self-assigned this Sep 26, 2017
@andrewboie andrewboie mentioned this issue Sep 26, 2017
Merged
@andrewboie andrewboie added area: Kernel bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug labels Sep 26, 2017
@nashif
Copy link
Member

nashif commented Jul 25, 2018

@andrewboie is this still valid? we do not have k_object_grant_access anymore :)

@andrewboie
Copy link
Contributor Author

@nashif it is, the API is called k_object_access_grant()

@andrewboie
Copy link
Contributor Author

Not a problem any more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andrewboie andrewboie

Labels
area: Kernel bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nashif @andrewboie