On 3/23/22 14:01, Eelco Chaudron wrote:
> 
> 
> On 23 Mar 2022, at 11:43, Abhiram R N wrote:
> 
>> When enabling offload for ERSPAN we are seeing one error as below
>>
>> netdev_offload_tc|INFO|init: failed to get ifindex for erspan0:
>> Operation not supported
>> netdev_offload|INFO|erspan0: No suitable flow API found.
>>
>> Adding the NETDEV_VPORT_GET_IFINDEX to ERSPAN device resolves this
>> error
> 
> The change looks good to me, but there is no real unit test to verify this, 
> and I have no real setup to test this.
> 
> So based on a pure visible and compile only check:
> 
> Acked-by: Eelco Chaudron <[email protected]>
> 
>> Signed-off-by: Abhiram R N <[email protected]>
>> ---

I don't see why this can cause any problems or why this callback
wasn't set from the beginning, so applied.  Thanks!

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to