From: Jakub Kicinski <kuba@kernel•org>
To: Paolo Abeni <pabeni@redhat•com>
Cc: davem@davemloft•net, netdev@vger•kernel.org, edumazet@google•com,
shuah@kernel•org, jiri@resnulli•us,
linux-kselftest@vger•kernel.org
Subject: Re: [PATCH net] selftests: netdevsim: add a config file
Date: Tue, 16 Jan 2024 10:34:30 -0800 [thread overview]
Message-ID: <20240116103430.600fdb9c@kernel.org> (raw)
In-Reply-To: <397aaa82985c749f03d0c6dc034e479d49df1b32.camel@redhat.com>
On Tue, 16 Jan 2024 18:40:49 +0100 Paolo Abeni wrote:
> On Tue, 2024-01-16 at 07:43 -0800, Jakub Kicinski wrote:
> > netdevsim tests aren't very well integrated with kselftest,
> > which has its advantages and disadvantages.
>
> Out of sheer ignorance I don't see the advantage?!?
>
> > But regardless
> > of the intended integration - a config file to know what kernel
> > to build is very useful, add one.
>
> With a complete integration we could more easily ask kbuild to generate
> automatically the kernel config suitable for testing; what about
> completing such integration?
My bad, I didn't have the right words at my fingertips so I deleted
the explanation of advantages.
make run_tests doesn't give us the ability to inject logic between
each test, AFAIU. The runner for netdevsim I typed up checks after
each test whether the VM has any crashes or things got otherwise
out of whack. And if so kills the VM and starts a new one to run
the next test. For make run_tests we can still more or less zero
in on which test caused an oops or crash, but the next test will
try to keep going. Even if we force kill it after we see a crash
I didn't see in the docs how to continue testing from a specific
point.
So all in all, yeah, uniformity is good, the hacky approach kinda
works. Converting netdevsim to make run_tests is not a priority..
next prev parent reply other threads:[~2024-01-16 18:34 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-16 15:43 [PATCH net] selftests: netdevsim: add a config file Jakub Kicinski
2024-01-16 17:40 ` Paolo Abeni
2024-01-16 18:34 ` Jakub Kicinski [this message]
2024-01-17 9:32 ` Paolo Abeni
2024-01-17 15:39 ` Jakub Kicinski
2024-01-18 10:49 ` Paolo Abeni
2024-01-18 11:00 ` patchwork-bot+netdevbpf
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=20240116103430.600fdb9c@kernel.org \
--to=kuba@kernel$(echo .)org \
--cc=davem@davemloft$(echo .)net \
--cc=edumazet@google$(echo .)com \
--cc=jiri@resnulli$(echo .)us \
--cc=linux-kselftest@vger$(echo .)kernel.org \
--cc=netdev@vger$(echo .)kernel.org \
--cc=pabeni@redhat$(echo .)com \
--cc=shuah@kernel$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox