[Intel-gfx] [PATCH i-g-t] tests: Add kms_plane_blinker

[Maarten Lankhorst]

Op 15-12-16 om 18:26 schreef Ville Syrjälä:
> On Thu, Dec 15, 2016 at 06:42:08PM +0200, Ville Syrjälä wrote:
>> On Thu, Dec 15, 2016 at 04:41:51PM +0100, Maarten Lankhorst wrote:
>>> Op 15-12-16 om 16:36 schreef Ville Syrjälä:
>>>> On Thu, Dec 15, 2016 at 04:28:52PM +0100, Maarten Lankhorst wrote:
>>>>> Op 15-12-16 om 16:23 schreef Ville Syrjälä:
>>>>>> On Thu, Dec 15, 2016 at 04:17:45PM +0100, Maarten Lankhorst wrote:
>>>>>>> Op 12-12-16 om 21:35 schreef ville.syrjala at linux.intel.com:
>>>>>>>> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>>>>>>>>
>>>>>>>> Add a test to try out all the different plane enable/disable
>>>>>>>> order permutations.
>>>>>>>>
>>>>>>>> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
>>>>>>> Didn't look through the test, but sounds like
>>>>>>>
>>>>>>> kms_atomic_transitions.plane-*-transition-* ?
>>>>>>>
>>>>>>> Although that one tests a few more edge cases like modeset disable and nonblocking updates..
>>>>>> I don't immediately see where it would try all the permutations, nor can
>>>>>> I see any crc_nonblock() stuff so doesn't seem like it could even spot
>>>>>> transient errors.
>>>>>>
>>>>> I haven't done any crc test there yet, but the double loop in run_transition_test handles all combinations of planes.
>>>> permutations > combinations
>>>>
>>> It permutates too, I used it for some basic wm testing before. :)
>> Does it? I'm too lazy to reverse engineed it, so I just tried to run it
>> but it didn't want to run, so meh.
> Needed i915.nuclear_pageflip=1. Didn't even know we had such a knob.
That was always needed for any atomic test in i915. :)
> Anywyas, all it tells me is
> "Running test on pipe A with resolution 1920x1080 and sprite size 1920x1080 alpha 1"
> and similar lines, and mostly I just see a black screen or a blinking backlight.
> So I can't tell what it's actually doing.
>
Well you can tell with --debug, it shows all atomic commits done.

It's a nice test for a 4k HDMI display, because in that case I had
big enough planes that having all planes enabled would cause a
underrun because the fifo size was too small. Patch 14 solves this
issue.

~Maarten