summaryrefslogtreecommitdiff
path: root/tests
AgeCommit message (Collapse)Author
2016-04-25tests/kms_flip_event_leak: Open DRM device with DRIVER_ANYTomeu Vizoso
So that this test can be run in drivers other than i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Series-version: 1 Series-to: intel-gfx Series-cc: padovan, daniels, marcheu, seanpaul, xexaxo, fedkem, mvlad, danvet Series-prefix: i-g-t Cover-letter: Make more tests generic Hi, these patches allow a few more tests to run on drivers other than i915, mainly by removing the last usage of DRM_IOCTL_I915_GET_PIPE_FROM_CRTC_ID and removing superfluous dependencies on bufmgr and tiled BOs. Thanks, Tomeu END
2016-04-25tests/kms_flip_event_leak: Use non-tiled formatsTomeu Vizoso
As the test doesn't actually need tiled BOs, drop the tiled formats so the test can run on drivers other than i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25tests/kms_flip: Open DRM device with DRIVER_ANYTomeu Vizoso
So that this test can be run in drivers other than i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25tests/kms_flip: Move bufmgr requirement into subtestsTomeu Vizoso
Because bufmgr is currently a i915-only thing and it's only needed in a subset of the subtests, require it only in the subtests that actually need it so that the other subtests aren't skipped without a reason. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25tests/kms_flip: Create tiled BOs only when neededTomeu Vizoso
Because attempts to create a tiled BO will cause a igt_require call to fail on drivers that don't support tiling, do so in the subtest that actually needs it so that other subtests aren't skipped without reason. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25tests/kms_render: Move dependency on i915 into subtestTomeu Vizoso
Batchbuffers are only needed in the subtest that does the blit on the GPU, so move that dependency into it so the other subtest can be ran on !i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25tests/drm_read: Drop DRM_IOCTL_I915_GET_PIPE_FROM_CRTC_IDTomeu Vizoso
So the test runs on other drivers, drop the usage of the i915-specific DRM_IOCTL_I915_GET_PIPE_FROM_CRTC_ID ioctl. Wait for a vblank event on pipe0 and if we get it, then the test can proceed (code copied from kms_vblank). Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25kms_flip: Shorten basic tests a bitTvrtko Ursulin
Seems like an overkill for a basic test to keep flipping for a full minute. Dial it down a bit. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-04-25igt/gem_close_race: Restore threads test to BAT statusChris Wilson
Let's try it again because it would have caught a bug in a patch I sent to the ml... References: https://bugs.freedesktop.org/show_bug.cgi?id=95048 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-25igt/gem_busy: Test parallel executionChris Wilson
Parallel but not concurrent! Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-22igt/gem_busy: Discard the indirect batchbuffersChris Wilson
The stack of batchbuffers is myth. In general there are only 2 levels of stack for HEAD (ringbuffer, batchbuffer) and chaining up the batch buffer just updates the lowest level of the stack. A BATCH_BUFFER_END at any depth then returns to the ring. So be creative and modify the batch buffer on the fly... Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-22igt/gem_exec_nop: Apply missed interrupt detectionChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-22igt/gem_close_race: Import the scratchChris Wilson
Rather than create a new scratch buffer every pass, import the original as this provides stress upon less commonly trod paths (i.e. handling objects with many vma) as well as many objects in general. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-22igt/gem_close_race: Avoid using threads, use signals insteadChris Wilson
Emulate the behaviour of the second thread killing fd at random by having a signal fire at a random time instead. Only one thread and so we do not have the issue of accessing another valid handle on another fd and so executing a blank buffer - triggering GPU hangs. References: https://bugs.freedesktop.org/show_bug.cgi?id=95048 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-21tests/gem_close_race: Remove basic-threads from BAT.Marius Vlad
Currently this test causes some machines to hang and segfaults on others. Rename it for now until we figure out the root cause. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95048 Acked-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-04-20igt/gem_close_race: BatifyChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-20tests/kms_rotation_crc: Adding rotation tests using COMMIT_ATOMIC pathpvishwak
Modified kms_rotation_crc to follow COMMIT_ATOMIC path if ATOMIC support is available from driver. If ATOMIC support is not present, tests will follow COMMIT_UNIVERSAL path. v2: (Marius) Updated commit message https://patchwork.freedesktop.org/patch/76040/ v3: (Marius) Added description in commit message https://patchwork.freedesktop.org/patch/79492/ Signed-off-by: Pratik Vishwakarma <pratik.vishwakarma@intel.com> [mcvlad: Re-worked a little comment description] Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-04-20kms_atomic: Fix crtc_id comparison failing.Robert Foss
Fixed ctrc_id comparison failing due to bad initialization of crtc variable. Signed-off-by: Robert Foss <robert.foss@collabora.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-04-20tests: Separate tests with lots of subtestsFeceoru, Gabriel
Currently, when trying to run i-g-t tests with piglit, it takes up to 20s until the first test is executed. The main reason is that gem_concurrent_all has ~500k subtests, overkilling piglit. This patch separates gem_concurrent_* tests from the rest of the tests, creating two files test-list.txt and test-list-full.txt. The piglit can now enumerate the i-g-t tests within a decent few seconds. The second list could be used when trying to execute specific these tests. v2: Rebased Signed-off-by: Gabriel Feceoru <gabriel.feceoru@intel.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-04-20tests/kms_pipe_crc_basic: Don't suspend the machine if the pipe is not present.marius vlad
suspend-read-crc-pipe will perform a suspend and then skip the test in case the pipe is not present or is fused w/ another pipe. Skip the test before doing the suspend. v2: Somehow the subject was trimmed. Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-04-20ksm_pipe_color: Set legacy gamma values inside loop.Bob Paauwe
When testing multple outputs, make sure to set the gamma values before testing the output. Otherwise we're testing using the gamma values that were reset after last output was tested. Without this, the first output passes, but each output after that will fail. Signed-off-by: Bob Paauwe <bob.j.paauwe@intel.com> Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-04-20igt/gem_concurrent_blit: Ensure we only shrink the child buffersChris Wilson
As we reuse the parent arrays under the assumption that the child uses fewer buffers, make sure that is true. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-18igt/gem_exec_nop: Fix time units for assertion printfChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-18igt/gem_ctx_param: RelabelChris Wilson
Checking parameters is not interesting for BAT, it is a simple ABI test that only provides coverage of the CONTEXT_GETPARAM and CONTEXT_SETPARAM ioctls, and no deeper. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-16gem_exec_lut_handle: Fix presumed_offset to force relocation on full-ppgttChris Wilson
If the object is at offset 0, quite likely using full-ppgtt, then the presumed_offset set also to 0 causes the relocation to be skipped. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-15igt/gem_mocs_settings: Check for pollution of default contextsChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-15igt/drv_module_reload_basic: Do not require snd-hda-intelTvrtko Ursulin
Recent refactoring has made absence of snd-hda-intel a test failure do the respective modprobe being at the end of a reload function now. Only fail in this case if module was previously unloaded. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Cc: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-15tests/kms_addfb_basic: Set tiling in addfb25 subtestTomeu Vizoso
When moving the call to gem_set_tiling into the subtests, the one needed by addfb25-framebuffer-vs-set-tiling was left out. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2016-04-15igt/gem_pwrite: Add backwards/random access patternsChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-15igt/gem_busy: Slow down the writerChris Wilson
Add a few more (128) loops to the page full of MI_STORE_DWORD in an attempt to try and slow down the execution to the point where a full-debug kernel can beat the GPU. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-15igt/gem_exec_nop: Correct %3.f typoChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-14igt/gem_exec_nop: Explain the parallel execution assertionChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-14tests: Open any driverTomeu Vizoso
For those tests that now pass on drivers other than i915, call drm_open_driver_master with DRIVER_ANY. Also do so from igt_enable_connectors. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-04-14kms_addfb_basic: Get intel gen from within subtestTomeu Vizoso
Because determining the Intel GFX generation requires a call to DRM_IOCTL_I915_GETPARAM, move the code that requires it to a subtest that can be skipped on drivers other than i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-04-14kms_addfb_basic: Move calls to gem_set_tiling to the subtestsTomeu Vizoso
So they don't cause unrelated subtests to be skipped when testing drivers other than i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-04-14kms_addfb_basic: Split tiling_tests offTomeu Vizoso
Move tests requiring tiled BOs to the end so they don't cause unrelated subtests to be skipped when testing drivers with only dumb buffer support. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-04-14kms_addfb_basic: move tiling functionality into each subtestTomeu Vizoso
Because calls to gem_set_tiling will cause the subtest to be skipped on drivers other than i915, move them to each subtest that needs them so the other subtests aren't skipped as well. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-04-14kms_addfb_basic: call igt_create_bo_with_dimensionsTomeu Vizoso
Many tests can do their work on drivers other than i915 and even with just dumb buffers, so call igt_create_bo_with_dimensions instead of gem_create which will paper out the differences and call the proper ioctls or cause the subtest to be skipped if that's not possible. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-04-14lib: Add wrapper for DRM_IOCTL_MODE_CREATE_DUMBTomeu Vizoso
In order to test drivers that don't have support for proper buffer objects, add a wrapper for creating dumb buffer objects that will be called from the lib code for those subtests that don't need to care. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-04-14igt/gem_ctx_create: Broaden per-engine testingChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-12igt/gem_exec_create: Add a "leak" testChris Wilson
Don't close the objects, but just mark them as unneded. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-12igt: Add a basic test to stress allocation of batchesChris Wilson
Run for 20s and don't expect to crash from memory exhaustion or sillies. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-12test/gem_mocs_settings: Testing MOCS register settingsPeter Antoine
The MOCS registers were added in Gen9 and define the caching policy. The registers are split into two sets. The first set controls the EDRAM policy and have a set for each engine, the second set controls the L3 policy. The two sets use the same index. The RCS registers and the L3CC registers are stored in the RCS context. The test checks that the registers are correct by checking the values by directly reading them via MMIO, then again it tests them by reading them from within a batch buffer. RCS engine is tested last as it programs the registers via a batch buffer and this will invalidate the test for workloads that don't use the render ring or don't run a render batch first. v2: Reorganised the structure. Added more tests. (Chris Wilson) v3: Fixed a few bugs. (Chris Wilson) v4: More Tidy-ups. (Chris Wilson) SKL does does not have a snoop bit. (Peter Antoine) Signed-off-by: Peter Antoine <peter.antoine@intel.com>
2016-04-11igt/gem_busy: fix compaction of multiple store commandsChris Wilson
Missed increment of the batch offset between commands. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-11igt/gem_concurrent_blit: Tweak num_buffers to just exceed targetChris Wilson
Remember that we allocate and use twice as many buffers as specified (we have a num_buffers array of src and dst) and so adjust the computation such that the combined allocation matches the target. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-10igt/gem_concurrent_blit: Purge the libdrm caches before cloningChris Wilson
When forking, the entire VM is cloned including the VMA backed by GEM objects from the libdrm buffer cache. This is not only slow, but can also cause the system to exceed the maximum number of VMA and premature test failure. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-10igt/gem_shrink: Fix execution object offsetChris Wilson
An off-by-one caused us to execute the blank object rather than the batch. References: https://bugs.freedesktop.org/show_bug.cgi?id=94801 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-09lib: Remove defunct stop_ringsChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-09igt/kms_flip: Convert over to real hang injectionChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-09igt/kms_pipe_crc_basic: Replace stop_rings with igt_hang_ringChris Wilson
We can inject a real GPU hang for greater effect! Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>