Re: [ovs-dev] [PATCH v3 2/2] Add test cases for ingress_policing parameters

2021-04-13 Thread Simon Horman
On Tue, Apr 13, 2021 at 03:54:32PM +0200, Simon Horman wrote: > On Tue, Apr 13, 2021 at 09:06:16PM +0800, Tonghao Zhang wrote: > > On Fri, Apr 9, 2021 at 7:52 PM Simon Horman > > wrote: > > > > > > From: Tianyu Yuan > > > > > > tests/ovs-vsctl.at: Add ingress_policing test in ovs-vsctl unit test

Re: [ovs-dev] [PATCH v3 2/2] Add test cases for ingress_policing parameters

2021-04-13 Thread Simon Horman
On Tue, Apr 13, 2021 at 09:06:16PM +0800, Tonghao Zhang wrote: > On Fri, Apr 9, 2021 at 7:52 PM Simon Horman > wrote: > > > > From: Tianyu Yuan > > > > tests/ovs-vsctl.at: Add ingress_policing test in ovs-vsctl unit test > > tests/system-offloads-traffic.at: Check ingress_policing with offloads

Re: [ovs-dev] [PATCH v3 2/2] Add test cases for ingress_policing parameters

2021-04-13 Thread Tonghao Zhang
On Fri, Apr 9, 2021 at 7:52 PM Simon Horman wrote: > > From: Tianyu Yuan > > tests/ovs-vsctl.at: Add ingress_policing test in ovs-vsctl unit test > tests/system-offloads-traffic.at: Check ingress_policing with offloads > enabled and disabled > > Exercise OVS setting of ingress_policing parameter

Re: [ovs-dev] [PATCH v3 2/2] Add test cases for ingress_policing parameters

2021-04-09 Thread 0-day Robot
Bleep bloop. Greetings Simon Horman, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers:

[ovs-dev] [PATCH v3 2/2] Add test cases for ingress_policing parameters

2021-04-09 Thread Simon Horman
From: Tianyu Yuan tests/ovs-vsctl.at: Add ingress_policing test in ovs-vsctl unit test tests/system-offloads-traffic.at: Check ingress_policing with offloads enabled and disabled Exercise OVS setting of ingress_policing parameters using ovs-vsctl and verify that the correct values are stored o