summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2014-01-29igt/gem_close_race: Fix blit commands for BDWHEADmasterChris Wilson1-15/+26
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73958 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-01-28iosf: make iosf_rw take a larger addrJesse Barnes1-25/+14
For higher DPIO ranges for example. Also fix it up to use intel_register_read/write. Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
2014-01-28dpio: make dpio reg read/write output like normal reg read/write outputJesse Barnes2-1/+3
Just for consistency. Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
2014-01-28mmio: drop unused i915_loaded fnJesse Barnes1-16/+0
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
2014-01-28mmio: use intel_iosf.c for DPIO reads and writesJesse Barnes5-134/+42
This makes it a bit more like the kernel, so we can go poke at DPIO and other IOSF regs a bit more easily. Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
2014-01-27tests/pm_rps: rename basic test to "basic-api"Daniel Vetter1-1/+1
Jeff plans to add more tests ... Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-27pm_rps: Make frequency logging more compactJeff McGee1-17/+18
Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-27pm_rps: Remove repeat sysfs readsJeff McGee1-47/+51
Storing values avoids some unnecessary overhead but more importantly allows all of our processing to be atomic. Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-27pm_rps: Expand on min and max config testingJeff McGee1-29/+77
Add a function that methodically varies min and max to exercise several valid and invalid combinations. Allow the caller to define what is to be checked between each step. Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-27NEWS: update with nomodeset register access changeJesse Barnes1-0/+3
2014-01-27quick_dump: more vlv dpio regsJesse Barnes1-0/+9
2014-01-27mmio: allow register init even if debugfs isn't available and i915 isn't loadedJesse Barnes1-4/+1
This lets us dump regs even if modeset=0 for example. Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
2014-01-26igt/gem_concurrent_blit: Scale resource usage to RAM correctlyChris Wilson1-16/+33
Note that we use twice the number of buffers, and so we need to restrict num_buffers appropriately to fit within RAM. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72255 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-01-23intel_error_decode: Protect against missing ring registersChris Wilson1-5/+4
A dodgy kernel may miss printing out the ring registers leading to a FPE. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-01-23tools: Allow building on Android after noinst_PROGRAMS is not defined any moreTvrtko Ursulin1-1/+1
Looks like filter-out macro gets silently unhappy about an undefined variable. Reviewed-by: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2014-01-22tests/gem_reset_stats: stop only one ring when submitting hangMika Kuoppala1-3/+20
If we stop all the rings, we can end up blaming the innocent rings on hangcheck. Reference: https://bugs.freedesktop.org/show_bug.cgi?id=73652 Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-21gem_storedw_batches_loop: Fix for pre-BDWBen Widawsky1-1/+2
My git failures are truly remarkable. I ended up pushing the wrong commit here: commit 1552aa21124cabe762862bb414490510415a2b2d Author: Ben Widawsky <benjamin.widawsky@intel.com> Date: Mon Jan 13 06:28:45 2014 -0800 gem_storedw_batches_loop: Fix for BDW This puts the offset of the reloc in the wrong place for pre-BDW Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73866 Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2014-01-20kms_flip: Try to consume all fences after a flipChris Wilson1-2/+54
As a flip is outstanding, there is an issue that the kernel may not be able to release one of the fences that userspace requires and erroneous report EDEADLK (on gen2, gen3). Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73696
2014-01-20quick_dump/watermarks: lp3, misc and debugBen Widawsky1-1/+5
I forgot to git add this originally. Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2014-01-20quick_dump/broadwell: Use haswell_otherBen Widawsky1-0/+1
For now, the power controls and watermarks seem to be the same offsets. So just reuse haswell_other.txt Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2014-01-20quick_dump/hsw: Watermark debugBen Widawsky1-0/+12
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2014-01-20quick_dump: Allow comments in the register filesBen Widawsky1-0/+14
Simple addition to the parser to add the following full line comments: {';', '#', "//"} Empty lines will also be ignored Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2014-01-20quick_dump: Fix the indentationBen Widawsky2-67/+70
Fix the spaces to use [the python standard] 4 soft spaces for tabe. While here, add the proper vim tag so we don't do it again. Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2014-01-20.gitignore: Add pycacheBen Widawsky1-1/+1
quick_dump which is python, generates files in __pycache__ which are the moral equivalent of object files. Don't let people add them to the index accidentally. Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2014-01-19tests/gem_fd_exhaustion: Make it workDaniel Vetter1-14/+22
- We need to drop root to actually hit the limits. This requires us to fork the actual test since otherwise the exit handlers (which require root) fail the entire test. - Don't assert that the gem create ioctl succeeds, it won't on the final run of the loop. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-19lib: add __gem_createDaniel Vetter2-0/+16
Useful for when a gem_create ioctl is expected to fail. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-19tests/pm_rps: Add missing staticDaniel Vetter1-1/+1
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-19debugger: avoid global variable shadowingDaniel Vetter1-8/+8
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-19lib: extract igt_drop_rootDaniel Vetter3-13/+16
Useful in other tests. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-19tests: Add gem_fd_exhaustionDaniel Vetter3-0/+85
Exhausts the system limit on open files and then tries to create a new shmem-backed gem object. Linus Torvalds reported that this blows up on a null obj->base.filp, but I can't reproduce this here: http://lists.freedesktop.org/archives/intel-gfx/2014-January/038433.html Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-19tests: Move pm_rps to the right Makefile targetDaniel Vetter1-1/+1
If it's not in the multi-test target group testrunners won't pick up on the fact that they need to enumerate subtests first. Cc: jeff.mcgee@intel.com Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-18pm_rps: Fix verbose option and streamline its useJeff McGee1-10/+42
Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-18pm_rps: Convert to subtest structureJeff McGee1-57/+62
Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-18pm_rps: Add read back checking on sysfs writesJeff McGee1-1/+6
Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-17tools: Install them allDaniel Vetter1-3/+1
No reason really not too, especially since we install manpages for some of them. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=66656 Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-15tests/gem_gtt_hog: Use igt_simple_mainDaniel Vetter1-4/+1
Results in the compiler complaining about wrong exits and return values. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-15tests/primve_nv_pcopy: Comment out unused check_swizzle functionDaniel Vetter1-1/+1
I've botched this in my patch to disable the swizzled pcopy test. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-15gem_gtt_hog: use a normal exit()Chris Wilson1-1/+1
Since we don't execute any subtests, using igt_exit leads to inconsistent behaviour. In the future, this may be converted. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73641 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-01-14igt: tests/Android.mk: fix LOCAL_PATH usageRobert Beckett1-3/+4
Fix usage of shared variable LOCAL_PATH in deferred variable expansion area. In Makefile language, rule and dependency definitions use immediate expansions of variables, so they get expanded as soon as the rule is created (1st pass). Rule implementation (a.k.a recipe) use deferred expansion (2nd pass). Android effectively makes all Android.mk files a single makefile by including them all in a big tree from the toplevel makefile. The rules are all evaluated in the first pass and targets are generated. Then the 2nd pass happens and the required target's recipes are run. At this point, LOCAL_PATH has been assigned the value from the last evaluated Android.mk in the 1st phase that defined LOCAL_PATH (most Android.mk use this variable). In my particular case, it was the bootloader's Android.mk that was evaluated last and had defined LOCAL_PATH to it's path. The errors are rather misleading due to it looking like a bug in another module's Android.mk rather than this one :) Basically, if you want to use a variable that any other Android.mk defines, then you can only use it in an immediate expansion context, not a deferred expansion context as it will likely be re-defined by the time the 2nd pass happens. This patch stores it in a unique variable that should not be being used by other Android.mk files. An alternative fix would be to use $@ and $< as the files in question are target and dependency, but I never like using those as they can easily break if dependencies are added etc. I prefer variable to be explicitly named to make them obvious. See gnu make manual for explanation of deferred vs immediate expansion of variables : http://www.gnu.org/software/make/manual/make.html#Reading-Makefiles Reviewed-by: Damien Lespiau <damien.lespiau@intel.com> Signed-off-by: Robert Beckett <robert.beckett@intel.com>
2014-01-14tests/gem_reset_stats: run tests on all ringsMika Kuoppala1-42/+90
Run all relevant tests on all rings. Suggested-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-14tests/gem_reset_stats: split ctx testsMika Kuoppala1-30/+108
Split context related tests from non-context ones and cleanup the naming. Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-14tests/prime_nv_pcopy: disable swizzle testsDaniel Vetter1-1/+3
It's pure luck that nv can detile some of the intel layouts since one of the video MC formats matches it. Since we can't possible fix this comment the test out. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73538 Acked-by: Maarten Lankhorst <bugs@mblankhorst.nl> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-13gem_storedw_batches_loop: Fix for BDWBen Widawsky1-3/+2
Existing code was trying to be too clever and wasn't properly emitting the high dword, or the correct length. Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
2014-01-13tests: Drop glib.h include from kms_cursor_crc and kms_fbc_crcVille Syrjälä2-4/+1
kms_cursor_crc and kms_fbc_crc don't need glib.h. This was just some copy-paste error on my part. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2014-01-10pm_rps: Use igt exit handler for restoreJeff McGee1-19/+23
Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-10pm_rps: Fix test to target original min and maxJeff McGee1-4/+9
The goal of the test is to confirm that gt_cur_freq_mhz can be forced to the boundaries of the frequency range by collapsing gt_min_freq_mhz and gt_max_freq_mhz to the target value. But we miss testing the upper end of the range by targetting the current value of max after it has been set equal to min. So fix by targetting orginal max instead of current max. This correction exposes a problem in setfreq where min is always set to target before max, which should fail if the target value is greater than max. So fix that too. Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-10pm_rps: Assert that valid sysfs writes return successJeff McGee1-8/+15
do_writeval now always checks the return value, whether we expect success or a specific error. Also add new macro writeval_inval to simplify repeated use of do_writeval to test for EINVAL return code. Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-10pm_rps: Use unbuffered I/O on sysfs filesJeff McGee1-5/+2
Bionic C library may not re-read a buffered, read-only file which results in failure to monitor changes in gt_cur_freq_mhz. Signed-off-by: Jeff McGee <jeff.mcgee@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-01-10tests/gem_pipe_control_store_loop: Delete bogus assertionChris Wilson1-3/+1
target_bo->offset was just being used to verify that the batch submission worked and fortunately was not being relied upon for any subsequent conditions. However, address 0 is valid and so the assertion itself was bogus as it is possible (almost assured with full-ppgtt) for the target_bo to be located at address 0. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72984 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2014-01-10gem_gtt_hog: Print elapsed time in msChris Wilson1-2/+2
Missed one last diff before pushing
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-15 23:37:48 +0000