[PATCH v3] app/testpmd: enable cli for programmable action
    Dumitrescu, Cristian 
    cristian.dumitrescu at intel.com
       
    Fri Oct  6 14:35:48 CEST 2023
    
    
  
Hi Qi,
<snip>
> 
> +#define ACTION_PROG_NAME_SIZE_MAX 32
> +#define ACTION_PROG_ARG_NUM_MAX 16
> +#define ACTION_PROG_ARG_VALUE_SIZE_MAX 32
Let's be a bit more generous with some of these sizes, please.
The action name might be hierarchical, i.e. include the control block and table path separated by dots, e.g. "dev5.ingress.forwarding.routing_table.set_next_hop", so:
#define ACTION_PROG_NAME_SIZE_MAX 256
Some parameters could be 128-bit IPv6 addresses, which may be specified in hex with additional characters such as '0x' or ':', so:
#define ACTION_PROG_ARG_VALUE_SIZE_MAX 64
<snip>
Regards,
Cristian
    
    
More information about the dev
mailing list