Merge ~leftyfb/plainbox-provider-checkbox:1659929-fix-virt-cpu into plainbox-provider-checkbox:master

Proposed by Mike Rushton
Status: Merged
Approved by: Jeff Lane 
Approved revision: 3cffaa7ed9942476651c56557ef251bebde7c990
Merged at revision: b5d5b5baa998bd8252689e18fa653fa01bd86518
Proposed branch: ~leftyfb/plainbox-provider-checkbox:1659929-fix-virt-cpu
Merge into: plainbox-provider-checkbox:master
Diff against target: 13 lines (+1/-1)
1 file modified
bin/virtualization (+1/-1)
Reviewer Review Type Date Requested Status
Jeff Lane  Approve
Review via email: mp+315823@code.launchpad.net

Description of the change

Fix for bug #1659929

Replaced hardcoded cpu type entry in virtualization script from 'POWER8' to 'host' to match the cpu type of the host.

To post a comment you must log in.
Revision history for this message
Jeff Lane  (bladernr) wrote :

Tested on standard Power8 an Power8NVL.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/bin/virtualization b/bin/virtualization
index 748c60c..0ee74e9 100755
--- a/bin/virtualization
+++ b/bin/virtualization
@@ -112,7 +112,7 @@ QEMU_ARCH_CONFIG = {
112 'qemu_extra_args': [112 'qemu_extra_args': [
113 '-enable-kvm',113 '-enable-kvm',
114 '-machine', 'pseries,usb=off',114 '-machine', 'pseries,usb=off',
115 '-cpu', 'POWER8',115 '-cpu', 'host',
116 ],116 ],
117 },117 },
118 's390x': {118 's390x': {

Subscribers

People subscribed via source and target branches