Re: [PATCH v2 0/3] Add support for -q <n> unconditional loop

["Nirjhar Roy (IBM)" <nirjhar.roy.lists@gmail.com>]

On 4/3/25 14:28, Nirjhar Roy (IBM) wrote:
> This series introduces "-q <n>" option to support unconditional looping.
> Along with this, it also brings some improvements to output data both, in stdout
> and in result.log.
>
> -q <n> option can be useful for fast unconditional looping (compared to
> -I <n>) to test a certain flakey test and get it's pass/fail metrics.
>
> This is similar to -L <n>, although with -L the test only loops if it
> fails the 1st time.
> The individual patches can provide more details about other changes and
> improvements.

Can I please get some reviews on this patch series?

--NR

>
> [v1] -> [v2]
>   1. This patch series removes the central config fs feature from [v1] (patch 4
>      patch 5) so that this can be reviewed separately. No functional changes.
>   2. Added R.B tag of Darrick is patch  1/5.
>
> [v1] https://lore.kernel.org/all/cover.1736496620.git.nirjhar.roy.lists@gmail.com/
>
> Nirjhar Roy (IBM) (3):
>    tests/selftest: Add a new pseudo flaky test.
>    check: Add -q <n> option to support unconditional looping.
>    check: Improve pass/fail metrics and section config output
>
>   check                  | 113 ++++++++++++++++++++++++++++++-----------
>   tests/selftest/007     |  21 ++++++++
>   tests/selftest/007.out |   2 +
>   3 files changed, 105 insertions(+), 31 deletions(-)
>   create mode 100755 tests/selftest/007
>   create mode 100644 tests/selftest/007.out
>
> --
> 2.34.1
>
-- 
Nirjhar Roy
Linux Kernel Developer
IBM, Bangalore