On Tuesday 20 June 2017 01:05 PM, 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.
Hmmm, By git blame, Its a very early commit(af75078fe : first public release). And I think this patch can't be back-ported. IMO, I would prefer to remove Cc: stable@ from git description area, suggestion? is it fine. Yes, I will reword the git description.
