Hi Thomas, >-----Original Message----- >From: Thomas Monjalon <[email protected]> >Sent: Tuesday, January 19, 2021 12:18 AM >To: Xueming(Steven) Li <[email protected]> >Cc: Ferruh Yigit <[email protected]>; Andrew Rybchenko ><[email protected]>; Olivier Matz <[email protected]>; >[email protected]; Slava Ovsiienko <[email protected]>; Asaf Penso ><[email protected]> >Subject: Re: [dpdk-dev] [PATCH v4 2/9] ethdev: support representor port list > >18/01/2021 12:16, Xueming Li: >> To support extended representor syntax, this patch extends the >> representor list parsing to support for representor port range in >> devargs, examples: >> representor=[1,2,3] - single list >> representor=[1,3-5,7,9-11] - list with singles and ranges >> >> Signed-off-by: Xueming Li <[email protected]> >> Acked-by: Viacheslav Ovsiienko <[email protected]> > >Parsing functions are usually difficult to read. >If you need to do another version, I would recommend adding an oneline >comment on top of parsing functions to give a hint about what it is really >doing. > >Here, > >> +static int >> +rte_eth_devargs_enlist(uint16_t *list, uint16_t *len_list, >> + const uint16_t max_list, uint16_t val) > >here, > >> +static char * >> rte_eth_devargs_process_range(char *str, uint16_t *list, uint16_t *len_list, >> const uint16_t max_list) > >here, > >> +static char * >> +rte_eth_devargs_process_list(char *str, uint16_t *list, uint16_t *len_list, >> + const uint16_t max_list) > >and also for this old one: > >> @@ -121,6 +115,9 @@ rte_eth_devargs_parse_representor_ports(char *str, >> void *data) > >
Will add them in next version, thanks.
