tests: add test to read all debugfs files

Catches a bunch of forcewake warnings on snb+

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

testdisplay: compile fix for !DRM_PLANES

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: add gem_unref_active_buffers

The crucial trick to reproduce the bug is that we need to have
a decent pile of active bos to retire. Because we unref the bo
after having it moved off the active list, our recursion depth
in fdo bug #42180 is limited by the number of active objects that
can retire at the same time.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=42180
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

testdisplay: allow overlay plane to be moved & scaled at runtime

For additional testing.

Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>

intel_reg_dumper: handle 3 pipe configs when dumping HDMI config

Could be on pipe A, B, or C.

Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>

testdisplay: add stub disable_planes() function

To fix the compile with non-plane libdrms.

Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>

tools/decode: decode XY_SCANLINES_BLT

tools/decode: decode XY_SETUP_* cmds

And extract decode code for br00 and br01.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

testdisplay: add sprite scaling and location support

Allow testdisplay to locate the sprite on the screen and potentially
scale it using different crtc width/height vs the source width/height
(determined by the resolution on the target pipe).

Also fix exit, making sure we properly disable all the planes.

Signed-off-by: Jesse Barnes <jbarnes@virtuougseek.org>

tools/intel_error_decode: decode gen4+ fences

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools/decode: properly decode gen6+ PIPE_CONTROL flags

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

top: Avoid FPE by removing meaningless metric

intel_reg_checker: New tool to accumulate checks for HW workarounds.

The goal of this tool is to be usable as a test for whether something
(suspend/resume, GPU reset, bugs, whatever) may have lost various
required hardware setup specified in the docs.

Reviewed-By: Eugeni Dodonov <eugeni.dodonov@intel.com>

intel/decode: print out chipset gen

... instead of i965+ for almost everything that Intel is still selling.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools/decode: don't move around the display register section

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools/decode: don't forget to print the name of the last ring

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Allow intel_dump_decode to read data from stdin.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

tools/intel_dump_decode: make devid parsing more useful

We use hex for these!

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools/decode: retain the ring name

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: add gem_hangcheck_forcewake

Provokes the forcewake warning when the hangcheck runs and no
one waits for the gpu (and hence holds the dev->struct_mutex).

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests/gem_fence_trash: some retuning

We actually want to test lru behaviour, so do a bit of work with
the fence before yielding to the next thread (we use twice as many
fences as there are, so yielding always is pretty bad, no matter how
clever our fence stealing).

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests/gem_*_blits: reduce buffer count to not trash swap

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Allow to output statistics to stdout.

This allows intel_gpu_top to run both in statistics-collecting mode
(collecting the per-ring statistics in gnuplot-friendly format) and
ncurses top-like mode at the same time.

It also allows to output the statistics directly to stdout, by using "-o
-", so the results can be parsed directly via a popen() parsing.

If you are using intel_gpu_top as previously (without any command-line
arguments), it should change nothing for you. If you were using its
logging facilities (e.g., the '-o file'), note that the logging will keep
running, but the detailed top-like interface will be on the screen at the
same time.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

bios_reader: Allow forcing the device ID with DEVICE=0xNNNN

As a pleasant side effect you can now feed it an opregion dump and it'll
decode without crashing.

Signed-off-by: Adam Jackson <ajax@redhat.com>

intel_reg_dumper: use none instead of NULL if no port is assigned to TRANS_DP

Makes the output a little more readable.

Treat PPT like CPT as we do in the kernel

Set the pch type to CPT if we detect a PPT as well.

tests: disable storedw tests on !render

According to docs, this should work with ppgtt and just convert to
noops otherwise. In reality, it hangs the machine.

So just disable these for the moment.

Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools/decode: add missing shift to MI_SEM_MBOX decoder

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

testdisplay: return 0 when successful

We use the "ret" variable to check function return values, and these
values are not necessarily 0 on the successful cases, so we need to
reset "ret" after we finish using it.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

getstats: remove useless assertion

getstats.c:48:2: warning: comparison of unsigned expression >= 0 is
always true

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

drm_open_any: explain why we are abort()ing

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
[danvet: pimped error msg]
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

testdisplay: remove wrong return statement

It seems that the kernel patches required by the TEST_PLANES feature are
still not upstream, so I didn't test this patch.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

testdisplay: don't enter the main loop if dump_info or test_all_modes

If calls inside update_display fail, the function returns and we don't
quit the program if dump_info or test_all_modes. So we enter the main
loop and keep waiting for user input, even on cases where we are not
supposed to require user input. To fix this, we move the check to
outside the update_display function.  As a side effect, we also do
cleanup instead of just exit(0) and we return 1 in case update_display
failed.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

testdisplay: test all modes if no option is provided

Previously, when called without any arguments, the application required
user input to finish.  However, testdisplay is ran by "make check", and
it is not a good idea to run a program that requires user input in "make
check". So we change the default behavior to something that doesn't
require user input. You can still access the previous behavior using the
"-m" argument.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Add missing files to .gitignore

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

testdisplay: fix compiler warnings

testdisplay.c:117:1: warning: comparison between signed and unsigned
integer expressions
testdisplay.c:125:1: warning: comparison between signed and unsigned
integer expressions
testdisplay.c:145:1: warning: comparison between signed and unsigned
integer expressions
testdisplay.c:1060:16: warning: comparison between signed and unsigned
integer expressions

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests/gem_mmap_gtt: also test gtt pwrite paths

This needs a properly pre-faulted dst bo.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: build all tests by default

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: separate testcases in gem_bad_length

Creating a zero-length bo should fail, so check that. Still run
the minimal batchbuffer, but without the zero-length reloc.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: disable gem_bad_blit

Hangs gen3 and simply writes garbage into the unmappable part of
gtt on gen4+, which might cause issues later on.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests/gem_bad_length: dont throw garbage at the gpu

It doesn't like it. Really.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

intel: we don't do distcheck for the debugger

intel_gpu_dump: really kill it

This is getting a bit hilarious ...

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

intel_gpu_dump: also kill manpage

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools: remove intel_gpu_dump

Superseded by the kernel's i915_error_state dumping facility.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools: beef up intel_dump_decode

- now also handles ascii dumps like i915_ringbuffer_data
- getoptified
- totally lazy binary detector

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: exit(77) to properly mark tests as SKIP

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: mark gen3-only tests as such

... instead of hanging the gpu.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: add basic test for missed irqs

Didn't catch any known bug, but can't hurt.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: check pipe_control coherency

v2: proper support for gen6+

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools: add intel_gen

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: basic ring<->cpu and ring<->ring tests

Using a dummy reloc that doesn't matter to trick the kernel into
synchroizing the rings.

v2: properly apply MI_NOOP workaround to MI_FLUSH_DW and
switch to MI_COND_BATCH_BUFFER_END as a dummy command on the
render ring to avoid PIPE_CONTROL errata.

v3: somebody clever decided that in C, you cound from 1,
i.e. I915_EXEC_RENDER == 1. It works now ...

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: split up gem_storedw_loop into ring specific variations

Also start to shortly explain testcases with an easily-greppable
header like this:

/*
 * Testcase:
 *
 * [Possible further explanation.]
 *
 */

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests/gem_storedw: disable on pre-gen6

Doesn't work.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

lib/batch: hackish interface to run batches on specific rings

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

test/gem_storedw_*: tune some more

MI_*/PIPE_CONTROL writes need to be in DOMAIN_INSTRUCTION, because
that is what mesa uses and I plan to use this to work around a
gen6 ppgtt issue.

Also testing with intentionally b0rked GFX_MODE on my snb shows that
we need to increase the loop counter a bit to reliably hit the tlb
invalidation problem. Test still completes within a few seconds.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools: decode some more MI_ ops

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools: decode MI_SEMAPHORE_MBOX

Juggle the length check up to avoid duplication.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tools/intel_gpu_dump: add hackish --devid parsing

For bugreports and whatnotelse.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

gem_vmap_blits: fix compilation if I915_PARAM_HAS_VMAP is not defined

Previously, "make check" failed because the main() function was not
defined.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Tested-By: Eugeni Dodonov <eugeni.dodonov@intel.com>
Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

intel_gpu_top: access hardware before running profiling command

Without this patch, intel_gpu_top will try to get access to the hardware
after running the profiling command in the background. In case such access
fails, the main process quits, but the background one will continue
running.

So let's move this test up, so it is done before we fork.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

intel_gpu_tools: describe -e option in usage screen

This patch adds a short description of '-e' option to the usage screen.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

intel_gpu_top: adopt to kernel coding style

This patch fixes adopts the code to the kernel coding_style.

intel_gpu_top: support profiling user-specified commands

This patch adds support for running intel_gpu_top to profile specific
commands. The required command will be carried out in separate process,
and main intel_gpu_top will leave when the child process will exit.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

intel_gpu_top: support non-interactive mode

This patch adds support for non-interactive mode, invoked by running with
'-o output' switch. In this case, no interactive output is being
performed, but the execution statistics are being saved into the output
file.

The output file is generated in both human and gnuplot-readable format.

Unlike interactive mode, where non-supported pipes and non-active
registers are skipped, the content of such pipes and registers is recorded
into the log file to simplify parsing and standardize the list of columns.

Also, unlike interactive mode, the registers are not sorted according to
the usage - this way, their variation over time can be analysed offline.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

This patch initializes the last_stats[] for registers prior to starting
the monitoring itself. This way, the first measure will already contain
the difference from the previous value instead of non-initialized value.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

Revert "intel_gpu_top: initialize monitoring statistics at startup"

This reverts commit 431fe7803d37f344275fdaceb57c1fbaeee8541c.

Will be replaced with more fine-grained commits.

intel_gpu_top: initialize monitoring statistics at startup

This patch initializes the last_stats[] for registers prior to starting
the monitoring itself. This way, the first measure will already contain
the difference from the previous value instead of non-initialized value.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

intel_gpu_tool: initial support for non-screen output

This patch adds initial support for non-stdio output, to be used for
non-interactive monitoring.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

intel_gpu_top: suport command line parameters and variable samples per
second

This patch adds support for getopt, and adds two default parameters to it:
-h to show usage notes; and -s to allow user to define number of samples
to acquire per second.

Manpage documentation is also adjusted accordingly.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

intel_gpu_top: account for time spent in syscalls

This allows intel_gpu_top to properly account for time spent inside system
calls. Effectively, with previous implementation, intel_gpu_top could
spent longer than 1s between consecutive measures. This attempts to
minimize the extra time spent while polling for data.

Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

tests: add store dword tests

Add a couple of simple store dword tests to test memory coherence.

gem_storedw_loop simply executes a batch that continually stores an
incremented value to a target buffer object, checking the results after
each batch completes.

gem_storedw_batches_loop does the same thing, but creates a new command
batch buffer for each iteration, which can exercise the buffer creation
code.  This test is based on one from Andrzej Kacprowski from Intel.

Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>

testdisplay: for tiled mode test, the height should be aligned

Signed-off-by: Hai Lan <hai.lan@intel.com>

Add missing copyright info to intel_decode.c

tools: readme addition for shader debugging

tools/range handing: forgot the register map file :(

intel-gpu-tools/debugging: shader debugger

high level summary of the files:
  * debug_rdata - get current state from debug registers. Helpful
    when developing the debugger, and could serve some purpose in the
    future.
  * eudb - the debugger itself
  * eviction_macro - generate the proper macro to flush the EU render
    cache until I get control flow working
  * pre_cpp - an evaluating c preprocesser like thing, to be used before
    cpp
  * sr - the system routine, exception handler which runs on the EU
  * test - a very basic test system routine
  * debug.h

intel-gpu-tools/debugging: add important debug regs

Cc: Chris Wilson <chris@chris-wilson.co.uk>

intel-gpu-tools/forcewaked: sample forcewake app

This app is required for debug features which seem to (undocumented)
reset themselves if/when the GT goes to sleep.

It is also useful for those doing general debugging or profiling from
userspace when they do not wish to have the GT sleep.

v2: made forcewaked more daemon-like

Cc: Chris Wilson <chris@chris-wilson.co.uk>

intel-gpu-tools/range handling: register range handling

Hooks to allow safe accesses from userspace. Can revert to old behavior
by using unsafe access.

forcewake: Add mmio code to do proper forcewake stuff for gen6

Add intel_backlight

A trivial tool to adjust the backlight registers (gen5+ only atm).

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel_decode: Handle a few gen7 blocks

gem_stress: ducttaping mandatory

In other news: We've been missing a unmapping_mapping_range somewhere
in the kernel. But lazy me never came around to digging up the real
cause.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

testdisplay: Test the first mode if there is no preferred mode.

For a TV device, there  might be no preferred mode. In this case,
we can test the first mode.

Signed-off-by: Hai Lan <hai.lan@intel.com>

tests/gem_vmap_blits: Fix typo

Why, oh why, do these only become obvious after pushing upstream?

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

intel_reg_dumper: Slightly tidy the reporting of pipe configs

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

Add a test case for coherency of vmaps

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

gem_stress: Assert we have no pending ops when using the CPU

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

gem_gtt_speed: Add option for variable buffer size

gem_stress: Add an option to test handling of signals

As signals cause the syscalls to be interrupted, we often need to clean
up partial state before returning to userspace. Often a source of
unamusing bugs, so encourage gem_stress to provoke them.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

tests: Add an exercise for zero-length buffers

Triggers an OOPS with dmar enabled currently.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

tests: Add a simple exercise of GTT mmaps

Test copying between 2 mappings and reading/writing from and to.

References: https://bugs.freedesktop.org/show_bug.cgi?id=38115
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

gem_stress: some stats about incoherencies

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

gem_stress: option to disable render copyfunc check

Too noisy - which is an another way of saying too broken :(

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

gem_stress: option for tile_size

I want to know how large these corruptions can get!

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

gem_stress: option to set tiles per buffer

gem_stress -p1 is much more evil than gem_stress -c1, it also manages
to tear appart untiled workloads!

Now duct-taping over it still works (--apply-duct-tape) ... hm.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

gem_stress: option to continue despite failures

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

gem_stress: option to override the number of rounds to complete

gem_stress -c 1

... sometimes takes a while to hit a problem.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

scripts: Add throttle.py

Parses a trace.dat and works out how long each throttle was and how many
batches retired within that period (and their average duration).

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>