On 02/11/2022 15:01, Simon Horman wrote:
> On Wed, Nov 02, 2022 at 01:52:18PM +0100, Eelco Chaudron wrote:
>>
>>
>> On 2 Nov 2022, at 13:46, Roi Dayan wrote:
>>
>>> From: Paul Blakey <[email protected]>
>>>
>>> Currently ethertype to prio hmap is static and the first ethertype
>>> being used gets a lower priority. Usually there is an arp request
>>> before the ip traffic and the arp ethertype gets a lower tc priority
>>> while the ip traffic proto gets a higher priority.
>>> In this case ip traffic will go through more hops in tc and HW.
>>> Instead, reserve lower priorities for ip ethertypes.
>>>
>>> Signed-off-by: Paul Blakey <[email protected]>
>>> Reviewed-by: Roi Dayan <[email protected]>
>>> Reviewed-by: Simon Horman <[email protected]>
>>> Acked-by: Eelco Chaudron <[email protected]>
>>
>> Thanks for fixing the nits also, just for the record:
>>
>> Acked-by: Eelco Chaudron <[email protected]>
>
> Thanks, I'll plan to apply this.
> Are there any thoughts on backporting?
> It seems more of an enhancement than a fix to me,
> but I'm open to suggestions.
yes its more for traffic performance and not fixing a bug.
branch 3.0 is pretty new and the cherry-pick is clean. so can consider there.
branch 2.9 doesn't have meter/police tc support and not the first commit
introduced tc priority so its not event relevant there.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
