[PATCH] test/py: Add a report show test durations

Simon Glass sjg at chromium.org
Sat Jan 25 18:14:05 CET 2025


Hi Tom,

On Fri, 24 Jan 2025 at 12:21, Tom Rini <trini at konsulko.com> wrote:
>
> On Fri, Jan 24, 2025 at 11:56:04AM -0700, Simon Glass wrote:
>
> > Execution time varies widely with the existing tests. Provides a summary
> > of the time taken for each test, along with a histogram.
> >
> > Example:
> >
> >    Duration : Number of tests
> >    ======== : ========================================
> >        <1ms :  1
> >        <8ms :  1
> >       <20ms : # 20
> >       <30ms : ######## 127
> >       <50ms : ######################################## 582
> >       <75ms : ####### 102
> >      <100ms : ## 39
> >      <200ms : ##### 86
> >      <300ms : # 29
> >      <500ms : ## 42
> >      <750ms : # 16
> >       <1.0s : # 15
> >       <2.0s : # 23
> >       <3.0s :  13
> >       <5.0s :  9
> >       <7.5s :  1
> >      <10.0s :  6
> >      <20.0s :  12
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> >
> >  test/py/conftest.py | 77 +++++++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 77 insertions(+)
>
> Whitespace issues aside,

Do you mean the blank lines? pylint wants those for top-level items.

 we should probably not do this every time, just
> when requested.

I'd like to see it on each run, actually. It doesn't take any time to
calculate and it lets us see where the time is going.

Regards,
Simon


More information about the U-Boot mailing list