summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2016-08-24igt/kms_cursor_legacy: Reduce scope of basic-cursor-vs-flipChris Wilson
Since we are experiencing too much noise in BAT from what just looks like scheduling delays in inspecting the vblank, reduce the basic test to the fundamentla: check that the cursor ioctl following the nonblocking flip/modeset occurs within the same vblank. Hopefully, CI + debug builds are fast enough to do get-vblank; flip; cursor; get-vblank within a single vblank period. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Imre Deak <imre.deak@intel.com> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2016-08-24igt/kms_busy: Fix subtest enumerationChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-23igt/kms_cursor_legacy: Add missing newlineChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-23lib/kms: Force connector probing on first useChris Wilson
In order for igt to run completely standalone, it must coldplug connectors on first use by forcing the probe. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-23Revert "tests/gem_sync: Skip basic-store-each sub-test on BDW."Chris Wilson
This reverts commit 38f84e30e699451cac6c7b45cd603e67b1287f15.
2016-08-23igt/prime_vgem: Test both front/back flipChris Wilson
Check that we neither stall nor flip too early given active front/back buffers. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-23igt/prime_vgem: Detect when the call to pageflip blocksChris Wilson
Identify whether it is the nonblockling page-flip request that blocks or the event is fired too early. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-23igt/prime_vgem: Perform connector probe on startupChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-23Split out the kms tests from gem_busy to avoid the cairo dependencyChris Wilson
Make kms_busy a separate set of tests so that gem_busy is kept within the core set and not thrown out from Android due to the cairo dependency of rendering the fb. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-22igt/gem_busy: Test flipping on a still active bufferChris Wilson
Flips should wait until all rendering is complete to a framebuffer before applying. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-22Revert "tests/prime_vgem: Remove BAT sub-tests that cause intermittent fails."Chris Wilson
This reverts commit 5188745c4b531d09508cf0897fb90a3634299283.
2016-08-22kms_atomic_transition: Add modeset tests for all plane combinations.Maarten Lankhorst
With the new patch series by Lyude for fixing SKL watermarks it was shown that watermarks for disabled planes may not be updated correctly on modeset. Ensure that this is tested by doing a test similar to plane-all-transition, but with modesets! Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2016-08-19igt/gem_ppgtt: Remember to flush operations before assertionChris Wilson
We have to flush any queued operations before making an assertion such as the VMA being removed from the ppgtt. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-18lib: Preallocate framebuffersChris Wilson
So for reasons unfathomable CRC appear to be miscalculated when the plane is not in the mappable aperture. Worrisome, but for now preallocate space in the mappable region for the new framebuffer and design some specific tests to explore the issue further. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-17igt/gem_exec_big: Check walking backwards through the objChris Wilson
Test slightly less predictable access to the relocation buffer. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-13vc4: Add a test for BO lookup failure error path.Eric Anholt
Causes an oops in the current kernel. Signed-off-by: Eric Anholt <eric@anholt.net>
2016-08-11kms_atomic_transition: Add nonblocking tests for plane transitionsMaarten Lankhorst
There is a potential race between place transitions and nonblocking modesets, so try nonblocking tests for those too. Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2016-08-11kms_atomic_transition: Add modeset tests too.Maarten Lankhorst
Try a series of tests in which we try to enable any combination of crtc enabled. {1,2,3}x-modeset-transitions will run any combination of enabled crtc's to and from the number being tested. Any lower amount is skipped. Extra -nonblocking tests are added, which perform the same modeset tests with nonblocking commit. Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2016-08-11kms_cursor_legacy: Try vrefresh amount of flips in cursor-vs-flipMaarten Lankhorst
When vrefresh is 30 the tests run for twice as long, set it to vrefresh to make the tests just as long anywhere Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2016-08-11kms_cursor_legacy: Add tests for nonblocking modesets vs legacy cursorMaarten Lankhorst
This probably won't pass by design on platforms that need to reallocate global resources like watermarks on gen9+, but it's a good test anyway. Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2016-08-10intel_error_decode: Skip inflation for uncompressed ascii85 streamsChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-10intel_error_decode: Fix buildChris Wilson
Fixes f984bc7de204 ("tools/Makefile: Don't build tools that depend on libdrm_intel") Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-10gem_largeobject: Fix for clang compilerDerek Morton
The clang compiler generates a build error for binaries with >128MB data segments because "The gap between stack and binary is only guarenteed to be 128MB on x86_64" Simple solution is to allocate the required memory from the heap. [v2: moved malloc to after skip_on_simulation] Signed-off-by: Derek Morton <derek.j.morton@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-09igt/gem_busy: Stress test gem_busy() vs gem_close()Chris Wilson
When doing lockless lookups using gem_busy(), one of the largest complications is ensuring that the bo doesn't disappear as we read it. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-05lib: Fix typo s/huma-readable/human-readable/Ville Syrjälä
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2016-08-05tools/intel_reg: Dump DP_BUFTRANS registers on ILK-IVBVille Syrjälä
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2016-08-05tools/intel_reg: Dump fence registers on ILKVille Syrjälä
Currently ILK doesn't get its fences dumped. Let's fix that. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2016-08-05tools/intel_reg: Don't reuse stale decoded results for later registersVille Syrjälä
In case the ->debug_output() function skips decoding the register it just returns, which means the caller will reuse whatever it already has in the tmp buffer as the decoded result for this result. What it usually has in there is the decoded result of some previous register. Showing incorrect decoded results is no good, so let's allow ->debug_output() to actually return how many bytes it wrote, and the caller can then skip showing the decoded results if zero bytes were produced. We'll make a variant of snprintf() that's safe to call without having to check the return value for the case when it didn't have enough space to do its work, that is, make it return 0 in case no bytes were written. v2: Document the _DEBUGSTRING() funcion (Jani) Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Jani Nikula <jani.nikula@intel.com>
2016-08-05igt/drv_missed_irq: Clear pending IRQs before testingChris Wilson
Upon starting, the IIR may contain stale user interrupts which then trigger a wakeup when another interrupt occurs (e.g. execlists context switch). This spurious wakeup causes the waiter to unstick and proceed - preventing the missed interupt test from suceeeding. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-05igt/drv_missed_irq: Coordinate child/parentChris Wilson
In the child, make sure the parent is about to sleep before allowing the GPU to complete. In theory, we are created as an inferior process on the same CPU and the scheduler shouldn't be executing the child before the parent, but seems to anyway. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-05igt/gem_madvise: Quick fixup of expectation around mmap()Chris Wilson
We don't do any santiy checking at the point of crating the mmap(), that is deferred until the pointer is actually dereferenced (as the state of the object may well change between mmap and pagefault). Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-04igt_kms: Use proper panning coordinates for legacy primary plane updatesMatt Roper
A copy/paste error resulted in us using src_x for both the x and y panning coordinates; make sure we use src_y instead for the appropriate parameter. Fixes: 0e29ce3265b ("igt_kms: Remove pan members from igt_plane, v2.") Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Signed-off-by: Matt Roper <matthew.d.roper@intel.com> Reviewed-by: Bob Paauwe <bob.j.paauwe@intel.com>
2016-08-04Revert "tests/prime_busy: Remove unstable sub-test from BAT/add flag to ↵Chris Wilson
bring them back when they're stable to be run in CI." This reverts commit 860b3c14f9e98face626e6ca025daf22844dd156.
2016-08-04igt/gem_mocs_settings: adding RC6 testsPeter Antoine
This change adds a RC6 test for the MOCS. The MOCS registers are loaded and saved as part of the RC6 cycle but not all the registers are saved/restored. This tests that those registers are correctly restored. Signed-off-by: Peter Antoine <peter.antoine@intel.com>
2016-08-04igt/gem_mocs_settings: Remove direct register testsPeter Antoine
On some platforms the MOCS values are not always saved and restored on RC6 enter/exit. The rational is that the context with restore these values. On these platforms the test will fail as it tests the values by directly reading the MOCS registers. So this change removes the direct testing of the values. Signed-off-by: Peter Antoine <peter.antoine@intel.com>
2016-08-04lib/Makefile: Replace automake specific names of listings in Makefile.sourcesRobert Foss
Replace the automake specific variable names for listings in Makefile.sources with something not automake specific. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04lib/tests/Makefile: Replace automake specific names of listings in ↵Robert Foss
Makefile.sources Replace the automake specific names of listings with something that isn't automake specific. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04tools/Makefile: Replace automake specific name of listings in Makfile.sourcesRobert Foss
Replace the automake specific names of listings in Makefile.sources with something not automake specific. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04benchmarks/Makefile: Replace automake specific name of listing in ↵Robert Foss
Makfile.sources Replace the automake specific name of listings in Makefile.sources with something not automake specific. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04demos/Makefile: Replace automake specific name of listing in Makfile.sourcesRobert Foss
Replace the automake specific name of listings in Makefile.sources with something not automake specific. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04lib/stubs: Add stubs for intel_bufmgrRobert Foss
This patch provides stubs for functionality otherwise provided by intel_bufmgr. The stubbed functions all fail with a call to igt_require_f(false,""). Defines and enums have been copied from libdrm_intel. Due to the stubbed tests failing with an igt_require_f() call, these stubs are not well suited for non-tests, since tools/benchmarks/etc 'skipping' execution is unhelpful. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04demos/Makefile: Don't build tools that depend on libdrm_intelRobert Foss
Use the HAS_INTEL automake flag to avoid building tools that won't compile unless libdrm_intel is available in the build system. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04tools/Makefile: Format whitespaceRobert Foss
Harmonize tabs/spaces etc. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04tools/Makefile: Don't build tools that depend on libdrm_intelRobert Foss
Use the HAS_INTEL automake flag to avoid building tools that won't compile unless libdrm_intel is available in the build system. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04benchmarks/Makefile: Don't build benchmarks that depend on libdrm_intelRobert Foss
Use the HAS_INTEL automake flag to avoid building benchmarks that won't compile unless libdrm_intel is available in the build system. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04Enable HAVE_LIBDRM_INTEL unconditionally for AndroidRobert Foss
Always set HAVE_LIBDRM_INTEL to true for Android targets. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04configure.ac: Harmonize HAVE_XXX flag for all drm platforms to HAVE_LIBDRM_XXXRobert Foss
Replace the automake flag HAVE_XXX for VC4/NOUVEAU with HAVE_LIBDRM_XXX in order for the flags to be more descriptive and also follow the same convention as HAVE_LIBDRM_INTEL. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-04configure.ac: Test for libdrm_intel and build for it if presentRobert Foss
Test for libdrm_intel and build for it if present. Also expose the HAVE_INTEL #define to allow code to be conditionally compiled. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-08-01igt/gem_mmap_gtt/wc: Reduce test runtimeChris Wilson
Add a new iterator macro to run for a specified number of milliseconds. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-01igt/gem_mmap_gtt: Ignore effects of CPU caches for comparing WBChris Wilson
When measuring WB performance we have the issue that it allows CPU caching which is many times faster than main memory! Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>