Change in vdsm[master]: virt: add logic for POWER cpu xml element

mpolednik at redhat.com mpolednik at redhat.com
Tue Jun 23 08:45:11 UTC 2015


Martin Polednik has uploaded a new change for review.

Change subject: virt: add logic for POWER cpu xml element
......................................................................

virt: add logic for POWER cpu xml element

Current flow in appendCpu caused ppc to be skipped, generating empty
element (<cpu/>). In order to specify cpu model on ppc, we have to use
libvirt's host-model element and power6, power7 or power8 model.

Change-Id: I9303b76904ef1344508136343104b37c09d2a370
Signed-off-by: Martin Polednik <mpolednik at redhat.com>
---
M vdsm/virt/vmxml.py
1 file changed, 7 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/36/42736/1

diff --git a/vdsm/virt/vmxml.py b/vdsm/virt/vmxml.py
index 38d2200..94af918 100644
--- a/vdsm/virt/vmxml.py
+++ b/vdsm/virt/vmxml.py
@@ -366,6 +366,13 @@
                     elif feature[0] == '-':
                         featureAttrs['policy'] = 'disable'
                     cpu.appendChildWithArgs('feature', **featureAttrs)
+        elif self.arch in (caps.Architecture.POWER):
+            cpu.setAttrs(mode='host-model')
+
+            features = self.conf.get('cpuType', 'power8').split(',')
+            model = features[0]
+
+            cpu.appendChildWithArgs('model', text=model)
 
         if ('smpCoresPerSocket' in self.conf or
                 'smpThreadsPerCore' in self.conf):


-- 
To view, visit https://gerrit.ovirt.org/42736
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9303b76904ef1344508136343104b37c09d2a370
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>


More information about the vdsm-patches mailing list