From: Oliver Hartkopp <socketcan@hartkopp.net>
To: Felix Maurer <fmaurer@redhat.com>,
Marc Kleine-Budde <mkl@pengutronix.de>,
linux-can@vger.kernel.org
Cc: Davide Caratti <dcaratti@redhat.com>,
Filippo Storniolo <fstornio@redhat.com>
Subject: Re: Advancing the testing for the CAN subsystem
Date: Fri, 21 Feb 2025 09:04:39 +0100 [thread overview]
Message-ID: <79891c7f-4a8b-4909-af6e-55598bf024e5@hartkopp.net> (raw)
In-Reply-To: <0feda774-853b-42c3-8bae-5d84f7881171@redhat.com>
Hi Felix, all,
by now I was only aware of syzbot who is sometimes showing up with some
splats
and the Linux Test Project (LTP)
https://github.com/linux-test-project/ltp/tree/master/testcases/network/can
which e.g. provides CAN filter tests which you can also find in the
can-tests repo.
On 20.02.25 14:03, Felix Maurer wrote:
> Hi Marc, Oliver, and linux-can community,
>
> we are reaching out to you because we would like to advance the testing
> of the kernel CAN subsystem. We, that's Davide, Filippo and I, are
> volunteering to provide the patches for this, but would like to get your
> feedback and opinions first.
I would definitely like it and support it :-)
> We know about the can-tests repository[1] and think this is a good
> starting point for our efforts. Currently, there are two main activities
> we'd like to do:
>
> - Promote the test cases in can-tests to become part of the kernel
> selftests: This would mainly get the tests closer to the upstream kernel
> development, both in terms of maintenance and actually running them. CI
> systems like LKFT and CKI could easily be continuously running the
> tests.
Ok. Just for my understanding: Bringing the test cases into
tools/testing/selftests would be the enabler to run LKFT and CKI, right?
Does it have any impact or improvement for syzbot or LTP too, e.g. that
we can also improve their test quality?
> The downside is that existing automation depending on can-tests
> (which we don't know about) would need to be modified.
I don't know about any existing automation based in can-tests either.
The reason for the can-test was mostly to document my own PoCs when
adding new features.
I have no objections to stay with this repo as a PoC playground that
might also help developers to get inspirations - and start something new
with the current code base which aims to fuel the kernel selftests.
> - Extend the coverage of the tests: This could include testing for,
> e.g., vcan, vxcan, and the cangw netlink interface. But we're open to
> feedback here if you see any pressing areas.
I see no pressing areas right now. But the recently integrated CAN XL
support might get some attention ;-)
> Like I stated in the beginning, this message should get the discussion
> started and we are looking forward to your feedback or concerns on these
> ideas.
Many thanks Felix.
Best regards,
Oliver
next prev parent reply other threads:[~2025-02-21 8:10 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-20 13:03 Advancing the testing for the CAN subsystem Felix Maurer
2025-02-21 8:04 ` Oliver Hartkopp [this message]
2025-02-24 14:17 ` Felix Maurer
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=79891c7f-4a8b-4909-af6e-55598bf024e5@hartkopp.net \
--to=socketcan@hartkopp.net \
--cc=dcaratti@redhat.com \
--cc=fmaurer@redhat.com \
--cc=fstornio@redhat.com \
--cc=linux-can@vger.kernel.org \
--cc=mkl@pengutronix.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox