[Bug 109225] [CI][DRMTIP] igt at kms_atomic_transition@plane-all-modeset-transition* - fail - Last errno: 28, No space left on device

[bugzilla-daemon at freedesktop.org]

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

Martin Peres <martin.peres at free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #16 from Martin Peres <martin.peres at free.fr> ---
(In reply to Stanislav Lisovskiy from comment #15)
> I think it stopped appearing right after my patch, in the link which Martin
> posted stack trace is still with old igt code(with my changes, there is no
> igt_display_commit at line 501). So I'm afraid, there was no reason to
> reopen it :D

Here is the patch from Stan:

commit 91908d36d0d5c90eea86e29736d2748d5ec55335
Author:     Stanislav Lisovskiy <stanislav.lisovskiy at gmail.com>
AuthorDate: Tue Feb 19 11:38:00 2019 +0200
Commit:     Petri Latvala <petri.latvala at intel.com>
CommitDate: Wed Mar 6 14:53:51 2019 +0200

    igt/tests: Fix error checking in kms_atomic_transition

    There is no guarantee that error return value will be
    always EINVAL, made a check more general as it can be
    ERANGE, ENOSPC, EINVAL and probably others, which all
    mean the same in context of this test case: i.e this sprite
    size is not valid.

    v2: Added macro to make check look a bit nicer.
    v3: Removed redundant debug line.
    v4: Added assertion if error is not EINVAL as expected,
        other errors except EINVAL are considered now a failures.

    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109225
    Signed-off-by: Stanislav Lisovskiy <stanislav.lisovskiy at intel.com>
    Reviewed-by: Stuart Summers <stuart.summers at intel.com>


And the last failure was:

commit 27027cf078e5e8c4ced3b7d941890659e4adf1cd
Author:     Nischala Yelchuri <nischala.yelchuri at intel.com>
AuthorDate: Fri Mar 1 11:49:00 2019 -0800
Commit:     Chris Wilson <chris at chris-wilson.co.uk>
CommitDate: Sat Mar 2 20:25:43 2019 +0000

    tests/kms_cursor_legacy: Add missing munmap

    Added munmap and replaced hard-coded values with PAGE_SIZE macro.

    Cc: Easwar Hariharan <easwar.hariharan at intel.com>
    Cc: Ashutosh Dixit <ashutosh.dixit at intel.com>
    Signed-off-by: Nischala Yelchuri <nischala.yelchuri at intel.com>
    Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>

Given that we had a 100% reproduction rate on drmtip and that the last failure
was seen 10 runs ago (drmtip_236 and we are now at drmtip_246), I think it is
safe to close it again!

Sorry for the noise, Stan!

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