On Tue, Nov 08, 2022 at 06:58:15AM -0500, Simon Horman wrote:
> On Tue, Nov 08, 2022 at 10:42:19AM +0200, Roi Dayan wrote:
> >
> >
> > On 02/11/2022 15:08, Ilya Maximets wrote:
> > > On 11/2/22 14: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.
> > >
> > > Looks like an enhancement to me as well.
> > >
> > > Best regards, Ilya Maximets.
> >
> >
> > Hi Simon,
> >
> > Just checking if the commit was forgotten maybe.
>
> Yes, it seems so.
> Sorry about that.
> I'll look into it ASAP.
Applied:
* c230c7579c14 ("netdev-offload-tc: Reserve lower tc prios for ip ethertypes")
https://github.com/openvswitch/ovs/commit/c230c7579c14
Sorry once again for the delay.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
