On 19/09/2017 14:13, Ville Syrjälä wrote:
On Tue, Sep 19, 2017 at 04:01:42PM +0300, Ville Syrjälä wrote:
On Tue, Sep 19, 2017 at 03:31:21PM +0300, Marta Lofstedt wrote:
The kms_rotation_crc@sprite-rotation-*-flip subtests, would need
display engine blending to be setup inorder to work in the same
manner as the respective tests for the primary plane.
Hmm. I don't see anything really blending related in there. It's
just using regular old XRGB framebuffers which means blending will
be off.
OK. So the actual problem is that the test calls drmModePageFlip()
expecting it to magically do something for the sprite plane.
drmModePageFlip() by definition only operates on the primary plane of
the crtc. So the fix looks correct (ie. get rid of the "flip" tests for
the sprite planes) but the commit message is incorrect. This also
explains why you only had to remove the tests with flip==1 and didn't
have to remove the flip==0 tests.
Is it possible to flip on the sprite planes? If so would there be value
with keeping the tests? Just replacing the flip with some other magic
ioctl so those code paths are checked as well.
Also, important thing to note is that with 90/270 the test fails before
the CRC check with a failure from drm_framebuffer_check_src_coords. Not
sure if that is something to investigate or another test code failure?
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
