- May 31, 2016
-
-
Marius Vlad authored
Signed-off-by:
Marius Vlad <marius.c.vlad@intel.com>
-
Tomeu Vizoso authored
When reworking igt_create_fb_with_bo_size to use igt_create_bo_with_dimensions, we mistakenly stopped acknowledging non-zero size parameters. To fix this, we move the core of the code to create_bo_for_fb and teach it to use the GEM APIs when a size is passed. igt_create_bo_with_dimensions ends up calling just create_bo_for_fb with a zero size because now the later is more generic than the former. Also, create_bo_for_fb now returns the handle of the BO that was created, as there's no point anymore in having it be a parameter passed by reference. Signed-off-by:
Tomeu Vizoso <tomeu.vizoso@collabora.com>
-
- May 27, 2016
-
-
Matt Roper authored
Some validation teams seem to run tests out of source directories that have been nfs mounted or rsync'd to different locations on the target machine. This causes the igt_srcdir that the tests were built with to be invalid on the machine the tests get run on. Add the current directory as a final fallback for data file searches. Reference: https://bugs.freedesktop.org/show_bug.cgi?id=92248 Cc: Humberto Israel Perez Rodriguez <humberto.i.perez.rodriguez@intel.com> Signed-off-by:
Matt Roper <matthew.d.roper@intel.com> Signed-off-by:
Marius Vlad <marius.c.vlad@intel.com>
-
- May 25, 2016
-
-
Chris Wilson authored
igt_fixture and its byzantine requirements Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
Just to compare handling of plane->mutexes Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
- May 24, 2016
-
-
Chris Wilson authored
Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
Looking at the uncontended cases reveals a major contention cliff. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Derek Morton authored
drm_read gem_exec_blt prime_mmap_kms Have cairo dependency through igt_kms.c so add to skip_tests_list to fix android build errors. Signed-off-by:
Derek Morton <derek.j.morton@intel.com> Reviewed-by:
Tim Gore <tim.gore@intel.com> Signed-off-by:
Marius Vlad <marius.c.vlad@intel.com>
-
Feceoru, Gabriel authored
Repairing the damage I caused not reading properly Daniel's comment in: https://patchwork.freedesktop.org/patch/81600/ Leaving gem_concurrent_all only in the EXTRA set Cc: Daniel Vetter <daniel.vetter@intel.com> Cc: Marius Vlad <marius.c.vlad@intel.com> Acked-By:
Daniel Vetter <daniel.vetter@intel.com> Signed-off-by:
Gabriel Feceoru <gabriel.feceoru@intel.com> Signed-off-by:
Marius Vlad <marius.c.vlad@intel.com>
-
Switched from drm_XXX aliases drm_intel_XXX aliases for symbols where that switch is possible. 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>
-
Switched from drm_XXX aliases drm_intel_XXX aliases for symbols where that switch is possible. 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>
-
Chris Wilson authored
Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
- May 23, 2016
-
-
Chris Wilson authored
Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Simona Vetter authored
Noticed while discussing CRC tests with Ville that this was totally wrong. v2: Ville pointed out that it only does not block when opened using igt_pipe_crc_new_nonblocking. Still different from igt_pipe_crc_collect_crc, which will always block. v3: Fix type (Ville). Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Acked-by:
Ville Syrjälä <ville.syrjala@linux.intel.com> Signed-off-by:
Daniel Vetter <daniel.vetter@intel.com>
-
- May 21, 2016
-
-
Chris Wilson authored
Set bit 21 to enable qword writes for Broadwell. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
- May 20, 2016
-
-
Chris Wilson authored
First try an illegal instruction before the infinite loop to try and trigger a gpu hang more gracefully as the infinite loop is causing some older machines issues as the GPU hogs the entire systems and makes the machine laggy, unresponsive for long periods. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
Since we do not use the allow-48b GTT per-object flag, even on Broadwell+ we will be constrained to 4GiB of usage GTT. Also only do a single pass for BAT as suggested by Marius Vlad. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
At its heart gem_exec_whisper is a race detector between submission between multiple rings and the execution. It is made even more unreliable by debug kernels making it nigh impossible to submit batches faster than the GPU can execute. As such, it's relevance for BAT is dubious. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
Rely on testing every engine in parallel to detect the most common synchronisation problems ("missed interrupt syndrome") for BAT. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
- May 19, 2016
-
-
Chris Wilson authored
Oops, because -nightly doesn't handle 64bit alignments correctly, this was not working quite as expected. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
Since we want to focus on the issue of running with a full-gtt, and part of that is in handling the eviction, do a warm-up pass before we start the clock that allocates the objects and fills the gtt. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
If we specify an unobtainable alignment (e.g, 63bits) the kernel will eviction the object from the GTT and fail to rebind it. We can use this, to measure how long it takes to move objects around in the GTT by running execbuf followed by the unbind. For small objects, this will be dominated by the nop execution time, but for larger objects this will be ratelimited by how fast we can rewrite the PTE. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
With the introduction of gem_exec_gttfill, we have a basic test that fulfils the role that I was using gem_cs_prefetch for (stressing the allotment of the full GTT) without the requirement for carefully checking for a prefetch layout bug. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
This is an attempt to emulate the stressful nature of gem_cs_prefetch without being as slow i.e. without trying to detect the GPU prefetch issues and just emphasive the basic correctness in handling enough batches to fill the GTT similar to gem_ringfill aims to fill a ring. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Tomeu Vizoso authored
Allow users of autogen.sh to skip the call to configure, which is needed when building out of tree. Signed-off-by:
Tomeu Vizoso <tomeu.vizoso@collabora.com>
-
Tomeu Vizoso authored
So that this test can be run in drivers other than i915. Signed-off-by:
Tomeu Vizoso <tomeu.vizoso@collabora.com>
-
Tomeu Vizoso authored
Paint the color key with cairo, so the test doesn't have to map the BO by itself, which depends on the driver being tested. Signed-off-by:
Tomeu Vizoso <tomeu.vizoso@collabora.com>
-
Chris Wilson authored
The *-before-* tests were designed to detect a potential issue with long speculative prefetches that would load the map[] before we have even called execbuf(). As CPUs get smarter and able to look ahead further we are starting to see failures and as we know the code is deliberately buggy exclude the tests from BAT. (I want to keep the tests in the full suite so that we can record which processors are affected, and look for related issues.) Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95378 References: https://bugs.freedesktop.org/show_bug.cgi?id=95488 Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
- May 17, 2016
-
-
Jani Nikula authored
Now that we have actual functions for kms_test_*_str since commit 2d432fc5 Author: Jani Nikula <jani.nikula@intel.com> Date: Wed May 11 12:42:06 2016 +0300 lib/igt_aux: define actual functions for kmstest_*_str move also the gtk-doc comments next to the definitions, for consistency. Acked-by:
Daniel Vetter <daniel@ffwll.ch> Signed-off-by:
Jani Nikula <jani.nikula@intel.com>
-
- May 16, 2016
-
-
Feceoru, Gabriel authored
basic-flip-vs-wf_vblank subtest sometimes fails asserting counted frames to be aproximately equal with the estimated number. This is a false negative, one of the reasons being the precision lost when truncating a fractional number. Fixed this by using floating point arithmetic. Cc: Jani Nikula <jani.nikula@intel.com> Cc: Daniel Vetter <daniel.vetter@intel.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95380 [mvlad] Signed-off-by:
Gabriel Feceoru <gabriel.feceoru@intel.com> Signed-off-by:
Marius Vlad <marius.c.vlad@intel.com>
-
marius vlad authored
Introduced by a633ad03 (tests: Separate tests with lots of subtests). v2: Fix commit typo (Jani Nikula). Signed-off-by:
Marius Vlad <marius.c.vlad@intel.com>
-
- May 13, 2016
-
-
Mika Kuoppala authored
Use large enough datatype so that we get the accurate mem requirements and thus skips on 48bit ppgtt. References: https://bugs.freedesktop.org/show_bug.cgi?id=94146 Acked-by:
Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by:
Mika Kuoppala <mika.kuoppala@intel.com>
-
Jani Nikula authored
Be consistent with exit status and printing errors to stderr. Signed-off-by:
Jani Nikula <jani.nikula@intel.com>
-
Jani Nikula authored
Make the output nicer. Do not print the header if a specific block is requested. Signed-off-by:
Jani Nikula <jani.nikula@intel.com>
-
Jani Nikula authored
Signed-off-by:
Jani Nikula <jani.nikula@intel.com>
-
Chris Wilson authored
Since 9ba9af2e ("lib: Skip execution in gem_has_ring()"), we no longer do a double execbuf for each hang, as the gem_require_ring() check now does a dummy execution that does not contribuee to the pending batch count. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
Chris Wilson authored
When reading reports, it is much easier for the test to tell us how it will be run than try and remember the configuration of a random machine. Signed-off-by:
Chris Wilson <chris@chris-wilson.co.uk>
-
- May 12, 2016
-
-
Signed-off-by:
Marius Vlad <marius.c.vlad@intel.com> Signed-off-by:
Jani Nikula <jani.nikula@intel.com>
-
Jani Nikula authored
Signed-off-by:
Jani Nikula <jani.nikula@intel.com>
-
Jani Nikula authored
Since we no longer store pointers to previous blocks, we can free them. Signed-off-by:
Jani Nikula <jani.nikula@intel.com>
-