From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2] test/test_mbuf: remove mempool global var Date: Tue, 20 Jun 2017 09:35:07 +0200 Message-ID: <2118823.ueSLhbdIFl@xps> References: <20170608142831.28166-1-santosh.shukla@caviumnetworks.com> <2170817.cQF1WCe9mM@xps> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, olivier.matz@6wind.com, shreyansh.jain@nxp.com, stable@dpdk.org To: santosh Return-path: In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 20/06/2017 06:14, santosh: > On Tuesday 20 June 2017 02:07 AM, Thomas Monjalon wrote: > > > 08/06/2017 16:28, Santosh Shukla: > >> Let test_mbuf alloc and free mempool. > >> > >> Cc: stable@dpdk.org > >> Signed-off-by: Santosh Shukla > > Why Cc stable? > > Is it fixing something? > > > w/o this fix, application can't run more than once. > Reason: Static allocation of resources and exiting w/o freeing so leak. > > Patch makes app resource handling dynamic and Now user could run > test more than once and app exits gracefully. thats why Cc: stable a need (IHMO). > Thanks. OK So we need a Fixes: tag in order to be able to guess which release it should be backported to.