[LTP] [RFC][PATCH 0/2] Merge runtest files

[Li Wang]

On Thu, Apr 3, 2025 at 6:19 PM Petr Vorel <pvorel@suse.cz> wrote:

> Hi,
>
> besides runtest/ipc, there are other small runtest files.  I know we are
> slowly working towards removing runtest files, but until it happen we
> might want to optimize (=> merge) small runtest files into others.
>

I agree with optimizing the runtest file to organize cases better.
Since we now mainly rely on the runtest files to triage test categories.



> With this change we have in the runtest files (ignoring comments):
>
> $ for i in runtest/*; do echo "$(grep -v '#' $i | wc -l): $i"; done | sort
> -rn
>
>
> $ wc -l runtest/* | sort -nr # not taking into account the comments
> 1823: runtest/syscalls
> 392: runtest/controllers
> 140: runtest/scsi_debug.part1
> 140: runtest/ltp-aiodio.part1
> 137: runtest/net.nfs
> ...
> 9: runtest/watchqueue
> 9: runtest/tracing
> 9: runtest/ima
> 7: runtest/dma_thread_diotest
> 7: runtest/cpuhotplug
> 6: runtest/net.ipv6_lib
> 6: runtest/input
> 6: runtest/capability
> 5: runtest/power_management_tests_exclusive
> 5: runtest/power_management_tests
> 5: runtest/kvm
> 4: runtest/net.multicast
> 3: runtest/uevent
> 3: runtest/hyperthreading
> 3: runtest/crashme
> 3: runtest/can
> 2: runtest/nptl
> 1: runtest/s390x_tests
> 1: runtest/irq
> 1: runtest/fcntl-locktests
> 0: runtest/staging
>
> I would suggest to:
>
> * move runtest/can to runtest/kernel_misc (better
> name would be kernel_drivers, but I don't want to do renames, even
> merging runtest files will disturb folks testing with LTP.
>
> * move runtest/fcntl-locktests to runtest/net.nfs.
>
> * runtest/nptl is NPTL test for pthread_cond_timedwait(), can this be
>   moved to runtest/sched? (there are other pthread_* tests). Not sure if
>   tests are relevant now.
>
> * runtest/irq Obviously we would need more IRQ tests (IMHO none even in
>   kselftest)
>
> I wonder if runtest/dma_thread_diotest. runtest/crashme,
> runtest/power_management_tests and runtest/power_management_tests_exclusive
> are still relevant or that much outdated, that it should be removed.
>
> I think vmcp_m.sh (the only one in runtest/s390x_tests) should be removed.
>
> Other should probably be kept separate.
>
> runtest/staging is expected to be often 0.
>

Sounds good to me, above.

-- 
Regards,
Li Wang