summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2014-11-06Build the docs directory lastThomas Wood
Ensure that other components are built before generating the documentation. v2: build the docs directory after any optional directories Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-11-06docs: add some test descriptionsThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-11-06lib: add the ability to include a description with a testThomas Wood
The IGT_TEST_DESCRIPTION macro can be used to define a description for a test. v2: Remove semicolon from end of macro (Damien Lespiau) Add API documentation for the macro. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-11-06configure: fix issues when running outside the source treeThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-11-06testdiplay: add a "--help" optionThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-11-06testdisplay: ensure invalid long option exit status is non-zeroThomas Wood
The optopt variable is not set if an invalid long option is used, so check the current option character instead. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-11-06lib/tests: don't use hard error status to indicate test failureThomas Wood
An exit status of 99 is reported separately in the test summary as an error, rather than as a test failure. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-11-06lib/tests: fix tests and test list locationThomas Wood
The igt_command_line.sh script was moved by commit 685e577 (Move library selftests to lib/tests), but the location of the tests and the test lists was not updated. Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-11-06tools/quick_dump: don't include generated sources in the distributionThomas Wood
This also ensures that "make dist" works even if SWIG is not available. BUILT_SOURCES is not required because the files are already explicit dependencies of other rules. v2: make sure SWIG is not invoked with the incorrect output filename Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-11-06Mediafill/skl: Remove redundant field to fix GPU hangZhao Yakui
After applying the commit(982f7eb238a0898c456e0574dee7c4507738d75f), the OUT_RELOC is updated on Broadwell and later, which is to handle the 64-bit field of gfx address internally. In such case some commands should be fixed, otherwise GPU hang will be triggered when running gem_media_fill. (It is already fixed on Broadwell) Reviewed-by: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Zhao Yakui <yakui.zhao@intel.com> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-11-06Rendercopy/skl: Remove redundant field to fix GPU hangZhao Yakui
After applying the commit(982f7eb238a0898c456e0574dee7c4507738d75f), the OUT_RELOC is updated on Broadwell and later, which is to handle the 64-bit field of gfx address internally. In such case some commands should be fixed, otherwise GPU hang will be triggered when running rendercopy. (It is already fixed on Broadwell) Reviewed-by: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Zhao Yakui <yakui.zhao@intel.com> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-11-05tests/drv_hangman: skip a few asserts when using the cmd parserBrad Volkin
This test has a few checks that batch buffer addresses in the error state match the expected address for the userspace supplied batch. But the batch buffer copy piece of the command parser means that the logged addresses are actually _supposed_ to be different. So skip just those checks. Signed-off-by: Brad Volkin <bradley.d.volkin@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-11-04tests: Remove spurious binaries from gem_tiled_wb commitChris Wilson
commit c45216b26d223ffd7f68c5794870214445a64b5c Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Tue Nov 4 07:30:57 2014 +0000 igt/gem_tiled_wb: Exercise CPU mmaps with swizzling brought along a few interlopers.
2014-11-04tests/prime_nv_test: Skip cpu mmap testcaseDaniel Vetter
It's not implemented yet, so no point in running the test really. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68638 Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2014-11-04igt/gem_tiled_wb: Remove extraneous mmap(wc) requirementChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-11-04igt/gem_tiled_wb: Exercise CPU mmaps with swizzlingChris Wilson
This exercises the the extended get_tiling ioctl in order to determine proper swizzling for direct access to objects through WB. Userspace cannot handle bit17 swizzling through wc mmaps (because bit17 requires swizzling based on the actual physical address of the page - which is unknown to userspace) and so we need an extended get_tiling ioctl to report the actual as well as the logical swizzling on an object. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-11-04ioctl_wrappers: Pass in offset to CPU mmapsChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-11-04igt/gem_linear_blits: Add sufficient RAM checkChris Wilson
References: https://bugs.freedesktop.org/show_bug.cgi?id=85834 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-10-31lib/igt_core.h: add debug messages for test requirementsThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-31lib/igt_core.h: fix igt_skip_on_f requirement messageThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-29lib/tests: use the "check_" prefix for testsThomas Wood
The "check_" prefix ensures the test programs are not installed and are only built when "make check" is run. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-29build: fix distcheckThomas Wood
Fix distcheck issues introduced by commit 685e577 (Move library selftests to lib/tests). Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-29lib: ensure the library is built before the testsThomas Wood
This fixes the build problems introduced by commit 685e577 (Move library selftests to lib/tests). Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-29demos/intel_sprite_on: Added support to display all sprites.Gagandeep S Arora
Extended intel_sprite_on functionality to display all the available sprite planes on a particular connector. Signed-off-by: Gagandeep S Arora <gagandeep.s.arora@intel.com> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-29tests: Reenable testdisplayDaniel Vetter
This seems to have been accidentally disabled in commit 982f7eb238a0898c456e0574dee7c4507738d75f Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Fri Aug 29 15:19:57 2014 +0100 Prepare for 64bit relocation addresses Apparently no one noticed. Cc: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2014-10-29tests/testdisplay: Fix test status check fumbleDaniel Vetter
This is a regression from 4306538d1d3f60877866c39c9ca953cc5e541dae is the first bad commit commit 4306538d1d3f60877866c39c9ca953cc5e541dae Author: Daniel Vetter <daniel.vetter@ffwll.ch> AuthorDate: Thu Oct 2 11:18:20 2014 +0200 Commit: Daniel Vetter <daniel.vetter@ffwll.ch> CommitDate: Thu Oct 2 11:34:55 2014 +0200 tests: Sprinkle missing igt_exit() where needed. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=85582 Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2014-10-29Add CONTRIBUTING fileDaniel Vetter
i-g-t has become a fairly big project with lots of people involved, so lets document the basics and formalize the current process a bit. Also use this opportunity to announce Thomas Wood as igt maintainer once more. v2: Recommend --subject-prefix="PATCH i-g-t" as suggested by Damien. v3: Clean out contributing-related information from README. Cc: Thomas Wood <thomas.wood@intel.com> Cc: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2014-10-29Move library selftests to lib/testsDaniel Vetter
Again they're not really igt testcases so are in the way of running spatch unconditionally. Move them someplace else. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-10-29Move watermark code from tests to toolsDaniel Vetter
They're now igt tests, and so if you blindly run lib/igt.cocci with spatch on tests/*c they get mangled. Move them away, but still keep them as noinst targets. Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2014-10-27tests/pm_rpm: add system-suspend-{execbuf,modeset} subtestsPaulo Zanoni
We were having a problem where the system-suspend test was passing, but then the next test - gem-execbuf-stress - was failing because of bugs caused by the suspend subtest. So add a single test that emulates the same problem, and another test that just sets a mode after resume. This way, we should be reproducing the problem even if you don't run the tests in the order they are defined. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2014-10-26skl_ddb_allocation: Respect the minimum number of blocksDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-10-24tests/kms_flip: Make flip-vs-panning-vs-hang change DSPSURFVille Syrjälä
Make sure DSPSURF will change during the panning operation in flip-vs-panning-vs-hang. This will now test agains bugs between the kernel's mmio vs. CS flip race handling and GPU resets. If the kernel is buggy if will fail to notice that the panning operation changed the base address before the GPU reset had a chance to deal with the pending page flips, and so the flip would never complete due to DSPSURFLIVE not matching the expected value. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-10-24igt/gem_userptr_blits/dmabuf: Map the right pointer for !llcChris Wilson
References: https://bugs.freedesktop.org/show_bug.cgi?id=85354#c2 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-10-23tests/gem_workarounds: adapt to constant wa list from driverMika Kuoppala
Driver now returns the constant values that were set once on ring initialization time. Extend the simple interface check to all gens (i915_wa_registers needs to be present). Due to delay when render context is visible on cpu side, wait gpu to execute something on default context to get rid of undeterministic behaviour by sometimes getting all ones. Cc: Arun Siluvery <arun.siluvery@linux.intel.com> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
2014-10-23tests/*: lib/igt.cocci found something!Daniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-10-23kms_rotation_crc: Add test for cursor rotation (v2)Sonika Jindal
v2: Slight rebasing onto latest i-g-t codebase (Matt). Signed-off-by: Sonika Jindal <sonika.jindal@intel.com> Reviewed-by: Matt Roper <matthew.d.roper@intel.com> Signed-off-by: Matt Roper <matthew.d.roper@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-10-23lib: fix #define maxMika Kuoppala
Regression from: commit be4710a541b517b5f8663448bffed5656d59b47b Author: Thomas Wood <thomas.wood@intel.com> Date: Fri Oct 10 11:20:35 2014 +0100 lib: add common min and max macros Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=85218 Tested-by: Guo Jinxian <jinxianx.guo@intel.com> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
2014-10-23igt/gem_userptr_blits/dmabuf: Provide partial coverage on !llc platformsChris Wilson
Without LLC, we cannot assume coherency between system RAM and the GTT. If we try to use the GTT with snooped memory, the machine may lock up, so the kernel prevents us from doing so, causing the test to fail. However, we still do want to verify that we can access the userptr through a dmabuf, so just check that we can make a copy from it using the GPU and ignore the GTT cross-checking. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=85354 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-10-22lib/igt_aux: move audio RPM code to igt_setup_runtime_pm()Paulo Zanoni
If we don't enable audio runtime PM, the audio driver won't release its reference, the refcount won't ever become zero, so we will never actually runtime suspend. So move this code from pm_rpm.c to igt_aux.c, so kms_flip - and any other IGT test case using RPM - can benefit from it. Previously, if you ran pm_rpm before running the other tests - or if you just didn't have snd_hda_intel loaded - you wouldn't notice this bug. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78893 Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2014-10-21tests/gem_exec_parse: test for chained batch buffersBrad Volkin
libva makes extensive use of chained batch buffers. The batch buffer copy portion of the command parser has the potential to break chained batches, so add a simple test to make sure that doesn't happen. Signed-off-by: Brad Volkin <bradley.d.volkin@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-10-17tests/kms_cursor_crc: HSW/BDW only have square cursorsPaulo Zanoni
When I look at cursor_size_ok() (from the Kernel's intel_display.c), I see that only 845g and i865g support non-square cursors, so SKIP the tests on HSW/BDW instead of failing them. This problem happened because support for non-square cursors was being developed for HSW/BDW, but it ended up not being merged due to a small problem and priority changes. Let's make those tests SKIP until we have proper support on the Kernel: after that, we can revert this patch. v2: Improve commit message. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=84268 Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2014-10-17lib/igt_aux: make igt_wait_for_pm_status() resist the signal helperPaulo Zanoni
If the signal helper is active, the usleep() calls return earlier, and we may end up returning false way before the 10s timeout, failing the subtests. This currently happens on the kms_flip RPM interruptible subtests. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78893 Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2014-10-17tests/kms_force_connector: fix assertion when VGA is already connectedThomas Wood
Compare the number of modes available when the edid is reset with the number before the edid was overridden. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=82230 Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-17tests: add a check for command line handlingThomas Wood
Check that command line handling works consistently across all tests. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-17drm_lib.sh: add standard command line optionsThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-17configure: include the skylake quick_dump files in EXTRA_DISTThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-17quick_dump: fix distcheckThomas Wood
Commit 711a17a (quick_dump: Move base_display.txt to indivual platforms) renamed base_display.txt to common_display.txt, but didn't update Makefile.am accordingly. Since common_display.txt is now included in the platform files, it is automatically included in $(QUICK_DUMP_EXTRA_DIST) and therefore just the reference to base_display.txt needs removing. Cc: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-17lib: add common min and max macrosThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-17lib: various documentation fixesThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2014-10-15skl_ddb_allocation: Make 'end' exclusive in the DDB allocation entryDamien Lespiau
Ville suggested that we should use the same semantics as C arrays to reduce the number of those pesky +1/-1 in the allocation code. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>