summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2016-01-27igt/gem_concurrent_blit: Tidy blt-fill using gem wrappersChris Wilson1-17/+7
Replace the open-coded ioctls with the thin gem wrappers. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-26tests/Android.mk: Make intel_residency CAIRO dependantDerek Morton1-0/+1
intel_residency has a cairo dependency through igt_fb.c. Remove it if ANDROID_HAS_CAIRO is not defined. Signed-off-by: Derek Morton <derek.j.morton@intel.com> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2016-01-26lib/igt_kms, tests/testdisplay: allow probing of new connector modesJesse Barnes5-14/+67
Fixup some fallout from the connector probing changes so testdisplay -m will pick up newly hotplugged displays correctly. Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org.
2016-01-26igt/gem_exec_alignment: Reduce GTT usage if !full-ppgttChris Wilson1-0/+2
If we have to share the GTT with others, we cannot rely on being able to fill it and have to factor in some slack for others. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-26igt/gem_exec_alignment: Actually check aligned locations after manyChris Wilson1-1/+3
As well as ensuring the kernel doesn't simply crash when asked to do lots of objects, check it actually aligns them. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-25igt: Disable igt_clflush_range() implementation on ARM buildsEric Anholt1-0/+4
Daniel has suggested that I put vc4 testing into igt, since it's got the piglit integration and KMS coverage already. This gets the ccore building so that I can start writing tests. Signed-off-by: Eric Anholt <eric@anholt.net> Reviewed-by: Daniel Stone <daniels@collabora.com>
2016-01-25build: Disable x86-specific utilities on non-x86Daniel Stone2-12/+35
Some bits can't be built on non-x86 architectures, mostly because they require x86-specific assembly primitives. Disable these by default on non-x86 architectures. Signed-off-by: Daniel Stone <daniels@collabora.com> Reviewed-by: Eric Anholt <eric@anholt.net>
2016-01-25lib/ioctl_wrappers: Add gem_has_softpinMichał Winiarski3-21/+31
We can move it from softpin test into lib, and since softpin support is highly unlikely to go away in-between getparam ioctl calls, let's just do a single call and store the value. v2: rebase Signed-off-by: Michał Winiarski <michal.winiarski@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-01-25lib/ioctl_wrappers: Add gem_gtt_type exposing raw HAS_ALIASING_PPGTT paramMichał Winiarski12-62/+52
No functional changes. While I'm here, let's also rename gem_uses_aliasing_ppgtt (since it's being used to indicate if we are using ANY kind of ppgtt) and introduce gem_uses_full_ppgtt to drop some unnecessary code from tests that were previously calling getparam directly instead of using ioctl wrapper. v2: drop gem_uses_full_48b_ppgtt since it's no longer used anywhere, s/48b/64b (Chris) v3: rebase Cc: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Michał Winiarski <michal.winiarski@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-01-25Promote eviction memory sizes and buffer counts to uint64_tChris Wilson4-31/+35
References: https://bugs.freedesktop.org/show_bug.cgi?id=93849 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-25tools: add intel_residencyPaulo Zanoni3-0/+710
After the recent discussions regarding the effects of the vblank disabling policies on PC state residencies, I started running some experiments to reevaluate some non-intuitive conclusions I had reached. In order to help me do this, I decided to write this tool. The idea is very simple: the tool puts the system on an screen-on idle state, checks which PC state residency is the deepest we can reach, measures its residency, then does some not-so-idle tests and measures the residencies. You can use the tool to compare different Kernel trees and you can also use the tool to compare enabled vs disabled features. It's obvious that these cases do not represent real-world use cases of our driver, but they are already enough to highlight differences between the many patches I wrote. I was even able to catch a bug in one of my patches by spotting an unexpected regression in the residencies. I've been using this tool for FBC, but I expect it to also be useful for PSR, DRRS and similar features. I've been measuring the effects of different optimizations I wrote, and I've also been measuring the FBC vs no-FBC cases. It is also important to highlight that if your system is not properly configured for efficient power savings the tool may not be able to show differences between the results. On my Broadwell machine, for example, if I don't run "powertop --auto-tune" before running the tool, I get PC2 as the deepest state, and 90%+ residency for every workload. After properly configuring the machine, I get PC7 as the deepest state, which is the expected. So far I only tested this tool on BDW and SKL, and it may hit some unexpected assertions for older platforms. I only implemented the cases that are immediately useful for me, but we may also expand the tool in the future. We can add more important workloads. We can add support for screen-off cases, so we can compare the effects of runtime PM and other screen-off features. There's a lot we can do, but none of this is on my current priority list. And remember: /usr/bin/paste is your friend when comparing results. v2: - Be more idle at setup_idle(). - Improve printing for /usr/bin/paste usage. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
2016-01-24igt/gem_softpin: Keep last_handle around to vary hole generationChris Wilson1-1/+5
If we don't close the handle from the last pass, we don't free up the previous pass's vma immediately, changing the hole allocation. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-23igt/gem_softpin: Exercise snoop+uncached abuttingChris Wilson1-7/+66
snooped objects are not allowed to abutt uncached objects on older gen (!llc and global GTT) or else the GPU may hang if it prefetches across a page boundary into a different memory type (i.e. CS reading from snoop). The kernel should be checking the alignment rules as normal. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-23igt/gem_concurrent_blit: Switch to a shared mmap for userptrChris Wilson1-5/+8
If we use a MAP_SHARED mmaping for the our backing storage for userptr, then it will be inherited across the fork with the same address. ideal for continuity testing of children. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-23igt/gem_concurrent_blit: Allocate a private batch cache for the childChris Wilson1-1/+13
We have to avoid the COW alias for the intel_bufmgr and intel_batch cache as the child may close the object (in its local cache) leaving an alias in the parent cache pointing to a stale object. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt/gem_concurrent_blit: Disable userptr+child testsChris Wilson1-1/+2
The issue here is that the pointer inherited upon the child is copied-on-write, i.e. the pointer is private to each process, but the handle is shared. This means that writes and reads in the child are going to a different set of pages than the GPU's object - the test is simply broken. To overcome this we would need to mmap the shared buffer into the child. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt/gem_concurrent_blit: Check inheritance of buffersChris Wilson1-0/+14
The current forked modes recreate their handles in the children and just look at any complications arising from contention. This mode looks at inheriting the fd+handles from the parent into the child and seeing if we can use them within the child. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt/gem_ringfill: Mark the write object as EXEC_OBJECT_WRITEChris Wilson1-0/+1
After setting the flag for NORELOC (to avoid having to pay the cost of validating the relocations on every pass), we need to make sure that we set EXEC_OBJECT_WRITE so that we do track the outstanding writes. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt/gem_ringfill: Disable MI_STORE_DATA_IMM on BSD/gen6Chris Wilson1-0/+3
It's broken, avoid at all costs. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt/gem_userptr_blits: Exercise applying relocations to a userptr boChris Wilson1-0/+42
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22lib: Refactor common detection of missed interruptsChris Wilson5-73/+42
As we have the same function in a few places to read the debugfs/i915_ring_missed_irq file, move it to the core. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt/gem_softpin: Fix MI_STORE_DATA_IMM for gen3Chris Wilson1-3/+6
We need both a secure batch and to flag it to use the virtual GTT address. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt: Fix use MI_STORE_DATA_IMM on gen3Chris Wilson2-4/+13
For the older gen, MI_STORE_DATA_IMM is a privileged command so we need to set the "secure" batch flag, and we also need to instruct the command to use the GTT virtual address. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22lib: Print memory requirements as MiBChris Wilson2-8/+9
Since we need a lot of memory, trim off the less significant digits for easier human consumption. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt/gem_ringfill: Reduce ringfill to just filling the ringsChris Wilson1-184/+148
The objective of this test is to check how the driver handles a full ring. To that end we need only submit enough work to fill the ring by submitting work faster than the GPU can execute it. If we are more careful in our batch construction, we can feed them much faster and achieve the same results much quicker. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt/gem_exec_alignment: Tweaks count for large alignmentsChris Wilson1-2/+3
We can fit a few more objects in at high alignment, so do so. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-22igt/gem_cs_tlb: Use softpin to remove GTT layout assumptionsChris Wilson1-79/+66
With softpin we can explicitly manage the layout of the objects to be executed, deliberately forcing the reuse of active pages in an attempt to spot misbehaviour in the CS TLBs. Being explicit allows us to eliminate a lot of the CPU overhead between execbuf, hopefully increasing the likelihood of a conflict. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-21igt/gem_concurrent_blit: Add userptr backing storage testsChris Wilson1-1/+104
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-21igt/gem_softpin: Fix MI_STORE_DWORD_IMM for gen2-3Chris Wilson1-1/+4
Before gen4, MI_STORE_DWORD was just 3 dwords long (cmd, offset, value). Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-20lib: Expand igt_hang_ring() to select target context and various optionsChris Wilson4-13/+106
Some potential callers want to inject a hang into a particular context, some want to trigger an actual ban and others may or may not want to capture the associated error state. Expand the hang injection interface to suit all. v2: Disable the new kernel API, but push to provide a missing piece of infrastucture to unbreak compilation. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-20tests: Drop the superfluous igt_require_hang_ring()Chris Wilson4-40/+7
As the hang injection now itself checks for validity before use, the tests don't need to do so themselves. Except in certain situations! If the test forks, it should do requirement checks before the fork (so that we don't anger the igt gods) and if the test plays around i915.reset then it needs to do an early igt_require_hang_ring() that is not affected by the changes to i915.reset. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-20lib: Always double check igt_require_hang_ring() on useChris Wilson2-12/+46
If we move the igt_require() into the hang injector, this makes simple test cases even more convenient. More complex test cases can always do their own precursory check before settting up the test. However, this does embed the assumption that the first context we are called from is safe (i.e no i915.enable_hangcheck/i915.reset interferrence). v2: A couple of environment variables to skip hang testing or to force hang injection even if the GPU cannot be reset. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-20igt/gem_partial_pwrite_pread: Fix range computationChris Wilson1-41/+37
The range we chose to overwrite in the target had an off-by-one error that could cause it to compute a size that went past the end of the buffer (and so trigger EINVAL). Fortuituously with our seed this did not occur. Whilst changing the range calculation, update the error logging to include the range information. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-20igt/gem_softpin: Remove false dependencies on esoteric featuresChris Wilson1-989/+324
For softpinning, we do not require either userptr or extended ppgtt, so remove those requirements and make the tests work universally. (Certain ABI tests require large GTT, or per-process GTT.) In the process, make the tests more extensive - validate overlapping handling more careful, explicitly test no-relocation support, validate more ABI handling. And for fun, cause a kernel GPF. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-20igt/gem_exec_alignment: Convert to subtestsChris Wilson1-5/+7
Allow both parts (single, many) to be run independently. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19igt/gem_exec_alignment: Fix off-by-one in evalation of find_last_bit()Chris Wilson1-4/+4
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19igt/gem_exec_alignment: Remember to require 48b objects!Chris Wilson1-8/+18
In order to actually use the high space we need to set the can-use-48bit flag. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19igt/gem_exec_alignment: Test multiple objects with alignment as wellChris Wilson1-8/+104
Compute the largest alignment for the most number of objects we can create, then trying an execbuf with them. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19igt/gem_exec_alignment: Test all possible pot alignmentsChris Wilson1-1/+16
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19igt/gem_ring_sync_loop: Be explicit!Chris Wilson1-62/+72
The test just aims to execute batches on alternating rings with a write target such that every batch must be executed after the previous completes. This stresses the inter-ring synchronisation, which is interrupt driven if the gpu does not support semaphores, and so is a good stress tests for detecting "missed interrupt syndrome". Make that detection explicit. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19igt/gem_storedw_loop: Add a few more iterationsChris Wilson1-0/+5
Whilst still keeping the runtime down, extend the pipeline slightly. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19igt/gem_storedw_loop: Remove libdrm crutchesChris Wilson1-65/+84
Make the behaviour of the test more explicit wrt to the handle management, mmap and domain handling. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19igt/gem_cs_prefetch: Replace explicit sync with implicit syncChris Wilson1-68/+74
We can trade off the explicit sync (presumably to avoid some resource starvation issue?) with the implicit sync of having to perform a relocation. Using an implicit sync helps stress core kernel code, besides being much faster! Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19igt/gem_mmap_wc: Test cpu mmap vs wc mmap coherencyChris Wilson4-52/+88
Similar to the cpu mmap vs gtt mmap coherency test. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19gem_concurrent_all: Add a "quick" tiny passChris Wilson1-1/+10
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19gem_concurrent_blit: Add a pread/pwrite variant to only set part of the bufferChris Wilson1-7/+38
The idea is to check partial cacheline reads/writes. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-19gem_concurrent_blit: Don't call igt_require() outside of a subtest/fixtureChris Wilson3-38/+85
gem_concurrent_blit tries to ensure that it doesn't try and run a test that would grind the system to a halt, i.e. unexpectedly cause swap thrashing. It currently calls intel_require_memory(), but outside of the subtest (as the tests use fork, it cannot do requirement testing within the test children) - but intel_require_memory() calls igt_require() and triggers and abort. Wrapping that initial require within an igt_fixture() stops the abort(), but also prevents any further testing. This patch restructures the requirement checking to ordinary conditions, which though allowing the test to run, also prevents listing of subtests on machines which cannot handle them.
2016-01-17tests: Add basic gem_sync testChris Wilson2-0/+161
A very basic test of functionality, execute a nop and wait for it to complete. It should be very effective at stimulating the "missed interrupt syndrome" on all devices. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-16igt/gem_streaming_writes: Set the initial CPU write domainChris Wilson1-0/+2
Remove one assumption from the test and amek the domain management explict - when we write through the CPU to construction the batch, mark it as having been written. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-01-15tests: Add gem_busyChris Wilson2-0/+259
Exercise the busy-ioctl and verify it reports the right active engines using the execbuffer notation. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-17 23:58:24 +0000