[Piglit] [PATCH v2 1/2] piglit-run.py: record whether the test run was concurrent

Ken Phillis Jr kphillisjr at gmail.com
Mon Nov 18 15:08:11 PST 2013


The only suggestion I can make is to combine both patches since the two
patches are co-dependent.


Also, it may be good to have the html summary status page state whether or
not the test is concurrent or serial.

For both patches:
Reviewed by: Kenney Phillis <kphillisjr at gmail.com>


On Fri, Nov 15, 2013 at 10:55 AM, Dylan Baker <baker.dylan.c at gmail.com>wrote:

> cc: ben at bwidawsk.net
> Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
> ---
>  piglit-run.py | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/piglit-run.py b/piglit-run.py
> index 1d63cd4..4aa5929 100755
> --- a/piglit-run.py
> +++ b/piglit-run.py
> @@ -142,6 +142,7 @@ def main():
>      json_writer.write_dict_item('profile', profileFilename)
>      json_writer.write_dict_item('filter', args.include_tests)
>      json_writer.write_dict_item('exclude_filter', args.exclude_tests)
> +    json_writer.write_dict_item('concurrency', args.concurrency)
>      json_writer.close_dict()
>
>      json_writer.write_dict_item('name', results.name)
> --
> 1.8.3.2
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20131118/8793b761/attachment.html>


More information about the Piglit mailing list