[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Zj8LhK28QvLAEhEq@Laptop-X1>
Date: Sat, 11 May 2024 14:09:08 +0800
From: Hangbin Liu <liuhangbin@...il.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: davem@...emloft.net, netdev@...r.kernel.org, edumazet@...gle.com,
pabeni@...hat.com, Vladimir Oltean <vladimir.oltean@....com>,
shuah@...nel.org, linux-kselftest@...r.kernel.org
Subject: Re: [PATCH net-next v2] selftests: net: local_termination: annotate
the expected failures
On Fri, May 10, 2024 at 06:32:36PM -0700, Jakub Kicinski wrote:
> Vladimir said when adding this test:
>
> The bridge driver fares particularly badly [...] mainly because
> it does not implement IFF_UNICAST_FLT.
>
> See commit 90b9566aa5cd ("selftests: forwarding: add a test for
> local_termination.sh").
>
> We don't want to hide the known gaps, but having a test which
> always fails prevents us from catching regressions. Report
> the cases we know may fail as XFAIL.
>
> Reviewed-by: Vladimir Oltean <vladimir.oltean@....com>
> Tested-by: Vladimir Oltean <vladimir.oltean@....com>
> Signed-off-by: Jakub Kicinski <kuba@...nel.org>
> ---
> CC: liuhangbin@...il.com
> CC: shuah@...nel.org
> CC: linux-kselftest@...r.kernel.org
>
> v2:
> - remove duplicated log_test_xfail
> v1: https://lore.kernel.org/all/20240509235553.5740-1-kuba@kernel.org/
> ---
> .../net/forwarding/local_termination.sh | 21 ++++++++++++++-----
> 1 file changed, 16 insertions(+), 5 deletions(-)
Reviewed-by: Hangbin Liu <liuhangbin@...il.com>
Powered by blists - more mailing lists