chipset updates

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

test: fix up hangman script and enable by default

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

tests: add gem_unfence_active_buffers

Unfortunately this requires slab poisoning to catch anything :(

Also add a new helper to drmtest to get the available fence count.

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

lib: Fix render copy on gen2

tests/gem_ringfill: add progress indicator

This thing takes ages on older chips.

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

tests: re-enable commented-out testcase

Meh, I suck.

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

tests: add gem_set_tiling_vs_pwrite

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

tests/gem_set_tiling_vs_gtt: update copyright header

Meh, I've forgotten these.

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

tests: add gem_set_tiling_vs_gtt

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

tests/gem_set_tiling_vs_blt: fix on gen4+

Oops, the new checks need correctly tiled blts to work.

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

tests: rename gem_tiled_after_untiled_blt into gem_set_tiling_vs_blt

... it checks much more now.

Also update the comments a bit in the testcase.

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

tests: pimp tiled_after_untiled_blt

Now with more tiling transition tests. Changes:
- refactoring to reuse the tiling transition functions.
- add a tiled->untiled transition
- add a tiled->tiled transition with different stride
- add a check at the end to ensure that the tiling has indeed changed
  on the test_bo

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

Add tools/intel_infoframes

This is a command-line tool that allows us to display and modify the
InfoFrames we send.

v2: use argv instead of stdin

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

tests: skip rc6_residency test if not supported

Fixes spurious failures.

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

Use mmap64() for performing the GTT mapping

Although the address space for GTT mappings may only be 32-bits, we need
to use the explicit 64-bit mmap interface so that on a 32-bit platform
the offset we pass is not truncated to 31-bits.

Fixes gem_mmap_offset_exhaustion on 32-bit platforms.

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

tests: add gem_gtt_concurrent_blit

Same test as Chris Wilson's gem_cpu_concurrent_blit, but for
gtt mmaps.

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

tests: add sysfs_rc6_residency to .gitignore

tests/sysfs_rc6_residency: add some sleep before testing

That way I can run i-g-t on my ivb without spurious failures.

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

tests: rc6 residency test

This is meant to test the sysfs entry for showing rc6 residency in
milliseconds. Remember, sysfs is a permanent interface.

v2: use new get_card interface to try "all" devices
check rc6p and rc6pp in addition to rc6

v3: rename rc6_residency.c to sysfs_rc6_residency.c
print better error messages
skip test if rc6 isn't enabled

v4: update to new sysfs names

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Ben Widawsky <benjamin.widawsky@intel.com>

drm/i915: extract card getting

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

build: make sure we have asprintf

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

test/gem_cpu_concurrent_blit: Limit memory usage to half aperture

So that we don't cause needless thrashing on older devices and spoil the
test.

test: Exercise concurrent GPU read/write with CPU domain access

Designed to exercise this patch to i915.ko:

diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index fbf1118..57ae1f2 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drivers/gpu/drm/i915/i915_gem.c
@@ -3181,9 +3181,11 @@ i915_gem_object_set_to_cpu_domain(struct drm_i915_gem_obj
        if (ret)
                return ret;

-       ret = i915_gem_object_wait_rendering(obj);
-       if (ret)
-               return ret;
+       if (write || obj->pending_gpu_write) {
+               ret = i915_gem_object_wait_rendering(obj);
+               if (ret)
+                       return ret;
+       }

        i915_gem_object_flush_gtt_write_domain(obj);

By exercising the conditions whereby should either of the checks be missed
an error is detected.

tests: explicit use bash

We use bashisms, which debian's default sh dash doesn't like.

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

tests: add wedging tests

Used to check that i915_error_state works. Unfortunately this
kills the gpu because wedging is permanent.

Base on a patch by Ben Widawsky.

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

tests: fixup drm_lib.sh sourcing

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

tests: use / as path separator instead of _

Oops. Originally by me, noticed while reviewing Ben Widawsky's patch.

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

tests: use common code for scripts

This removes cut and pasted code and uses a more central source.

Signed-off-by: Ben Widawsky <benjamin.widawsky@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Make the assertions guarding syscalls (drmIoctl in particular) verbose

Currently all we see is gem_read: ret == 0 failed, where it would help
to see the errno and/or the ret.

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

tests: use gem_create and gem_close more

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

tests/gem_fence_thrash: make some good use of drmtest helpers

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

tests/gem_exec_faulting_reloc: kill superflous mmap_bo

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

tests: add gem_tiled_after_untiled_blt

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

tests: adapt storedw tests to ppgtt

MI_MEM_VIRTUAL actually means use global gtt now, not setting the bit
means use ppgtt. On previous gens, not setting the bits ment 'use
physical memory'. So what, the usual confusion.

Note that for some odd reasong this is broken on gen6, but only on the
bsd ring. Unexpected.

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

tests: don't just bail out when there's not enough RAM

... but actually run the test with reduced memory requirements,
as the messages claims. Additional print it to stdout, stderr
seems to imply FAIL in our QA's testing rig.

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

tests: add gem_mmap_offset_exhaustion

This also adds a gem_madvise helper to lib/drmtest.c

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

update .gitingore

intel_sprite_on: kill overtly verbose and totally content-free comments

/me just hates fluff

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

intel_sprite_on: remove copy&pasted gem functions

Can we just please stop this?

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

intel_gpu_top: Trust the register access is safe

Fixes intel_gpu_top on gen3 which otherwise refuses to do mmio.

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

decode: Use the correct start mask for gen3 fence registers

A cut'n'paste error from gen2 apparently.

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

configure: Add AC_SYS_LARGEFILE for mmap64 support on 64-bit

Without enabling largefile support the prototype for mmap64() is broken
on Linux/x86_64 with the result being a 32-bit integer sign-extended
to fill a 64-bit pointer. Hilarity ensues.

intel_reg_read: add a flag to simplify bit decoding

This allows to specify '-d' parameter which will decode individual bits in
each register being read.

The register bits are printed horizontally for space reasons. This
requires more than 80x25 terminal to see them all. An alternative solution
would be to print them vertically, but this will become much more
difficult to read when printing multiple registers at the same time.

v2: fix spacing to get us a bit closer to the code nirvana.

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

intel_reg_read: support reading multiple registers

The registers must be passed on the command line and will be read
sequentially, one at a time.

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

intel_reg_read: add support for getopt

This will allow us to pass more options to it in the future.

v2: fix whitespacing issues and improve scary warning text as suggested by
Paul Menzel.

Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>

tests: some more tuning on the tiled partial pwrite/pread test

... 8 pages is just not enough to somewhat reliably get one page that
needs bit17 swizzling.

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

tests/gem_tiled_partial_pread/write: fixes

Astonishing how dense I sometimes am ...
- increase the bo size so that we don't have any round-up to next tile
  size areas (which the 2d blit go over, but the 1d pread/pwrite calls
  will notice).
- correctly set tiling bits when copying back to a linear buffer.
- read back through a tiled bo to avoid messing with swizzling.

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

tests: add tiled partial pwrite/pread test

And indeed, this blows up.

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

lib: Downgrade error for unknown 'swap' not 'ram'

Oops, my mistake for copying a compile fix from another machine...

If we can't detect how much RAM we have, 0 is not a suitable default, so
keep the error for the time being and only downgrade the error for
unknown swap.

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

lib: Downgrade error for failing to determine swap interface to a warning

If we can't detect how much swap is available, presume none.

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

testdisplay: remove unnecessary passing of width, height, stride, depth

These are globals, no need to shadow them everywhere.

testdisplay: remove plane test support

This is better done in intel_sprite_on and was broken anyway.

demos/intel_sprite_on: build warning fixes

... from Armin Reese.

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

tests/gem_ringfill: shut up the compiler

Double-include ftl and local variable shadowing. While fixing the
later I've noticed that we mix up width and height in the blt copy
function.

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

tests/gem_ringfill: Exercise all rings

On SandyBridge, the BLT commands were split from the RENDER commands as
well as the BSD split inherited from Ironlake. So we need to make sure
we do exercise each ring, and in order to do so we also need to make
sure each batch takes longer to execute than it takes for us to
submit it.

v2: Exercise each ring sequentially.

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

gem_ringfill: Make this actually test that all the batches executed.

I was looking into some strange behavior in Mesa that looks like
batches maybe being skipped, but this test didn't catch it.

intel_reg_dumper: Add dumping of GPU turbo regs.

I was interested in finding why my IVB system is not getting GPU turbo
after suspend/resume.  The piece that looks weird to me is that
INTERRUPT_THRESHOLD is sitting at 0, whereas pre-suspend it's
0x12000000.

reg_dumper: Refactor duplicated reg-dumping code.

instdone: Fix fallthroughs to invalid instdone bits on newer hardware.

instdone: Add support for one of the IVB instdone regs.

[danvet: added GAM and GAFM bits.]

tests/intel_reg_dumper: s/pipe/disp_pipe/ to shut off the compiler

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

Add intel_panel_fitter tool

The tool allows you to change the panel fitter settings, so you can
change the size of the screen being displayed on your monitor without
changing the real pixel size of your desktop. The biggest use case for
this tool is to work around overscan done by TVs and some monitors in
interlaced mode.

v2: reviews by Ben, Chris and Rodrigo
 - don't install
 - use intel_register_access_init
 - check for maximum X and Y values
 - add a disclaimer saying this is not the real solution
 - print less when pf is disabled and option '-l' is used

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

reg_dumper: change and fix behavior when using dump files

Before this patch, handling dump files was wrong:
- when HAS_PCH_SPLIT was specified, intel_reg_dumper segfaulted inside
  intel_check_pch()
- the "devid" variable was used but not set
- there was no way to specify the device id of the machine used to
  generate the dump file

This patch fixes this behavior with the following changes:
- the HAS_PCH_SPLIT variable is gone
- there is now a '-d' argument that can be used to specify the device id
  used to interpret the results
- when a dump file is used but the '-d' argument is not provided, an
  Ironlake machine is assumed

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

reg_dumper: add interlaced information to TRANSCONF

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

reg_dumper: add interlaced and rotation information to PIPECONF

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

tests: fixup storedw tests

We need to use _INSTRUCTION as the reloc domain because otherwise the
ppgtt pipe_control w/a for snb won't kick in and the test fails.

The storedw tests for blt and bsd are still disabled because the
corresponding patch to flag ppgtt support isn't merged upstream yet.
Without ppgtt these hang my snb here.

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

test/gem_tiled_pread_pwrite: add progress indicator

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

remove old .c files

Oopps, I've forgotten about this in

commit 9b32894937af27f9ba95ea572ac857d376fe9034
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Sat Feb 11 16:52:26 2012 +0100

    prepend 'intel_' to installed programms

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

lib/drmtest: fixup copy&paste printf format issue

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

prepend 'intel_' to installed programms

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

Remove pointless Android ifdefs for sys/fcntl.h.

On my system, sys/fcntl.h contains exactly one line:

   #include <fcntl.h>

So there's really no need to #ifdef it.  Also, intel_mmio.c already
included <fcntl.h>; there's no need to include it twice.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Added support for Android

 Added the Android.mk file as per Android make system. Also had to modify
 the headers with the proper location for fcntl.h as per the Android
 bionic headers location

Signed-off-by: Sateesh Kavuri <sateesh.kavuri@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Release 1.2

lib/rendercopy: fixup make distcheck

And complete the gem_stress->rendercopy rename that I've forgotten
about.

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

Fix program name in intel_bios_dumper.man SYNOPSIS

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Update generation checks to provide basic support for Ivybridge.

There may be some updates required, but assuming Ivybridge is similar to
Sandybridge is a decent start; previously it fell through to the Gen2/3
case and nothing worked.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

add sprite demo from Armin Reese

Also fixed up the copyright header a bit. No comments on the coding
styled used ;-)

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

Move free(cmd) to after last use of cmd in intel_gpu_top

Error: Use after free (CWE 416)
   Use after free of pointer 'cmd' in call to fprintf
        at line 496 of tools/intel_gpu_top.c in function 'main'.
          Previously freed at line 491 with free.
[ This bug was found by the Parfait 0.4.2 bug checking tool.
  For more information see http://labs.oracle.com/projects/parfait/ ]

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

tests/testdisplay.c: rewite function dump_connectors and dump_crtcs.

Replace the globe variable drm_fd with local variable passed from parameter.

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

tests/testdisplay.c: Fix the black screen issue while force mode.

Replace the 8 globe force mode variables with a struct drmModeInfo variable.
Next step, we could reduce the number of globe veriables, to be convenient to call the functions.

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

gem_partial_pwrite_pread: add progress indicator

Without optimized pread/pwrite this test takes forever.

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

intel_reg_dumper: add TRANS_VSYNCSHIFT

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

intel_reg_write: use register access init/fini

This will do the proper forcewake stuff.

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

intel_reg_read: use register access init/fini

This will do the proper forcewake stuff.

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

mmio: refactor reg init/fini

Rearrange register access init and fini so that we properly accomodate
platforms without forcewake.

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

i-g-t: silence -Winline

In this case, the code already has a comment about the problem. We don't
need to flood the build with this less than useful message.

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

intel_bios_reader: Sanitize input to ensure all data blocks are within bounds

Running intel_bios_reader upon itself causes the reader to crash and
burn. It obviously finds a VBT signature inside the binary, but then
does not rigorously check that all data blocks are valid before
dereferencing them.

Reported-by: Emanuel Bronshtein
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=45205
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

drmtest: add progress indicator

Some test that trash the aperture necessarily need to take forever, so
add a little progress indicator to keep worried minds at peace.

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

tests: add gem_cs_prefetch

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

intel_error_decode: fixup glibc free warning

Usually some random stack garbage doesn't equal some other
random stack garbage, leading to the filename != path check
succeeding.

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

Add Solaris implementation of intel_get_total_swap_mb()

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

Make benchmarks also link against libpciaccess

Fixes Solaris build error on build of intel_upload_blit_large:

Undefined first referenced
 symbol       in file
pci_device_probe                    ../lib/.libs/libintel_tools.a(intel_pci.o)  (symbol belongs to implicit dependency libpciaccess.so.0)
pci_system_init                     ../lib/.libs/libintel_tools.a(intel_pci.o)  (symbol belongs to implicit dependency libpciaccess.so.0)
pci_device_find_by_slot             ../lib/.libs/libintel_tools.a(intel_pci.o)  (symbol belongs to implicit dependency libpciaccess.so.0)
ld: fatal: symbol referencing errors. No output written to intel_upload_blit_large

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

testdisplay: make udev optional

Hopefully this makes Solaris happy.

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

testdisplay: extract hotplug code

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

lib: forcewake is gen6+

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

tests/gem_tiled_swapping: be more conservative with the swap requirements

Blew up and angered the OOM killer on one of my machines.

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

tests: add gem_tiled_swapping

Also add a drmtest_exchange_int helper, might come handy at other
places.

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

lib: extract drmtest_permute_array

Lots of tests need to create havoc to LRUs in the kernel or otherwise
need to shuffle things around a bit. So make a small array permutation
function available.

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

lib: add intel_get_total_swap_mb

Test that try to exercise the swap paths need to check whether swap is
available, otherwise they'll just oom.

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

Revert "tests/gem_tiled_pread_pwrite: actually try to use more than total ram"

This test is actually not really designed to test swap.

This reverts commit 6d755c39d38e9f0e5a991933cc82e015d1ccd272.

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