Skip to content

RDKEMW-17755: Update RTK specific kernel settings with defaults and o…#820

Open
plaksh002c-public wants to merge 2 commits intosupport/8.4.4.0from
topic/RDKEMW-17755
Open

RDKEMW-17755: Update RTK specific kernel settings with defaults and o…#820
plaksh002c-public wants to merge 2 commits intosupport/8.4.4.0from
topic/RDKEMW-17755

Conversation

@plaksh002c-public
Copy link
Copy Markdown
Contributor

…ver-ride settings.

Copilot AI review requested due to automatic review settings May 1, 2026 02:26
@plaksh002c-public plaksh002c-public requested review from a team as code owners May 1, 2026 02:26
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the RDK sysctl recipe/template so RTK-specific VM kernel parameters (swappiness, vfs_cache_pressure, watermark_scale_factor) are machine-configurable with sensible defaults, while preserving the prior default runtime behavior.

Changes:

  • Added default BitBake variables for vm.swappiness, vm.vfs_cache_pressure, and vm.watermark_scale_factor.
  • Updated do_install() to substitute the new variables into the generated 98-sysctl-mw.conf.
  • Converted the sysctl config file entries from hard-coded values to template placeholders.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
recipes-core/sysctl/rdksysctl.bb Introduces defaults and adds sed substitutions to generate sysctl config with per-machine overrides.
recipes-core/sysctl/files/98-sysctl-mw.conf Replaces hard-coded VM values with template tokens consumed by the recipe.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread recipes-core/sysctl/rdksysctl.bb Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants