Skip to content
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

Make syscall internal implementation visibility consistent #12220

Closed
wants to merge 3 commits into from
Closed

Make syscall internal implementation visibility consistent #12220

wants to merge 3 commits into from

Commits on Dec 22, 2018

  1. kernel: syscalls internal implementation inside an object 

    Syscall internal implementations were spread across headers, as static
inline, and C sources. The generated syscall code was declaring all
internal _impl_ as an extern function because of the ones implemented
in C sources. This was violating MISRA-C rule 8.8 in the cases
internal functions were implemented in the headers.

This patch put all internal implementations inside C sources.

MISRA-C rule 8.8

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
    Flavio Ceolin committed Dec 22, 2018
    Configuration menu
    Copy the full SHA
    0c3ce95 View commit details
    Browse the repository at this point in the history

  2. kernel: Avoid redefine functions 

    When userspace was not defined the functions k_object_access_revoke and
k_object_access_all_grant were being defined twice and with different
visibility.

MISRA-C rule 8.8

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
    Flavio Ceolin committed Dec 22, 2018
    Configuration menu
    Copy the full SHA
    78cd946 View commit details
    Browse the repository at this point in the history

  3. doc: kernel: Fix k_object_access_revoke doc 

    API brief explanation was wrong. It was saying that this function
grant, instead of revoking, access to a thread.

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
    Flavio Ceolin committed Dec 22, 2018
    Configuration menu
    Copy the full SHA
    495965b View commit details
    Browse the repository at this point in the history