2023-08-03 15:15:35

by Zhangjin Wu

Subject: [PATCH v1 1/3] selftests/nolibc: allow report with existing test log

After the tests finish, it is valuable to report and summarize with
existing test log.

This avoid rerun or run the tests again when not necessary.

Signed-off-by: Zhangjin Wu <[email protected]>
---
tools/testing/selftests/nolibc/Makefile | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/tools/testing/selftests/nolibc/Makefile b/tools/testing/selftests/nolibc/Makefile
index 16cc33db54e3..51fef5e6a152 100644
--- a/tools/testing/selftests/nolibc/Makefile
+++ b/tools/testing/selftests/nolibc/Makefile
@@ -225,6 +225,10 @@ rerun:
$(Q)qemu-system-$(QEMU_ARCH) -display none -no-reboot -kernel "$(srctree)/$(IMAGE)" -serial stdio $(QEMU_ARGS) > "$(CURDIR)/run.out"
$(Q)$(REPORT) $(CURDIR)/run.out

+# report with existing test log
+report:
+ $(Q)$(REPORT) $(CURDIR)/run.out
+
clean:
$(call QUIET_CLEAN, sysroot)
$(Q)rm -rf sysroot
--
2.25.1

2023-08-04 16:43:20

by Thomas Weißschuh

Subject: Re: [PATCH v1 1/3] selftests/nolibc: allow report with existing test log

On 2023-08-03 22:44:47+0800, Zhangjin Wu wrote:
> After the tests finish, it is valuable to report and summarize with
> existing test log.
>
> This avoid rerun or run the tests again when not necessary.
>
> Signed-off-by: Zhangjin Wu <[email protected]>

Reviewed-by: Thomas Weißschuh <[email protected]>

> ---
> tools/testing/selftests/nolibc/Makefile | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/tools/testing/selftests/nolibc/Makefile b/tools/testing/selftests/nolibc/Makefile
> index 16cc33db54e3..51fef5e6a152 100644
> --- a/tools/testing/selftests/nolibc/Makefile
> +++ b/tools/testing/selftests/nolibc/Makefile
> @@ -225,6 +225,10 @@ rerun:
> $(Q)qemu-system-$(QEMU_ARCH) -display none -no-reboot -kernel "$(srctree)/$(IMAGE)" -serial stdio $(QEMU_ARGS) > "$(CURDIR)/run.out"
> $(Q)$(REPORT) $(CURDIR)/run.out
>
> +# report with existing test log
> +report:
> + $(Q)$(REPORT) $(CURDIR)/run.out
> +
> clean:
> $(call QUIET_CLEAN, sysroot)
> $(Q)rm -rf sysroot
> --
> 2.25.1
>