summaryrefslogtreecommitdiff
path: root/benchmarks
AgeCommit message (Collapse)Author
2017-12-05meson: build a full dependency for lib_igt_perfDaniel Vetter
meson prefers packages dependencies over passing arount static libraries, because those also include linker flags, include dirs and everything else. While at it pull the special cases out from the common build stanzas like we do with other special cases. Just a bit of ocd to keep everything polished. v2: Don't forget to add perf_pmu to the test list (Petri). Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2017-11-29igt: Remove Android supportArkadiusz Hiler
This patch gets rid of the Android support, deleting all the hacks and moving code around to the places it belongs. Android build is not really maintained properly and rots rather fast. With recent push for Meson here and Android going for Soong it will only accelerate. It's a good time to drop the illusion of providing any support. Cc: Daniel Vetter <daniel.vetter@intel.com> Cc: Kalyan Kondapally <kalyan.kondapally@intel.com> Cc: Petri Latvala <petri.latvala@intel.com> Cc: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Acked-by: Petri Latvala <petri.latvala@intel.com>
2017-11-22gem_syslatency: Avoid arithmetic on void * warningTvrtko Ursulin
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
2017-11-22automake: include drm-uapi headers in EXTRA_DISTDaniel Vetter
This breaks make distcheck otherwise. Cc: Eric Anholt <eric@anholt.net> Cc: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> Reviewed-by: Petri Latvala <petri.latvala@intel.com> Tested-by: Petri Latvala <petri.latvala@intel.com>
2017-11-22gem_wsim: Busy stats balancersTvrtko Ursulin
Add busy and busy-avg balancers which make balancing decisions by looking at engine busyness via the i915 PMU. And thus are able to make decisions on the actual instantaneous load of the system, and not use metrics that lag behind by a batch or two. In doing so, each client should be able to greedily maximise their own usage of the system, leading to improved load balancing even in the face of other uncooperative clients. On the other hand, we are only using the instantaneous load without coupling in the predictive factor for dispatch and execution length. v2: * Commit text. (Chris Wilson) * Rename get_stats to get_pmu_stats. (Chris Wilson) * Fix PMU readout in VCS remap mode. v3: * Integrated Petri's meson build recipe. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Petri Latvala <petri.latvala@intel.com>
2017-11-21benchmarks/gem_exec_nop: fix engines selectionDmitry Rogozhkin
Code on loop() function shadowed function parameter which led to inability to try out different engines in the test: we always loaded RCS0. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103804 Signed-off-by: Dmitry Rogozhkin <dmitry.v.rogozhkin@intel.com> Cc: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
2017-11-21lib: Use drm-uapi/i915_drm.h instead of local defines.Eric Anholt
The MMAP_V2 is replaced by just using MMAP, since the official header has the updated struct. The gem_create_v2 and gem_get_aperture are left as is, because they seem to not be reflected in the UABI header! Signed-off-by: Eric Anholt <eric@anholt.net> Acked-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2017-10-12benchmark/gem_busy: Compare polling with syncobj_waitChris Wilson
v2: Hook the syncobj array to the execbuf! Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-10-03Fix rlim_cur compiler warnings when building on ARM.Eric Anholt
Signed-off-by: Eric Anholt <eric@anholt.net> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
2017-10-02meson: Distribute meson build system filesPetri Latvala
Signed-off-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2017-10-02benchmarks: Add ezbench.d files to distPetri Latvala
Signed-off-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2017-10-02benchmarks: Add wsim files to distPetri Latvala
Signed-off-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2017-09-29benchmarks: Actually build LIBDRM_INTEL_BENCHMARKSPetri Latvala
Adding LIBDRM_INTEL_BENCHMARKS into benchmarks_PROGRAMS had a tab before the assignment, making the line be inserted into the Makefile as is instead of being evaluated by automake. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2017-09-28benchmarks/gem_exec_fault: Update for tryhard kernels.Chris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Matthew Auld <matthew.auld@intel.com> Reviewed-by: Matthew Auld <matthew.auld@intel.com>
2017-09-28benchmarks/gem_syslatency: Apply vmpressure, measure page allocationChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
2017-09-08meson: basic build system supportDaniel Vetter
Why? Because it's fast. Like really, really fast. Some data (from a snb laptop, so rather lower-powered): - Incremental build after $ touch lib/igt_core.c with meson: 0.6s It notices that the symbol list of the libigt.so hasn't changed and doesn't bother re-linking the almost 300 binaries we have. make -j 6 for the same scenario takes 44s. - Incremental build with nothing changed: make: 0.7s, meson: 0.2s This means stuff like --disable-git-hash is entirely pointless with meson, it's faster than a make ever can be (with 0.6s). - Reconfigure stage: ninja reconfigure 0.8s vs. ./configure 8.6s) - Running tests, after a full build: ninja test 6s vs. make check 24s - Full build (i.e. including ./autogen.sh respectively meson build), including tests, from a pristine git checkout. automake 2m49s vs. meson 44s. Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: Eric Anholt <eric@anholt.net> Cc: Daniel Stone <daniel@fooishbar.org> Acked-by: Jani Nikula <jani.nikula@intel.com> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Stone <daniels@collabora.com> Acked-by: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2017-09-08build: remove _GNU_SOURCE from source filesDaniel Vetter
We are, the build system takes care of that. Reviewed-by: Eric Anholt <eric@anholt.net> Acked-by: Jani Nikula <jani.nikula@intel.com> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Stone <daniels@collabora.com> Acked-by: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2017-09-08build: Define _GNU_SOURCE in Makefile.amDaniel Vetter
In meson I want to just set this everywhere (no reason not to), and doing so will allow us to clean up a few things. But that means autofoo needs to follow suit. v2: Rebase. Reviewed-by: Eric Anholt <eric@anholt.net> Acked-by: Jani Nikula <jani.nikula@intel.com> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Stone <daniels@collabora.com> Acked-by: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2017-07-11igt: Add LOCAL defines for distro compatibilityArkadiusz Hiler
Each Linux distro takes a different spin on providing kernel's uapi headers (especialy the *drm*.h). You can get them with linux-headers, you can get them with libdrm. Sometime you can even get them twice, from both sources. Sometimes the headers match your kernel version, sometimes you end up stuck with headers matching the kernel that the given release of the distro came out with. This makes things harder for code that does not depend on libdrm, as we cannot have sane ./configure-time checks. So let's define LOCAL_ version for FENCE and EXECBUFFER2_WB defines until all the stable distros catch up (+/- some epsilon). Cc: Petri Latvala <petri.latvala@intel.com> Cc: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
2017-06-09benchmarks/gem_busy: Trim unused comparesChris Wilson
Keep userspace light and don't spend time testing the result if we don't use it. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2017-06-08gem_wsim: More interesting workloadsTvrtko Ursulin
These ones demonstrate fence usage and also mixing them with data dependencies. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-06-08gem_wsim: Add some randomness to the 17i7 workloadTvrtko Ursulin
It was the only one with no randomness. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-06-08gem_wsim: Asymmetrical 1-to-n workloadsTvrtko Ursulin
Simulates a single decoder feeding multiple processing and encoding pipelines. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-06-08gem_wsim: Fix heartbeats in GT2 modeTvrtko Ursulin
VCS2 needs to be redirected to BCS in the remapping mode otherwise the update will get missed. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-06-08gem_wsim: Detect invalid duration rangeTvrtko Ursulin
Maximum has to be larger than minimum and all values positive. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-06-08gem_wsim: Late balancing decision modeTvrtko Ursulin
An approximation of in kernel balancing where balancing decision is postponed to just before submitting the batch for execution. This is implemented by waiting on data dependencies in userspace before submitting each batch. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-06-08gem_wsim: Add static context balancerTvrtko Ursulin
This one assigns VCS engines for each created context in a round- robin fashion. By default each client starts the assignment from the first engine. In global mode the engines are assigned in the round-robin fashion between all the clients - which is identical to the i915 behaviour. Important difference is that doing it from gem_wsim means the assignment always start from a known point so the run-to-run behaviour is identical and repeatable, contrary to when i915 round- robin assignment is used. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-06-08gem_wsim: Added missing -H help textTvrtko Ursulin
Also check that the usage makes sense. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-06-07benchmarks/gem_wsim: Add LOCAL fence definesArkadiusz Hiler
Same as in other benchmarks using them, as not every system is updated with libdrm having those defines. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
2017-06-07Android.mk: Use drm stubsArkadiusz Hiler
Use drm stubs that sit under lib/stubs. Also drop strange, nonexistent additions to LOCAL_C_INCLUDES. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
2017-06-07Android.mk: Fix libkmod useArkadiusz Hiler
On Android libkmod.h is nested under libkmod directory, so we should include appropriately. Also we need to link with it. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
2017-05-31lib: Moving gem_execbuf_wr to ioctl_wrappersLukasz Fiedorowicz
gem_execbuf_wr was duplicated in multiple places. Moving everything to lib/ Signed-off-by: Lukasz Fiedorowicz <lukasz.fiedorowicz@intel.com> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-23gem_wsim: Add global balancing modeTvrtko Ursulin
In this mode ('-G' on the command line) all balancing operations are routed via the first client so the complete balancing state is shared. In other words the overall balancing behaviours is like there is only one client submitting the aggregate workload. This can help with the observed metrics and lead to better balancing decisions in a lot of cases. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-23gem_wsim: Move seqno status page access to helpersTvrtko Ursulin
Refactoring for upcoming changes. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-23gem_wsim: Move seqno allocation and query into helpersTvrtko Ursulin
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-23gem_wsim: Convert clients to threadsTvrtko Ursulin
Will make the userspace balancing daemon simulation easier. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-19gem_wsim: Add missing help text for -pTvrtko Ursulin
Document priority support in the help text. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-17wsim: Allow assigning priorities to each workloadChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2017-05-17wsim: Fix reporting of workload/s for slavesChris Wilson
Slaves just keep on running, far beyond the repeat target of their master. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2017-05-17gem_wsim: Fix client exit with more than one background workloadTvrtko Ursulin
Need to close more pipe ends to support master with more than one background workloads. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-17wsim: Only require execbuf wr ioctl for FENCE_OUTChris Wilson
Just a micro-optimisation to avoid copying back the struct to userspace if we aren't looking for an output. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2017-05-16gem_wsim: Implement sw sync point supportTvrtko Ursulin
Two new workload commands are added, 'f' and 'q.<idx>' which enable creation and signalling of non i915 fences. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-16gem_wsync: Clearer step metadata handlingTvrtko Ursulin
Introduce an anonymous union so each step type can use its own name for the metadata. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-16gem_wsim: Fence supportTvrtko Ursulin
Add sync fence dependency support to workload steps. Only one sync fence dependency per step is supported at the moment. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-16gem_wsim: Simplify batch offset block a bitTvrtko Ursulin
Just compact it a bit by avoiding the min != max check duplication and change get_duration to change w_step. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-16gem_wsim: Add append workloadTvrtko Ursulin
Used with the '-a' command line switch which follows the same usage as '-w' and '-W', it enables to add append workload steps to the end of all normal workloads. This for example allows running any workload in the real-time mode: gem_wsim -w <some-workload> -a p.16667 Makes a workload to be run with the 60 Hz period. At the same time fix the periodic mode execution with dropped frames, or almost dropped frames. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-16gem_wsim: Two small tidiesTvrtko Ursulin
Tidy last in the array presumed offset setting even though this code path is not used at the moment. Also use READ_ONCE on all fields we are trying to read from the status page. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-11gem_wsim: Minimize startup gapTvrtko Ursulin
Do the GPU quiesce as close as possible in time to the start of the workload in order to minimize the gap in engine utilization tracking. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2017-05-11wsim: Set the seqno/time stamp on each batch to every engineChris Wilson
We need to track queue depth not just when we are asked to load balance but for specific engine requests so that we include those in our balancing. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2017-05-10wsim: Use a loop over engines to calculate RTChris Wilson
Simple code tidy to reduce some duplication. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>