On 8/14/24 09:57, Hangbin Liu wrote:
After commit d7db7775ea2e ("net: veth: do not manipulate GRO when using
XDP"), there is no need to load XDP program to enable GRO. On the other
hand, the current test is failed due to loading the XDP program. e.g.

  # selftests: net: udpgro.sh
  # ipv4
  #  no GRO                                  ok
  #  no GRO chk cmsg                         ok
  #  GRO                                     ./udpgso_bench_rx: recv: bad 
packet len, got 1472, expected 14720
  #
  # failed

  [...]

  #  bad GRO lookup                          ok
  #  multiple GRO socks                      ./udpgso_bench_rx: recv: bad 
packet len, got 1452, expected 14520
  #
  # ./udpgso_bench_rx: recv: bad packet len, got 1452, expected 14520
  #
  # failed
  ok 1 selftests: net: udpgro.sh

After fix, all the test passed.

  # ./udpgro.sh
  ipv4
   no GRO                                  ok
   [...]
   multiple GRO socks                      ok

Fixes: d7db7775ea2e ("net: veth: do not manipulate GRO when using XDP")
Reported-by: Yi Chen <[email protected]>
Closes: https://issues.redhat.com/browse/RHEL-53858
Signed-off-by: Hangbin Liu <[email protected]>

LGTM,

Acked-by: Paolo Abeni <[email protected]>


Reply via email to