platform/upstream/intel-gpu-tools.git
10 years agoassembler/bdw: Add the support of align1 register-indirect addressing mode on Gen8
Zhao Yakui [Tue, 9 Apr 2013 01:59:15 +0000 (09:59 +0800)]
assembler/bdw: Add the support of align1 register-indirect addressing mode on Gen8

Otherwise it can't compile the following GPU shader that uses the
register-indirect addressing mode.
  >add.sat (16) r[a0.5,0]<1>:uw     r[a0.5,0]<16;16,1>:uw  0x0080:uw
  >add.sat (16) r[a0.5,32]<1>:uw    r[a0.5,32]<16;16,1>:uw 0x0080:uw

Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: SEND instruction
Xiang, Haihao [Fri, 22 Feb 2013 03:14:13 +0000 (11:14 +0800)]
assembler/bdw: SEND instruction

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Small cleanup
Ben Widawsky [Fri, 16 Aug 2013 22:24:25 +0000 (15:24 -0700)]
assembler/bdw: Small cleanup

This was originally part of:

commit 62298329350b965e4bbfc558e5a4b1b3646742ea
Author: Xiang, Haihao <haihao.xiang@intel.com>
Date:   Wed Aug 14 14:21:16 2013 -0700

    assembler: error for the wrong syntax of SEND instruction on GEN6+

I merged that patch separately, but this tiny hunk was leftover. In
order to not muck in changing too much history, I am leaving this as a
discrete patch, but with the changed commit message

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Check & Refinement Engine message
Xiang, Haihao [Fri, 22 Feb 2013 03:14:11 +0000 (11:14 +0800)]
assembler/bdw: Check & Refinement Engine message

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Video Motion Estimation(VME) message
Xiang, Haihao [Fri, 22 Feb 2013 03:14:10 +0000 (11:14 +0800)]
assembler/bdw: Video Motion Estimation(VME) message

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Thread Spawn message
Xiang, Haihao [Fri, 22 Feb 2013 03:14:09 +0000 (11:14 +0800)]
assembler/bdw: Thread Spawn message

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Data port message
Xiang, Haihao [Fri, 22 Feb 2013 03:14:08 +0000 (11:14 +0800)]
assembler/bdw: Data port message

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Set thread switch for multiple branch instructions
Xiang, Haihao [Fri, 22 Feb 2013 03:14:07 +0000 (11:14 +0800)]
assembler/bdw: Set thread switch for multiple branch instructions

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Set jip/uip offsets used by flow control instructions
Xiang, Haihao [Fri, 22 Feb 2013 03:14:06 +0000 (11:14 +0800)]
assembler/bdw: Set jip/uip offsets used by flow control instructions

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Disable mask control for advanced mode
Xiang, Haihao [Fri, 22 Feb 2013 03:14:05 +0000 (11:14 +0800)]
assembler/bdw: Disable mask control for advanced mode

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Set math function
Xiang, Haihao [Fri, 22 Feb 2013 03:14:04 +0000 (11:14 +0800)]
assembler/bdw: Set math function

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Use gen8_set_exec_size() to set the execution size
Damien Lespiau [Tue, 5 Feb 2013 11:34:10 +0000 (11:34 +0000)]
assembler/bdw: Use gen8_set_exec_size() to set the execution size

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Preliminary gen8 send & msgtarget support
Damien Lespiau [Thu, 31 Jan 2013 18:25:45 +0000 (18:25 +0000)]
assembler/bdw: Preliminary gen8 send & msgtarget support

Still some work needed there, but enough for rendercopy.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Add the start of a gen8 disassembler
Damien Lespiau [Thu, 31 Jan 2013 15:31:27 +0000 (15:31 +0000)]
assembler/bdw: Add the start of a gen8 disassembler

Directly taken from Mesa.

v2 (Ben): Updated copyright

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Make the validation functions take a brw_program_instruction
Damien Lespiau [Thu, 31 Jan 2013 01:55:48 +0000 (01:55 +0000)]
assembler/bdw: Make the validation functions take a brw_program_instruction

This allows to use the same functions to validate operands on gen8 for
now.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Support some basic gen8 intructions
Damien Lespiau [Thu, 31 Jan 2013 02:16:08 +0000 (02:16 +0000)]
assembler/bdw: Support some basic gen8 intructions

We should now support alu2 intructions with direct register addressing.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agoassembler/bdw: Add gen8_instruction from mesa
Damien Lespiau [Wed, 30 Jan 2013 17:19:57 +0000 (17:19 +0000)]
assembler/bdw: Add gen8_instruction from mesa

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: Port storedw_loop_vebox to gen8
Ben Widawsky [Wed, 9 Oct 2013 22:51:34 +0000 (15:51 -0700)]
tests/bdw: Port storedw_loop_vebox to gen8

I chose not to implement this in the same way as Zhao Yakui because I
was lazy.

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: Port storedw_loop_blt to gen8
Zhao Yakui [Tue, 9 Apr 2013 01:29:19 +0000 (09:29 +0800)]
tests/bdw: Port storedw_loop_blt to gen8

The code is from the storedw_loop_render.

v2 (by Ben): Flush on the correct ring

Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: Port storedw_loop_bsd to gen8
Zhao Yakui [Tue, 9 Apr 2013 01:29:18 +0000 (09:29 +0800)]
tests/bdw: Port storedw_loop_bsd to gen8

The code is from the storedw_loop_render.

v2 (by Ben): Flush on the correct Ring

Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: Port storedw_loop_render to bdw
Damien Lespiau [Thu, 14 Feb 2013 15:32:33 +0000 (15:32 +0000)]
tests/bdw: Port storedw_loop_render to bdw

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: pwrite_pread
Ben Widawsky [Wed, 9 Oct 2013 02:48:35 +0000 (19:48 -0700)]
tests/bdw: pwrite_pread

support gen8 style blits

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: gem_linear_blits
Ben Widawsky [Wed, 9 Oct 2013 15:41:52 +0000 (08:41 -0700)]
tests/bdw: gem_linear_blits

support gen8 style blits

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: gem_pin
Ben Widawsky [Wed, 9 Oct 2013 15:41:52 +0000 (08:41 -0700)]
tests/bdw: gem_pin

support gen8 style blits

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: gem_exec_blt
Ben Widawsky [Wed, 9 Oct 2013 15:41:52 +0000 (08:41 -0700)]
tests/bdw: gem_exec_blt

support gen8 style blits

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: gem_evict_*
Ben Widawsky [Wed, 9 Oct 2013 15:41:52 +0000 (08:41 -0700)]
tests/bdw: gem_evict_*

support gen8 style blits

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: gem_cpu_reloc
Ben Widawsky [Wed, 9 Oct 2013 01:15:21 +0000 (18:15 -0700)]
tests/bdw: gem_cpu_reloc

support gen8 style blits

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agotests/bdw: gem_exec_faulting_reloc
Ben Widawsky [Wed, 9 Oct 2013 00:38:43 +0000 (17:38 -0700)]
tests/bdw: gem_exec_faulting_reloc

support gen8 style blits

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agobdw: Update obvious missing blit support
Ben Widawsky [Tue, 8 Oct 2013 22:02:07 +0000 (15:02 -0700)]
bdw: Update obvious missing blit support

This provides a macro that allows us to update all the arbitrary blit
commands we have stuck throughout the code. It assumes we don't actually
use 64b relocs (which is currently true). This also allows us to easily find
all the areas we need to update later when we really use the upper dword.

This block was done mostly with a sed job, and represents the easier
in test blit implementations.

v2 by Oscar: s/OUT_BATCH/BEGIN_BATCH in BLIT_COPY_BATCH_START

CC: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Oscar Mateo <oscar.mateo@intel.com>
10 years agobdw: Add gen8 specific instdone bits
Damien Lespiau [Thu, 14 Feb 2013 15:32:32 +0000 (15:32 +0000)]
bdw: Add gen8 specific instdone bits

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agobdw: Add gen8 to intel_gen()
Damien Lespiau [Thu, 14 Feb 2013 15:32:31 +0000 (15:32 +0000)]
bdw: Add gen8 to intel_gen()

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agobdw: Add gen8 to the IS_9XX() macro
Damien Lespiau [Thu, 14 Feb 2013 15:32:30 +0000 (15:32 +0000)]
bdw: Add gen8 to the IS_9XX() macro

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agopciid/bdw: Add Broadwell PCI ids
Ben Widawsky [Mon, 3 Dec 2012 19:36:46 +0000 (11:36 -0800)]
pciid/bdw: Add Broadwell PCI ids

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agochipset: IS_I9XX macro
Ben Widawsky [Tue, 4 Dec 2012 16:38:46 +0000 (08:38 -0800)]
chipset: IS_I9XX macro

This isnt useful in IGT, but it will allow us to keep the merge process
with libdrm simpler.

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agobump version to 1.5 and add the release date intel-gpu-tools-1.5
Rodrigo Vivi [Wed, 6 Nov 2013 15:06:11 +0000 (13:06 -0200)]
bump version to 1.5 and add the release date

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
10 years agotests: pm_psr
Rodrigo Vivi [Tue, 1 Oct 2013 18:12:37 +0000 (15:12 -0300)]
tests: pm_psr

Check on debugfs if PSR is supported by panel and matching all conditions in
hardware. In this case PSR must be enabled and performance counting increasing

v2: check if performance counter is really increasing.
v3: respect new naming convention

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
10 years agoNEWS: drop_caches improvements
Daniel Vetter [Wed, 6 Nov 2013 14:06:15 +0000 (15:06 +0100)]
NEWS: drop_caches improvements

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agoquick_dump/vlv: add DPIO_CTL to the dump
Jesse Barnes [Tue, 5 Nov 2013 15:44:20 +0000 (07:44 -0800)]
quick_dump/vlv: add DPIO_CTL to the dump

Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
10 years agolib/drmtest: Retire requests via drop caches after gem_quiescent_gpu
Oscar Mateo [Tue, 5 Nov 2013 14:15:19 +0000 (14:15 +0000)]
lib/drmtest: Retire requests via drop caches after gem_quiescent_gpu

This helps make sure that the GPU is really quiescent by getting
rid of any residual stuff.

Signed-off-by: Oscar Mateo <oscar.mateo@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agotests/gem_ctx_bad_exec: Check the errno, too
Daniel Vetter [Tue, 5 Nov 2013 10:56:50 +0000 (11:56 +0100)]
tests/gem_ctx_bad_exec: Check the errno, too

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agogem_flink_race: Assure no pending requests before object counting
Oscar Mateo [Tue, 5 Nov 2013 10:57:31 +0000 (10:57 +0000)]
gem_flink_race: Assure no pending requests before object counting

Same thing that was done for prime_self_import.

v2: Move igt_drop_caches_set() call inside get_object_count() to make
it clearer why we want this.

Signed-off-by: Oscar Mateo <oscar.mateo@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agoprime_self_import: Assure no pending requests before object counting
Oscar Mateo [Tue, 5 Nov 2013 10:56:24 +0000 (10:56 +0000)]
prime_self_import: Assure no pending requests before object counting

We don't want a previously used object to be freed in the middle of a
before/after object counting operation (or we would get a "-1 objects
leaked" message). We have seen this happening, e.g., when a context
from a previous run dies, but its backing object is alive waiting for
a retire_work to kick in.

v2: Use igt_debugfs facilities for drop cache.
v3: Move igt_drop_caches_set() call inside get_object_count() to make
it clearer why we want this.

Signed-off-by: Oscar Mateo <oscar.mateo@intel.com>
Cc: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agoNEWS: Roll in updates.
Daniel Vetter [Sun, 3 Nov 2013 10:28:50 +0000 (11:28 +0100)]
NEWS: Roll in updates.

10 years agotests: establish core_ prefix
Daniel Vetter [Sat, 2 Nov 2013 11:42:21 +0000 (12:42 +0100)]
tests: establish core_ prefix

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agotests: Use kms_ prefix a bit more
Daniel Vetter [Sat, 2 Nov 2013 11:35:44 +0000 (12:35 +0100)]
tests: Use kms_ prefix a bit more

I was a bit on the fence about the basic pipe CRC test since that
doesn't really test kms, but debug infrastructure in debugfs.

Otoh running this one for a full kms testrun is always good, to make
sure that all the other (real) CRC based tests work sanely.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agotests: establish pm_ prefix
Daniel Vetter [Sat, 2 Nov 2013 11:27:24 +0000 (12:27 +0100)]
tests: establish pm_ prefix

Imo power management, power consumption and performance are tightly
enough coupled that we can throw them all into one bin.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agotests: estbalish drv_ prefix
Daniel Vetter [Sat, 2 Nov 2013 11:22:37 +0000 (12:22 +0100)]
tests: estbalish drv_ prefix

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agotests: Start to document naming conventions
Daniel Vetter [Sat, 2 Nov 2013 11:17:49 +0000 (12:17 +0100)]
tests: Start to document naming conventions

Just a start and we need more work here. Some follow-up patches will
clear up some of the historical confusion.

While at it rename the pc8 "basic" test to "rte".

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agoUpdate .gitignore a bit
Daniel Vetter [Sat, 2 Nov 2013 11:54:35 +0000 (12:54 +0100)]
Update .gitignore a bit

- Ignore build-aux/
- Cleanup ignores for assembler/

10 years agolib: drop return value from igt_drop_caches
Daniel Vetter [Mon, 4 Nov 2013 17:11:29 +0000 (18:11 +0100)]
lib: drop return value from igt_drop_caches

No one actually cares, everyone expects it to just work.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agolib: Add igt_drop_caches_set()
Oscar Mateo [Mon, 4 Nov 2013 16:30:47 +0000 (16:30 +0000)]
lib: Add igt_drop_caches_set()

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>
10 years agotests: Fix "-thrashing" and "-thrash-inactive" distinction
Oscar Mateo [Mon, 4 Nov 2013 16:30:46 +0000 (16:30 +0000)]
tests: Fix "-thrashing" and "-thrash-inactive" distinction

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>
10 years agotests: igt_no_subtest
Daniel Vetter [Fri, 1 Nov 2013 21:18:20 +0000 (22:18 +0100)]
tests: igt_no_subtest

Yet another check for the library.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agotests: Add a Makefile comment about the expected failures in igt_ tests
Daniel Vetter [Fri, 1 Nov 2013 17:31:59 +0000 (18:31 +0100)]
tests: Add a Makefile comment about the expected failures in igt_ tests

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agotests: Add a test template
Daniel Vetter [Thu, 31 Oct 2013 16:05:28 +0000 (17:05 +0100)]
tests: Add a test template

Requested by Paulo.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agolib: add igt_main macro
Daniel Vetter [Thu, 31 Oct 2013 15:23:26 +0000 (16:23 +0100)]
lib: add igt_main macro

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>
10 years agolib: make igt_install_exit_handler never fail
Daniel Vetter [Thu, 31 Oct 2013 16:16:33 +0000 (17:16 +0100)]
lib: make igt_install_exit_handler never fail

Most callers didn't bother checking, so just move the asserts into
the function itself.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agolib: make igt_set_vt_graphics_mode never fail
Daniel Vetter [Thu, 31 Oct 2013 16:02:41 +0000 (17:02 +0100)]
lib: make igt_set_vt_graphics_mode never fail

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>
10 years agolib: consolidate pipe crc exit handler
Daniel Vetter [Thu, 31 Oct 2013 15:09:46 +0000 (16:09 +0100)]
lib: consolidate pipe crc exit handler

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>
10 years agotests/kms_cursor_crc: rework to auto-skip connectors
Daniel Vetter [Thu, 31 Oct 2013 15:06:40 +0000 (16:06 +0100)]
tests/kms_cursor_crc: rework to auto-skip connectors

Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agotests/debugfs_pipe_crc: set mode before creating CRC helper
Daniel Vetter [Thu, 31 Oct 2013 14:37:05 +0000 (15:37 +0100)]
tests/debugfs_pipe_crc: set mode before creating CRC helper

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>
10 years agotests: use "auto" pipe CRC source
Daniel Vetter [Thu, 31 Oct 2013 13:02:44 +0000 (14:02 +0100)]
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>
10 years agolib: add more self-tests around the igt_exit checks.
Daniel Vetter [Fri, 1 Nov 2013 17:29:33 +0000 (18:29 +0100)]
lib: add more self-tests around the igt_exit checks.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agolib: fix igt_exit assert when only listing subtests.
Daniel Vetter [Fri, 1 Nov 2013 17:21:08 +0000 (18:21 +0100)]
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>
10 years agolib: Don't forget to close the pipe ctl fd in igt_pipe_crc_reset()
Damien Lespiau [Fri, 1 Nov 2013 16:25:10 +0000 (16:25 +0000)]
lib: Don't forget to close the pipe ctl fd in igt_pipe_crc_reset()

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
10 years agogitignore: Ignore recently introduced tools and tests
Damien Lespiau [Fri, 1 Nov 2013 15:47:42 +0000 (15:47 +0000)]
gitignore: Ignore recently introduced tools and tests

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
10 years agomodule_reload: remove snd_hda_intel
Paulo Zanoni [Mon, 21 Oct 2013 13:45:41 +0000 (11:45 -0200)]
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>
10 years agoAdd VGA register read/write tools
Ville Syrjälä [Thu, 19 Sep 2013 15:14:54 +0000 (18:14 +0300)]
Add VGA register read/write tools

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
10 years agodebugfs_pipe_crc: skip on simulation
Ben Widawsky [Thu, 31 Oct 2013 21:16:41 +0000 (14:16 -0700)]
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>
10 years agotests/pc8: remove some printfs
Paulo Zanoni [Tue, 29 Oct 2013 16:02:43 +0000 (14:02 -0200)]
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>
10 years agotests/pc8: clarify modeset_subtest arguments
Paulo Zanoni [Tue, 29 Oct 2013 13:14:50 +0000 (11:14 -0200)]
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>
10 years agotests/pc8: add modeset -stress-no-wait tests
Paulo Zanoni [Tue, 29 Oct 2013 13:00:45 +0000 (11:00 -0200)]
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>
10 years agotests/pc8: add modeset-lpsp and modeset-non-lpsp
Paulo Zanoni [Mon, 28 Oct 2013 21:11:31 +0000 (19:11 -0200)]
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>
10 years agogem_suspend: Skip on simulation
Ben Widawsky [Wed, 30 Oct 2013 03:01:43 +0000 (20:01 -0700)]
gem_suspend: Skip on simulation

Simulation has proven flaky across both reset, and s3/4

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
10 years agogem_close_race: Fix linker flags
Ben Widawsky [Wed, 30 Oct 2013 03:01:14 +0000 (20:01 -0700)]
gem_close_race: Fix linker flags

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>
10 years agogem_close_race: Also test random closing of active fd
Chris Wilson [Tue, 29 Oct 2013 14:21:29 +0000 (14:21 +0000)]
gem_close_race: Also test random closing of active fd

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
10 years agolib/drmtest: Scream harder when igt_exit isn't called for subtest tests
Daniel Vetter [Tue, 29 Oct 2013 10:29:00 +0000 (11:29 +0100)]
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>
10 years agotests: Use igt_exit for tests with subtest
Daniel Vetter [Tue, 29 Oct 2013 10:15:43 +0000 (11:15 +0100)]
tests: Use igt_exit for tests with subtest

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agogem_close_race: Bump the workload
Chris Wilson [Tue, 29 Oct 2013 09:25:44 +0000 (09:25 +0000)]
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()

10 years agogem_close_race: Tidy up call to execbuffer
Chris Wilson [Tue, 29 Oct 2013 08:38:30 +0000 (08:38 +0000)]
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>
10 years agotests/kms_cursor_crc: Use igt_exit
Daniel Vetter [Tue, 29 Oct 2013 06:47:33 +0000 (07:47 +0100)]
tests/kms_cursor_crc: Use igt_exit

Otherwise the exit codes are all bogus.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
10 years agotests: Convert the ctx test to use render nodes when possible
Damien Lespiau [Thu, 24 Oct 2013 14:27:12 +0000 (15:27 +0100)]
tests: Convert the ctx test to use render nodes when possible

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
10 years agotests/gem_render_copy: Use render nodes
Damien Lespiau [Thu, 24 Oct 2013 14:21:59 +0000 (15:21 +0100)]
tests/gem_render_copy: Use render nodes

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
10 years agolib: Add a drm_open_any_render() that will try to use render nodes
Damien Lespiau [Thu, 24 Oct 2013 14:19:32 +0000 (15:19 +0100)]
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>
10 years agolib: Close non intel fds in drm_get_card()
Damien Lespiau [Thu, 24 Oct 2013 14:04:26 +0000 (15:04 +0100)]
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>
10 years agolib: Remove stale comment
Damien Lespiau [Thu, 24 Oct 2013 13:08:17 +0000 (14:08 +0100)]
lib: Remove stale comment

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
10 years agotests: Add gem_close_race
Chris Wilson [Mon, 28 Oct 2013 09:10:46 +0000 (09:10 +0000)]
tests: Add gem_close_race

The intention is to exercise #70784. Yet the first challenge is make the
test suite happy.

10 years agogem_render_linear_blits: Remove aub dump support
Damien Lespiau [Tue, 22 Oct 2013 14:03:48 +0000 (15:03 +0100)]
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>
10 years agotests/gem_render_copy: Add aub dump support
Damien Lespiau [Tue, 22 Oct 2013 13:57:57 +0000 (14:57 +0100)]
tests/gem_render_copy: Add aub dump support

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
10 years agotests/gem_render_copy: Only dump pngs when the -d option is given
Damien Lespiau [Tue, 22 Oct 2013 13:26:38 +0000 (14:26 +0100)]
tests/gem_render_copy: Only dump pngs when the -d option is given

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
10 years agotests/gem_render_copy: Add a simple render copy test
Damien Lespiau [Tue, 22 Oct 2013 13:20:10 +0000 (14:20 +0100)]
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>
10 years agolib: Add a function to dump a scratch buf into a png
Damien Lespiau [Tue, 22 Oct 2013 13:18:26 +0000 (14:18 +0100)]
lib: Add a function to dump a scratch buf into a png

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
10 years agobuild: include intel_l3_parity.h to its _SOURCES to fix make distcheck
Rodrigo Vivi [Tue, 22 Oct 2013 14:35:59 +0000 (12:35 -0200)]
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>
10 years agobuild: Fix small typo that was breaking make distcheck
Rodrigo Vivi [Tue, 22 Oct 2013 13:28:56 +0000 (11:28 -0200)]
build: Fix small typo that was breaking make distcheck

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
10 years agokms_flip: Add test for -ENOENT
Ville Syrjälä [Wed, 16 Oct 2013 20:07:34 +0000 (23:07 +0300)]
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>
10 years agokms_cursor_crc: Add a cursor test using CRCs
Ville Syrjälä [Fri, 18 Oct 2013 14:44:42 +0000 (17:44 +0300)]
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>
10 years agolib: Check pipe source validity in igt_pipe_crc_new()
Ville Syrjälä [Mon, 21 Oct 2013 15:45:03 +0000 (18:45 +0300)]
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>
10 years agolib: Allow pipe_crc_free(NULL)
Ville Syrjälä [Mon, 21 Oct 2013 16:37:33 +0000 (19:37 +0300)]
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>
10 years agolib/drmtest: Add kmstest_paint_color_alpha()
Ville Syrjälä [Fri, 18 Oct 2013 18:43:14 +0000 (21:43 +0300)]
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>
10 years agointel_reg_dumper: fix "enable" at hsw_debug_lp_wm
Paulo Zanoni [Mon, 21 Oct 2013 15:19:57 +0000 (13:19 -0200)]
intel_reg_dumper: fix "enable" at hsw_debug_lp_wm

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>