summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2013-11-04lib: drop return value from igt_drop_cachesDaniel Vetter4-14/+8
No one actually cares, everyone expects it to just work. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-04lib: Add igt_drop_caches_set()Oscar Mateo4-22/+51
This is basically a "drop cache" interface to the igt_debugfs facilities. Also, update existing users. Signed-off-by: Oscar Mateo <oscar.mateo@intel.com> Cc: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-04tests: Fix "-thrashing" and "-thrash-inactive" distinctionOscar Mateo2-2/+2
A typo in the relocation tests made both sub-tests perform the same action: drop *all* caches. Signed-off-by: Oscar Mateo <oscar.mateo@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01tests: igt_no_subtestDaniel Vetter3-0/+41
Yet another check for the library. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01tests: Add a Makefile comment about the expected failures in igt_ testsDaniel Vetter1-0/+2
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01tests: Add a test templateDaniel Vetter3-0/+82
Requested by Paulo. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01lib: add igt_main macroDaniel Vetter40-182/+49
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-11-01lib: make igt_install_exit_handler never failDaniel Vetter4-11/+10
Most callers didn't bother checking, so just move the asserts into the function itself. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01lib: make igt_set_vt_graphics_mode never failDaniel Vetter9-12/+15
All tests agree that this is a letal failure, so no point to pass it back to callers. Also add some igt_require calls for the drm_fd where I've noticed that it's missing. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01lib: consolidate pipe crc exit handlerDaniel Vetter4-41/+30
No need to sprinkle this all over: - exit handlers will only be registered once - they're always called when exiting, so no need to explictly call them. This allows us to hide all the pipe crc cleanup in the library. Cc: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01tests/kms_cursor_crc: rework to auto-skip connectorsDaniel Vetter1-12/+17
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01tests/debugfs_pipe_crc: set mode before creating CRC helperDaniel Vetter1-3/+5
Otherwise the automagic skipping for DP ports on gmch platforms won't work. v2: We also need to just skip that connector, not the entire subtest. kms_cursor_crc still needs to be updated. Cc: Damien Lespiau <damien.lespiau@intel.com> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01tests: use "auto" pipe CRC sourceDaniel Vetter4-19/+8
Makes stuff work on DP ports on gmch platforms automatically. Cc: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01lib: add more self-tests around the igt_exit checks.Daniel Vetter4-0/+88
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01lib: fix igt_exit assert when only listing subtests.Daniel Vetter4-1/+46
Ooops. Reported by Paulo. Also add a new testcase for make check to make sure this actually works. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01lib: Don't forget to close the pipe ctl fd in igt_pipe_crc_reset()Damien Lespiau1-0/+2
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-11-01gitignore: Ignore recently introduced tools and testsDamien Lespiau2-0/+3
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-11-01module_reload: remove snd_hda_intelPaulo Zanoni1-0/+6
The audio driver uses the power well provided by our driver, so on Haswell we can't "rmmod i915" if we don't "rmmod snd_hda_intel" first. The problem with removing snd_hda_intel is that we also need to kill its users. On the specific machine I tested, the only user seem to be alsactl, but on other machines this may change. IMHO we should leave the "kill user space" step to whoever is running the script, but Daniel asked me to put it here so we have a better chance of Just Working on QA's machines. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70336 Requested-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2013-11-01Add VGA register read/write toolsVille Syrjälä3-1/+189
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2013-10-31debugfs_pipe_crc: skip on simulationBen Widawsky1-0/+2
As with other display tests, they're not working (and in this case, cause hangs). Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-10-30tests/pc8: remove some printfsPaulo Zanoni1-8/+0
They pollute the output, which makes it harder to see which tests failed or succeeded. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2013-10-30tests/pc8: clarify modeset_subtest argumentsPaulo Zanoni1-13/+15
My tiny little brain can't properly process 3 unnamed boolean arguments. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2013-10-30tests/pc8: add modeset -stress-no-wait testsPaulo Zanoni1-6/+12
Used to trigger a racing condition between mode setting and enabling/disabling PC8. The modeset-lpsp-stress-no-wait test should fail on Kernels without the fix I just wrote today. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2013-10-30tests/pc8: add modeset-lpsp and modeset-non-lpspPaulo Zanoni1-1/+51
The goal of these tests is to exercise the combination of power well enabling/disabling and PC8 enabling/disabling. The modeset-non-lpsp test exposes a silent bug on the current code that can just be detected with the Runtime D3 patches. The Kernel patch that fixes the bug will add a WARN to unmute it on the PC8-only case. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2013-10-29gem_suspend: Skip on simulationBen Widawsky1-0/+2
Simulation has proven flaky across both reset, and s3/4 Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-10-29gem_close_race: Fix linker flagsBen Widawsky1-0/+1
Seems the -lpthread was missed in commit 8f771f3facae133bb72216fa05c74bc817920b6d Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Tue Oct 29 14:21:29 2013 +0000 gem_close_race: Also test random closing of active fd Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-10-29gem_close_race: Also test random closing of active fdChris Wilson1-10/+137
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2013-10-29lib/drmtest: Scream harder when igt_exit isn't called for subtest testsDaniel Vetter1-2/+2
We really need this since otherwise the magic return value handling for running testcases with piglit (or on QA's validation infrastructure) doesn't work properly. We need to be careful though to only install this check on success. See also the previous commits to sprinkle igt_exit() calls over all the tests that missed it. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-10-29tests: Use igt_exit for tests with subtestDaniel Vetter3-3/+3
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-10-29gem_close_race: Bump the workloadChris Wilson1-5/+7
Emit more work per client and many more clients in order to increase the chance of racing i915_gem_release() and i915_gem_retire_requests()
2013-10-29gem_close_race: Tidy up call to execbufferChris Wilson1-22/+13
Pass the right pointer to the execlist would be a good start. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2013-10-29tests/kms_cursor_crc: Use igt_exitDaniel Vetter1-1/+1
Otherwise the exit codes are all bogus. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-10-28tests: Convert the ctx test to use render nodes when possibleDamien Lespiau5-6/+6
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-28tests/gem_render_copy: Use render nodesDamien Lespiau1-1/+1
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-28lib: Add a drm_open_any_render() that will try to use render nodesDamien Lespiau2-0/+59
I was fedup with having to run my tests as root and not being able to use my usual setup for tests that only exercise the GT part of the GPU. Render nodes to the rescue! Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-28lib: Close non intel fds in drm_get_card()Damien Lespiau1-1/+3
When going through card%u devices, close the ones that we were able to open but weren't intel devices. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-28lib: Remove stale commentDamien Lespiau1-2/+0
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-28tests: Add gem_close_raceChris Wilson3-0/+150
The intention is to exercise #70784. Yet the first challenge is make the test suite happy.
2013-10-24gem_render_linear_blits: Remove aub dump supportDamien Lespiau1-16/+0
It's much easier to follow the new gem_render_copy test and acquire a aub dump from it. Suggested-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-24tests/gem_render_copy: Add aub dump supportDamien Lespiau1-3/+18
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-24tests/gem_render_copy: Only dump pngs when the -d option is givenDamien Lespiau1-3/+18
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-24tests/gem_render_copy: Add a simple render copy testDamien Lespiau3-0/+129
The goal is here to both: demonstrate a simple usage of render copy with the possibility to write pngs to visualize what it's doing and to provide a test bed to port the render copy function to new architectures. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-24lib: Add a function to dump a scratch buf into a pngDamien Lespiau3-0/+52
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-22build: include intel_l3_parity.h to its _SOURCES to fix make distcheckRodrigo Vivi1-0/+1
and make sure intel_l3_parity.h will be included in tarball. Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
2013-10-22build: Fix small typo that was breaking make distcheckRodrigo Vivi1-1/+1
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
2013-10-21kms_flip: Add test for -ENOENTVille Syrjälä1-10/+20
Make sure we get -ENOENT when we pass a non-existing fb_id to the page flip and setcrtc ioctls. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2013-10-21kms_cursor_crc: Add a cursor test using CRCsVille Syrjälä2-0/+391
Use the display CRC support to validate cursor plane functionality. The test will position the cursor plane either fully onscreen, partially onscreen, or fully offscreen, using either a fully opaque or fully transparent surface. In each case it then reads the PF CRC and compares it with the CRC value obtained when the cursor plane was disabled. v2: Loop over all connectors and crtcs Use igt_wait_for_vblank Test more corner cases Add some progress indication ala. kms_flip Make it work for pf and pipe sources Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2013-10-21lib: Check pipe source validity in igt_pipe_crc_new()Ville Syrjälä2-24/+41
Have igt_pipe_crc_new() check whether the selected source is actually supported. v2: Make debugfs_pipe_crc.c not crash Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2013-10-21lib: Allow pipe_crc_free(NULL)Ville Syrjälä1-0/+3
Prevent pipe_crc_free() from segfaulting on NULL ptr. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2013-10-21lib/drmtest: Add kmstest_paint_color_alpha()Ville Syrjälä2-0/+10
kmstest_paint_color_alpha() just like kmstest_paint_color() except you also get to specify the alpha channel. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>