Re: [PATCH v5 0/2] Add config file support for l3fwd

[Bruce Richardson <bruce.richardson@intel.com>]

On Tue, Feb 08, 2022 at 09:49:04AM -0800, Stephen Hemminger wrote:
> On Tue, 8 Feb 2022 16:15:15 +0000
> "Medvedkin, Vladimir" <vladimir.medvedkin@intel.com> wrote:
> 
> > >> Or have a generic library for reading LPM entries.  L3fwd is supposed
> > >> to be as small as possible (it no longer is), and the real work should
> > >> be done by libraries to make it easier to build other applications.  
> > > 
> > > I never heard users ask about such thing,
> > > but if there is a demand for that, then I suppose it could be considered.
> > > CC-ing LPM/FIB maintainers to comment.
> > > Though I believe it should be a subject of separate patch and discussion
> > > (I think many questions will arise - what format should be, how to support
> > > different types of user-data, to make it generic enough, etc.).  
> > 
> > Agree, it is very application specific, so it could be really difficult 
> > to make it generic.
> 
> But several other also have LPM tables, so why not have common code for other applications.
> 
> examples/l3fwd-power/main.c
> examples/ipsec-secgw/rt.c
> examples/ip_fragmentation/main.c
> examples/l3fwd/l3fwd_lpm.c
> examples/ip_reassembly/main.c

Could we perhaps add a "load" function to the lpm library itself?