From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <yuanhan.liu@linux.intel.com>
Received: from mga11.intel.com (mga11.intel.com [192.55.52.93])
 by dpdk.org (Postfix) with ESMTP id A63A82931;
 Mon, 19 Sep 2016 16:47:14 +0200 (CEST)
Received: from orsmga005.jf.intel.com ([10.7.209.41])
 by fmsmga102.fm.intel.com with ESMTP; 19 Sep 2016 07:47:13 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.30,362,1470726000"; d="scan'208";a="10959376"
Received: from yliu-dev.sh.intel.com (HELO yliu-dev) ([10.239.67.162])
 by orsmga005.jf.intel.com with ESMTP; 19 Sep 2016 07:47:11 -0700
Date: Mon, 19 Sep 2016 22:47:43 +0800
From: Yuanhan Liu <yuanhan.liu@linux.intel.com>
To: Christian Ehrhardt <christian.ehrhardt@canonical.com>
Cc: Bruce Richardson <bruce.richardson@intel.com>, stable@dpdk.org,
 Yury Kylulin <yury.kylulin@intel.com>, helin.zhang@intel.com,
 "Ananyev, Konstantin" <konstantin.ananyev@intel.com>,
 jingjing.wu@intel.com, wenzhuo.lu@intel.com, dev <dev@dpdk.org>
Message-ID: <20160919144743.GK10323@yliu-dev.sh.intel.com>
References: <1472489448-15911-1-git-send-email-yury.kylulin@intel.com>
 <20160919132932.GA45256@bricha3-MOBL3>
 <CAATJJ0Jw3FGczyhtX035erAE3q_2zw=KBgPWjS3ZNRT61Fx1Ug@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAATJJ0Jw3FGczyhtX035erAE3q_2zw=KBgPWjS3ZNRT61Fx1Ug@mail.gmail.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Subject: Re: [dpdk-dev] [dpdk-stable] [PATCH 0/2] fix mbufs leakage during
 Rx queue release on ixgbe/i40e
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Mon, 19 Sep 2016 14:47:15 -0000

On Mon, Sep 19, 2016 at 03:52:25PM +0200, Christian Ehrhardt wrote:
> On Mon, Sep 19, 2016 at 3:29 PM, Bruce Richardson <
> bruce.richardson@intel.com> wrote:
> 
> > > This patch set fixes problem when mbufs are not released back to the
> > > original memory pool during Rx queue relese call leading to mbufs
> > > "leakage".
> > >
> > > This issue happens when device stop (rte_eth_dev_stop()) will be called
> > > after device start (rte_eth_dev_start()) without any incoming packets
> > > received since device start. In this case both rxq->rx_tail and
> > > rxq->rxrearm_start equal to 0 and release cycle will not be executed.
> > > On the next device start (rte_eth_dev_start()) new set of mbufs for the
> > > RX queue will be allocated.
> > >
> > > Yury Kylulin (2):
> > >   net/ixgbe: fix mbufs leakage during Rx queue release
> > >   net/i40e: fix mbufs leakage during Rx queue release
> > >
> > Applied to dpdk-next-net/rel_16_11
> >
> 
> I don't know the underlying code well enough,

Me, neither.

> but given the description I'd
> expect that qualifies for the stable tree?

But, yes, I agree with Christian.

> Set stable@dpdk on cc.

Thanks for doing that.

	--yliu