summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2015-09-08tests/gem_bad_reloc: use correct page table sizeDaniele Ceraolo Spurio
2 subparts of gem_bad_reloc check that the reloc address is below the global gtt boundary. However, when executing from ppgtt the reloc address can be greater than that and still be a valid address. To be sure that we're using the right upper limit, select it based on the ppgtt mode. Cc: Michel Thierry <michel.thierry@intel.com> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> Reviewed-by: Michel Thierry <michel.thierry@intel.com> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-09-08tests/gem_pwrite_snooped: disable const cast warningThomas Wood
Disable -Wcast-qual temporarily to allow memchr_inv to return non-const data (similar to memchr), without causing a compiler warning. Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-09-08tests: remove gem_storedw_loop_* testsThomas Wood
These have been replaced by subtests in gem_storedw_loop. Signed-off-by: Thomas Wood <thomas.wood@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2015-09-08build: fix unused-result warningsThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-09-08lib: various documentation fixesThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-09-08intel_reg: ensure "intel_reg help" always worksThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-09-08tools: install the register definition filesThomas Wood
Install the register definition files and use them by default in intel_reg. v2: remove redundant path check Suggested-by: Jani Nikula <jani.nikula@linux.intel.com> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-09-08tools: remove quick_dumpThomas Wood
Remove quick_dump as it has been replaced by the intel_reg tool and move the register definition files to tools/registers. Signed-off-by: Thomas Wood <thomas.wood@intel.com> Acked-by: Jani Nikula <jani.nikula@intel.com>
2015-09-08tests/gem_storedw_loop: remove redundant ppgtt checkThomas Wood
All tests require ppgtt, so checking for it later on has no effect. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-09-08tests/gem_storedw_loop: skip on gen6 bsdThomas Wood
MI_STORE_DATA is broken on gen6 bsd. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-09-08tools/null_state_gen: fix various compiler warningsThomas Wood
Add the debug and warning flags to cflags and fix the resulting issues. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-09-08tests/tools: add --as-needed to linker flagsThomas Wood
Add --as-needed to the linker flags to reduce the number of shared library dependencies, since not all the tests and tools use all the libraries required by the helper library (for example, many tests do not use cairo). This helps portability of the binaries and also makes a very small improvement to the execution speed and memory consumption. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-08-25lib/gpgpu_fill.c: fix file permissionsThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-08-25gem_storedw_loop: Skip test if device doesn't have requested ringAnder Conselvan de Oliveira
The VEBOX ring is not available in generations before Haswell, so make tests that use it skip instead of fail in previous gens. Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-08-25tests/gem_ctx_exec.c: only hang the ring we are testingTim Gore
In the reset-pin-leak test we were calling igt_set_stop_rings(STOP_RING_DEFAULTS) which sets the stop_rings bits for all gpu engines. But we only submit work to the render engine. When TDR is enabled (as it is in Android currently) only the render engine gets reset, which clears the stop_rings bit for the render engine but not for the other engines. This causes the test to fail on the second iteration because stop_rings is not clear. So just set the stop_rings bit for the engine we are going to hang, namely the render engine. Signed-off-by: Tim Gore <tim.gore@intel.com> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-08-25kms_frontbuffer_tracking: print the status when it's not what we expectPaulo Zanoni
Printing the status on an unexpected failure is always the first thing I do. So do this always. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-25kms_frontbuffer_tracking: add slow_draw() subtestPaulo Zanoni
It fails on my SKL machine. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-25kms_frontbuffer_tracking: wait for PSR to be HW Enabled & ActivePaulo Zanoni
Make sure PSR is fully there. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-25kms_frontbuffer_tracking: add options for setting the shared fb X/YPaulo Zanoni
I'm using this to debug some aspects of the GTT tracking. While at it, do a small rename and fix the ASCII art. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-21benchmarks/gem_exec_reloc: Allow profiling 0 relocsChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2015-08-21lib: fix distcheck issuesThomas Wood
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-08-21lib: add a single include headerThomas Wood
Add a header that includes all the headers for the library. This allows reorganisation of the library without affecting programs using it and also simplifies the headers that need to be included to use the library. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-08-21assembler: remove built sources with make cleanThomas Wood
Built sources are generated by "make all", so should be removed by "make clean". This also ensures "distcleancheck" passes. Signed-off-by: Thomas Wood <thomas.wood@intel.com>
2015-08-19tests/drv_hangman: Adjust to 64bit bb offsetsMika Kuoppala
commit e1f123257a1f7d3af36a31a0fb2d4c6f40039fed Author: Michel Thierry <michel.thierry@intel.com> Date: Wed Jul 29 17:23:56 2015 +0100 drm/i915: Expand error state's address width to 64b changed the batch buffer address to be 64b. Fix the parsing of gtt offset accordingly. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91638 Cc: Akash Goel <akash.goel@intel.com> Cc: Michel Thierry <michel.thierry@intel.com> Reviewed-by: Michel Thierry <michel.thierry@intel.com> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
2015-08-17igt: remove deprecated reg access tools in favor of intel_regJani Nikula
intel_iosf_sb_read, intel_iosf_sb_write, intel_reg_dumper, intel_reg_read, intel_reg_snapshot, intel_reg_write, intel_vga_read, and intel_vga_write have been deprecated in favor of intel_reg. Remove the deprecated tools. intel_reg does everything they do, and more. Signed-off-by: Jani Nikula <jani.nikula@intel.com>
2015-08-16overlay: Fix min frequency detection on BYT/BSWChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2015-08-14kms_frontbuffer_tracking: reduce the FBC wait timeout to 2sPaulo Zanoni
Just like we did for PSR, let's do it for FBC. FBC gets reenabled in just 50ms, so the 5000ms timeout is huge. On the other hand, we only pay the 5000ms timeout full price 9 times when running kms_frontbuffer_tracking --fbc-only, so this change shouldn't save too much time. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: reduce the PSR wait timeout to 2sPaulo Zanoni
PSR only takes up to 100ms to be enabled, but we were using 5000ms timeouts. The problem with PSR is that the MMAP_GTT tests have to assert that PSR is disabled and stays disabled during the whole timeout, so that 5s cost is a little to high when we consider that we do the full 5s wait 74 times when running kms_frontbuffer_tracking --psr-only. By reducing to 2s we still make sure we're safe, since we're waiting 20x the time PSR needs to get reenabled, and we reduce the runtime of running kms_frontbuffer_tracking --psr-only from 14m10s to 10m29s. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14benchmark/gem_exec_trace: Inline everythingChris Wilson
Avoid the globals and make the dispatch one huge function and hope GCC works some magic. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2015-08-14benchmark/gem_exec_tracer: Tweak to handle SNAChris Wilson
SNA starts by feeding in deliberately bad ioctls in order to detect the kernel interface versions. A quick solution is to always feed it to the ioctl and only record the trace if it is valid. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2015-08-14kms_frontbuffer_tracking: add subtests for rgb565 and rgb101010Paulo Zanoni
Make sure valid formats work properly and invalid formats keep the features disabled. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: fix psr-farfromfence assertionsPaulo Zanoni
We're using GTT draws and they disable PSR. This test is mostly for FBC anyway. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: split prepare_subtest() in two functionsPaulo Zanoni
And remove duplicated code by calling prepare_subtest_data() on rte_subtest(). Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: remove duplicated pairs on multidrawPaulo Zanoni
If we already tested the combination of m1+m2, don't test m2+m1 later: the drawing pattern used already has 4 squares, so we're already testing the "m2 followed by m1" case. This should reduce the test time from about 60s to about 30s. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: SKIP if not enough stolen is availablePaulo Zanoni
This is not an FBC bug. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: use all drawing methods on multidraw_subtestPaulo Zanoni
Instead of having a single pair of methods per subtest. Having this in pairs is not very useful since we end doing all the drawing methods per subtest anyway. This saves a few modesets, which makes eDP slightly faster - about 22s on my local machine. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: improve support for multiple pixel formatsPaulo Zanoni
We now have per-pixel-format FBs and CRC values. Also t->format to indicate the format we're using on the subtest. We still only use the default RGB888 format (AKA FORMAT_DEFAULT) for all tests since multiplying the current number of tests by the number of new pixel formats doesn't sound like a great idea. With this, it should be really easy to add new tests that use different pixel formats. Also, if you want a full IGT run on a specific pixel format, all you need to do is to change FORMAT_DEFAULT. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: add information about more formatsPaulo Zanoni
We're not using those formats yet, but a simple change to create_fb() allows us to use these formats without problems now. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: introduce enum colorPaulo Zanoni
We want to add support for different pixel formats and the current hardcoded pixel values won't work with that. So add enum color and its auxiliary functions so we can decide the pixel values based on the chosen color enum. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: don't pass the format to create_fb()Paulo Zanoni
It is create_fb() who is going to decide the buffer format based on the arguments provided by the caller. This is another step for the non-XRGB8888 support. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: add a wrapper for igt_create_fb()Paulo Zanoni
The wrapper will do a few additional things when we add support for formats different than XRGB8888. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14kms_frontbuffer_tracking: fix MMAP_WC assertions on PSRPaulo Zanoni
Now that the MMAP_WC operations call the dirty ioctl, PSR gets reenabled after some time. So we have to adjust op_disables_psr() to take that into account. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14lib/igt_draw: break if we already wrote every pixelPaulo Zanoni
Due to the nature of accessing a tiled buffer in an untiled way, we used to loop through the whole buffer all the time. Add a small mechanism to just break in case we know we already wrote every pixel we should have written. On kms_frontbuffer_tracknig/fbc-2p-primscrn-pri-shrfb-draw-pwrite (with a 3200x1800 primary screen and a 1920x1080 secondary screen), I could reduce the runtime from ~7.53s to ~6.01s. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14lib/igt_draw: fix pwrite_tiled()Paulo Zanoni
Don't forget to flush in case we're in the last iteration of the loop. This fixes failures of kms_frontbuffer_tracking when used with --use-small-modes on eDP monitors. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14lib/igt_draw: add support for RGB565 and XRGB2101010Paulo Zanoni
We need to test those pixel formats on the FBC code, so let's make sure the drawing library works on them first. v2: Update the gtkdoc (Daniel). Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14lib/igt_core: use print("%s", #expr) instead of print(#expr)Paulo Zanoni
If I have a program with the following: igt_skip_on(i % 2 == 0); igt_skip_on_f(i % 2 == 0, "i:%d\n", i); igt_require(i % 2 == 0); igt_require_f(i % 2 == 0, "i:%d\n", i); then I'll get compiler error messages complaining about format conversions related to the '%' character used in the mod operation. So put the whole string as a %s argument to avoid interpreting '%' and any other possible chars. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2015-08-14tests/kms_setmode: mark simple clone test as basicJesse Barnes
Should cover simple, single CRTC mode sets. Reviewed-by: Daniel Vetter <daniel@ffwll.ch> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
2015-08-14tests/kms_flip: add basic tests for flip, flip vs dpms, and flip modeset v2Jesse Barnes
Simple variants that don't do multiple output or interruptible testing. v2: add vblank variant, remove duplication (Daniel) Reviewed-by: Daniel Vetter <daniel@ffwll.ch> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
2015-08-14tests/pm_rpm: mark RTE and D3 tests as basicJesse Barnes
These always need to pass for basic PM functionality. Reviewed-by: Daniel Vetter <daniel@ffwll.ch> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
2015-08-14tests/pm_backlight: mark simple test as basicJesse Barnes
We should be able to adjust the backlight and observe changes in sysfs. Reviewed-by: Daniel Vetter <daniel@ffwll.ch> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-08 07:48:51 +0000