Message ID | 20240315195508.95099-7-eric@garver.life |
---|---|
State | Changes Requested |
Delegated to: | Eelco Chaudron |
Headers | show |
Series | dpif: probe support for OVS_ACTION_ATTR_DROP | expand |
Context | Check | Description |
---|---|---|
ovsrobot/apply-robot | success | apply and check: success |
ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
ovsrobot/intel-ovs-compilation | success | test: success |
On 15 Mar 2024, at 20:55, Eric Garver wrote: > Verify that the explicit drop action is re-probed if the hw-offload flag > is changed. > > Signed-off-by: Eric Garver <eric@garver.life> Thanks for adding this additional test. With the one small change in 5/6, the series looks good to me. Acked-by: Eelco Chaudron <echaudro@redhat.com>
diff --git a/tests/system-offloads-traffic.at b/tests/system-offloads-traffic.at index 6bd49a3eef30..899a8e9bca2f 100644 --- a/tests/system-offloads-traffic.at +++ b/tests/system-offloads-traffic.at @@ -921,3 +921,15 @@ AT_CHECK([tc -d filter show dev ovs-p0 ingress | grep -q "csum (iph)"], [0]) OVS_TRAFFIC_VSWITCHD_STOP AT_CLEANUP + +AT_SETUP([offloads - re-probe drop action]) +OVS_TRAFFIC_VSWITCHD_START() +OVS_CHECK_DROP_ACTION() +AT_KEYWORDS(drop_action) + +dnl trigger a re-probe of the explicit drop action +AT_CHECK([ovs-vsctl set Open_vSwitch . other_config:hw-offload=true]) +OVS_WAIT_UNTIL([grep -q "Datapath does not support explicit drop action" ovs-vswitchd.log]) + +OVS_TRAFFIC_VSWITCHD_STOP +AT_CLEANUP
Verify that the explicit drop action is re-probed if the hw-offload flag is changed. Signed-off-by: Eric Garver <eric@garver.life> --- tests/system-offloads-traffic.at | 12 ++++++++++++ 1 file changed, 12 insertions(+)