Re: [PATCH] buildstats.sh: add ELAPSED option

[Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>]

On Tue, 2017-06-06 at 05:20 +0200, liu.ming50@gmail.com wrote:
> From: Ming Liu <peter.x.liu@external.atlascopco.com>
> 
> This allows users to be able to get the elapsed time of specific
> tasks from buildstats, sometimes could be useful.
> 
> Signed-off-by: Ming Liu <peter.x.liu@external.atlascopco.com>
> ---
>  scripts/contrib/bb-perf/buildstats.sh | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/scripts/contrib/bb-perf/buildstats.sh b/scripts/contrib/bb-perf/buildstats.sh
> index 8d7e248..caf34a6 100755
> --- a/scripts/contrib/bb-perf/buildstats.sh
> +++ b/scripts/contrib/bb-perf/buildstats.sh
> @@ -40,6 +40,7 @@
>  
>  # Stats, by type
>  TIME="utime:stime:cutime:cstime"
> +ELAPSED="Started:Ended:Elapsed time"

This is useful, thanks. Just keep in mind that this is not wall time:
these timestamps come from the bitbake engine (not from /proc/[PID]),
which depending on the build parallelisms, times can change. 


Leo

>  IO="IO wchar:IO write_bytes:IO syscr:IO read_bytes:IO rchar:IO syscw:IO cancelled_write_bytes"
>  RUSAGE="rusage ru_utime:rusage ru_stime:rusage ru_maxrss:rusage ru_minflt:rusage ru_majflt:\
>  rusage ru_inblock:rusage ru_oublock:rusage ru_nvcsw:rusage ru_nivcsw"
> @@ -113,6 +114,9 @@ for stat in ${STATS}; do
>  	    TIME)
>  		stats="${stats}:${TIME}"
>  		;;
> +	    ELAPSED)
> +		stats="${stats}:${ELAPSED}"
> +		;;
>  	    IO)
>  		stats="${stats}:${IO}"
>  		;;
> -- 
> 2.7.4
>