cpu_scaling_test fails even if measured speedup exceeds expected speedup
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Checkbox |
Fix Released
|
Medium
|
Brendan Donegan | ||
Bug Description
As per this output:
ubuntu@
System Capabilites:
-------
System has 4 cpus
Supported CPU Frequencies:
2501 MHz
2500 MHz
2400 MHz
2300 MHz
2200 MHz
2100 MHz
2000 MHz
1900 MHz
1800 MHz
1700 MHz
1600 MHz
Supported Governors:
conservative
ondemand
userspace
powersave
performance
Current governors:
cpu0: userspace
cpu1: userspace
cpu2: userspace
cpu3: userspace
Userspace Governor Test:
-------
Setting governor to userspace
Setting CPU frequency to 1600 MHz
Running CPU load test...
pid 13619's current affinity list: 0-3
pid 13619's new affinity list: 0
Minimum frequency load test time: 4.56
Setting CPU frequency to 2501 MHz
Running CPU load test...
pid 13619's current affinity list: 0
pid 13619's new affinity list: 0
Maximum frequency load test time: 2.22
Note: found ida flag, increasing expected speedup by 8.0%
CPU Frequency Speed Up: 1.70
Measured Speed Up: 2.05
Percentage Difference 20.9%
Error: measured speedup vs expected speedup is 20.9% and is not within 10.0% margin.
On Demand Governor Test:
-------
Setting governor to ondemand
Waiting 5 seconds... done.
Running CPU load test...
pid 13619's current affinity list: 0
pid 13619's new affinity list: 0
On Demand load test time: 2.23
Percentage Difference vs. maximum frequency: 0.5%
Waiting 5 seconds... done.
Performance Governor Test:
-------
Setting governor to performance
Running CPU load test...
pid 13619's current affinity list: 0
pid 13619's new affinity list: 0
Performance load test time: 2.22
Percentage Difference vs. maximum frequency: 0.0%
Conservative Governor Test:
-------
Setting governor to conservative
Waiting 10 seconds... done.
Running CPU load test...
pid 13619's current affinity list: 0
pid 13619's new affinity list: 0
Conservative load test time: 4.55
Percentage Difference vs. minimum frequency: 0.2%
Restoring original governor to userspace
ubuntu@
1
The cpu_scaling_test script can fail if the measured speedup actually *exceeds* the expected speedup. Obviously this is not a cause for concern so the script should only fail if the measured speedup is less than the expected speedup by a significant amount.
Related branches
- Zygmunt Krynicki (community): Approve
-
Diff: 35 lines (+8/-2)2 files modifieddebian/changelog (+3/-0)
scripts/frequency_governors_test (+5/-2)
- Ubuntu branches: Pending requested
-
Diff: 114866 lines (+22292/-18992)125 files modifiedcheckbox_qt/qt_interface.py (+7/-2)
debian/changelog (+51/-0)
debian/po/ast.po (+2/-2)
debian/po/cs.po (+2/-2)
debian/po/de.po (+2/-2)
debian/po/en_AU.po (+2/-2)
debian/po/en_GB.po (+2/-2)
debian/po/es.po (+2/-2)
debian/po/fr.po (+2/-2)
debian/po/he.po (+2/-2)
debian/po/hu.po (+2/-2)
debian/po/id.po (+2/-2)
debian/po/it.po (+2/-2)
debian/po/ja.po (+2/-2)
debian/po/nl.po (+2/-2)
debian/po/oc.po (+2/-2)
debian/po/pl.po (+2/-2)
debian/po/pt_BR.po (+2/-2)
debian/po/ro.po (+2/-2)
debian/po/ru.po (+2/-2)
debian/po/tr.po (+2/-2)
debian/po/uk.po (+2/-2)
debian/po/zh_CN.po (+2/-2)
debian/po/zh_TW.po (+2/-2)
jobs/audio.txt.in (+6/-6)
jobs/networking.txt.in (+4/-1)
jobs/suspend.txt.in (+5/-4)
patches/0.14.2 (+2/-1)
po/ace.po (+199/-198)
po/af.po (+199/-198)
po/am.po (+199/-198)
po/ar.po (+219/-203)
po/ast.po (+199/-198)
po/az.po (+199/-198)
po/be.po (+199/-198)
po/bg.po (+199/-198)
po/bn.po (+199/-198)
po/bo.po (+199/-198)
po/br.po (+199/-198)
po/bs.po (+199/-198)
po/ca.po (+199/-198)
po/ca@valencia.po (+199/-198)
po/ckb.po (+199/-198)
po/cs.po (+199/-198)
po/cy.po (+199/-198)
po/da.po (+199/-198)
po/de.po (+179/-178)
po/dv.po (+199/-198)
po/el.po (+199/-198)
po/en_AU.po (+415/-414)
po/en_CA.po (+199/-198)
po/en_GB.po (+891/-504)
po/eo.po (+202/-201)
po/es.po (+462/-216)
po/et.po (+199/-198)
po/eu.po (+199/-198)
po/fa.po (+199/-198)
po/fi.po (+202/-201)
po/fr.po (+1576/-489)
po/ga.po (+199/-198)
po/gd.po (+199/-198)
po/gl.po (+261/-208)
po/he.po (+199/-198)
po/hi.po (+199/-198)
po/hr.po (+199/-198)
po/hu.po (+199/-198)
po/hy.po (+199/-198)
po/id.po (+199/-198)
po/is.po (+199/-198)
po/it.po (+199/-198)
po/ja.po (+199/-198)
po/jbo.po (+199/-198)
po/ka.po (+199/-198)
po/kk.po (+199/-198)
po/km.po (+199/-198)
po/kn.po (+199/-198)
po/ko.po (+199/-198)
po/ku.po (+199/-198)
po/ky.po (+199/-198)
po/lt.po (+199/-198)
po/lv.po (+199/-198)
po/mk.po (+199/-198)
po/ml.po (+199/-198)
po/mr.po (+199/-198)
po/ms.po (+666/-251)
po/my.po (+199/-198)
po/nb.po (+199/-198)
po/nds.po (+199/-198)
po/ne.po (+199/-198)
po/nl.po (+199/-198)
po/nn.po (+199/-198)
po/oc.po (+204/-199)
po/pl.po (+199/-198)
po/ps.po (+199/-198)
po/pt.po (+266/-257)
po/pt_BR.po (+626/-236)
po/ro.po (+199/-198)
po/ru.po (+629/-222)
po/sd.po (+199/-198)
po/shn.po (+199/-198)
po/si.po (+199/-198)
po/sk.po (+199/-198)
po/sl.po (+288/-225)
po/sq.po (+217/-198)
po/sr.po (+199/-198)
po/sv.po (+199/-198)
po/ta.po (+199/-198)
po/te.po (+199/-198)
po/th.po (+199/-198)
po/tr.po (+204/-203)
po/ug.po (+199/-198)
po/uk.po (+218/-217)
po/ur.po (+199/-198)
po/uz.po (+199/-198)
po/vi.po (+199/-198)
po/zh_CN.po (+199/-198)
po/zh_HK.po (+199/-198)
po/zh_TW.po (+199/-198)
qt/frontend/qtfront.ui (+2/-2)
scripts/audio_settings (+12/-2)
scripts/color_depth_info (+1/-1)
scripts/frequency_governors_test (+5/-2)
scripts/graphics_driver (+4/-4)
scripts/graphics_modes_info (+1/-1)
scripts/network_device_info (+95/-44)
| Sylvain Pineau (sylvain-pineau) wrote | #1 |
| Changed in checkbox: | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| Changed in checkbox: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Brendan Donegan (brendan-donegan) |
| Changed in checkbox: | |
| status: | In Progress → Fix Committed |
| Changed in checkbox: | |
| status: | Fix Committed → Fix Released |
The test: "if differenceSpeedUp > self.speedUpTol
Warning:
Since Rev 1490, scripts/