-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
kernel: Add missing marshalling header for k_reschedule #83615
base: main
Are you sure you want to change the base?
Conversation
Hello @thughes, and thank you very much for your first pull request to the Zephyr project! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm... how is it not using the function? This should be followed by an include of the generated "...mrsh.c" marshalling code which references it (the marshalling function then gets stuffed into the global table of syscalls handlers).
I suspect the real problem is elsewhere. Did the include get forgotten?
Yeah, that seems like it. Just copy over the marshalling boilerplate from another function. This is a common goof; we'll add a syscalls and test coverage for it, but then forget to have coverage calling it from a userspace process. |
39f91a6
to
a1a3573
Compare
Without this header, compiling the kernel.poll test with -Werror=unused-function fails. Signed-off-by: Tom Hughes <tomhughes@chromium.org>
a1a3573
to
aa6c27c
Compare
Without this header, compiling the kernel.poll test with
-Werror=unused-function
fails.