From: Petr Machata <petrm@nvidia.com>
To: Jakub Kicinski <kuba@kernel.org>
Cc: Petr Machata <petrm@nvidia.com>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Paolo Abeni <pabeni@redhat.com>, <netdev@vger.kernel.org>
Subject: Re: [PATCH net-next] selftests: forwarding: Add missing config entries
Date: Mon, 29 Jan 2024 11:45:07 +0100 [thread overview]
Message-ID: <87le88l6qz.fsf@nvidia.com> (raw)
In-Reply-To: <20240126112538.2a4f8710@kernel.org>
Jakub Kicinski <kuba@kernel.org> writes:
> On Fri, 26 Jan 2024 17:36:16 +0100 Petr Machata wrote:
>> The config file contains a partial kernel configuration to be used by
>> `virtme-configkernel --custom'. The presumption is that the config file
>> contains all Kconfig options needed by the selftests from the directory.
>>
>> In net/forwarding/config, many are missing, which manifests as spurious
>> failures when running the selftests, with messages about unknown device
>> types, qdisc kinds or classifier actions. Add the missing configurations.
>>
>> Tested the resulting configuration using virtme-ng as follows:
>>
>> # vng -b -f tools/testing/selftests/net/forwarding/config
>> # vng --user root
>> (within the VM:)
>> # make -C tools/testing/selftests TARGETS=net/forwarding run_tests
>
> Thanks a lot for fixing this stuff! The patch went into the
> net-next-2024-01-26--18-00 branch we got: pass 94 / skip 2 / fail 15
>
> https://netdev.bots.linux.dev/contest.html?branch=net-next-2024-01-26--18-00&executor=vmksft-forwarding&pw-y=0
>
> Clicking thru a handful of the failures it looks like it's about a 50/50
> split between timeouts and perf mismatch.
Looking at some recent runs. A number of failures are probably due to
the system failing to oversubscribe the interface with the tested
qdiscs. That's sch_ets, sch_tbf_ets, sch_tbf_prio, sch_tbf_root,
tc_police.
Not sure what to do about it. Maybe separate out heavy traffic tests,
and add a make run_lotraf_tests?
tc_actions started getting a passible deadlocking warning between Jan 27
00:37 and Jan 28 18:27:
https://netdev-2.bots.linux.dev/vmksft-forwarding/results/438201/108-tc-actions-sh/
https://netdev-2.bots.linux.dev/vmksft-forwarding/results/438566/109-tc-actions-sh/
So either something landed that broke it, or the host kernel now has
more debugging enabled, so it now gives a citation.
ip6gre_inner_v6_multipath is just noisy? It failed the last run, but
passed several before.
router_multicast and router get a complaint about a missing control
socket. I think at first approximation they need:
# mkdir -p /usr/local/var/run
But even then I'm getting a fail. This and the others seem to all be in
IPv6 multicast.
next prev parent reply other threads:[~2024-01-29 11:27 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-26 16:36 [PATCH net-next] selftests: forwarding: Add missing config entries Petr Machata
2024-01-26 19:25 ` Jakub Kicinski
2024-01-29 10:45 ` Petr Machata [this message]
2024-01-30 2:31 ` Jakub Kicinski
2024-01-30 12:45 ` Paolo Abeni
2024-01-30 13:00 ` patchwork-bot+netdevbpf
2024-01-30 14:30 ` Jiri Pirko
2024-01-30 18:14 ` Jakub Kicinski
2024-02-13 17:17 ` Petr Machata
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87le88l6qz.fsf@nvidia.com \
--to=petrm@nvidia.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=kuba@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.