From mboxrd@z Thu Jan  1 00:00:00 1970
From: Yuanhan Liu <yuanhan.liu@linux.intel.com>
Subject: Re: [dpdk-stable] [PATCH 0/2] fix mbufs leakage during
 Rx queue release on ixgbe/i40e
Date: Mon, 19 Sep 2016 22:47:43 +0800
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
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>
To: Christian Ehrhardt <christian.ehrhardt@canonical.com>
Return-path: <dev-bounces@dpdk.org>
Content-Disposition: inline
In-Reply-To: <CAATJJ0Jw3FGczyhtX035erAE3q_2zw=KBgPWjS3ZNRT61Fx1Ug@mail.gmail.com>
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>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

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