Skip to content

Conversation

smuppand
Copy link
Contributor

@smuppand smuppand commented Jun 3, 2025

This patch refactors the CPUFreq_Validation run.sh script to use cpufreq policies instead of per-core logic, ensuring compatibility across various SoCs with shared frequency domains.

Key improvements:

  • Automatically detects all cpufreq policies under /sys/devices/system/cpu/cpufreq/
  • Associates and validates frequency scaling across all CPUs in each policy group
  • Uses the 'userspace' governor for deterministic frequency control
  • Tracks and logs detailed per-policy results (e.g., "CPU0-3 [via policy0] = PASS")
  • Improves CI stability by filtering out false negatives due to platform-specific quirks
  • Incorporates ShellCheck style and POSIX safety fixes

This update enhances portability, clarity, and reliability of CPU frequency testing across Qualcomm platforms and beyond.

@vnarapar

@smuppand smuppand requested a review from vnarapar June 3, 2025 12:29
…acking

This update replaces the per-core cpufreq walker with a more robust,
policy-aware implementation. The new script:
- Identifies cpufreq policies and associated CPU cores via sysfs
- Validates frequency scaling for each policy, using 'userspace' governor
- Logs per-policy results with PASS/FAIL based on actual frequency behavior
- Improves compatibility across SoCs with shared cpufreq domains

Signed-off-by: Srikanth Muppandam <[email protected]>
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 4d23ccb into qualcomm-linux:main Jun 4, 2025
6 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