From mboxrd@z Thu Jan 1 00:00:00 1970 From: santosh Subject: Re: [PATCH v2] test/test_mbuf: remove mempool global var Date: Tue, 20 Jun 2017 09:44:15 +0530 Message-ID: References: <20170608142831.28166-1-santosh.shukla@caviumnetworks.com> <2170817.cQF1WCe9mM@xps> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org, olivier.matz@6wind.com, shreyansh.jain@nxp.com, stable@dpdk.org To: Thomas Monjalon Return-path: In-Reply-To: <2170817.cQF1WCe9mM@xps> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 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.