On 8/22/2017 6:03 AM, Li Han wrote: > in parse_ringnuma_config/parse_portnuma_config functions,port_id > should less than RTE_MAX_ETHPORTS,but port_id_is_invalid check > assumes that port_id may be 255.
testpmd using RTE_PORT_ALL [1], which is in valid port_id range, as special meaning [2] making things tricky. Only above 255 is no more valid since port_id is not 16bits, it should be 65535. Except from above detail, Reviewed-by: Ferruh Yigit <ferruh.yi...@intel.com> And if you are familiar with this code, above a few lines there is a "int_fld[i] > 255" check, is the intention to check port_id limit there? If so this is no more valid check, and would you mind sending a patch to fix if you have time? Thanks, ferruh [1] #define RTE_PORT_ALL (~(portid_t)0x0) [2] Meaning all enabled ethdev ports, like: start_port(RTE_PORT_ALL); > > Signed-off-by: Li Han <han....@zte.com.cn> <...>