[P9] virsh capabilities does not show certain tags "model", "vendor" for power9 boston

Bug #1690209 reported by bugproxy
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Ubuntu-power-systems project
Fix Released
Undecided
Canonical Server
libvirt (Ubuntu)
Fix Released
Undecided
Ubuntu on IBM Power Systems Bug Triage

Bug Description

virsh capabilities output does not show certain tags like "model", "vendor" for power9 boston and leads to failure of many testcases

E:g:-
ERROR 213-guest_cpu.cpu_compare.qemu.qcow2.virtio_scsi.smp2.virtio_net.HostOS.2.0.ppc64le.powerkvm-libvirt.virsh.cpu_compare.host_cpu.no_cpu_match.modify_feature_policy.require_policy -> TestSetupFail: Error in Getter for property "model", element tag "model" not found on parent at xpath "/host/cpu" in XML

Actual:
#virsh capabilities
<capabilities>

  <host>
    <uuid>7dd9b002-b3d9-4cc2-ac92-124428b013df</uuid>
    <cpu>
      <arch>ppc64le</arch>
      <topology cores="16" sockets="1" threads="1" />
      <pages size="64" unit="KiB" />
      <pages size="2048" unit="KiB" />
      <pages size="1048576" unit="KiB" />
    </cpu>
....
....
....

Expected:

#virsh capabilities
<capabilities>

  <host>
    <uuid>7dd9b002-b3d9-4cc2-ac92-124428b013df</uuid>
    <cpu>
      <arch>ppc64le</arch>
      <model>POWER9</model>
      <vendor>IBM</vendor>
      <topology cores="16" sockets="1" threads="1" />
      <pages size="64" unit="KiB" />
      <pages size="2048" unit="KiB" />
      <pages size="1048576" unit="KiB" />
    </cpu>
....
...
...

Patch accepted upstream with commit id 9cdf3a1c06189b8d0cab9f8f62b5b626cf8a150c.

bugproxy (bugproxy)
tags: added: architecture-ppc64le bugnameltc-154411 severity-high targetmilestone-inin1710
Changed in ubuntu:
assignee: nobody → Ubuntu on IBM Power Systems Bug Triage (ubuntu-power-triage)
affects: ubuntu → libvirt (Ubuntu)
Frank Heimes (fheimes)
Changed in ubuntu-power-systems:
assignee: nobody → Canonical Server Team (canonical-server)
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

I asked on other channels, but since this related here - is there a known minimum libvirt version for p9 and if so which one?

Changed in libvirt (Ubuntu):
status: New → Incomplete
tags: added: p9-virt-stack
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Note: Related Qemu bug which very likely is a prereq is bug 1692452

Manoj Iyer (manjo)
tags: added: ubuntu-17.04
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Ping - Polling for related commit ID's or libvirt versions for p9 support

Frank Heimes (fheimes)
Changed in ubuntu-power-systems:
status: New → Incomplete
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Hi,
I'm working on the just released libvirt 3.5 - will the patches be in there already?
If not what is your timeline on this?

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

I beg your pardon - on this one in particular the commit was listed (there were many P9 bugs I had to check, I missed that on just this one the commit was known).

9cdf3a1c06189b8d0cab9f8f62b5b626cf8a150c is in there since 3.4 this bug would be solved by having libvirt 3.5.

Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (14.4 KiB)

This bug was fixed in the package libvirt - 3.5.0-1ubuntu1

---------------
libvirt (3.5.0-1ubuntu1) artful; urgency=medium

  * Merged with Debian unstable (3.5)
    This closes several bugs:
    - improved handling of host-model since libvirt 3.2 (LP: #1673467)
    - Adding POWER9 cpu model to cpu_map.xml (LP: #1690209)
  * Remaining changes:
    - Disable sheepdog (universe dependency)
    - Disable libssh2 support (universe dependency)
    - Disable firewalld support (universe dependency)
    - Disable selinux
    - Enable esx support
      + Add build-dep to libcurl4-gnutls-dev (required for esx)
    - Set qemu-group to kvm (for compat with older ubuntu)
    - Regularly clear AppArmor profiles for vms that no longer exist
    - Additional apport package-hook
    - Modifications to adapt for our delayed switch away from libvirt-bin (can
      be dropped >18.04).
      + d/p/ubuntu/libvirtd-service-add-bin-alias.patch: systemd: define alias
        to old service name so that old references work
      + d/p/ubuntu/libvirtd-init-add-bin-alias.patch: sysv init: define alias
        to old service name so that old references work
      + d/control: transitional package with the old name and maintainer
        scripts to handle the transition
    - Backwards compatible handling of group rename (can be dropped >18.04).
    - config details and autostart of default bridged network. Creating that is
      now the default in general, yet our solution provides the following on
      top as of today:
      + nat only on some ports <port start='1024' end='65535'/>
      + autostart the default network by default
      + do not autostart if 192.168.122.0 is already taken (e.g. in containers)
    - d/p/ubuntu/Allow-libvirt-group-to-access-the-socket.patch: This is
      the group based access to libvirt functions as it was used in Ubuntu
      for quite long.
      + d/p/ubuntu/daemon-augeas-fix-expected.patch fix some related tests
        due to the group access change.
    - ubuntu/parallel-shutdown.patch: set parallel shutdown by default.
    - d/p/ubuntu/enable-kvm-spice.patch: compat with older Ubuntu qemu/kvm
      which provided a separate kvm-spice.
    - d/p/ubuntu/storage-disable-gluster-test: gluster not enabled, skip test
    - d/p/ubuntu/ubuntu-libxl-qemu-path.patch: this change was split. The
      section that adapts the path of the emulator to the Debian/Ubuntu
      packaging is kept.
    - d/p/ubuntu/ubuntu-libxl-Fix-up-VRAM-to-minimum-requirements.patch: auto
      set VRAM to minimum requirements
    - d/p/ubuntu/xen-default-uri.patch: set default URI on xen hosts
    - Add libxl log directory
    - libvirt-uri.sh: Automatically switch default libvirt URI for users on
      Xen dom0 via user profile (was missing on changelogs before)
    - d/p/ubuntu/apibuild-skip-libvirt-common.h: drop libvirt-common.h from
      included_files to avoid build failures due to duplicate definitions.
    - Update README.Debian with Ubuntu changes
    - Convert libvirt0, libnss_libvirt and libvirt-dev to multi-arch.
    - Enable some additional features on ppc64el and s390x (for arch parity)
      + systemtap, zfs, numa and numad on s390x.
      + sys...

Changed in libvirt (Ubuntu):
status: Incomplete → Fix Released
Frank Heimes (fheimes)
Changed in ubuntu-power-systems:
status: Incomplete → Fix Released
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2017-09-18 11:43 EDT-------
Nageswara, Aline,

Can we go ahead and close this bug?

Revision history for this message
bugproxy (bugproxy) wrote :

------- Comment From <email address hidden> 2017-09-19 01:31 EDT-------
Tested this on DD2.0 Boston machine and the here is the result from 'virsh capabilities'

user@ltc-boston122:~$ virsh capabilities
<capabilities>

<host>
<uuid>78f11ba1-c10e-487f-b06f-dad1b39a13bf</uuid>
<cpu>
<arch>ppc64le</arch>
<model>POWER9</model>
<vendor>IBM</vendor>
<topology sockets='1' cores='16' threads='4'/>
<pages unit='KiB' size='64'/>
<pages unit='KiB' size='2048'/>
<pages unit='KiB' size='1048576'/>
</cpu>

libvirt-daemon 3.6.0-1ubuntu4

This bugzilla can be closed.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.