Cannot get hugepages on PPC64

Bug #1617081 reported by Vincent JARDIN
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Wishlist
Vincent JARDIN

Bug Description

We cannot spawn VMs with hugepages on PPC64, for example with the IBM P8 systems.
The analysis shows that it is due to
  virt/libvirt/driver.py
     _has_hugepage_support()

which has only:
   supported_archs = [arch.I686, arch.X86_64]

 arch.PPC64LE is missing here.

Tags: libvirt
Changed in nova:
assignee: nobody → Vincent JARDIN (vincent-jardin)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/360822

Revision history for this message
Vincent JARDIN (vincent-jardin) wrote :

An alternate fix is available her:
  https://review.openstack.org/#/c/303564
is includes the feature for both PPC64 and PPC64LE

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Vincent Jardin (vincent.jardin@6wind.com) on branch: master
Review: https://review.openstack.org/360822
Reason: Right, let's give up. I see that Breno did refresh his patch. Let's apply Breno's one.

Side question: how to link Breno's patch with this bug https://bugs.launchpad.net/nova/+bug/1617081 ?

tags: added: libvirt
Revision history for this message
Vincent JARDIN (vincent-jardin) wrote :

https://review.openstack.org/#/c/303564 seems good to go, waiting for +2.

Changed in nova:
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

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

commit abc24acfa1982a0ffccbe08a006ac7c7a9f4ecda
Author: Breno Leitao <email address hidden>
Date: Fri Aug 26 18:36:00 2016 -0300

    libvirt: add hugepages support for Power

    Power architectures (arch.PPC64LE and arch.PPC64) support huge pages and
    transparent huge pages. This patch just enables it on nova libvirt driver. A
    reno note was also added to track this new feature.

    This change also enables the test_does_want_hugepages unit test to run on the
    architectures that support huge pages.

    Closes-bug: #1617081
    Change-Id: I22bc57a0b244667c716a54ca37c175f26a87a1e9

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 14.0.0.0rc1

This issue was fixed in the openstack/nova 14.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/388064

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 14.0.0.0rc1

This issue was fixed in the openstack/nova 14.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (stable/mitaka)

Change abandoned by Lee Yarwood (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/388064
Reason: Hello Corey,

stable/mitaka has now entered phase II support [1][2], only accepting critical bugfixes and security patches. As this review does not meet these criteria it is being abandoned at this time.

However please reopen this review if you feel it is still suitable for stable/mitaka and the nova-stable-maint team will revisit this decision.

[1] http://docs.openstack.org/project-team-guide/stable-branches.html#support-phases
[2] https://releases.openstack.org/#release-series

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.