Hi Yuanhan, On Tuesday 20 June 2017 01:52 PM, Yuanhan Liu wrote:
> On Tue, Jun 20, 2017 at 09:35:07AM +0200, Thomas Monjalon wrote: >> 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: sta...@dpdk.org >>>>> Signed-off-by: Santosh Shukla <santosh.shu...@caviumnetworks.com> >>>> 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. > Also, I would suggest you to include above info (the real issue) in > the commit log. > > --yliu Yes, We'll reword and post v3 but won't be a stable candidate.
