summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2014-01-10pm_rps: Use unbuffered I/O on sysfs filesJeff McGee1-5/+2
Bionic C library may not re-read a buffered, read-only file which results in failure to monitor changes in gt_cur_freq_mhz. Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-10tests/gem_pipe_control_store_loop: Delete bogus assertionChris Wilson1-3/+1
target_bo->offset was just being used to verify that the batch submission worked and fortunately was not being relied upon for any subsequent conditions. However, address 0 is valid and so the assertion itself was bogus as it is possible (almost assured with full-ppgtt) for the target_bo to be located at address 0. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72984 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-01-10gem_gtt_hog: Print elapsed time in msChris Wilson1-2/+2
Missed one last diff before pushing
2014-01-10tests: Add gem_gtt_hogChris Wilson3-0/+179
The purpose of this test is to exercise the userspace latency hogs reported by Arjan van de Ven. He found some applications blocked the device by stalling on the GPU inside the pagefault handler.
2014-01-10tests/gem_largeobject: Prefault instead of pinDaniel Vetter1-2/+4
This doesn't test pin but pwriting to big objects ... Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72901 Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-10tests: fixup igt_simple_main falloutDaniel Vetter5-14/+7
Oops, I should look at compilier warnings a bit better. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-08tests/gem_ctx_exec: fix ram requirement fumbleDaniel Vetter1-1/+2
We need to half available ram, not double it ... Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72557 Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-08tests/NAMING-CONVENTION: s/snooped/snoop/Daniel Vetter1-1/+1
Oops. We should be a bit more consistent with which verb form we're picking. tile,tiled,tiling is even worse ... Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-08build: Added -UNDEBUG to tools/Android.mkDevon Davies1-1/+1
NDEBUG is defined by default in Android and causes assert(x) to be defined as void(x). This change ensures that asserts work as expected in tools. This flag is already inside tests/Android.mk Reviewed-by: Oscar Mateo <oscar.mateo@intel.com> Signed-off-by: Devon Davies <devon.davies@intel.com> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-01-07tools: Remove intel_disable_clock_gatingDamien Lespiau3-73/+0
This tool only supports ILK. I take the fact that nobody has felt the need to update for later platform a sign it's not very useful. Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-01-07tools: Removed unused tools/intel_iosf_read.cDamien Lespiau1-70/+0
Also intel_iosf_read() does not exist, and would need a bit more arguments. Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-01-07lib: Use INSTDONE_I965 and INSTDONE_1 for gen6Damien Lespiau2-5/+5
The GEN6_INSTDONE_1 and GEN6_INSTDONE_2 registers are just the old INSTDONE_I965 and INSTDONE_1 registers but renamed. Let's use the old names (this is what the kernel does). Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-01-07lib: Move the INSTDONE bit definitions to instdone.cDamien Lespiau2-237/+242
This is the only place where they are used and we've even started using 1 << n constants with gen 7. Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-01-07instdone: Add an assert to make sure we never overflow instdone_bitsDamien Lespiau1-0/+1
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-01-07intel_reg: Renamed INST_DONE to INSTDONEDamien Lespiau5-12/+12
That's how the registers are named in the kernel defines. Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-01-07tests/kms_flip: free the test_output struct when counting modesPaulo Zanoni1-4/+13
Looks like we have to do a lot of work just to count the number of modes... Caught by Valgrind. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2014-01-07tests/kms_flip: don't leak the connector_config structPaulo Zanoni1-1/+3
... in case we can't find a compatible mode. We already have config[0], we can't return without freeing it first. Caught by Valgrind. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2014-01-07tests/kms_flip: don't leak the CRTCPaulo Zanoni1-0/+5
The kms_flip program calls kmstest_get_connector_config, which returns a struct containing some allocated variables, including a pointer to the CRTC. The problem is that we copy the values returned by this structure to the test_output struct, but we ignore the CRTC pointer. So free the CRTC pointer instead of leaking it. Caught by Valgrind. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2014-01-07tests/kms_flip: don't leak the connector when setting DPMSPaulo Zanoni1-0/+2
Caught by Valgrind. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2014-01-07drmtest: don't leak memory when parsing the argumentsPaulo Zanoni1-0/+2
Found this while investigating memory leaks on kms_flip. Detected by Valgrind. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2014-01-07NEWS updatesDaniel Vetter1-0/+5
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-06intel_error_decode: Factor out common decoding codeDamien Lespiau1-31/+21
4 pieces of code were looking very similar. Let's factor out a common function in the not so unlikely case we need to tweak that code. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-01-02configure.ac: Fix shader debugger for realBen Widawsky1-2/+1
I tried to get fancy before, but ended up pushing something which I didn't test. Keep the thing simple and stupid, and just make it work. Reported-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-12-31kmstest: Fix up lifetimes of cairo objectsChris Wilson11-34/+45
cairo_t is the short lived drawing context, whereas cairo_surface_t is the heavyweight object that persists and is also tied to underlying GEM objects. So make the kmstest API reflect the different weights and fix the lifetime and underlying object reference leaks. Based on the fix by Paulo Zanoni. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2013-12-30configure.ac: Fixup conditional shader debuggerBen Widawsky1-3/+6
The original patch was incomplete. Testing no assembler build is a pain for me. Sorry for the mess. Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-12-30configure.ac: Remove GEN4ASM defineBen Widawsky1-1/+0
This was a leftover hunk that got accidently merged. Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-12-24gem_exec_blit: gen8 updates for blitBen Widawsky1-1/+12
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-12-20shader-debugger: Use in tree gen4asmBen Widawsky2-11/+11
Reported-by: Patrick McMunn <doctorwhoguy@gmail.com> Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-12-20README: Fix spelling failDaniel Vetter1-2/+2
Thanks to Jani for spotting this. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-20README: Document how to avoid Waffle with piglitDaniel Vetter1-0/+10
Suggested by Paulo. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-16tests/pm_pc8: check if we can't become DRM masterPaulo Zanoni1-0/+3
QA reported a failure that I believe happened because we couldn't become DRM master, so add code that checks for this and prints a nice error message. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2013-12-13build: Create version and config information files in AndroidOscar Mateo1-2/+33
Also, skip the compilation of pm_lpsp in Android (uses Cairo) Signed-off-by: Oscar Mateo <oscar.mateo@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-13tests/gem_close_race: Adapt the test for Full PPGTTOscar Mateo1-2/+2
With Full PPGTT, each new fd creates a new context and thus a new PPGTT, so we have to reduce the number of simultaneous fds or face OOM problems. For every new PPGTT, its PDEs are stored in the GGTT which imposes a limit of 1024 new contexts. We want to leave at least 1/4 of the GGTT available for "important" stuff like scanout buffers, so never open more than 768 fds. v2: Unconditionally use the lower limit, as suggested by Daniel Vetter Signed-off-by: Oscar Mateo <oscar.mateo@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-13lib: Include the necessary stdint.h in gen6_render.hDamien Lespiau1-0/+2
Instead of assuming someone else will do it. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-12-12tests: Document the Makefile variables a bit betterDaniel Vetter1-0/+7
Also, this is a test for the patchwork hook. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-12tests/gem_reset_stats: add reverse order in close-pending-forkMika Kuoppala1-14/+70
Use own copy of gem_quiescent_gpu() so that test still works if it gets changed. Further improve the test by posting a batch to rings in reverse order. Suggested-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-12tests: Move gem_reset_stats to the right placeDaniel Vetter1-1/+3
Binaries should only be added to the HANG target if they're completely broken. If they just cause a recoverable gpu hang as part of the test then they need to be somewhere else. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-11tests/kms_setmode: avoid 2 connectors on the same encoderPaulo Zanoni1-0/+15
Don't try to set modes on two connectors that share the same encoder. That will just fail. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68463 Reviewed-by: Imre Deak <imre.deak@intel.com> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2013-12-10kms_flip: consolidate run_test_on_{crtc,crtc_pair}Imre Deak1-100/+40
No functional change. Signed-off-by: Imre Deak <imre.deak@intel.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-10kms_flip: keep drm_fd open for the dpms restore exit handlerImre Deak1-2/+4
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=69161 Signed-off-by: Imre Deak <imre.deak@intel.com>
2013-12-10kms_flip: fix seq_step for 2xcrtc subtestsImre Deak1-4/+10
This also raises the absolute wait-for-vblank sequence step from 5 to 10, just to make it consistent with the corresponding 1x subtests. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=69161 Signed-off-by: Imre Deak <imre.deak@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-10rendercopy/bdw: Fix length of a bunch of zeroed out packets.Kenneth Graunke1-6/+20
A bunch of packets were too short. Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> Reviewed-by: Damien Lespiau <damien.lespiau@intel.com> Cc: Ben Widawsky <ben@bwidawsk.net> Cc: Damien Lespiau <damien.lespiau@intel.com>
2013-12-10rendercopy/bdw: Emit 3DSTATE_WM_HZ_OP.Kenneth Graunke2-0/+12
We don't want depth/stencil fast clears or HiZ resolves; we want normal drawing. Without this, the pixel pipeline doesn't work. Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> Cc: Ben Widawsky <ben@bwidawsk.net> Cc: Damien Lespiau <damien.lespiau@intel.com>
2013-12-10ignore version.hDaniel Vetter1-0/+2
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-10tests: roll out igt_simple_init/igt_simple_mainDaniel Vetter56-169/+89
Also use igt_skip a bit more to simplify some of the tests. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-10tests: gem_close_race has subtests!Daniel Vetter1-3/+3
Oops. v2: Found more. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-10lib: igt_simple_main/init for subtest-less testsDaniel Vetter3-3/+17
Atm only used to print the version information. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-10lib: print uname information in the version stringDaniel Vetter1-2/+8
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-10lib: Add host platform to version stringDaniel Vetter2-1/+4
This is import on systems where we compile 32bit tests that run on 64bit platforms. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-12-10intel-gpu-tools: Version informationBen Widawsky3-1/+38
Provide two arguments version, and verbose, which allow printing from an arbitrary igt test. It will show system information (from build time, not runtime), as well as the git SHA being used. This will help reduce errors when people try to reproduce problems. As an example if I want to verify someone is running the correct version of a test, I could ask them to do: bwidawsk@ironside ~/intel-gfx/intel-gpu-tools (master)$ sudo ./tests/gem_exec_nop --verbose gem_exec_nop-git-3c5423b (Linux ironside 3.12.0-1-ARCH #1 SMP PREEMPT Wed Nov 6 09:06:27 CET 2013 x86_64 GNU/Linux) Time to exec x 1: 35.000µs (ring=render) Time to exec x 2: 28.000µs (ring=render) Time to exec x 4: 20.000µs (ring=render) Time to exec x 8: 14.625µs (ring=render) Time to exec x 16: 11.188µs (ring=render) Time to exec x 32: 11.125µs (ring=render) Time to exec x 64: 10.328µs (ring=render) Time to exec x 128: 10.172µs (ring=render) Time to exec x 256: 10.234µs (ring=render) Time to exec x 512: 10.232µs (ring=render) Time to exec x 1024: 10.121µs (ring=render) Time to exec x 2048: 10.151µs (ring=render) Time to exec x 4096: 11.474µs (ring=render) Time to exec x 8192: 9.432µs (ring=render) Time to exec x 16384: 6.003µs (ring=render) Time to exec x 32768: 5.029µs (ring=render) Time to exec x 65536: 4.206µs (ring=render) Time to exec x 131072: 3.630µs (ring=render) Subtest render: SUCCESS --verbose is provided for completeness, but doesn't seem too useful at the moement. bwidawsk@ironside ~/intel-gfx/intel-gpu-tools (master)$ sudo ./tests/gem_exec_nop --version gem_exec_nop-git-3c5423b (Linux ironside 3.12.0-1-ARCH #1 SMP PREEMPT Wed Nov 6 09:06:27 CET 2013 x86_64 GNU/Linux) I've put version.h in the root directory so that any subdir can access it. I've added the tests usage since it's immediately useful, and done easily via Daniels igt infrastructure work. v2: - Always print the version number. - We want to print uname at runtime. - Also prefix the i-g-t release version. Signed-off-by: Ben Widawsky <ben@bwidawsk.net> (v1) Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
generated by cgit v1.2.3 (git 2.39.1) at 2024-11-22 10:39:30 +0000