25/04/2018 11:40, Remy Horton: > > On 24/04/2018 20:53, Thomas Monjalon wrote: > [..] > > But I would like to review the devargs you are standardizing. > > Unfortunately, I cannot find a documentation about it. > > How users are supposed to use it? > > Can you, at least, describe the syntax in the commit log, please? > > The patch follows this pseudo-BNF: > > cfg := pair (',' pair)* > pair := (key '=' value) > key := 'port' | 'representor' > value := range | list > range := int ('-' int)? > int := [0-9]+ > list := '[' range (',' range)* ']'
OK Please can you add it as a comment in the parsing code? We will need one or two examples in the commit message too. Can you show a complete command line please? How do you give ethdev properties without the new devargs syntax (in progress by Gaetan)?