Skip to content

RDKB-63485: Update RFC Defaults for meminsight#243

Open
gomathishankar37 wants to merge 1 commit intodevelopfrom
feature/RDKB-63485
Open

RDKB-63485: Update RFC Defaults for meminsight#243
gomathishankar37 wants to merge 1 commit intodevelopfrom
feature/RDKB-63485

Conversation

@gomathishankar37
Copy link
Contributor

No description provided.

@gomathishankar37 gomathishankar37 requested a review from a team as a code owner February 26, 2026 08:12
Copilot AI review requested due to automatic review settings February 26, 2026 08:12
@gomathishankar37 gomathishankar37 requested a review from a team as a code owner February 26, 2026 08:12
Copy link
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

This PR updates RFC defaults for the meminsight tool across three platform-specific system default files. The changes standardize the comment from "xMemInsight" to "meminsight" and introduce a new xMemTrigger configuration variable with a default value of "stop". This is a follow-up to RDKB-60957 which initially added meminsight defaults.

Changes:

  • Standardized comment from "Defaults for xMemInsight" to "Defaults for meminsight"
  • Added new $xMemTrigger=stop default variable for xd4 and arm platforms
  • Added commented #xMemTrigger=stop for bci platform (following platform-specific convention)

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
source/scripts/init/defaults/system_defaults_xd4 Updated comment and added active xMemTrigger default
source/scripts/init/defaults/system_defaults_bci Updated comment and added commented xMemTrigger default
source/scripts/init/defaults/system_defaults_arm Updated comment and added active xMemTrigger default

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

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