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

Reply via email to