From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qk0-f182.google.com (mail-qk0-f182.google.com [209.85.220.182]) by dpdk.org (Postfix) with ESMTP id 098166CD4 for ; Wed, 6 Jul 2016 14:38:04 +0200 (CEST) Received: by mail-qk0-f182.google.com with SMTP id t127so313193792qkf.1 for ; Wed, 06 Jul 2016 05:38:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical-com.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to; bh=6z6n3H3YZposSRcOfseWb/Qq+jr728ofKFC1RA9wOOw=; b=rFp09dgLNjDSUZSS7TXNppfH4T8w1OFpbvR846yM88WzHvJ2AsypJdndLtTC+SrUmS HKOsb8fX3+Kw0n3YNpwQAUyP/wxB7xtwk01fOoAtHkIu+8hUv2XhyiEimSq4lhk9eyS1 1/9IM4k5pmvVPWpeUk6MF2ocUh8uhuUVEIhXINXPW+9aYnJpH8l8iAqU2HDRM85tSuaO V1/402F43Wsy+eAi2y8gB/AigSBurMySjQmvYbmw573ZtJK5MNSla5go3tZvSjXCUHgp hb8C4UR1+sjLQSImA0PMc7hKQPZsFSi1kZyhy+EqIcmHtLUbsLPOQcb/d0RbkQd/6TZ4 KsIw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to; bh=6z6n3H3YZposSRcOfseWb/Qq+jr728ofKFC1RA9wOOw=; b=cf5VspawM+3fxsTdfRV6eVXkOVpkJlnFLrq6dWoxlay+luleQroqJmE6cKpCt6298f y2xu86fsvaSYlr5sx8AiQJRI6OAiuNi+/E4in1bQqiKVx2BEif0TyfOIZYJfwBlCjV6S iGAQ6gJIxP9+uApsnllpw7AiU8tD/iZpjMsNVLm28C76igxITQvMlg1vUC3+0xXhesvr M6qWJhcQV33xll3rruc3Zqa6+JqCpaJoxtm3OUwT7z813UYV5MiJeqF1JJJc3ew11WTp xMLvCP6UJBO8PdXb8quKYkBlNgrJxVjSL2LaOB9YPHhDAIX1/hAPoNxKnolXI2nQ+5Sq RLiw== X-Gm-Message-State: ALyK8tIbuAWAFTUGTrJlRm5hejjPnmjsX1b/7v5SNzRpcrrNf1N0sYUsA7ueLTlV2+I5lC9iSPhDo6nbK6SafK0y X-Received: by 10.55.186.193 with SMTP id k184mr31074509qkf.184.1467808683416; Wed, 06 Jul 2016 05:38:03 -0700 (PDT) MIME-Version: 1.0 Received: by 10.55.23.77 with HTTP; Wed, 6 Jul 2016 05:37:43 -0700 (PDT) In-Reply-To: References: <1467807898-27772-1-git-send-email-christian.ehrhardt@canonical.com> From: Christian Ehrhardt Date: Wed, 6 Jul 2016 14:37:43 +0200 Message-ID: To: Christian Ehrhardt , Patrik Andersson R , Thomas Monjalon , dev , Yuanhan Liu , "Xie, Huawei" Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH v2] Memory leak when adding/removing vhost_user ports X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Jul 2016 12:38:04 -0000 Now let us really get to this old discussion. Once again - sorry for mixing this up :-/ Found too much at once today. ## Ok - refocus ## Back then in http://dpdk.org/dev/patchwork/patch/12103/ http://dpdk.org/dev/patchwork/patch/12118/ We came up with a nice solution for the leak. But it was never picked up upstream. There were a lot of changes to all of that code, especially vhost client/server. Lacking an openvswitch for DPDK 16.07 I can't test if the issue still shows up, but looking at the code suggests it still does. Unfortunately the internal structures and scopes are slightly different so that I couldn't come up with a working forward port of the old patch. Attached is a patch as close as I got (obviously not working). I'm convinced that Yuanhan Liu and Xie Huawei with their deep context knowledge of dpdk's vhost_user will quickly know: - if the root cause still applies - what the best new way of fixing this would be As this makes long term usage of dpdk aborting by a leak I hope we have a chance to get this into 16.07 still. Kind Regards, Christian Christian Ehrhardt Software Engineer, Ubuntu Server Canonical Ltd On Wed, Jul 6, 2016 at 2:30 PM, Christian Ehrhardt < christian.ehrhardt@canonical.com> wrote: > This is the series this really belongs to > http://dpdk.org/dev/patchwork/patch/11581/ > Message ID < > 1458292380-9258-1-git-send-email-patrik.r.andersson@ericsson.com> > > Should I wait for a v2 to point the patch at the right ID or do you prefer > a fixed resubmit right away? > > Christian Ehrhardt > Software Engineer, Ubuntu Server > Canonical Ltd > > On Wed, Jul 6, 2016 at 2:26 PM, Christian Ehrhardt < > christian.ehrhardt@canonical.com> wrote: > >> Sorry, >> please ignore the two links, the cover letter has - they belong to a >> different issue I have to bring up again. >> Everything else still applies. >> >> Christian Ehrhardt >> Software Engineer, Ubuntu Server >> Canonical Ltd >> >> On Wed, Jul 6, 2016 at 2:24 PM, Christian Ehrhardt < >> christian.ehrhardt@canonical.com> wrote: >> >>> Hi, >>> while checking for dpdk 16.07 what backports are accepted in the >>> meantime so I >>> can drop them I found this particular discussion has been silently >>> forgotten by >>> all of us. >>> >>> Back then we had the patch and discussion first in >>> http://dpdk.org/dev/patchwork/patch/12103/ >>> and then >>> http://dpdk.org/dev/patchwork/patch/12118/ >>> >>> Things worked fine as I reported and I integrated the patch in our >>> packaging as >>> it fixed a severe issue. Since it was reported by someone else I do not >>> seem to >>> be the only one :-) >>> >>> So today I rebased the patch including my updates I made based on our >>> discussion >>> and I think it would make as much sense as it made back then to fix this. >>> >>> Christian Ehrhardt (1): >>> vhost_user: avoid crash when exeeding file descriptors >>> >>> lib/librte_vhost/vhost_user/fd_man.c | 11 ++++++----- >>> lib/librte_vhost/vhost_user/vhost-net-user.c | 19 +++++++++++++++++-- >>> 2 files changed, 23 insertions(+), 7 deletions(-) >>> >>> -- >>> 2.7.4 >>> >>> >> >