On 4/8/19 4:36 PM, Dekel Peled wrote:
Regarding Andrew's suggestion: "Shouldn't these action be
RTE_FLOW_ACTION_TYPE_MOD_TCP_{ACK,SEQ} with singed 32-bit integer parameter (negative to
decrement, positive to increment)?"
I will leave the actions as is, the action names indicate the operation they
perform.
I think it is an overkill to have two actions for the purpose: DEC
(value) == INC ((uint32_t)-value)
If it is really important to have DEC and INC, please, make it clear
from actions documentation why.
Thanks,
Andrew.
