summaryrefslogtreecommitdiff
path: root/tests/gem_storedw_batches_loop.c
AgeCommit message (Collapse)Author
2017-04-08tests: More require GEM markupChris Wilson
A few more tests that use i915/gem without first checking that the GPU is working. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-25lib/ioctl_wrappers: Add gem_gtt_type exposing raw HAS_ALIASING_PPGTT paramMichał Winiarski
No functional changes. While I'm here, let's also rename gem_uses_aliasing_ppgtt (since it's being used to indicate if we are using ANY kind of ppgtt) and introduce gem_uses_full_ppgtt to drop some unnecessary code from tests that were previously calling getparam directly instead of using ioctl wrapper. v2: drop gem_uses_full_48b_ppgtt since it's no longer used anywhere, s/48b/64b (Chris) v3: rebase Cc: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Michał Winiarski <michal.winiarski@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2015-09-11convert drm_open_any*() calls to drm_open_driver*(DRIVER_INTEL) calls with cocciMicah Fedke
Apply the new API to all call sites within the test suite using the following semantic patch: // Semantic patch for replacing drm_open_any* with arch-specific drm_open_driver* calls @@ identifier i =~ "\bdrm_open_any\b"; @@ - i() + drm_open_driver(DRIVER_INTEL) @@ identifier i =~ "\bdrm_open_any_master\b"; @@ - i() + drm_open_driver_master(DRIVER_INTEL) @@ identifier i =~ "\bdrm_open_any_render\b"; @@ - i() + drm_open_driver_render(DRIVER_INTEL) @@ identifier i =~ "\b__drm_open_any\b"; @@ - i() + __drm_open_driver(DRIVER_INTEL) Signed-off-by: Micah Fedke <micah.fedke@collabora.co.uk> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-08-21lib: add a single include headerThomas Wood
Add a header that includes all the headers for the library. This allows reorganisation of the library without affecting programs using it and also simplifies the headers that need to be included to use the library. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-06-08tests/gem_storedw_batches_loop: add subtest for cached mappingsImre Deak
v2: - add a subtest for uncached mappings too for LLC platforms where the default is cached mapping (Chris) Signed-off-by: Imre Deak <imre.deak@intel.com>
2015-03-06igt.cocci: Replace igt_assert() with igt_assert_CMP() where possibleMatt Roper
The integer comparison macros give us better error output by including the actual values that failed the comparison. Signed-off-by: Matt Roper <matthew.d.roper@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2014-05-22Revert "Revert "store_dw_loop: make loops smaller""Daniel Vetter
This reverts commit 3005ac3ee8d7aede73a3e63d2068a6074156ebe5. QA has done the testing we've wanted. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-05-19Revert "store_dw_loop: make loops smaller"Daniel Vetter
This reverts commit f00efff326610fdba92dbc91d951790a3320052e. This is a temporary revert since I want QA to first test with the original testcase whether it got faster again. This is to test the effects of commit 227f782e4667fc622810bce8be8ccdeee45f89c2 Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Thu May 15 10:41:42 2014 +0100 drm/i915: Retire requests before creating a new one We should revert this revert again as soon as QA has completed testing. References: https://bugs.freedesktop.org/show_bug.cgi?id=78024 Cc: Jesse Barnes <jbarnes@virtuousgeek.org> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-05-19store_dw_loop: make loops smallerJesse Barnes
These tests are really for catching TLB or GTT mapping failures due to bad programming in the kernel driver. We've never needed more than a few pages worth of data write to actually see those.
2014-05-14tests: sprinkle igt loggingDaniel Vetter
All the cases that simply dump some debug information and couldn't be converted to some of the fancier macros. Some information output removed when it's redundant with the subtest status. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-03-22lib: rename intel_gpu_tools.h to intel_io.hDaniel Vetter
With the header cleanup we can now give this header a suitable name, since it now really only contains register access and other I/O functions and assorted definitions. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-03-22lib: unnecessary header removal for drmtest.h, part 1Daniel Vetter
Brought a few missing headers to light in ioctl_wrappers.h, too. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-03-22tests|lib: remove assert.h includesDaniel Vetter
Only the igt core and non-test tools should have asserts to catch internal errors, tests and helper libraries should all user igt_asert instead. Fix things up where assert instead of igt_assert was used. One tiny step towards header sanity. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-21gem_storedw_batches_loop: Fix for pre-BDWBen Widawsky
My git failures are truly remarkable. I ended up pushing the wrong commit here: commit 1552aa21124cabe762862bb414490510415a2b2d Author: Ben Widawsky <benjamin.widawsky@intel.com> Date: Mon Jan 13 06:28:45 2014 -0800 gem_storedw_batches_loop: Fix for BDW This puts the offset of the reloc in the wrong place for pre-BDW Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73866 Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2014-01-13gem_storedw_batches_loop: Fix for BDWBen Widawsky
Existing code was trying to be too clever and wasn't properly emitting the high dword, or the correct length. Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-11-11gem_storedw_batches_loop: Fix for real this timeBen Widawsky
Last patch to fix the breakage when adding gen8 had bugs in the reloc offset, and size field. Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-11-07gem_storedw_batches_loop: Update for gen8Ben Widawsky
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-11-01lib: add igt_main macroDaniel Vetter
In the past new testcases with subtest often forgot to add the call to igt_exit at the end of their main() function. That is now caught with a bit more obnoxious asserts, but it's still a nuissance. This little igt_main macro takes care of that (and also of calling the subtest machinery initialization code correctly). If no one objects I'll roll this out for all the simple cases (i.e. those tests that don't have additional argv parsing on top of the subtest machinery). v2: Roll it out across the board. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-10-29tests: Use igt_exit for tests with subtestDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-09-19lib/drmtest: Improve printf-like igt_skip_on/requireDaniel Vetter
Ben Widawsky suggested to use vasprintf, which perfectly fits the bill. Also fix the logic conversion bug in tests/gem_storedw_batches_loop that crept in again :( Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-09-19lib/drmtest: igt_assert|require with format stringsDaniel Vetter
v2: Add a comment about the pitfalls around va_list handling. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-09-19tests/gem_stored_batches_loop: use igt_assert moreDaniel Vetter
Terser code ftw! Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-09-18tests/gem_storedw_batches_loop: Add testcase to check secure dispatchDaniel Vetter
v2: Use the mrb_exec function since otherwise we can't pass flags. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-08-13tests: use igt_fail instead of exit(param != 0)Daniel Vetter
Mostly a sed job with too manual fixups: - one case of using _exit instead of exit - and one case which under some conditions use 77, so convert that check to an igt_skip. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-08-13tests: s/assert/igt_assertDaniel Vetter
Just a wholesale rollout for now, we can refine later on. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-08-12s/drmtest_/igt_/Daniel Vetter
Requested-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-07-18tests: Instrument tests run in simulation to run quicklyDamien Lespiau
We tweak the tests marked as runnable in simulation to run more quickly, more often then not at the expense of stress testing (which is of an arguable interest for the initial bring up in simulation). Hopefully the values chosen still test something, which is not always straightforward. It does run quickly, the number on an IVB machines are: $ time sudo IGT_SIMULATION=0 ./piglit-run.py tests/igt.tests foo [...] real 2m0.141s user 0m16.365s sys 1m33.382s Vs. $ time sudo IGT_SIMULATION=1 ./piglit-run.py tests/igt.tests foo [...] real 0m0.448s user 0m0.226s sys 0m0.183s Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-07-18tests: Black list tests we don't want to run on simulationDamien Lespiau
Let's start by a small set of tests, to eventually consider running more. The current list should then be: gem_mmap gem_pread_after_blit gem_ring_sync_loop gem_ctx_basic gem_pipe_control_store_loop gem_storedw_loop_render gem_storedw_loop_blt gem_storedw_loop_bsd gem_render_linear_blits gem_tiled_blits gem_cpu_reloc gem_exec_nop gem_mmap_gtt v2 add (Daniel Vetter) gem_exec_bad_domains gem_exec_faulting_reloc gem_flink gem_reg_read gem_reloc_overflow gem_tiling_max_stride prime_* Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-02-13tests: Add a quick variant to the gem_storedw_* testsDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2012-10-10fix warn: 'div' shadows a global declarationImre Deak
Signed-off-by: Imre Deak <imre.deak@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2012-10-04tests/*storedw*: add subcases with more outstanding batchesDaniel Vetter
Just in case this is better at hitting seqno signalling races ... Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2012-03-22tests: adapt storedw tests to ppgttDaniel Vetter
MI_MEM_VIRTUAL actually means use global gtt now, not setting the bit means use ppgtt. On previous gens, not setting the bits ment 'use physical memory'. So what, the usual confusion. Note that for some odd reasong this is broken on gen6, but only on the bsd ring. Unexpected. Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-11-27tests: properly terminate tests when values mismatchDaniel Vetter
Not much use running them in the test rig otherwise. Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-13tests: exit(77) to properly mark tests as SKIPDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-09tests/gem_storedw: disable on pre-gen6Daniel Vetter
Doesn't work. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-09test/gem_storedw_*: tune some moreDaniel Vetter
MI_*/PIPE_CONTROL writes need to be in DOMAIN_INSTRUCTION, because that is what mesa uses and I plan to use this to work around a gen6 ppgtt issue. Also testing with intentionally b0rked GFX_MODE on my snb shows that we need to increase the loop counter a bit to reliably hit the tlb invalidation problem. Test still completes within a few seconds. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-08-12tests: add store dword testsJesse Barnes
Add a couple of simple store dword tests to test memory coherence. gem_storedw_loop simply executes a batch that continually stores an incremented value to a target buffer object, checking the results after each batch completes. gem_storedw_batches_loop does the same thing, but creates a new command batch buffer for each iteration, which can exercise the buffer creation code. This test is based on one from Andrzej Kacprowski from Intel. Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>