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.

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

drivers: peci: Add verification handlers #25303

Merged
merged 1 commit into from
May 25, 2020

Conversation

ceolin
Copy link
Member

@ceolin ceolin commented May 13, 2020

Add verification handlers for syscalls defined in peci driver.

Signed-off-by: Flavio Ceolin flavio.ceolin@intel.com

Add verification handlers for syscalls defined in peci driver.

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
@ceolin ceolin added area: Userspace Userspace bug The issue is a bug, or the PR is fixing a bug labels May 13, 2020
@ceolin ceolin requested a review from andrewboie May 13, 2020 21:35
@ceolin ceolin linked an issue May 13, 2020 that may be closed by this pull request
@andrewboie
Copy link
Contributor

At some point we need to update the related test cases to run in user mode so we get code coverage of these handlers.

@ceolin
Copy link
Member Author

ceolin commented May 14, 2020

At some point we need to update the related test cases to run in user mode so we get code coverage of these handlers.

Agree ... Also I think I'll write some script to ensure that declared syscalls have verification handlers defined.

Copy link
Contributor

@franciscomunoz franciscomunoz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nashif nashif added this to the v2.3.0 milestone May 25, 2020
@carlescufi carlescufi merged commit c07f71e into zephyrproject-rtos:master May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Userspace Userspace bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add PECI subsystem user space handlers
6 participants