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

Documentation: add CONFIG_SCHEDSTATS to required kconfigs #26035

Merged
merged 1 commit into from Jun 15, 2023

Conversation

ti-mo
Copy link
Contributor

@ti-mo ti-mo commented Jun 8, 2023

This is a follow-up to #25795 (comment).

Commit 8531c5a ("bpf,datapath: read jiffies from /proc/schedstat") changed the way the kernel jiffy value is obtained. This procfs file is gated behind CONFIG_SCHEDSTATS.

Documentation: add CONFIG_SCHEDSTATS to required kconfigs

Commit 8531c5a ("bpf,datapath: read jiffies from /proc/schedstat")
changed the way the kernel jiffy value is obtained. This procfs file is
gated behind CONFIG_SCHEDSTATS.

Signed-off-by: Timo Beckers <timo@isovalent.com>
@ti-mo ti-mo requested review from a team as code owners June 8, 2023 13:23
@ti-mo ti-mo requested review from aspsk and zacharysarah June 8, 2023 13:23
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jun 8, 2023
Copy link
Contributor

@zacharysarah zacharysarah left a comment

Choose a reason for hiding this comment

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

LGTM from a docs perspective

@ti-mo ti-mo added the release-note/misc This PR makes changes that have no direct user impact. label Jun 14, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jun 14, 2023
@ti-mo ti-mo added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Jun 14, 2023
@ti-mo
Copy link
Contributor Author

ti-mo commented Jun 14, 2023

One-liner doc change, applying ready-to-merge.

@aanm aanm merged commit 7f4bf25 into cilium:main Jun 15, 2023
38 checks passed
@ti-mo ti-mo deleted the tb/document-schedstat-req branch June 18, 2023 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants