Skip to content

Conversation

smuppand
Copy link
Contributor

@smuppand smuppand commented Jun 3, 2025

This patch introduces the initial run.sh script for executing Kernel_Selftests within the qcom-linux-testkit framework. Key features include:

  • Basic support for executing kernel selftests from /kselftest based on a whitelist.

  • Logs individual test outcomes (PASS/FAIL/SKIP) to Kernel_Selftests.tests.

  • Summarizes overall result to Kernel_Selftests.res for CI integration.

  • Supports both run_test.sh scripts and standalone test binaries.

  • Architecture-based filtering via enabled_tests.list.

This provides the foundation for validating kernel selftests on Qualcomm platforms and integrates with existing test automation flows.

@vnarapar 

…nner

- Added (or improved) validation to ensure that the enabled_tests.list whitelist file is present and non-empty before executing kernel selftests.
- Script now also checks for the presence of the /kselftest directory and fails early with a clear error if missing.
- This prevents silent failures and improves CI robustness by guaranteeing that test configuration and dependencies are always validated before running.
- Logs explicit error messages for missing or empty enabled_tests.list and for missing /kselftest.
- Follows community best practices for test automation, as seen in major kernel CI flows.

Signed-off-by: Srikanth Muppandam <[email protected]>
Copy link

This pull request has been marked as stale due to 30 days of inactivity. To prevent automatic closure in 7 days, remove the stale label or add a comment. You can reopen a closed pull request at any time.

@github-actions github-actions bot added the Stale label Aug 12, 2025
@smuppand smuppand added bug Something isn't working and removed Stale labels Aug 12, 2025
Copy link
Contributor

@vnarapar vnarapar left a comment

Choose a reason for hiding this comment

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

LGTM

@vnarapar vnarapar merged commit a1ee177 into qualcomm-linux:main Sep 7, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants