On Thu, May 09, 2024 at 04:55:53PM -0700, Jakub Kicinski wrote:
> diff --git a/tools/testing/selftests/net/forwarding/local_termination.sh 
> b/tools/testing/selftests/net/forwarding/local_termination.sh
> index c5b0cbc85b3e..4bba9c78db3e 100755
> --- a/tools/testing/selftests/net/forwarding/local_termination.sh
> +++ b/tools/testing/selftests/net/forwarding/local_termination.sh
> @@ -73,6 +73,10 @@ check_rcv()
>       local pattern=$3
>       local should_receive=$4
>       local should_fail=
> +     local xfail_sw=$5
> +
> +     local kind=$(ip -j -d link show dev $if_name |
> +                      jq -r '.[].linkinfo.info_kind')
>  
>       [ $should_receive = true ] && should_fail=0 || should_fail=1
>       RET=0
> @@ -81,7 +85,14 @@ check_rcv()
>  
>       check_err_fail "$should_fail" "$?" "reception"
>  
> -     log_test "$if_name: $type"
> +     # If not a SW interface, ignore the XFAIL allowance
> +     [ "$kind" != veth ] && [ "$kind" != bridge ] && xfail_sw=
> +
> +     if [ $RET -ne 0 ] && [ "$xfail_sw" == true ]; then
> +         log_test_xfail "$if_name: $type"
> +     else
> +         log_test "$if_name: $type"
> +     fi
>  }

I just wish IFF_UNICAST_FLT was available to user space.

Reviewed-by: Vladimir Oltean <[email protected]>
Tested-by: Vladimir Oltean <[email protected]>

Reply via email to