fix warn in testdisplay: ignoring return value of 'chdir'

Signed-off-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

fix warn in prime_nv_*: ignoring return value of ‘fgets’

Signed-off-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

fix warn in gem_reg_read: 'read' shadows a global declaration

Signed-off-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

fix warn in sysfs_{rc6, rps}*: ignoring return value of 'fscanf'

Signed-off-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

fix warn in gem_readwrite: 'read/write' shadows a global declaration

Signed-off-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

fix warn in intel_reg_snapshot: ignoring return value of 'write'

Signed-off-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

fix warn in intel_error_decode: ignoring return value of 'asprintf'

Signed-off-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

fix warn: 'div' shadows a global declaration

Signed-off-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests/gem_exec_big: fixups

- don't emit a self-modifying batch, the kernel doesn't like those
- also change the buffer size, to exercise all corner-cases of the
  sg_table stuff

tests/gem_exec_big: add a reloc at the end of the big batch

Since that's the other place where the broken get_page blows up.

Actually it seems to only blow up on specific pages ...

tests: add big batchbuffer test

... and use it in the hangman to stress the error_state capture a bit more.

Uncovered a bug in the obj->pages ->obj->sg_list rework.

lib: remove auto-quiescenting of the gpu

In retrospective, this is an ugly idea. Any tests that needs this
can call it themselves.

lib: make drmtest_progress more robust

tests: update .gitignore

I've thought I've fixed this, but looks like I've failed to commit.

tests: add gem_cs_tlb

Should test tlb coherency, but somehow it fails to hit the bug I want
it to hit ... needs more work.

tests/gem_cs_prefetch: clarify what it tests

I've forgotten ;-)

tests/flip_test: check -EBUSY and -EINVAL behaviour

We want -EBUSY for a pending flip and -EINVAL if the pipe is off
(either dpms off or completely off). With the small exception that
someone thought it would be funny to return -EBUSY when the crtc is
fully off.

tests/*storedw*: add subcases with more outstanding batches

Just in case this is better at hitting seqno signalling races ...

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

flip_test: fix timestamp order

When grabbing the timestamp after a flip, it has a decent chance to
actually be after the flip. Surprising.

tests/gem_caching: only disable on vintage i965g/gm

Hooray for confusing macros!

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

tests: add drm_threaded_access_tiled

This catches parallel access to bo->virtual causing sigbus

Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
[danvet: applied some bikeshed
- changed prefix from drm_ to gem_, it's a kernel gem test
- added autohell magic to link with pthreads
- .gitignore entry
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

flip_test: add an explicit flag for timing checks

We'll need this for the wait_vblank test cases.

gem_cpu_reloc: Do another pass explicitly moving to the CPU write domain

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

gem_cpu_reloc: Use the mappable aperture size!

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

gem_cpu_reloc: And run the test in reverse to check with already bound batches

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

gem_cpu_reloc: Fix for running on SNB+

I work with these everyday and I still made a basic mistake.

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

flip_test: increase runtime of the plain flip to 15s

This ensures that we'll hit the stupid hpd poll handler in every run.

Also don't yell around about jitter for tv outputs ...

tests/flip_test: measure inter-frame ts jitter

Yell if it's wrong.

For some odd reason this blows up on my snb. And always on the same
o->count frame on the 2nd crtc ... And we have to thank the hpd poll
helper for that. Comment explaining this added, also made the error
non-fatal.

flip_test: race against modesets, too

Even slower than dpms switching ;-)

gem_wait_render_timeout: Fix SKIP return code

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

tests: Add gem_cpu_reloc

Attempt to stress test performing relocations whilst the batch is in the
CPU domain.

A freshly allocated buffer starts in the CPU domain, and the pwrite
should also be performed whilst in the CPU domain and so we should
execute the relocations within the CPU domain. If for any reason one of
those steps should land it in the GTT domain, we take the secondary
precaution of filling the mappable portion of the GATT.

In order to detect whether a relocation fails, we first fill a target
buffer with a sequence of invalid commands that would cause the GPU to
immediate hang, and then attempt to overwrite them with a legal, if
short, batchbuffer using a BLT. Then we come to execute the bo, if the
relocation fail and we either copy across all zeros or garbage, then the
GPU will hang.

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

tests/flip_test: also test pannning with dummy load

Needs a bit more relaxed select timeout to work (which is not
required when testing dummy_load vs. dpms, since the dpms forces the
sync, not the select timeout).

tests/flip_test: test interaction with panning

Since panning with set_crtc is synchronous, we need to stall for any
outstanding pageflips. This new testcase exercise that code. Unfortunately
we still need eyes to check whether we don't loose the offset :(

tests/flip_test: robustifications

- don't yell around about dropped frames on tv connectors (and explain
  why in a comment)
- wait a bit when using a test config that checks for dropped frames before
  starting. Also allow for 1% of fudge, this makes it reliably work
- make the dummy load more variable, took too long on older machines.

tests/flip_test: don't complain about timestamps in the future

those can happen due to the scanline vblank timestamp adjuster ...

tests/flip_test: actually try to check what I want to check

/me sucks

tests/flip_test: check whether timestamp isn't in the past

This is mostly useful for when we add support for other clock sources,
e.g. clock_monotic, to check whether the kernel picked the right
clock.

tests/flip_test: check whether the pageflip event took too long

We're suspecting that something is fishy with the event deliver/vblank
timestamp handling on gmch platforms. Unfortunately, this isn't it.

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

flip_test: count the number of flips and check we don't drop any

In the basic case of just flipping between two buffers with no
additional work, we should neve drop a frame.

tests/flip_test: re-enable disabled testcases

oops

tests/flip_test: new testcase with a busy load before the flip

I've hoped that this would be good enough to hang pch platforms, but
it looks like those just complete the flip immediately on disabled pipes.

fixup .gitignore for newly add sysfs_rps test

lib: helper to convert gem_hanlde to drm_intel_bo

drmtest: use do_or_die

tests/flip_test: Revamp to demonstrate race with modeset/dpms and pageflips

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

intel_infoframes: Dump HDMI vendor infoframes

Those infoframes are programmed when using stereo 3D modes.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Add option -o <number>, which can test only one specified mode.

Each mode line has a number just like '[i]'. So we can only test the specified mode with giving the number of mode to '-o' parameter.

Signed-off-by: Yi Sun <yi.sun@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests/module_reload: Silence warnings when unloaded shared modules

So long as we spit out critical errors during i915 reload, or detect any
oops the test will do its job.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=55040

sysfs rps test added

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

gem_gtt_speed: Add a test that caches the mmapping across iterations

Repeat the memset streaming performance test on the same mapping so that
we can factor out the overhead of establishing the GTT/CPU mmaps.

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

gem_gtt_speed: Use a memset() to test streaming performance

This make the reasonable assumption that the libc code for memset() can
saturate the memory bandwidth -- at any rate it should do better than
the copy.

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

intel_sprite_on: plane needs 16.16 width and height

This only ever worked because we used to have a bug in our driver which
was fixed months ago by:

commit b4db1e35ac59c144965f517bc575a0d75b60b03f
Author: Jesse Barnes <jbarnes@virtuousgeek.org>
Date:   Tue Mar 20 10:59:09 2012 -0700

    drm/i915: treat src w & h as fixed point in sprite handling code

Reported-by: Armin Reese <armin.c.reese@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

intel_sprite_on: coding style fixes

It was driving me nuts...

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

intel_sprite_on: Idententation + Copyright fix

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

tools: Update gitignore with intel_dpio_{read, write}

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

intel_reg_dumper: Add more information when dumping single registers

Now that we can dump registers giving a partial name, adding more
information about the dumped registers seems useful.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

intel_reg_dumper: Allow partial register names on the command line

Let people give just a part of the register name. Handy when not
remembering the exact name or if the register is defined with a
different name than the one in the spec being looked at.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

intel_reg_dumper: Also decode registers given by address

One can now give an address instead of a register name to decode a
single register.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

intel_reg_dumper: Add a single register decode mode

From time to time, one would like to decode a register value that have
been captured at a certain point in time (and say printed out with a
printk). intel_reg_dumper has all the knowledge to do that and this
patch adds a way to ask it to decode a value.

Example usage:

$ ./tools/intel_reg_dumper PCH_PP_CONTROL 0xabcd0002
       PCH_PP_CONTROL: 0xabcd0002 (blacklight disabled, power...

v2: friendlier invocation (Chris Wilson)
v3: remove unecessary casts and use strcasecmp (Jani Nikula)

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

clang: Fix static analysis warnings from clang

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

clang: Fix warnings found through clang.

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

tests/gem_cacheing: Exercise snoop coherency

This is based on tests/gem_partial_pwrite_pread which aims to detect
incoherency between CPU reads and writes to a bo whilst using it as a
source and target for GPU writes.

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

Add bare-metal interface to adjust cacheing (i.e. snoop status) of a bo

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

bump to 1.3 release

mmio: Limit the uc- mapping to only map the registers

In the future, we may like to enable wc mapping of at least the GATT,
and so causing a conflict if we attempt to map the entire bar as uc-
here. Obviously we need a better fallback plan, but for the moment only
attempt to map the portion of the pci space that we use for register
access.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

intel_gtt: Harden against changes to kernel mappings of the GTT

Rather than use the common mmio segment which will be in future
restricted to just the registers and so exclude the GTT portion on all
architectures, explicitly mmap the GTT ourselves. Repeat this mmapping
with a couple of flags until we matching the existing kernel mapping.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

s/NO_PID/NOP_ID

Alan typo'ed it, I've failed to notice :(

Add --disable-nouveau option for platforms without nouveau support

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

flip_test: add cast to pacify Solaris Studio compiler

Fixes build error:
"flip_test.c", line 180: improper pointer/integer combination: op "="

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Rename NOPID to NO_PID to avoid conflict with Solaris NOPID

Solaris <sys/types.h> already has #define NOPID (pid_t)(-1)

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Fall back to CLOCK_MONOTONIC on systems without CLOCK_MONOTONIC_RAW

Solaris has not yet adopted this Linux extension

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Revert "tests: disable nouveau if unavailable"

This reverts commit dd8325323bfcca1f742e864ca7d87101a0d59936.

I like the patch from Alan Coopersmith with an explicit disable
option better.

tests: disable nouveau if unavailable

tests/prime_nv_test: add missing call for test8

tests/prime_nv_pcopy: kill debug code that's not upstream

tests/prime_nv: don't enable buffer reuse

We share these suckers, hence the fd-local libdrn instance does not
have full control over the lifecycle of the object. Prevents the tests
from blowing up with

[drm:i915_gem_mmap_gtt] *ERROR* Attempting to mmap a purgeable buffer

and similar things.

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

Converted a display debug script for use by designers

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

intel_reg_read: use strtol instead of atoi

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

intel_reg_read: support -c option to read multiple dwords

Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

tools: Added intel_dpio_read and intel_dpio_write

In Valleyview the DPLL and lane control registers are accessible only
through side band fabric called DPIO. Added two tools to read and write
registers residing in this space.

v2: Moved the core read/write functions to lib/intel_dpio.c based on
Ben's feedback

Signed-off-by: Vijay Purushothaman <vijay.a.purushothaman@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

tests: add gem_gtt_cpu_tlb

This catches tlb invalidation issues on my snb mobile gt2+

intel_audio_dump: add Haswell audio dump support

Add Haswell audio registers definition and dump support.

Signed-off-by: Wang Xingchao <xingchao.wang@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

intel_audio_dump: fix wrong port definition

there're three Ports B/C/D used for selection by each transcoder A/B/C.

Signed-off-by: Wang Xingchao <xingchao.wang@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Fixup previous commit

ENODEV not being defined and return code wrong

And add another corner case skip

fix crash with insufficient permissions

satisfy danvet's OC

and add skip codes, but mostly satisfy danvet's OCD

Add intel <-> nouveau prime tests

tools/reg_dumper: really dump pipe C regs

Not just a copy of pipe B. Meh.

Also kill a few redudant #define for pipe B - they match pipe A.

Update the image file pass.png with higher correction level

Signed-off-by: Yi Sun <yi.sun@intel.com>

tests/testdisplay.c Remove an uncomfortable error output

Signed-off-by: Yi Sun <yi.sun@intel.com>

lib: add more Haswell PCI IDs

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>

tools/reg_dumper: dump pipe C regs

Also reorder the pipe B regs a bit to be consisten with pipe A.

tests/gem_reg_read: actually wait a bit between reading TIMESTAMP reg

QA could only hit this on ilk because the timer increase quite a bit
slower on that platform than on gen6+ (80ns vs 320ns, iirc).

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=52937

tests/testdisplay.c: Add a option '-r'.

With the option '-r', the testdisplay could paint a 2-D bar code(QR
bar code) on the screen. The word "pass" is hiden in the bar code
image.  Further more, with this option, testdisplay will wait until a
system signal 'SIGUSR1' coming after each mode setting. This function
is for another program to control testdisplay.

danvet: Fix up the missing static.

Signed-off-by: Yi Sun <yi.sun@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

build: Make autoreconf honour ACLOCAL_FLAGS

When running autoreconf, it's possible to give flags to the underlying
aclocal by declaring a ACLOCAL_AMFLAGS variable in the top level
Makefile.am.

Putting ${ACLOCAL_FLAGS} there allows the user to set an environment
variable up before running autogen.sh and pull in the right directories
to look for m4 macros, say an up-to-date version of the xorg-util macros.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>

test/flip_test: don't fail if we can't find a crtc

some connectors are forced to certain crcs ...

tests: fixup prime test paranoia

tests/prime: more paranoia

- check the old contents first in check_bo
- drop all GEM handles for the re-import test

tests: add missing stuff to .gitignore

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