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 <tho...@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?
