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>

overlay: Wrap the colour index around the rgba array

And stop assigning colours beyond the end of the array. Still needs a
better method for colour assignment, big list of good colours & ida?

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

overlay: Add missing include "debugfs.h"

Later versions of gcc are apparently stricter.

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

overlay: Auatomatically mount debugfs

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

quick_dump/bdw: use hex() to convert devid to hex string in error message

This patch is to avoid the error on device auto-detection failure
"TypeError: Can't convert 'int' object to str implicitly".

Reviewed-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Mengdong Lin <mengdong.lin@intel.com>

tests: add kms_setmode

Iterate through all valid/invalid crtc/connector combinations. At the
moment only clone configurations are tested as the single output cases
are tested already by testdisplay. Also from combinations where all
connectors are on the same crtc (clone-single-crtc) only those are
tested that are invalid, as I haven't found any machine that supports
these (have to be GT2 with dvo and vga output).

For configurations with one crtc per connector the FBs are per-crtc atm.

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

lib: add subtest extra command line option handling

At the moment any command line option handling done by tests will
interfere with the option handling of the subtest interface. To fix this
add a new version of the subtest_init function accepting optional short
and long command line options. Merge these together with the subtest
interface's own long options and handle both together in the same
getopt_long call.

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

lib: handle SIGSEGV similarly to other error signals

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

lib: shorten DP/eDP connector names

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

lib: export kmstest_get_connector_default_mode

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

lib/drmtest: igt_skip when drm_get_card fails

Also add printf support to igt_skip to make sure there's always a
reason why we fail a testcase.

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

lib/drmtest: extract gem_create_context

libdrm requirements are now up to it.

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

tests: roll out igt_require some more

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

lib/drmtest: rip out drm_open_any_master

It's unused. Also most of our tests failed to ask for the right type
of drm fd anyway. So it's imo better to just let them fall over when
they don't get master but want it, like they already do today.

This also allows us to garbage-collect the master parameter to
drm_get_card and associated code.

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

lib/drmtest: Properly skip if no intel gfx supported

drm/* kernel test nirvana!

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

lib/drmtest: make igt_fail work outside of subtests

With the abortable fixtures we can now correctly fail all subsequent
sutbests if we hit upon an igt_assert outside of a subtest.

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

lib/drmtest: skip fixtures after an igt_skip

This way we can just enclose all igt_skip/igt_require calls into
fixtures even when we have subtests, and still output correct SKIP
message for all of them. Wohoo, magic!

The only thing which doesn't work yet is enumerating failed subtests,
but I think that should work out on top of this.

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

overlay: Tweak label locations to include baseline offset.

Still approximately, too lazy to query the metrics.

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

overlay: Add a X11 window backend

Useful for remote hosts.

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

overlay: Tweak a couple of the fill colours to be slightly less horrid

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

overlay: Don't smooth gpu freq

This is supposed to be discrete jumps, so use straight lines to emphasis
this nature.

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

test/gem_dummy_reloc_loop: fix igt_require conversion

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

overlay: Remove the double ':' from object comm names

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

overlay: Use a more compact 2x2 layout for the graphs

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

overlay: Rearrange GUI state

Planning a new layout...

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

overlay: Print min/max GPU frequencies.

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

overlay: Convert gem objects to megabytes

Less confusing, I hope.

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

overlay: Parse gem objects

Condense the information and begin graphing it. Remaining todo for
memory is to measure bind/evict flux, and perhaps clflush.

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

overlay: Add GPU frequency

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

overlay: Include CPU usage in the overview chart

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

overlay: Count flips per plane

Stop the misleading double-accounting of flips when we have multiple
displays active.

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

overlay: Improve the horizontal fit of the charts

Notably fix the fill boundary to not extend past the right hand side of
the chart.

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

overlay: Correct layout of ring/seqno in raw sample

For seqno completion, the events are too coarse i.e. one event may
signal the completion of a few seqno. We will need to sort the events to
properly compute the busy times.

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

overlay: Accumulate busy times

Still a little too course as we add multiple overlapping seqnos and
waits.

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

overlay: Couple wait begin/end events together to fix accounting

Since the events may be processed out of order (due to per-cpu
ringbuffers) we need to be careful to associated wait pairs in order to
compute the correct elapsed time.

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

overlay: Perf output redirection must be done after mmap()

Effectively you choose to redirect the output of an event instead of
mmapping it directly - but the target must already be mmapped.

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

overlay: Show per-process wait times

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

overlay: Graph per-process requests over time

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

overlay: Track requests per-process

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

overlay: Drop unused mmap/comm events

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

overlay: Constify perf ringbuffer

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

overlay: Fix wraparound handling of perf ringbuffer

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

overlay: Primitive integration with perf

We can now record when a pageflip occurs by listening for the flip
tracepoint. Merely proof of principle at this point.

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

overlay: Show GPU waits

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

Introduce intel-gpu-overlay

A realtime display of GPU activity. Note, this is just at the point of
minimum usability...

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

quick_dump: add is_haswell()

This was missed beforehand.

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

gem_concurrent_blit: Purge the child bufmgr's cache

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

drmtest: Squelch the rude interrupter

Stop printing the debugging information of how many times we emitted a
signal.

gem_concurrent_blit: Fix the leak from the children.

As the children use the parent's fd, the kernel only has a single filp
for everyone. Therefore we cannot rely on the process termination
reaping the buffers we allocate for the children.

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

gem_concurrent_blit: Share total num_buffers between all children

Apparently not all machines have more than 4GiB of RAM available.
Spoilsports.

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

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

   predicate SEND execsize dst sendleadreg payload directsrcoperand instoptions
   predicate SEND execsize dst sendleadreg payload imm32reg instoptions
   predicate SEND execsize dst sendleadreg payload sndopr imm32reg instoptions
   predicate SEND execsize dst sendleadreg payload exp directsrcoperand instoptions

The above four syntaxes are only used on legacy platforms which support implied move
from payload to dst.

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Reviewed-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

gem_ctx_basic: Fix misc warnings

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

intel_infoframes: Dump 3D_Ext_Data when using Side-by-side (half)

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

intel_infoframes: Add support for decoding HDMI VICs

The HDMI vendor infoframe can contain a HDMI VIC (as of HDMI 1.4, only
used for 4k formats).

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Simon Farnsworth <simon.farnsworth@onelan.co.uk>

tests/prime: test dma-buf llseek support

This exercises a proposed patch from Christopher James Halse Rogers.

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

lib/drmtest: Use igt_assert in the do_ioctl/do_or_die helpers

abort results in "crash" test results in the piglit runner, igt gives
us neater output.

The downside is that we can now only use these in igt tests, but
that's already the case (and given their placement in drmtest.h their
intended use-case, too).

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

tests/gem_ctx_basic: Use igt_require

... instead of threading the 77 return value through threads. Even
better would be to check for requirements in the single-threaded setup
part of the test, since now it's done by multiple threads in parallel
and will result in interleaved and multiplied output on stdout.
Whatever.

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

Add intel_framebuffer_dump

A simple utility to capture the currently active framebuffers and record
them as a png.

lib/drmtest: fix subtest output when skipping all tests

(null) isn't too nice ...

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

tests/prime_nv_api: fixup conversion goof-up

Oops.

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

tests/primve_nv_test: use igt_assert for checking

... and drop the int return argument of the tests.

Also:
- make nv_write_i915_cpu_mmap_read functional again by dropping the
  goto.
- add checks to i915_import_pread_pwrite

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

lib/drmtest: documentation for igt_fixture

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

lib/drmtest: add igt_subtest_name and use it in kms_flip

Useful when the testname is constructed with igt_subtest_f.

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