Skip to content

Commit

Permalink
selftests/bpf: Fix up xdp bonding test wrt feature flags
Browse files Browse the repository at this point in the history
Adjust the XDP feature flags for the bond device when no bond slave
devices are attached. After 9b0ed89 ("bonding: do not report
NETDEV_XDP_ACT_XSK_ZEROCOPY"), the empty bond device must report 0
as flags instead of NETDEV_XDP_ACT_MASK.

  # ./vmtest.sh -- ./test_progs -t xdp_bond
  [...]
  [    3.983311] bond1 (unregistering): (slave veth1_1): Releasing backup interface
  [    3.995434] bond1 (unregistering): Released all slaves
  [    4.022311] bond2: (slave veth2_1): Releasing backup interface
  Freescale#507/1   xdp_bonding/xdp_bonding_attach:OK
  Freescale#507/2   xdp_bonding/xdp_bonding_nested:OK
  Freescale#507/3   xdp_bonding/xdp_bonding_features:OK
  Freescale#507/4   xdp_bonding/xdp_bonding_roundrobin:OK
  Freescale#507/5   xdp_bonding/xdp_bonding_activebackup:OK
  Freescale#507/6   xdp_bonding/xdp_bonding_xor_layer2:OK
  Freescale#507/7   xdp_bonding/xdp_bonding_xor_layer23:OK
  Freescale#507/8   xdp_bonding/xdp_bonding_xor_layer34:OK
  Freescale#507/9   xdp_bonding/xdp_bonding_redirect_multi:OK
  Freescale#507     xdp_bonding:OK
  Summary: 1/9 PASSED, 0 SKIPPED, 0 FAILED
  [    4.185255] bond2 (unregistering): Released all slaves
  [...]

Fixes: 9b0ed89 ("bonding: do not report NETDEV_XDP_ACT_XSK_ZEROCOPY")
Signed-off-by: Daniel Borkmann <[email protected]>
Reviewed-by: Toke Høiland-Jørgensen <[email protected]>
Message-ID: <[email protected]>
Signed-off-by: Alexei Starovoitov <[email protected]>
  • Loading branch information
borkmann authored and Alexei Starovoitov committed Mar 6, 2024
1 parent f267f26 commit 0bfc033
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tools/testing/selftests/bpf/prog_tests/xdp_bonding.c
Original file line number Diff line number Diff line change
Expand Up @@ -511,7 +511,7 @@ static void test_xdp_bonding_features(struct skeletons *skeletons)
if (!ASSERT_OK(err, "bond bpf_xdp_query"))
goto out;

if (!ASSERT_EQ(query_opts.feature_flags, NETDEV_XDP_ACT_MASK,
if (!ASSERT_EQ(query_opts.feature_flags, 0,
"bond query_opts.feature_flags"))
goto out;

Expand Down Expand Up @@ -601,7 +601,7 @@ static void test_xdp_bonding_features(struct skeletons *skeletons)
if (!ASSERT_OK(err, "bond bpf_xdp_query"))
goto out;

ASSERT_EQ(query_opts.feature_flags, NETDEV_XDP_ACT_MASK,
ASSERT_EQ(query_opts.feature_flags, 0,
"bond query_opts.feature_flags");
out:
bpf_link__destroy(link);
Expand Down

0 comments on commit 0bfc033

Please sign in to comment.