Add ppc architecture for NUMA

Bug #1516578 reported by Sudipta Biswas
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Sudipta Biswas
Liberty
Fix Released
Low
Sudipta Biswas

Bug Description

Post this commit: https://review.openstack.org/#/c/170780/11/nova/virt/libvirt/driver.py,

the NUMA topology reporting depends on the host architecture.

The current check includes arch.I686, arch.X86_64 only.

This bug is filed to include the ppc64/ppc64le architectures included to the list, since they use the same libvirt driver for KVM based systems.

Tags: libvirt numa ppc
Changed in nova:
assignee: nobody → Sudipta Biswas (sbiswas7)
Changed in nova:
status: New → In Progress
Sean Dague (sdague)
Changed in nova:
assignee: Sudipta Biswas (sbiswas7) → nobody
importance: Undecided → Wishlist
status: In Progress → Confirmed
Revision history for this message
Sudipta Biswas (sbiswas7) wrote :

https://review.openstack.org/#/c/245765/ - I don't know why the review was not updated with the bug.
This patchset has been posted out a few months back - and all review comments have been incorporated. I am just waiting for some cores to take a look at the same for a merge. Assigning it back and putting it in progress.

Changed in nova:
assignee: nobody → Sudipta Biswas (sbiswas7)
status: Confirmed → In Progress
Changed in nova:
assignee: Sudipta Biswas (sbiswas7) → Sean Dague (sdague)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/245765
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=1a2443ce67700c494275a3ea51e584c551f7490f
Submitter: Jenkins
Branch: master

commit 1a2443ce67700c494275a3ea51e584c551f7490f
Author: Sudipta Biswas <email address hidden>
Date: Mon Nov 16 19:06:24 2015 +0530

    libvirt: Add ppc to supported arch for NUMA

    This patch includes the ppc64/ppc64le architectures to the list
    of host architectures that support NUMA via the libvirt driver.

    This change is for the PowerPC based servers that use the libvirt
    driver. This also introduces the minimum libvirt version needed
    for PowerPC to support NUMA from Nova.

    Also note, that in order to retain the unit test that excluded
    PPC from the NUMA supported architectures, an arbitrary choice
    of architecture is made (S390) - that is not a part of the list
    that supports NUMA.

    Change-Id: I51dd3e160153b40072a04095e9bf3f15abd74252
    Closes-Bug: #1516578

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
Sudipta Biswas (sbiswas7) wrote :

Thanks Sean for the review and the quick update of the patch.

Changed in nova:
assignee: Sean Dague (sdague) → Sudipta Biswas (sbiswas7)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/287669

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/nova 13.0.0.0b3

This issue was fixed in the openstack/nova 13.0.0.0b3 development milestone.

Tony Breeds (o-tony)
tags: added: liberty-backport-potential
Matt Riedemann (mriedem)
Changed in nova:
importance: Wishlist → Low
tags: added: libvirt numa ppc
removed: liberty-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (stable/liberty)

Change abandoned by Matt Riedemann (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/287669
Reason: This looks abandoned.

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.