Hi,
> Subject: [dpdk-dev] [PATCH 4/4] ether: add packet modification aciton in flow
Typo in the commit message header "action"
> +/** RTE_FLOW_ACTION_TYPE_FIELD_INCREMENT
> + *
> + * Increment 1 on specific field of the packet.
> + *
> + * Typical usage: increase TTL
> + */
> +struct rte_flow_action_field_increment {
> + const struct rte_flow_item *item; /**< specify the data to modify. */
> + uint8_t layer;
> + /**< 0 means outermost matched pattern, 1 means next-to-
> outermost...
> +*/ };
> +
> +/** RTE_FLOW_ACTION_TYPE_FIELD_DECREMENT
> + *
> + * Decrement 1 on specific field of the packet.
> + *
> + * Typical usage: Decrease TTL
> + */
> +struct rte_flow_action_field_decrement {
> + const struct rte_flow_item *item; /**< Specify the data to modify. */
> + uint8_t layer;
> + /**< 0 means outermost matched pattern, 1 means next-to-
> outermost...
> +*/ };
> +
Can you have only one struct to represent both increment/decrements operations
instead of two structs? As I see both use same data types .
Thanks,
Reshma
