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>

tests/kms_cursor_crc: Use igt_exit

Otherwise the exit codes are all bogus.

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

tests: Convert the ctx test to use render nodes when possible

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

tests/gem_render_copy: Use render nodes

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

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>

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>

lib: Remove stale comment

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

tests: Add gem_close_race

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

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>

tests/gem_render_copy: Add aub dump support

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

tests/gem_render_copy: Only dump pngs when the -d option is given

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

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>

lib: Add a function to dump a scratch buf into a png

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

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>

build: Fix small typo that was breaking make distcheck

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>

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>

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>

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>

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>

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>

intel_reg_dumper: fix "enable" at hsw_debug_lp_wm

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

gitignore: Ignore intel_opregion_decode

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

intel_opregion_decode: decipher bclm table

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

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>

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>

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>

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>

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>

intel_error_decode: Fix X/Y fence for gen2/3

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>

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>

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>

tests/kms_flip: no absolute vblank wait on the first frame

We don't have a reference value for the vblank counter yet, so
will end up waiting for a long time for the vblank counter to
wrap around again.

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

tets/kms_flip: disable correlation check

It's broken since it doesn't take the order the events arrived into
account and so will fall over for longer seq_step values in the vblank
event: Since the flip completes right away, but the vblank later on
they won't have the same timestamp.

For now just give up, we could resurrect this by strictly alternating
between a flip and a vblank wait.

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

tests/debugfs_pipe_crc: new subtest to check CRC frame numbers

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

rendercopy: Add a way to dump an .aub file with the rendercopy bos

v2 (by Ben): Remove libdrm dependency since intel-gpu-tools now requires
a higher version anyway. Remove associated #ifdef ENABLE_AUB_DUMP

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Ben Widawsky <benjamin.widawsky@intel.com>

lib: fix the assert in igt_stop_helper

So much for developing different patches on different machines and
then not retesting after rebasing. Reported by Ben on irc.

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

lib: Don't wait for a vblank when enabling the CRCs

This was a previous attempt to solve the first CRC being bogus. We know
wait for it and discard it at the end of the function.

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

tests/debugfs_pipe_crc: Test the read CRCs are not null

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

tests/debugfs_pipe_crc: test all connectors

Diff looks a bit ugly, but it's just due to the added indentation.

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

tests/debugfs_pipe_crc: test all pipes

And skip them if there's no suitable connector.

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

tests/debugfs_pipe_crc: fix fb leak

Also remove the unused connector_t->mode.

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

tests/debugfs_pipe_crc: fall back to PIPE source

With PLANE1 and PIPE CRC sources the test will work on all currently
shipping (and planed fwiw) platforms.

Also add all the other new sources for non-ivb/hsw chips.

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

lib/drmtest: Check that helper processes have died correctly

If the test gets stopped sometimes a helper process falls over.
We need to report this. Since we currently don't track helper
process to precisely we can't shut up the 2nd test failure messge.
This shouldn't happen anyway.

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

tests/ZZ_missed_irq: Fixup SOURCE_PATH handling

Copy the trick from ZZ_hangman of just cd'ing into the old working
directory in a subshell.

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

gem_ctx_bad_exec: skip vebox when appropriate

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

README: list some of the dependencies

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Ben Widawsky <benjamin.widawsky@intel.com>

tests/debugfs_pipe_crc: correctly skip on unsupported platforms

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

debugfs_pipe_crc: Let's check CRCs!

Let's add a new test that sets a mode, wait for a few vblanks (3) and
then make sure we read 3 identical CRCs.

Some subtests check for various parsing errors.

In the process, improve the debugfs helpers to deal with CRCs.

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

lib: Add igt_wait_for_vblank() helper

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

lib: Make igt_debugfs_open() take the mode as argument

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

lib: Add a igt_display.h with a few enums and defines from the kernel

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

lib: Add kmstest_paint_color()

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

lib: Add a igt_assert_cmpint()

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

lib: Add igt_debugfs_fopen()

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

lib: Add a small helper to open debugfs files

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

Depend on libdrm_intel >= 2.4.47.

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

configure: Don't bail if libdrm_nouveau isn't available.

We were seriously *requiring* libdrm_nouveau unless explicitly disabled?

Acked-by: Ben Widawsky <ben@bwidawsk.net>
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>

tests/pc8: Readd verbose error message

I deemed them a bit redundant (assuming that developers are the only
ones that look at them anyway). But Paulo requested that I readd them
for the education of QA.

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

tests/kms_flip: fix igt_assert

I've blown the conversion in

commit 37e5b318c5dac290611099e2eec21df1e49fa71f
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Fri Oct 4 18:42:26 2013 +0200

    tests/kms_flip: use igt_assert more

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

tests/kms_flip: Skip if the kernel doesn't let y-tiled bo slip through

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

tests/kms_flip: exit handler needs a fixture

Since it can fail.

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

testdisplay: Allow getopt to print error messages

By not assigning opterr, getopt will print its own error message that
includes information about whether an option is unknown or just requires
an additional argument.

Signed-off-by: Thomas Wood <thomas.wood@intel.com>

tests/gem_suspend: test debugfs/sysfs reads while s/r

Just a very quick hack cobbled together with /bin/sh and exec. We
can't use system since that does stupid things with singals ... Still
we need to whack the child process pretty hard to get rid of it.

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

tests/module_reload: fail if the module didn't unload

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

gem_pin: Use iterator for blit batch setup

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

gem_exec_blt: Use iterator for blit batch setup

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

gem_evict_*: Use iterator for blit batch

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

gem_linear_blits: Use iterator for blit batch setup

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

gem_exec_faulting_reloc: use iterator for batchbuffer

This will help keep the code clean for new platforms which might have
differently sized relocations.

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

tests: add kms_addfb

Just a bunch of testcase to check that all the addfb sanity checks
work as they should.

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

lib/drmtest: extract rawer __gem_set_tiling

For tests that expect failures. Also apply the existing gem_set_tiling
helper a bit wider.

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

pwrite_pread: Get devid only once

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

pwrite_pread: Extract batch building

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

pwrite_pread: use execbuf.batch_len for size

This will make an upcoming change easier.

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

gem_tiled_blits: Squash bo leak on simulation

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

gem_tiled_blits: Squash memory leak on simulation

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

tests/kms_flip: Skip if no clone configuration could be found

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

configure.ac: require dri2proto for overlay

With the introduction of

commit f9a50de3dcc501e930de6c60983a4feb57121e7e
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Sat Aug 17 11:12:07 2013 +0100

    Introduce intel-gpu-overlay

dri2proto became a dependency (and there is no way to disable overlay
explicitly. The actual version chosen was arbitrarily stolen from mesa.

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

tests/pc8: Fail harder

If the test enviroment isn't properly set up we should fail the
testcase, since otherwise there's no way to make sure a feature
actually works.

To cut down on bug triaging time extract the basic test (which was
previously used to skip all subtests) into a "basic" subtest.

Also fail the test hard if the msr interface isn't available. And
switch all other check in setup_enviroment to igt_require.

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

tests/pc8+: Only skip i2c subtest if i2c /dev interface is unavailable

Through the magic of igt_skip we only need to fail this subtest, not
the entire thing.

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

tests/kms_flip: use igt_assert more

I've left the time delta checks as-is since those are tricky math and
my mind is feeble.

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

tests/kms_flip: check flip count in both directions

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

NEWS: Mention gen7 perf counters release

This is too great to be omitted!

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

tests: Simulate missed breadcrumb irqs

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

intel_perf_counters: Add support for Gen7 platforms.

We finally received permission to release this; the counters should be
properly documented in the Haswell PRMs.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>

tests/kms_flip: use igt_assert in the dpms helpers

No need to route the error code around really.

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

tests/kms_flip: Check the dpms confusion

Some kernels inadvertedly forwarded dpms changes to crtcs connected to
shared encoders even though that specific output wasn't enabled.
Hilarity ensued.

Note that we only have shared encoders on hsw (DP+HDMI) and with sdvo
cards (multi-function encoders).

v2: Do a full OFF->ON->OFF transition to make sure something actually
happens.

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

NEWS: testdisplay now with 3d support!

tests/kms_flip: scale test runtime

On machines with many outputs and many crtcs it takes too long.
So scale the runtime by the number of output combinations we're trying
to light up.

Compensate the test runtime a bit by doubling most subtests' duration.

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

tests/kms_flip: fail harder

At least for pipe A/B we should always fail the test if we can't light
up the preferred mode - the kernel should filter out impossible modes
and for our hw pipe A/B are the least constrained.

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

testdisplay: Print the stereo mode being tested

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

testdisplay: Make -o and -3 work together

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

testdisplay: Remove mode resetting now that we don't adjust the timings

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

testdisplay: Respect specified_disp_id in the stereo loop

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

testdisplay: Remove printing out the eyes geometry

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

lib: Adjust dump_modes() to the new stereo encoding

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

testdisplay: Use DRM_MODE_FLAG_3D_MASK

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

testdisplay: Remove the timing adjustements now that the kernel does it

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

intel_infoframe: Display the VIC in decimal

It's not customary to display the VIC in hexadecimal and lead me to
scratch my head for a couple of seconds. Print it in decimal instead.

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