On 24 Jun 2026, at 10:43, Eelco Chaudron wrote:
> On 11 Jun 2026, at 18:45, Tim Rozet via dev wrote:
>
>> When there are offload errors we see error messages that only include
>> the netdev such as:
>>
>> 2026-03-19T19:42:43.103Z|03122|dpif_netlink(handler367)|ERR|failed to
>> offload flow: Invalid argument: ovn-d2586f-0
>>
>> This error message lacks the flow information in order to debug why the
>> flow failed to be offloaded. This then requires a user to go reproduce
>> the problem and turn on debug logging to try to correlate the error to a
>> flow.
>>
>> Pass the provider error to log_flow_put_message() for flow put failures,
>> while keeping ENOSPC on the existing debug-only path. This makes the
>> offload layer emit the failed flow match/actions for cases like EINVAL,
>> so root-causing TC offload failures does not require enabling debug
>> logging.
>>
>> Signed-off-by: Tim Rozet <[email protected]>
>
> Thanks Tim for following up with the v2. The changes look good to me.
>
> Acked-by: Eelco Chaudron <[email protected]>
This patch was applied to main.
//Eelco
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
