Adding/removing dpdk ports leaks memory in openvswitch

Bug #1573091 reported by Christian Ehrhardt 
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openvswitch-dpdk (Ubuntu)
Fix Released
High
James Page
Nominated for Xenial by Christian Ehrhardt 

Bug Description

Similar to bug 1570466 but a different root cause.
It was identified that memory leaked on continuously adding/removing vhost_user ports in openvswitch-dpdk.
See the other bug for reproduction suggestion.

Fortunately for this bug a fix is already upstream in the openvswitch 2.5 branch.

In particulr for this bug we need:
d9df7b9206831631ddbd90f9cbeef1b4fc5a8e89 netdev-dpdk: Fix memory leak in netdev_dpdk_vhost_destruct().
But there are more critical ones related to openvswitch-dpdk:
a39aa7e10d033e3ed15b16bd101f655aa1db8135 dpif-netdev: report numa node number on pmd thread create failure
e099c86ca432d1daeaf8edb66225e7edf061c0b0 netdev-dpdk: vhost: Fix txq enabling in the absence of notifications.
3b70d8fcca23d9fed2f2a2c2e9eb428e4a202123 netdev-dpdk: Fix crash when changing the vhost-user port.
f61053debcb466d8bfec31070f0826e3c0e3c727 INSTALL.DPDK: remove issue with QEMU v2.4.0 and dpdkvhostuser
05a00f12f2c4e25077f0ca55f7f6734908932abf netdev-dpdk: fix mbuf leaks

I understand we generally wait for the 2.5.1 release which will get some extra test and checks, but I hope that shouldn't be too far away.

So please merge the 2.5.1 point release of openvswitch as soon as available to fix this issue.

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

FYI - we agreed to discuss the availability and approach to upload the point release when we meet in approx 1.5 week.
Crossing fingers that it gets release until then.

Changed in dpdk (Ubuntu):
status: New → Triaged
importance: Undecided → High
Changed in openvswitch-dpdk (Ubuntu):
status: New → Triaged
importance: Undecided → High
assignee: nobody → James Page (james-page)
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

As dpdk part is only open for tracking (dpdk portion fixed in last upload) setting that importance to wishlist

Changed in dpdk (Ubuntu):
importance: High → Wishlist
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Done for Yakkety via the upload of openvswitch 2.6.0~git20160822.2cff273-0ubuntu1
Closing DPDK which was only a tracker, but nominating a Xenial task until OVS 2.5.1 is released into Xenial

Changed in dpdk (Ubuntu):
status: Triaged → Invalid
Changed in openvswitch-dpdk (Ubuntu):
status: Triaged → Fix Released
no longer affects: dpdk (Ubuntu)
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.