Commit Graph

1874 Commits

Author SHA1 Message Date
Daniel Vetter 842007154d tests: use "auto" pipe CRC source
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-01 20:43:15 +01:00
Daniel Vetter 0b51de8dd0 lib: add more self-tests around the igt_exit checks.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-11-01 18:30:22 +01:00
Daniel Vetter e14596a6d0 lib: fix igt_exit assert when only listing subtests.
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-01 18:30:22 +01:00
Damien Lespiau 9e21c255e7 lib: Don't forget to close the pipe ctl fd in igt_pipe_crc_reset()
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-11-01 16:25:52 +00:00
Damien Lespiau b1945f8c46 gitignore: Ignore recently introduced tools and tests
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-11-01 15:47:42 +00:00
Paulo Zanoni bd0aa100ca module_reload: remove snd_hda_intel
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-01 12:55:46 -02:00
Ville Syrjälä 3c6a7e780b Add VGA register read/write tools
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2013-11-01 15:28:25 +02:00
Ben Widawsky 164b540936 debugfs_pipe_crc: skip on simulation
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-31 14:16:41 -07:00
Paulo Zanoni 2addb2744f tests/pc8: remove some printfs
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-30 19:37:21 -02:00
Paulo Zanoni b6645ee2e2 tests/pc8: clarify modeset_subtest arguments
My tiny little brain can't properly process 3 unnamed boolean
arguments.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2013-10-30 19:37:21 -02:00
Paulo Zanoni abb3340617 tests/pc8: add modeset -stress-no-wait tests
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-30 19:37:21 -02:00
Paulo Zanoni 7a5275f8a5 tests/pc8: add modeset-lpsp and modeset-non-lpsp
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-30 19:37:21 -02:00
Ben Widawsky 87cf5571fc gem_suspend: Skip on simulation
Simulation has proven flaky across both reset, and s3/4

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2013-10-29 20:01:43 -07:00
Ben Widawsky 5382aca1af gem_close_race: Fix linker flags
Seems the -lpthread was missed in
commit 8f771f3fac
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-29 20:01:36 -07:00
Chris Wilson 8f771f3fac gem_close_race: Also test random closing of active fd
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2013-10-29 14:22:18 +00:00
Daniel Vetter 5d24a67c1b lib/drmtest: Scream harder when igt_exit isn't called for subtest tests
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-29 11:34:32 +01:00
Daniel Vetter 780807099a tests: Use igt_exit for tests with subtest
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-10-29 11:15:43 +01:00
Chris Wilson ccb1b5888d gem_close_race: Bump the workload
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-29 09:27:28 +00:00
Chris Wilson 51783fc7f9 gem_close_race: Tidy up call to execbuffer
Pass the right pointer to the execlist would be a good start.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2013-10-29 08:39:53 +00:00
Daniel Vetter 7b8784dccf tests/kms_cursor_crc: Use igt_exit
Otherwise the exit codes are all bogus.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-10-29 07:47:33 +01:00
Damien Lespiau 020600b77d tests: Convert the ctx test to use render nodes when possible
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-28 11:00:07 +00:00
Damien Lespiau d71ecdbf5d tests/gem_render_copy: Use render nodes
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-28 11:00:07 +00:00
Damien Lespiau 69c200b0bb lib: Add a drm_open_any_render() that will try to use render nodes
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-28 11:00:07 +00:00
Damien Lespiau 8329acb752 lib: Close non intel fds in drm_get_card()
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-28 11:00:07 +00:00
Damien Lespiau 949ce5b8a1 lib: Remove stale comment
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-28 11:00:07 +00:00
Chris Wilson 90ecf062d1 tests: Add gem_close_race
The intention is to exercise #70784. Yet the first challenge is make the
test suite happy.
2013-10-28 09:13:09 +00:00
Damien Lespiau 37c02cfa5c gem_render_linear_blits: Remove aub dump support
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-24 13:44:32 +01:00
Damien Lespiau 7ad0cf585b tests/gem_render_copy: Add aub dump support
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-24 13:44:32 +01:00
Damien Lespiau 145722187c tests/gem_render_copy: Only dump pngs when the -d option is given
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-24 13:44:32 +01:00
Damien Lespiau 9cd092fac1 tests/gem_render_copy: Add a simple render copy test
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-24 13:44:32 +01:00
Damien Lespiau c3707ab387 lib: Add a function to dump a scratch buf into a png
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-24 13:44:32 +01:00
Rodrigo Vivi 13ebe3c014 build: include intel_l3_parity.h to its _SOURCES to fix make distcheck
and make sure intel_l3_parity.h will be included in tarball.

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
2013-10-22 12:35:59 -02:00
Rodrigo Vivi 50c2653381 build: Fix small typo that was breaking make distcheck
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
2013-10-22 11:28:56 -02:00
Ville Syrjälä fa05c0bced kms_flip: Add test for -ENOENT
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-21 20:23:18 +03:00
Ville Syrjälä 08c27e3e38 kms_cursor_crc: Add a cursor test using CRCs
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-21 20:00:32 +03:00
Ville Syrjälä 0786d5cc8f lib: Check pipe source validity in igt_pipe_crc_new()
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-21 20:00:32 +03:00
Ville Syrjälä 987bcd9a5b lib: Allow pipe_crc_free(NULL)
Prevent pipe_crc_free() from segfaulting on NULL ptr.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2013-10-21 20:00:32 +03:00
Ville Syrjälä 471ec64de8 lib/drmtest: Add kmstest_paint_color_alpha()
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>
2013-10-21 20:00:32 +03:00
Paulo Zanoni e98a06d593 intel_reg_dumper: fix "enable" at hsw_debug_lp_wm
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2013-10-21 13:28:11 -02:00
Damien Lespiau b7ccea08e6 gitignore: Ignore intel_opregion_decode
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2013-10-21 15:55:52 +01:00
Jani Nikula 6d3c917e89 intel_opregion_decode: decipher bclm table
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2013-10-21 09:43:20 -02:00
Jani Nikula 76be74975a intel_opregion_decode: new tool for decoding graphics opregion
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2013-10-21 09:43:05 -02:00
Jani Nikula a360e3993b intel_bios_reader: dump all sections, including unknown ones
We still need to dump some of the known sections explicitly due to
dependencies on information extracted, such as LFP data pointers and
panel_type.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2013-10-21 09:42:58 -02:00
Jani Nikula 419d053fd6 intel_bios_reader: dump all edp device info structs
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2013-10-21 09:42:50 -02:00
Jani Nikula a4180df32a intel_bios_reader: prepare for dumping all sections
Dump sections through a table based on the section id.

Hex dump the section. This works also for unknown sections.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2013-10-21 09:42:20 -02:00
Jani Nikula 8596a4b886 intel_bios_reader: add size temp variable as a shorthand for finfo.st_size
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2013-10-21 09:40:50 -02:00
Chris Wilson 58ac17d821 intel_error_decode: Fix X/Y fence for gen2/3 2013-10-21 09:32:34 +01:00
Daniel Vetter 4acaf0004d tests/kms_flip: More leeway for dummy load tests
At least on my old atom i945 they take forever. Also, metric tons of
pipe B underruns while running rcs-flip-vs-panning. No idea why that
happens.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-10-20 20:37:21 +02:00
Daniel Vetter 3d2d73ed45 kmstest: don't pollute stderr for impossible output configs
Otherwise on machines where the only output available is restricted to
one pipe we'll have tons of "warn" results for no gain in the piglit
runner. All tests that use the kmstest helpers already check the
return value and do something sensible (like skip the tests if there's
really no config available).

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-10-20 19:24:08 +02:00
Daniel Vetter 45cd89ab3d tests/kms_flip: Unconfuse the framenumber step checking
We run flips and vblank waits in parallel, which means the step
for each is the maximum. If we'd switch to an interleaved scheme
(to be able to check the correlation) we'd need to add them instead.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2013-10-20 16:11:57 +02:00