Re: [ovs-dev] [PATCH] system-offloads-traffic: Remove tc ingress pps check on meter offload.

2023-04-18 Thread David Marchand
On Tue, Apr 18, 2023 at 10:53 AM Simon Horman wrote: > > > 3. SUPPORT_TC_INGRESS_PPS is intended to check if > > >TC police action with PPS rate is supported by the kernel. > > > > Well, back to this topic then. > > SUPPORT_TC_INGRESS_PPS does not exist anymore. > > So I guess we should at

Re: [ovs-dev] [PATCH] system-offloads-traffic: Remove tc ingress pps check on meter offload.

2023-04-18 Thread Simon Horman
On Tue, Apr 18, 2023 at 10:34:22AM +0200, David Marchand wrote: > On Tue, Apr 18, 2023 at 10:20 AM Simon Horman > wrote: > > > > On Mon, Apr 17, 2023 at 06:28:13PM +0200, David Marchand wrote: > > > Caught during some code review. > > > The incriminated commit had put an unneeded check on tc

Re: [ovs-dev] [PATCH] system-offloads-traffic: Remove tc ingress pps check on meter offload.

2023-04-18 Thread David Marchand
On Tue, Apr 18, 2023 at 10:20 AM Simon Horman wrote: > > On Mon, Apr 17, 2023 at 06:28:13PM +0200, David Marchand wrote: > > Caught during some code review. > > The incriminated commit had put an unneeded check on tc ingress support > > for the meter offloading test. > > > > Note:

Re: [ovs-dev] [PATCH] system-offloads-traffic: Remove tc ingress pps check on meter offload.

2023-04-18 Thread Simon Horman
On Mon, Apr 17, 2023 at 06:28:13PM +0200, David Marchand wrote: > Caught during some code review. > The incriminated commit had put an unneeded check on tc ingress support > for the meter offloading test. > > Note: SUPPORT_TC_INGRESS_PPS had been reworked in the commit 5f0fdf5e2c2e > ("test: Move

[ovs-dev] [PATCH] system-offloads-traffic: Remove tc ingress pps check on meter offload.

2023-04-17 Thread David Marchand
Caught during some code review. The incriminated commit had put an unneeded check on tc ingress support for the meter offloading test. Note: SUPPORT_TC_INGRESS_PPS had been reworked in the commit 5f0fdf5e2c2e ("test: Move check for tc ingress pps support to test script."). Fixes: 5660b89a309d