On Tue, Oct 18, 2016 at 09:34:49AM +0200, Daniel Vetter wrote: > Re S4: If it indeed improves coverage (i.e. calls our shutdown hooks and > all these S4-only callbacks) then adding it to BAT sounds reasonable. > Still there's the issue of where to get the machine time from. I really do > think you need to first trade in some speed-up here (or throw out some > other tests) before you can add more tests. For the record, the runtimes of all tests in CI can be found in the result-runtimes.log file, within the test log directory. Over 10s runtimes on a randomly selected machine are: 31.92 igt@gem_ctx_switch@basic-default-heavy 22.91 igt@gem_busy@basic-hang-default 21.03 igt@kms_flip@basic-flip-vs-dpms 20.96 igt@kms_flip@basic-flip-vs-modeset 20.39 igt@gem_ctx_switch@basic-default 20.30 igt@gem_exec_create@basic 20.11 igt@gem_ctx_create@basic-files 19.64 igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a 18.93 igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b 18.91 igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c 18.76 igt@gem_exec_suspend@basic-s3 17.98 igt@kms_pipe_crc_basic@hang-read-crc-pipe-c 16.70 igt@kms_flip@basic-flip-vs-wf_vblank 14.17 igt@kms_pipe_crc_basic@hang-read-crc-pipe-a 14.14 igt@gem_exec_nop@basic-parallel 14.12 igt@gem_exec_nop@basic-series 13.76 igt@kms_pipe_crc_basic@hang-read-crc-pipe-b 12.70 igt@gem_ringfill@basic-default-hang 11.34 igt@drv_hangman@error-state-basic 10.91 igt@gem_sync@basic-many-each 10.81 igt@gem_close_race@basic-threads 10.73 igt@kms_flip@basic-plain-flip 10.33 igt@gem_sync@basic-store-each 10.29 igt@gem_sync@basic-store-all 10.11 igt@gem_sync@basic-each 10.08 igt@gem_sync@basic-all Does any of these strike as a low-hanging fruit? -- Petri Latvala _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
