summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2016-09-02Updated NEWS, and bumped version to 1.16.Marius Vlad
Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-09-02lib/igt_core: Wrap print_backtrace_sig_safe() with HAVE_LIBUNWIND.Marius Vlad
Reported-by: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-09-01igt/pm_rps: Remove reliance on guessing the busy workloadChris Wilson
Create an unbounded batch in order to ensure that the workload doesn't disappear before the wait and so we should be given the RPS waitboost. References: https://bugs.freedesktop.org/show_bug.cgi?id=97564 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-09-01igt/gem_ctx_bad_exec: Begone invalid negative testChris Wilson
Stop looking for an error for a valid combination. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97562 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-09-01igt/prime_vgem: Simplify inter-process flip checksChris Wilson
Avoid having both the child and parent do the same "did the flip" happen check with each looking for the same event on the same fd. The problem being that the child may fall asleep and by the time it wakes up to do its check, the parent has already eaten the event. So leave the checking that the flip does occur to the parent. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-09-01tests/Makefile.am: Add -Wno-unused-result to testsmarius vlad
Removes useless warnings if a caller of a function does not use its returned value(s). Mostly aesthetic. Signed-off-by: Marius Vlad <marius.c.vlad@intel.com> CC: Chris Wilson <chris@chris-wilson.co.uk> CC: Daniel Vetter <daniel@ffwll.ch>
2016-09-01lib/igt_core: Print stacktrace when receiving one of the crash signals.marius vlad
While at it add SIGFPE as a crash signal. v3: Remove calls to igt_assert_eq() as these are not async-safe. As one user of this method remove the function pointer and recursive call. (Chris Wilson) v2: Added some helpers to avoid printf() inside a signal handler. (Chris Wilson) Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-09-01lib/intel_chipset: Fix compilation when enabling the debugger.marius vlad
Add IS_SANDYBRIDGE() macro used by debugger/eudb. Signed-off-by: Marius Vlad <marius.c.vlad@intel.com> CC: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-09-01autotools/: Allow check target to be invoked w/o the need to issue a build.Marius Vlad
We need to have the test list generated before running the check target. Migrated igt_command_line.sh to tests/ from lib/tests/, which allows to building the tests and execute the script. This would allow cleaning followed by a make check. Also assembler/ directory needs also to be adjusted in order for this to work. Kept the possibility to invoke tests/igt_command_line.sh to determine which test is failing. Signed-off-by: Marius Vlad <marius.c.vlad@intel.com> Url: https://patchwork.freedesktop.org/series/6539/ Reviewed-By: Chris Wilson <chris@chris-wilson.co.uk>
2016-09-01lib: Make igt_command_line.sh selftest standaloneChris Wilson
Allow the casual user to run igt_command_line.sh to discover what make check is complaining about. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-30prime_vgem: Fix fence flippingChris Wilson
On the move to doing front/back flips, I managed to completely break the test by forgetting to pass the fence to be signaled. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-29benchmarks/gem_busy: Measure polling of sync_fileChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-29lib: Avoid using 2 writes to /proc/sys/vm/drop_cachesChris Wilson
As the second write is ignored (leading to lack of memory freeing and spuriously failing tests), just do everything from the one write. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-28benchmarks/gem_busy: Compare using wait-ioctl for busyness checkChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-28benchmarks/gem_busy: Exercise the busy ioctlChris Wilson
And include poll(dmabuf) for comparison. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-26igt/kms_flip: There's no such thing as bo-too-bigChris Wilson
Since we can now use the entire global GTT for display, we can handle framebuffers that no longer fit into the mappable aperture. Update the kms_flip/bo-too-big expectations. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97502 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-08-25tests: Add kms_properties test.Maarten Lankhorst
This is a simple test that only tries to set the current property values back. It exposes the issue that some connector properties only work when set through the legacy path, because i915 doesn't handle atomic connector properties yet. The other way around is true too: The atomic CRTC_ID connector property cannot be set through legacy means yet. This causes the connector tests to fail on i915. Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2016-08-25igt_kms: Populate more members of connectors.Maarten Lankhorst
It's possible to make use of disconnected connectors, for example when overriding the mode or testing connector properties that don't need it to be connected. Support this by zeroing the mode in that case. We still return false, but this is ignored by commit. Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
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>