[Bug 106701] [CI] igt at kms_available_modes_crc@available_mode_test_crc - fail - Failed assertion: __gem_create(fd, size, &handle) == 0

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=106701

--- Comment #7 from Juha-Pekka Heikkilä <juhapekka.heikkila at gmail.com> ---
I think problem part is this piece of code in kms_available_modes_crc.c,
setup_fb():

        switch (fillers[i].bpp) {
        case NV12:
        case BYTES_PP_1:
                bpp = 8;
                break;

        case P010:
        case BYTES_PP_2:
                bpp = 16;
                break;

        case SKIP4:
        case BYTES_PP_4:
                bpp = 32;
                break;
        }

it's missing default case and leave 'bpp' to zero. To me it look like correct
fix is to change in table 'fillers' last element from 

        { 0, 0, 0, 0 }

to

        { 0, 0, SKIP4, 0 }

I think this change will fix bug that would follow from creating default case.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20181102/40b9a28a/attachment.html>