tests/gem_tiled_swapping: fix igt_require conversion

I seem to be incompetent at logic ...

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

tests/gem_mmap_gtt: fix access checks

Reading manpages advisable ;-)

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

tests/gem_mmap_gtt: clarify access check checks a bit

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

lib/drmtest: Print info when children died due to signals

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

tests/gem_evict_everything: add swapping and forked subtests

Much better at hitting the list corruption here on my machines
than what we have thus far.

Note that somehow I just can't reproduce the bug any more. No idea
why. But I guess it's time to simply push this pile out.

v2: Limit threads and rounds to something reasonable.

v3: Use igt_permute_array to avoid EINVAL due to duplicated bo.

v4:
- Add a variant of the forked tests with multiple drm fds.
- Tune the swapped forked tests a bit to complete in a reasonable
  amount of time.

v5: Add some memory pressure from the cpu by using cpu mmaps (which
directly hit shmem, so bypass gem completely).

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

tests/gem_evict_*: Fix leak in copy

This turned out to be the reason one one of my tests was hitting
the list corruption bug - we need a good deal more memory pressure.
So I'll now add a new testcase for that.

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

tests/gem_tiled_swapping: Use igt_require

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

lib/drmtest: extract igt_fork from gem_concurrent_blt

Making sure that we correctly collect the exit codes from all children
is a bit a hassle. So add another magic igt codeblock for easy forking
and joining.

Note that children are (currently at least) not allowed to call igt_skip.
Add an assert to enforce this.

v2:
- Properly propagate the exit code.
- Fix the segfault.
- Add a child int and num_children paramter to the magic codeblock as
  suggested by Chris Wilson.
- Don't dump noise into stdout when a child thread fails, the parent
  will do that for us already.

v3: Now with some docs.

v4: Fixup igt_waitchildren to properly reset state so it can be used
again.

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

kms_flip: Remove debugging leftovers

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

kms_flip: Set everything to zero to disable a CRTC

Just setting fb=0 is not enough as the kernel thinks userspace is
insane.

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

kms_flip: Use the first mode if we find no matching modes for the crtc pair

We will check that we can set the mode on both crtcs before use, so
hopefully this will work...

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

kms_flip: Hook up primary events for page-flips

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

kms_flip: Run on pairs of connected outputs.

The goal is to flip the same framebuffer on a pair of CRTCs (clone mode)
and check that (a) the flip works in all combinations of workloads, and
(b) that we can hit the desired refresh rate under the simplest, most
ideal of conditions.

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

kms_flips: Operate on an array of crtc

This should be no functional change as we operate on an array of
crtc[1]. Later we shall test clone mode across a number of crtc.

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

kms_flip: Exercise flip-vs-render

For machine with split BCS/RCS rings, we also need to test whether we
correctly wait upon outstanding render work before flipping and changing
modes. This should also serve to exercise the ring selection code for
flips.

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

overlay: Increase idle timeout to 30s

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

overlay: Hide idle processes

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

overlay: Autohide Flip counter after a period of inactivity

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

overlay: Monitor per-ring context switch rate

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

overlay: Make it easier to renice

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

overlay: Generate unique name for snapshots

Since we no longer increment the counter every frame, we need to use a
timestamp instead.

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

overlay: Add number of running processes to CPU display

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

overlay: Include the core count in the cpu info line

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

overlay: Scale the CPU busy value by the numer of active cores

Having 100% always mean one processor fully utilised is much easier to
understand (than having to remember how many cores are on each machine
and scale appropriately).

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

overlay: Tidy presentation of ring busy values

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

overlay: Restore the correct default font size

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

overlay: read rc6 status from perf

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

overlay: Add support for multi-monitor positioning

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

overlay: Set the text color in case we do not have RC6

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

overlay: improve alignment of some labels

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

overlay: Read interrupts from perf

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

overlay: Read frequency from perf

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

overlay: Read power from perf_events

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

overlay: Align the process names in the memory table

And use a shade of gray to distinguish from the total instead.

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

lib/drmtest: restore in_fixture assert

I didn't really want to revert those, too ...

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

Revert "tests/gem_concurrent_blit: Fix segmentation fault"

This reverts commit 912a7d855600aadb937517ec5bab26bfd9b8953d.

Now fixed for real!

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

lib/drmtest: don't complete fixtures with a longjmp

Longjmp creates havoc with stack variables of the current stackframe.
And since fixtures should be used to set up such variables creating
havoc isn't a great idea. With this I can revert a bunch of
bogus patches I've done to paper over this by moving stack variables
to be global.

The same issue is actually a feature for subtest blocks since subtests
should be independant anyway.

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

tests/gem_mmap_gtt: Add testcase for the vma access manager

Currently fails since the patches aren't merged yet.

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

tests/gem_mmap: Convert to subtests

I wanted to write a testcase for gtt mmap access restriction checking
and only noticed after conversion to subtests that this here is for
cpu mmaps ;-)

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

tests/kms_setmode: don't save/restore vt graphics for dry-run

Signed-off-by: Imre Deak <imre.deak@intel.com>

lib/drmtest: fix handling of -h --help argument parsing

So far we handled -h or --help arguments properly only if the test
called igt_subtest_init_parse_opts(). Fix this for igt_subtest_init()
callers too. Make sure we still don't exit for any other
unknown options, which the caller may parse with a second getopt scan.

Signed-off-by: Imre Deak <imre.deak@intel.com>

overlay: Trim the padding slightly

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

overlay: Count number of semaphores used by each process

This required me to contract the per-process information considerably,
hopefully readability is not sacrificed too much.

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

overlay: Allow simple positioning and resizing

Using window.size=<width>x<height> or window.size=<scale>% in
the config file, or --size=<scale>% or --size=<width>x<height>

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

overlay: Monitor interrupts

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

overlay: Detach from terminal

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

overlay: Add a raw kms output

For when you don't have any display server, use brute force.

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

overlay: Draw the cpu busy line on top of the waits

By drawing it behind the waits, it is obscured by the translucent waits.

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

overlay: simplify types for object counts for 32/64bit porting

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

overlay: Expose sampling period/frequency as a user parameter

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

tests/gem_evict_*: Add interruptible modes

Also I've stumbled over igt_fixture again. Somehow gcc loves to wreak
havoc with stack variables set up in igt_fixtures ... Duct-tape in the
form of moving fd out to global scope applied.

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

overlay: Fix power reading from debugfs

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

overlay: Distinguish the root debugfs path and our dri node

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

overlay: Search for right debugfs path

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

tests/sysfs_rps: Fix igt_require conversion

Another fumble caught by QA.

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

overlay: Replace debug ticker with hostname

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

overlay: Close DRI2 device after authentication failure

So that we don't leave a master fd open when operating remotely.

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

overlay: Double buffer the x11 window

For pleasant remote transport.

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

overlay: Make the config parser more lenient

Allow a dot-notation outside of the [ini] style to make commandline
parameters easier to add.

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

tests/pc8: don't work around the eDP I2C bug

On the current Kernels, i2c operations on disabled eDP panels don't
work because we don't enable the panel before doing the i2c
transactions. Whenever we try these transactions we print an error
message on dmesg and fail. The current test suite was trying to work
around this problem: we count how many eDP outputs we have and add
them to the final result.

Daniel asked me to not work around the bug, so now the i2c test will
fail until we actually fix the Kernel bug (or stop using eDP panels).

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

tests/pc8: fix function return value

Looks like I forgot to push this chunk... The test was passing in
cases where you only have eDP monitors due to the fact that we
consider i2c won't work on eDP monitors.

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

overlay: Rudiments of config files and option parsing

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

overlay: Fill the blank charts with error messages

Instead of showing nothing, show a "space left intentionally blank"
message.

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

overlay: Remove busy estimation

Too inaccurate to be useful.

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

tests: ddx_intel_after_fbdev loads intel ddx after fbdev was loaded.

v2: Don't add to kernel tests.

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

overlay: Fix underflow for clamping initial values

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

overlay: i915 pmu doesn't require a specific frequency

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

overlay: Put a faint outline around each chart

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

overlay: Fade the graphs behind the text

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

tests: Add gem_evict_alignment

Clone gem_evict_everything and use it to exercise the misalignment
paths.

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

lib/drmtest: fix logic fumble in gem_context_create

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

overlay: Fix old mmio gpu busyness probing

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

overlay: Fix sampling of i915 perf event

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

overlay: Fix display of rc6 states

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

overlay: Add copyright statements

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

overlay: Use the new i915 PMU to query GPU busyness

And so avoid having to hold forcewake indefinitely.

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

overlay: Take a snapshot on SIGUSR1

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

tests/pc8: Update to latest igt infrastructure

- Use igt_require for magic SKIP results while still printing results
  for all subtests.

- Wrap setup/teardown code outside of subtests with igt_fixture. This
  will ensure that the so wrapped code isn't run when an
  igt_require/igt_assert failed (and we're essentially just printing
  the SKIP/FAIL result for all subtests) or when just enumerating
  subtests.

- Use igt_exit. Otherwise SKIP/FAIL results of subtests won't be propagated
  correctly.

- igt_subtest checks should be run unconditionally to make sure we
  always enumerate the same set of subtests. A bit on the line here
  since this is a subtest hidden behind a runtime flag.

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

tests: add pc8

This test chekcs our code that enables Package C8+. The environment
requirements for this test are quite complicated:
  - The machine needs to be properly configured to reach PC8+ when
    possible, which means all the power management policies and device
    drivers must be properly configured.
  - You need at least one output connected.
  - You need the /dev/cpu/0/msr file available.
  - You need the /dev/i2c-* files available.

v2: - Many many changes due to the latest review comments and rebase.

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

gitignore: Add a couple of recent binaries to .gitignore files

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

quick_dump: Use AM_V_GEN when generating the bindings

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

overlay: Include gpu-perf.h for distribution

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

assembler: Disable the declare test

It's not hitting a valid assertion that it tries to write an instruction
without a defined execution size (because the "default" exec_size never
end up being set).

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

assembler: Disable tests that where already failing in the gen4asm repo

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

assembler: Ignore make check output

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

assembler: Fix the path of intel-gen4asm

With the move to intel-gpu-tools, we need to update that as well.

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

assembler: Revert "Fix missing environment variables problem in test/run-test.sh"

Same as:

  commit 497814f2f2828efdc5bdd787ebc490d5083f61b8
  Author: Damien Lespiau <damien.lespiau@intel.com>
  Date:   Tue Aug 20 14:52:05 2013 +0100

      assembler: Revert "Automatically run all test cases."

make check will define srcdir and buildir variables for us.

This reverts commit 1c009349bc894bd195b5522540536898b0bee574.

assembler: Revert "Automatically run all test cases."

The tests where supposed to be run through make check, not running the
"run-test.sh" standalone. So revert that patch to have make check work
as intended.

This reverts commit 6983eebf47f37def8f2315d5af1800b81644f240.

assembler: Tune the error message for invalid send on gen6+

And be a bit more descriptive.

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

intel_infoframes: Be future-proof about showing 3D_Ext_Data

As Ville noted, future 3D_Struct must also send 3D_Ext_Data in the
vendor infoframe.

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

build: Fix the overlay build summary at the end of configure

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

lib: Make igt_assert() abort under gdb

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

gem_evict_everything: Fix the error code checking after drmIoctl

drmIoctl doesn't return the errno, so look it up after an error.

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

tests: Add gem_evict_everything

Exercise the eviction logic. This is just a naive test to ensure that we
can evict old buffers to accommodate new batches, and in the process
trigger the evict everything logic.

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

overlay: Sample power every second

Found the bug, but still only read the msr once every second - mainly as
an exercise in handling different sample rates.

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

overlay: use rc6_enable to drop useless information

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

overlay: Fix counting fail

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

overlay: Add graph for GPU power consumption

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

overlay: Add rc6 residency

Not much point at the moment since we use forcewake to continuously
probe the GPU busyness. But that will eventually change to a more
power-efficient in-kernel.

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

tests/gem_concurrent_blit: Fix segmentation fault

Apparently the simple changes introduced in

commit 78865847f9fae7e590960f9836f2ec8b611a190e
Author:     Daniel Vetter <daniel.vetter@ffwll.ch>
AuthorDate: Mon Aug 19 07:23:49 2013 +0200
Commit:     Daniel Vetter <daniel.vetter@ffwll.ch>
CommitDate: Mon Aug 19 08:09:25 2013 +0200

    lib/drmtest: skip fixtures after an igt_skip

pushed the stack space requirement for run_modes over some magic limit
(I guess where the strack grow logic starts to fail, but I didn't
check). Fix this by moving the data into .bss.

Also add some asserts to check that fixtures aren't abused while at
it (i.e. the useful parts of my debug printf/assert craze).

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