On 10/6/22 19:46, Mike Pattrick wrote:
> On Thu, Sep 1, 2022 at 6:16 AM Robin Jarry <[email protected]> wrote:
>>
>> Packets that could not be transmitted because the TXQ are full should be
>> taken into account in the global ovs_tx_failure_drops as it was the case
>> before commit 29b94e12d57d ("netdev-dpdk: Refactor the DPDK transmit
>> path.").
>>
>> netdev_dpdk_eth_tx_burst() returns the number of packets that were *not*
>> transmitted. Add that number to stats.tx_failure_drops and only include
>> the packets that were dropped in previous steps afterwards.
>>
>> Fixes: 29b94e12d57d ("netdev-dpdk: Refactor the DPDK transmit path.")
>> Signed-off-by: Robin Jarry <[email protected]>
> 
> Looks good to me.
> 
> Acked-by: Mike Pattrick <[email protected]>

Thanks!  Applied and backported down to 2.17.

Best regards, Ilya Maximets.

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to