cpu_offlining script doesn't work properly on ARM
Bug #1078897 reported by
Jeff Lane
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Checkbox |
Fix Released
|
Low
|
Jeff Lane | ||
Bug Description
Attempted to run the cpu_offlining script on the AC100 running the armhf image for Quantal. This is the result:
bladernr@
Beginning CPU Offlining Test
Offlining cpu0
./cpu_offlining: line 11: echo: write error: Operation not permitted
ERROR: Failed to offline cpu0
Offlining cpu1
Onlining cpu0
./cpu_offlining: line 25: echo: write error: Invalid argument
Onlining cpu1
it appears that the offlining and onlining of cpu1 is successful, but it attempts the same on cpu0 for some reason and gets a write error. The two lines in question are attempting to write bits into the online cpu hook in sys:
/sys/devices/
interestingly, on the AC100 at least, cpu0 has an online file... this is generally NOT present for cpu0 on x86/amd64
One solution woudl be to simply fix the script to skip processor 0 by default. However, it IS entirely possible, though rare, that cpu0 is not the primary core... there is always the possibility that CPU0 is Core X on the chip depending on how the chip was designed or which cores were cut during assembly and testing at the factory. Usually this is not an issue though, it's a very small corner case, IME.
Related branches
lp:~bladernr/checkbox/1078897-dont-offline-cpu0
- Daniel Manrique (community): Approve
-
Diff: 78 lines (+29/-14)2 files modifieddebian/changelog (+4/-1)
scripts/cpu_offlining (+25/-13)
lp:~roadmr/ubuntu/saucy/checkbox/0.16
- Daniel Manrique (community): Approve
- Ubuntu branches: Pending requested
-
Diff: 58223 lines (+46783/-2840)308 files modifiedVagrantfile (+25/-16)
bin/checkbox-hw-collection (+19/-0)
checkbox/contrib/xrandr.py (+132/-59)
checkbox/lib/fifo.py (+6/-1)
checkbox/lib/template.py (+4/-4)
checkbox/message.py (+17/-15)
checkbox/parsers/submission.py (+1/-1)
checkbox/parsers/tests/fixtures/submission_info_memory.xml (+1/-1)
checkbox/parsers/tests/test_cputable.py (+4/-4)
checkbox/parsers/tests/test_description.py (+17/-17)
checkbox/parsers/tests/test_dmi.py (+13/-13)
checkbox/parsers/tests/test_efi.py (+5/-5)
checkbox/parsers/tests/test_submission.py (+43/-50)
checkbox/parsers/tests/test_xinput.py (+21/-26)
checkbox/parsers/udevadm.py (+3/-2)
checkbox/test_resource.py (+347/-0)
checkbox/tests/__init__.py (+49/-0)
checkbox/tests/test_job.py (+5/-4)
checkbox/tests/test_message_files.py (+3/-2)
checkbox/tests/test_setup_files.py (+4/-2)
checkbox/user_interface.py (+6/-11)
checkbox/variables.py (+2/-1)
checkbox_cli/cli_interface.py (+3/-3)
checkbox_gtk/gtk_interface.py (+2/-2)
checkbox_qt/qt_interface.py (+3/-3)
checkbox_urwid/urwid_interface.py (+2/-2)
data/whitelists/hwsubmit.whitelist (+24/-0)
data/whitelists/sru.whitelist (+114/-0)
debian/changelog (+142/-0)
debian/checkbox-hw-collection.install (+2/-0)
debian/checkbox-hw-collection.links (+1/-0)
debian/checkbox-hw-collection.postinst (+7/-0)
debian/checkbox.install (+1/-0)
debian/control (+13/-0)
debian/po/ast.po (+3/-4)
debian/po/cs.po (+3/-4)
debian/po/de.po (+5/-5)
debian/po/en_AU.po (+3/-4)
debian/po/en_GB.po (+3/-4)
debian/po/es.po (+7/-6)
debian/po/fr.po (+3/-4)
debian/po/gl.po (+3/-4)
debian/po/he.po (+3/-4)
debian/po/hu.po (+3/-4)
debian/po/id.po (+3/-4)
debian/po/it.po (+3/-4)
debian/po/ja.po (+3/-4)
debian/po/nl.po (+3/-4)
debian/po/oc.po (+3/-4)
debian/po/pl.po (+3/-4)
debian/po/pt_BR.po (+3/-4)
debian/po/ro.po (+3/-4)
debian/po/ru.po (+3/-4)
debian/po/tr.po (+3/-4)
debian/po/uk.po (+3/-4)
debian/po/zh_CN.po (+3/-4)
debian/po/zh_TW.po (+3/-4)
debian/rules (+1/-0)
examples/checkbox-cli.ini (+3/-0)
examples/checkbox-hw-collection.ini (+18/-0)
examples/checkbox-qt.ini (+4/-0)
examples/checkbox-urwid.ini (+3/-0)
examples/network.cfg (+4/-0)
jobs/audio.txt.in (+4/-0)
jobs/benchmarks.txt.in (+68/-0)
jobs/camera.txt.in (+0/-1)
jobs/esata.txt.in (+1/-1)
jobs/firewire.txt.in (+1/-1)
jobs/graphics.txt.in (+3/-1)
jobs/hibernate.txt.in (+8/-2)
jobs/info.txt.in (+1/-1)
jobs/input.txt.in (+11/-1)
jobs/keys.txt.in (+8/-2)
jobs/led.txt.in (+45/-14)
jobs/local.txt.in (+7/-0)
jobs/mediacard.txt.in (+50/-33)
jobs/miscellanea.txt.in (+52/-1)
jobs/mobilebroadband.txt.in (+21/-0)
jobs/monitor.txt.in (+4/-4)
jobs/networking.txt.in (+4/-4)
jobs/power-management.txt.in (+57/-21)
jobs/rendercheck.txt.in (+1/-1)
jobs/resource.txt.in (+13/-0)
jobs/stress.txt.in (+15/-3)
jobs/suspend.txt.in (+96/-39)
jobs/usb.txt.in (+6/-6)
jobs/virtualization.txt.in (+16/-0)
jobs/wireless.txt.in (+13/-13)
plainbox/.coveragerc (+11/-0)
plainbox/MANIFEST.in (+5/-0)
plainbox/README.md (+0/-122)
plainbox/README.rst (+23/-0)
plainbox/Vagrantfile (+42/-0)
plainbox/daily-package-testing/README (+18/-0)
plainbox/daily-package-testing/Vagrantfile (+48/-0)
plainbox/daily-package-testing/test-in-vagrant.sh (+59/-0)
plainbox/docs/appdev/index.rst (+39/-0)
plainbox/docs/author/index.rst (+63/-0)
plainbox/docs/conf.py (+13/-1)
plainbox/docs/dev/architecture.rst (+7/-305)
plainbox/docs/dev/config.rst (+253/-0)
plainbox/docs/dev/index.rst (+12/-7)
plainbox/docs/dev/old.rst (+339/-0)
plainbox/docs/dev/reference.rst (+15/-0)
plainbox/docs/dev/resources.rst (+259/-0)
plainbox/docs/index.rst (+8/-5)
plainbox/docs/usage.rst (+29/-5)
plainbox/mk-interesting-graphs.sh (+2/-2)
plainbox/mk-venv.sh (+181/-26)
plainbox/plainbox/data/org.freedesktop.policykit.pkexec.policy (+30/-0)
plainbox/plainbox/data/plainbox-ci-mailer.conf (+44/-0)
plainbox/plainbox/data/plainbox.conf (+26/-0)
plainbox/plainbox/data/plainbox.desktop (+10/-0)
plainbox/plainbox/data/report/hardware-1_0.rng (+533/-0)
plainbox/plainbox/impl/applogic.py (+173/-0)
plainbox/plainbox/impl/box.py (+13/-386)
plainbox/plainbox/impl/checkbox.py (+161/-74)
plainbox/plainbox/impl/commands/check_config.py (+92/-0)
plainbox/plainbox/impl/commands/checkbox.py (+101/-0)
plainbox/plainbox/impl/commands/run.py (+319/-0)
plainbox/plainbox/impl/commands/special.py (+133/-0)
plainbox/plainbox/impl/commands/sru.py (+270/-0)
plainbox/plainbox/impl/commands/test_run.py (+28/-0)
plainbox/plainbox/impl/commands/test_sru.py (+63/-0)
plainbox/plainbox/impl/config.py (+538/-0)
plainbox/plainbox/impl/depmgr.py (+59/-5)
plainbox/plainbox/impl/exporter/__init__.py (+53/-8)
plainbox/plainbox/impl/exporter/json.py (+12/-5)
plainbox/plainbox/impl/exporter/rfc822.py (+5/-2)
plainbox/plainbox/impl/exporter/test_init.py (+22/-3)
plainbox/plainbox/impl/exporter/test_json.py (+13/-7)
plainbox/plainbox/impl/exporter/test_rfc822.py (+47/-0)
plainbox/plainbox/impl/exporter/test_text.py (+43/-0)
plainbox/plainbox/impl/exporter/test_xml.py (+96/-0)
plainbox/plainbox/impl/exporter/text.py (+3/-3)
plainbox/plainbox/impl/exporter/xml.py (+385/-0)
plainbox/plainbox/impl/integration_tests.py (+8/-5)
plainbox/plainbox/impl/job.py (+36/-46)
plainbox/plainbox/impl/mock_job.py (+1/-1)
plainbox/plainbox/impl/rfc822.py (+4/-19)
plainbox/plainbox/impl/runner.py (+90/-42)
plainbox/plainbox/impl/secure/__init__.py (+27/-0)
plainbox/plainbox/impl/secure/checkbox_trusted_launcher.py (+402/-0)
plainbox/plainbox/impl/secure/test_checkbox_trusted_launcher.py (+268/-0)
plainbox/plainbox/impl/session.py (+95/-9)
plainbox/plainbox/impl/test_applogic.py (+108/-0)
plainbox/plainbox/impl/test_box.py (+24/-9)
plainbox/plainbox/impl/test_checkbox.py (+83/-0)
plainbox/plainbox/impl/test_config.py (+153/-0)
plainbox/plainbox/impl/test_depmgr.py (+22/-2)
plainbox/plainbox/impl/test_job.py (+142/-66)
plainbox/plainbox/impl/test_result.py (+2/-2)
plainbox/plainbox/impl/test_rfc822.py (+27/-19)
plainbox/plainbox/impl/test_runner.py (+62/-0)
plainbox/plainbox/impl/test_session.py (+123/-11)
plainbox/plainbox/impl/testing_utils.py (+1/-1)
plainbox/plainbox/impl/transport/__init__.py (+95/-0)
plainbox/plainbox/impl/transport/certification.py (+132/-0)
plainbox/plainbox/impl/transport/test_certification.py (+183/-0)
plainbox/plainbox/impl/transport/test_init.py (+68/-0)
plainbox/plainbox/test-data/xml-exporter/example-data.json (+19037/-0)
plainbox/plainbox/test-data/xml-exporter/example-data.xml (+15749/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_binary_attachment.json (+7/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_binary_attachment.xml (+21/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_comments.json (+13/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_comments.xml (+30/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_hardware_info.json (+10/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_hardware_info.xml (+22/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_io_log.json (+14/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_io_log.xml (+30/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_text_attachment.json (+7/-0)
plainbox/plainbox/test-data/xml-exporter/test_dump_with_text_attachment.xml (+21/-0)
plainbox/plainbox/testing_utils/__init__.py (+50/-1)
plainbox/requirements/rtfd.txt (+5/-0)
plainbox/setup.py (+20/-2)
plainbox/test-in-vagrant.sh (+82/-0)
plainbox/test-with-coverage.sh (+4/-4)
plugins/environment_info.py (+7/-0)
plugins/hexr_prompt.py (+32/-0)
plugins/hexr_transport.py (+169/-0)
plugins/jobs_prompt.py (+0/-1)
plugins/launchpad_exchange.py (+1/-1)
plugins/launchpad_prompt.py (+3/-2)
plugins/launchpad_report.py (+2/-1)
plugins/lock_prompt.py (+16/-8)
plugins/message_info.py (+4/-2)
plugins/persist_info.py (+1/-1)
po/POTFILES.in (+1/-0)
po/ace.po (+3/-3)
po/af.po (+3/-3)
po/am.po (+3/-3)
po/ar.po (+3/-3)
po/ast.po (+3/-3)
po/az.po (+3/-3)
po/be.po (+3/-3)
po/bg.po (+3/-3)
po/bn.po (+3/-3)
po/bo.po (+3/-3)
po/br.po (+3/-3)
po/bs.po (+5/-5)
po/ca.po (+3/-3)
po/ca@valencia.po (+3/-3)
po/ckb.po (+3/-3)
po/cs.po (+13/-3)
po/cy.po (+3/-3)
po/da.po (+3/-3)
po/de.po (+3/-3)
po/dv.po (+3/-3)
po/el.po (+3/-3)
po/en_AU.po (+3/-3)
po/en_CA.po (+3/-3)
po/en_GB.po (+3/-3)
po/eo.po (+3/-3)
po/es.po (+113/-113)
po/et.po (+3/-3)
po/eu.po (+3/-3)
po/fa.po (+3/-3)
po/fi.po (+3/-3)
po/fr.po (+3/-3)
po/ga.po (+3/-3)
po/gd.po (+3/-3)
po/gl.po (+242/-213)
po/he.po (+3/-3)
po/hi.po (+3/-3)
po/hr.po (+3/-3)
po/hu.po (+3/-3)
po/hy.po (+3/-3)
po/id.po (+3/-3)
po/is.po (+3/-3)
po/it.po (+11/-10)
po/ja.po (+174/-160)
po/jbo.po (+3/-3)
po/ka.po (+3/-3)
po/kk.po (+3/-3)
po/km.po (+3/-3)
po/kn.po (+3/-3)
po/ko.po (+3/-3)
po/ku.po (+3/-3)
po/ky.po (+3/-3)
po/lt.po (+3/-3)
po/lv.po (+4/-4)
po/mk.po (+3/-3)
po/ml.po (+3/-3)
po/mr.po (+3/-3)
po/ms.po (+3/-3)
po/my.po (+3/-3)
po/nb.po (+3/-3)
po/nds.po (+3/-3)
po/ne.po (+3/-3)
po/nl.po (+3/-3)
po/nn.po (+3/-3)
po/oc.po (+5/-5)
po/pl.po (+3/-3)
po/ps.po (+3/-3)
po/pt.po (+20/-20)
po/pt_BR.po (+14/-14)
po/ro.po (+3/-3)
po/ru.po (+5/-5)
po/sd.po (+3/-3)
po/shn.po (+3/-3)
po/si.po (+3/-3)
po/sk.po (+3/-3)
po/sl.po (+43/-43)
po/sq.po (+3/-3)
po/sr.po (+3/-3)
po/sv.po (+3/-3)
po/ta.po (+3/-3)
po/te.po (+3/-3)
po/th.po (+3/-3)
po/tr.po (+3/-3)
po/ug.po (+3/-3)
po/uk.po (+3/-3)
po/ur.po (+3/-3)
po/uz.po (+3/-3)
po/vi.po (+3/-3)
po/zh_CN.po (+3/-3)
po/zh_HK.po (+3/-3)
po/zh_TW.po (+3/-3)
qt/frontend/qtfront.cpp (+11/-1)
qt/frontend/qtfront.h (+2/-1)
qt/frontend/qtfront.ui (+20/-4)
scripts/alsa_record_playback (+1/-1)
scripts/bluetooth_test (+1/-1)
scripts/camera_test (+76/-46)
scripts/cpu_offlining (+25/-13)
scripts/cpu_topology (+1/-1)
scripts/create_connection (+172/-55)
scripts/fwts_test (+26/-23)
scripts/glob_test (+121/-0)
scripts/gpu_test (+74/-71)
scripts/gputest_benchmark (+110/-0)
scripts/graphics_stress_test (+6/-0)
scripts/lsmod_info (+6/-3)
scripts/memory_compare (+6/-3)
scripts/mm-test (+527/-0)
scripts/network (+382/-0)
scripts/network_check (+5/-1)
scripts/network_device_info (+11/-10)
scripts/rendercheck_test (+7/-0)
scripts/sleep_test_log_check (+72/-0)
scripts/sources_test (+11/-8)
scripts/virtualization (+82/-29)
scripts/xen_test (+74/-0)
setup.cfg (+1/-0)
setup.py (+6/-6)
tarmac-verify (+15/-5)
test-in-vagrant.sh (+101/-20)
test_scripts/tests/test_gputest_benchmark.py (+82/-0)
| tags: | added: server |
Revision history for this message
| Jeff Lane (bladernr) wrote | #1 |
| Changed in checkbox: | |
| status: | New → In Progress |
| importance: | Undecided → Low |
| assignee: | nobody → Jeff Lane (bladernr) |
| Changed in checkbox: | |
| status: | In Progress → Fix Committed |
| Changed in checkbox: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
There was another bug 1053594 that dealt with this as well, and in the process of deleting my broken checkbox branch that addresses the issue, I seem to have deleted the bug itself :/
Luckily, that bug and this bug are essentially the same.
So the issue boils down to CPU0 being "special" for ARM, so to fix this, we'll just work around it by not trying to offline cpu0