> On Thu, 10 Oct 2024 17:17:16 +0800 > Chaoyong He <chaoyong...@corigine.com> wrote: > > > Fix one memory leak problem in the logic of VF initialization. > > > > Fixes: d81e2b514dc9 ("net/nfp: move device info into process private > > data") > > Cc: sta...@dpdk.org > > > > Signed-off-by: Chaoyong He <chaoyong...@corigine.com> > > Reviewed-by: Long Wu <long...@corigine.com> > > Reviewed-by: Peng Zhang <peng.zh...@corigine.com> > > --- > > The code as is looks fine, but would have been better to use rte_calloc() when > allocating eth_xstats_base.
Yes, I think what you said is better. Will do it in next version. Thanks for your review. > > Acked-by: Stephen Hemminger <step...@networkplumber.org>