Skip to content

shmbridge: avoid false failures from qcom_scm kernel cmdline#467

Merged
abbajaj806 merged 1 commit into
qualcomm-linux:mainfrom
smuppand:kernel-tests
Jun 1, 2026
Merged

shmbridge: avoid false failures from qcom_scm kernel cmdline#467
abbajaj806 merged 1 commit into
qualcomm-linux:mainfrom
smuppand:kernel-tests

Conversation

@smuppand
Copy link
Copy Markdown
Contributor

The shmbridge test scans current-boot kernel logs for qcom_scm-related errors. The existing match is too broad and can treat benign boot arguments such as qcom_scm.download_mode=1 as failures when other kernel command line parameters contain error-like words such as panic.

Update the log scan to:

  • collect qcom_scm/shmbridge-related lines
  • ignore Kernel command line entries
  • ignore qcom_scm.download_mode
  • ignore CONFIG_QCOM_SCM references
  • fail only when the remaining lines contain real error keywords

This avoids false LAVA failures while preserving detection of genuine qcom_scm or shmbridge errors.

The shmbridge test scans current-boot kernel logs for qcom_scm-related
errors. The existing match is too broad and can treat benign boot
arguments such as qcom_scm.download_mode=1 as failures when other kernel
command line parameters contain error-like words such as panic.

Update the log scan to:
- collect qcom_scm/shmbridge-related lines
- ignore Kernel command line entries
- ignore qcom_scm.download_mode
- ignore CONFIG_QCOM_SCM references
- fail only when the remaining lines contain real error keywords

This avoids false LAVA failures while preserving detection of genuine
qcom_scm or shmbridge errors.

Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>
Copy link
Copy Markdown
Contributor

@abbajaj806 abbajaj806 left a comment

Choose a reason for hiding this comment

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

LGTM

@abbajaj806 abbajaj806 merged commit 43ead87 into qualcomm-linux:main Jun 1, 2026
12 checks passed
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