Merge lp://qastaging/~pundiramit/lava-android-test/big-LITTLE into lp://qastaging/lava-android-test

Proposed by Amit Pundir
Status: Merged
Merged at revision: 174
Proposed branch: lp://qastaging/~pundiramit/lava-android-test/big-LITTLE
Merge into: lp://qastaging/lava-android-test
Diff against target: 42 lines (+38/-0)
1 file modified
lava_android_test/test_definitions/big-LITTLE.py (+38/-0)
To merge this branch: bzr merge lp://qastaging/~pundiramit/lava-android-test/big-LITTLE
Reviewer Review Type Date Requested Status
Andy Doan (community) Approve
Review via email: mp+111835@code.qastaging.launchpad.net

Description of the change

add big.LITTLE tests into lava-android-test

To post a comment you must log in.
Revision history for this message
Andy Doan (doanac) wrote :

Looks good. I've tested this under my FM dispatcher code on lava-server05. Some stuff works:

LAVA: (stdout) bl-mod-load01 : PASS
LAVA: (stdout) Verifying cpufreq driver module loaded...
LAVA: (stdout) bl-mod-load02 : PASS
LAVA: (stdout) Loading cpufreq driver module when already loaded...
LAVA: (stdout) insmod: init_module '/system/modules/arm-bl-cpufreq.ko' failed (File exists)
LAVA: (stdout) bl-mod-load03 : PASS
LAVA: (stdout) Unloading cpufreq driver module...
<6>arm-bl-cpufreq: cpufreq backend driver unloaded.
LAVA: (stdout) bl-mod-unload01 : PASS
LAVA: (stdout) Verifying cpufreq driver module unloaded...
LAVA: (stdout) bl-mod-unload02 : PASS

But I also see some failures:
LAVA: (stdout) Setting governor to userspace on all cores...
LAVA: (stdout) /system/bin/governor[38]: cannot create /sys/devices/system/cpu/cpu1/cpufreq/scaling_governor: No such file or directory
LAVA: (stdout) /system/bin/governor[38]: cannot create /sys/devices/system/cpu/cpu2/cpufreq/scaling_governor: No such file or directory

It could be I'm on a week old build or something, but regardless that issue is in the android code base. I think this test looks good. I'll merge it now.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches