Re: [RFC] MAINTAINERS: split kselftest entry into 'framework' and 'all'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/3/24 08:23, Jakub Kicinski wrote:
The testing effort is increasing throughout the community.
The tests are generally merged into the subsystem trees,
and are of relatively narrow interest. The patch volume on
linux-kselftest@xxxxxxxxxxxxxxx makes it hard to follow
the changes to the framework, and discuss proposals.


I agree with you that the linux-kselftest mailing list is high
volume list and it is hard to keep up with it.

Create a new ML for "all" of kselftests (tests and framework),
replacing the old list. Use the old list for framework changes
only. It would cause less churn to create a ML for just the
framework, but I prefer to use the shorter name for the list
which has much more practical use.


I am not sure if the split it helps with reducing the work for
maintainers and reviewers. We might end up watching both lists.
I know I have to.

It is hard enough with one list for people to cc the right people
and the list itself when they send patches. I am concerned that with
two lists it becomes harder.

selftests have grown year after year and we have about 109 subdirs
under selftests as of 6.12.
Some of these are high volume, active, and longer term citizens of
selftests: net, mm, and bpf. Developers and maintainers from these
subsystems are very engaged in test development and using the tests
for their development activities. This also includes they fit into
the selftest common infrastructure or core.

These are the subsystems that don't require my attention as much to
make sure builds and installs are working correctly. These tests
have been around a long time and the problems if any were are ironed
out. I sanity check the Makefile changes and if there is new test that
gets added.

There are several other subsystems that aren't highly active. I take
these patches through my tree. Whenever a new subsystem/directory gets
added, I review to make sure Makefile that anchors the test to selftest
core is correct.

Most subsystems' test patches get reviewed by their developers which
is how it should be. The tests should be reviewed by developers
and maintainers in those subsystems.

Can we solve the problem of hard to follow the changes to the framework,
and discuss proposals another way?

- Naming convention selftests/core for core/common framework patches
  We have to get people to do this. New ML has the same issue of getting
  people to use the right ML.


Signed-off-by: Jakub Kicinski <kuba@xxxxxxxxxx>
---
Posting as an RFC because we need to create the new ML.

CC: shuah@xxxxxxxxxx
CC: linux-kselftest@xxxxxxxxxxxxxxx
CC: workflows@xxxxxxxxxxxxxxx
---
  MAINTAINERS | 12 ++++++++++++
  1 file changed, 12 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index c27f3190737f..9a03dc1c8974 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12401,6 +12401,18 @@ S:	Maintained
  Q:	https://patchwork.kernel.org/project/linux-kselftest/list/
  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest.git
  F:	Documentation/dev-tools/kselftest*
+F:	tools/testing/selftests/kselftest/
+F:	tools/testing/selftests/lib/
+F:	tools/testing/selftests/lib.mk
+F:	tools/testing/selftests/Makefile
+F:	tools/testing/selftests/*.sh
+F:	tools/testing/selftests/*.h

I would add these somehow to core/framework. Some of the
issues CI runs into require fixes to individual test makefiles.

selftests/*testdirs*/Makefile

If we do decide on creating two lists, I would prefer creating
an alias for exiting linux-kselftest to linux-kselftest-all
and then create a new list kselftest-core

+
+KERNEL SELFTEST TESTS
+M:	Shuah Khan <shuah@xxxxxxxxxx>
+M:	Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>
+L:	linux-kselftest-all@xxxxxxxxxxxxxxx
+S:	Maintained
  F:	tools/testing/selftests/
KERNEL SMB3 SERVER (KSMBD)

thanks,
-- Shuah





[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux