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

Sepolicy for android_pm_tune script #344

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

amarjeet-intel
Copy link

Adding the required domain and rules for
executing android_pm_tune script to obtain
the power optimization during idle display
conditions.

Tests done : Logs

Tracked-On: OAM-125227

#allow android_pm_tune sysfs:file {getattr open write};
allow android_pm_tune sysfs_net:dir r_dir_perms;
allow android_pm_tune sysfs_net:file rw_file_perms;
allow android_pm_tune sysfs_net:file {getattr};
Copy link
Contributor

Choose a reason for hiding this comment

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

getattr is part of r_file_perms so no need to separately add it

allow android_pm_tune sysfs_net:file {getattr};
allow android_pm_tune sysfs_app_readable:file rw_file_perms;
allow android_pm_tune sysfs_app_readable:dir r_dir_perms;
allow android_pm_tune sysfs_app_readable:file {getattr};
Copy link
Contributor

Choose a reason for hiding this comment

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

same comment applies here

@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

Adding the required domain and rules for
executing android_pm_tune script to obtain
the power optimization during idle display
conditions.

Tests done : Boot Test done

Tracked-On: OAM-125227
Signed-off-by: amarjeet-intel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering Build Not Started Engineering Build Not Started Pending Developer Approval Pending Developer Approval Pending PR Review Pending PR Review Valid commit message
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants