[EXT] Re: [PATCH] test/security: fix buffer leaks in error path

[Akhil Goyal]

> > +#define FREE_PKTS(j, m) {		\
> > +	while (j--)			\
> > +		rte_pktmbuf_free(m[j]);	\
> > +}
> > +
> 
> Changes looks good. However do you want to define a static inline
> function here instead of macro?
Any specific reason for changing to inline function?
In this case macro is as good as inline function.