Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Zhangjin Wu <falcon@tinylab.org>
To:     thomas@t-8ch.de
Cc:     arnd@arndb.de, falcon@tinylab.org, linux-kernel@vger.kernel.org,
        linux-kselftest@vger.kernel.org, w@1wt.eu
Subject: Re: [PATCH v3 01/12] selftests/nolibc: allow report with existing test log
Date:   Sat, 29 Jul 2023 16:53:31 +0800
Message-Id: <20230729085331.7849-1-falcon@tinylab.org>
In-Reply-To: <aa604c61-281d-42b0-85dd-d7a08cdaee6d@t-8ch.de>
References: <aa604c61-281d-42b0-85dd-d7a08cdaee6d@t-8ch.de>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Feedback-ID: bizesmtp:tinylab.org:qybglogicsvrgz:qybglogicsvrgz5a-1
Precedence: bulk

> On 2023-07-28 04:22:42+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 <falcon@tinylab.org>
> > ---
> >  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 2e9694370913..75419b695f0d 100644
> > --- a/tools/testing/selftests/nolibc/Makefile
> > +++ b/tools/testing/selftests/nolibc/Makefile
> > @@ -207,6 +207,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_RUN_OUT)
> 
> Isn't REPORT_RUN_OUT gone in this revision?
>

Yeah, I moved it as the first generic patch but forgot it have used a
later macro, and therefore no recheck in this revision, thanks a lot.

Thanks
Zhangjin

> > +
> >  clean:
> >  	$(call QUIET_CLEAN, sysroot)
> >  	$(Q)rm -rf sysroot
> > -- 
> > 2.25.1
> >