Hello, pm-qa/powertop test fails on ARM, even though it succeeds for Intel.
on both platforms, powertop installed using command - "sudo apt-get install powertop" powertop_01.1: checking if reports are generated... Ok [on Intel] powertop_01.1: checking if reports are generated... Err [on ARM] It looks like "report_name=PowerTOP*.csv" does not work for ARM. On ARM, report name are lowercase. So report_name should be "powertop*.csv". If I build from git from (git.linaro.org), generated report file name is powertop.csv. So even then it needs to be lowercase. I made a patch to make it work on both platforms (even though the actual fix is probably to make the output filenames similar in both cases). Please comment. --------- diff --git a/powertop/powertop_01.sh b/powertop/powertop_01.sh index c66b725..2768924 100755 --- a/powertop/powertop_01.sh +++ b/powertop/powertop_01.sh @@ -34,6 +34,12 @@ run_powertop() { local seconds=10 local iterations=2 local report_name=PowerTOP*.csv + local processor=`uname -p` + + case "$processor" in + arm*) + report_name=powertop*.csv + esac # remove old reports if exists rm -f $report_name -- Thanks, -Meraj
_______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev