Re: [dpdk-dev] [PATCH 1/2] ethdev: add jump action to description table

[Yongseok Koh]

> On Oct 3, 2018, at 12:12 AM, Andrew Rybchenko <arybche...@solarflare.com> 
> wrote:
> 
> On 10/3/18 4:56 AM, Yongseok Koh wrote:
>> Signed-off-by: Yongseok Koh <ys...@mellanox.com>
>> 
>> ---
>>  app/test-pmd/config.c        | 2 +-
>>  lib/librte_ethdev/rte_flow.c | 1 +
>>  2 files changed, 2 insertions(+), 1 deletion(-)
>> 
>> diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
>> index 794aa5268..641ac5e17 100644
>> --- a/app/test-pmd/config.c
>> +++ b/app/test-pmd/config.c
>> @@ -1141,7 +1141,7 @@ static const struct {
>>      MK_FLOW_ACTION(END, 0),
>>      MK_FLOW_ACTION(VOID, 0),
>>      MK_FLOW_ACTION(PASSTHRU, 0),
>> -    MK_FLOW_ACTION(JUMP, 0),
>> +    MK_FLOW_ACTION(JUMP, sizeof(struct rte_flow_action_jump)),
>>      MK_FLOW_ACTION(MARK, sizeof(struct rte_flow_action_mark)),
>>      MK_FLOW_ACTION(FLAG, 0),
>>      MK_FLOW_ACTION(QUEUE, sizeof(struct rte_flow_action_queue)),
>> diff --git a/lib/librte_ethdev/rte_flow.c b/lib/librte_ethdev/rte_flow.c
>> index cff4b5209..00ed67b5a 100644
>> --- a/lib/librte_ethdev/rte_flow.c
>> +++ b/lib/librte_ethdev/rte_flow.c
>> @@ -80,6 +80,7 @@ static const struct rte_flow_desc_data 
>> rte_flow_desc_action[] = {
>>      MK_FLOW_ACTION(END, 0),
>>      MK_FLOW_ACTION(VOID, 0),
>>      MK_FLOW_ACTION(PASSTHRU, 0),
>> +    MK_FLOW_ACTION(JUMP, sizeof(struct rte_flow_action_jump)),
>>      MK_FLOW_ACTION(MARK, sizeof(struct rte_flow_action_mark)),
>>      MK_FLOW_ACTION(FLAG, 0),
>>      MK_FLOW_ACTION(QUEUE, sizeof(struct rte_flow_action_queue)),
>> 
> 
> I think it should have Fixes tag and Cc to stable.

Agree.

Thanks,
Yongseok