[dpdk-dev] [PATCH 3/4] ethdev: move representor parsing functions

Thomas Monjalon thomas at monjalon.net
Tue Oct 9 15:25:27 CEST 2018


09/10/2018 14:38, Remy Horton:
> 
> On 09/10/2018 03:18, Thomas Monjalon wrote:
> > The functions for representor devargs parsing were static
> > in the file rte_ethdev.c.
> > In order to reuse them in the file rte_class_eth.c,
> > they are moved to the files ethdev_private.c/.h.
> >
> > Signed-off-by: Thomas Monjalon <thomas at monjalon.net>
> 
> There is overlap between these functions and some of the rte_kvargs_*() 
> functions. Is it feasible/desirable to combine them?

Yes!
This is what I wanted to suggest.
I think we should parse list and range as part of the kvargs library.
Would you be interested to rework those functions in a next release?




More information about the dev mailing list