On 15/09/2020 15:41, David Marchand wrote: > On Mon, Sep 7, 2020 at 10:15 AM Ruifeng Wang <ruifeng.w...@arm.com> wrote: >> >> The rte_lpm structure is exported because lookup API is inlined. >> But most of the structure can be hidden. >> Discussion at: http://patches.dpdk.org/patch/72403/ >> This patch set aimed to hide the rte_lpm structure as much as possible. >> >> A data free issue was identified and fixed. >> >> Ruifeng Wang (2): >> lpm: fix free of data structure >> lpm: hide internal data >> >> lib/librte_lpm/rte_lpm.c | 154 +++++++++++++++++++++++---------------- >> lib/librte_lpm/rte_lpm.h | 7 -- >> 2 files changed, 92 insertions(+), 69 deletions(-) > > Cc: maintainers. > > Those changes look good to me. > Opinions? > >
LGTM, Gatekeeper confirmed they can handle this change. Acked-by: Kevin Traynor <ktray...@redhat.com> > Thanks. >
