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

Fix trace error caused by wifi modules #2065

Open
wants to merge 1 commit into
base: celadon/u/mr0/master
Choose a base branch
from

Conversation

RanderWang
Copy link

@RanderWang RanderWang commented Sep 29, 2024

This is a regession issue that caused by commit: c7bb847 "Load mac80211 module based on iwl7000_mac80211 module existence" . The reason is that wifi modules need to be in readtracefs group
to access trace node in /sys/kernel/debug/. This patch adds readtracefs group for this service.

Tracked-On: OAM-125337

@sysopenci
Copy link

one of the dependent pr's program name is not compatable with other dependent prs, for more details please check tracked_on

@sysopenci sysopenci added Developer Approved and removed Pending Developer Approval Pending Developer Approval labels Sep 29, 2024
This is a regession issue that caused by commit: c7bb847
"Load mac80211 module based on iwl7000_mac80211 module existence"
The reason is that wifi modules need to be in readtracefs group
to access trace node in /sys/kernel/debug/. This patch adds
readtracefs group for this service.

Tracked-On: OAM-125337
Signed-off-by: Rander Wang <[email protected]>
@sysopenci
Copy link

one of the dependent pr's program name is not compatable with other dependent prs, for more details please check tracked_on

Copy link
Contributor

@dyang23 dyang23 left a comment

Choose a reason for hiding this comment

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

LGTM

@sysopenci sysopenci added PR approved and removed Pending PR Review Pending PR Review labels Sep 29, 2024
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added Engineering Build Successful Engineering Build Successful and removed Engineering Build Not Started Engineering Build Not Started labels Sep 30, 2024
@sysopenci
Copy link

one of the dependent pr's program name is not compatable with other dependent prs, for more details please check tracked_on

@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants