summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2016-05-03igt/gem_exec_flush: Give even batch write a unique valueChris Wilson
One property lost in the expansion for various coherency checks was ensuring that every time we overwrote the batch it had a unique value (to ensure that the GPU was seeing the latest value). Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-03igt/gem_exec_flush: Stress a few more synchronisation paths for execbufChris Wilson
Look at handling of multiple batches within the buffer and avoiding as much synchronisation as possible. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-03kms_panel_fitting: Enable connectors with "scaling mode" property set.Robert Foss
Enable testing on all connectors that have the "scaling mode" property set. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93012 Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-05-03kms_panel_fitting: Enabled test on non-Intel hardware.Robert Foss
Switched from DRIVER_INTEL to DRIVER_ANY to enable test on all hardware. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-05-03kms_panel_fitting: Switched DRM format to a hardware agnostic alternative.Robert Foss
Changed the DRM format to LOCAL_DRM_FORMAT_MOD_NONE since it is hardware agnostic. Also fixed formatting/tabs. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-05-03kms_panel_fitting: Remove un-used variable.Robert Foss
ref_crc is never assigned or read, and can be safely removed. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-05-03kms_panel_fitting: Remove un-read variable pipe_crc.Robert Foss
pipe_crc in data_t is assigned an allocated memory space and then later free'd. But it is never used for any comparisons. It should therefore be safe to remove pipe_crc and the crc requirement. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-05-03kms_panel_fitting: Remove un-read variable.Robert Foss
Remove devid from data_t since it is never read. Also remove one assignment to devid. Signed-off-by: Robert Foss <robert.foss@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-05-03kms_panel_fitting: skips if there's a non-eDP display connected. Fix it.Robert Foss
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93012 Signed-off-by: Jani Nikula <jani.nikula@intel.com> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-05-03tests/kms_sink_crc_basic: clean up assert_color and provide more detailsJani Nikula
We can simply sscanf the crc in one go. Also split up the igt asserts to get better details about what went wrong. Signed-off-by: Jani Nikula <jani.nikula@intel.com>
2016-05-03tests: limit pm_backlight actual brightness tolerance between 0 and maxJani Nikula
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
2016-05-03tests: add context param to pm_backlight testsJani Nikula
We'll be adding more context for the subtests than just the max brightness. Signed-off-by: Jani Nikula <jani.nikula@intel.com>
2016-05-03tests: use igt_assert_lte to verify pm_backlight test resultsJani Nikula
Gives out better diagnostics than just igt_asssert. Signed-off-by: Jani Nikula <jani.nikula@intel.com>
2016-05-03lib: compute exitcode firstChris Wilson
Before we print the exitcode to the debug/kmsg logs, we should inspect what its final value will be. For example, in the case of running multiple subtests which all happen to be skipped, igt_exitcode is 0, but the final exit code will be 77. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Tested-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2016-05-03igt/gem_exec_flush: Test continuously rewriting the batchChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-03igt/gem_ringfill: Avoid CPU -> GTT -> CPU transitionChris Wilson
Avoid the second pair of full clflushes when setting up the batch. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-03tools/intel_reg_decode: drop confusing use of IS_965()Jani Nikula
Unlike in the kernel driver coding style, IS_965() matches the platform and all subsequent ones. Replace IS_965() with suitable but less confusing alternatives. Most occurences are on code paths that only get called for gens 2, 3 and 4, so replace those with IS_GEN4(). In the one other call site just flip the condition to check for gens 2 and 3 instead. Signed-off-by: Jani Nikula <jani.nikula@intel.com>
2016-05-03lib/intel_chipset: drop unused IS_9XX()Jani Nikula
It's also confusing as the style differs from the kernel (exact platform in the kernel vs. the platform and any later ones in igt). Signed-off-by: Jani Nikula <jani.nikula@intel.com>
2016-05-02tools: Add missing Kabylake codename strings.Rodrigo Vivi
No functional change and no change in the current format. Just introducing the missing Kabylake name strings. v2: Duh! forgot the ")"... Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2016-05-02igt/gem_exec_flush: Match gem_set_domain to pointer accessChris Wilson
When using the kernel set-domain cache management, we need to set the domain as appropriate for our pointer access. In this case we access the buffer through a CPU mmap, and so we must request access via the CPU domain. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-02igt/gem_exec_flush: Immediately repeat the same cachelineChris Wilson
When looking at a pair of GPU writes, where we want to make sure that the clean cacheline is invalidated automatically, we want to reuse that cacheline whilst we know it remains valid (i.e. repeat the test using a new value to the same location). Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-02lib: Fixup u64 multiply for computing nanosecondsChris Wilson
32bit builds ran into a silly multiplication issue when computing elapsed nanoseconds of more than 2s... Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-02igt/gem_exec_flush: Move assertions out of interruptible loopsChris Wilson
Since the value in the bo may be altered by the test, we only want to repeat phases of the test to avoid breaking the test itself. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-02lib: Tweak calibration of initial settimer delayChris Wilson
If we assume that the first settimer and clock_gettime() itself have appreciable overhead, try to exclude those from the calibration delay. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-02tools/intel_watermark: Dump linetime watermarks on hsw/bdwVille Syrjälä
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2016-05-02igt/gem_exec_flush: Mark mmap-wc dependencyChris Wilson
Add a requirement for mmap-wc so that failure on older kernels is explained. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-02benchmarks/gem_latency: Report throughputChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-02igt/gem_exec_flush: Use a cacheline strideChris Wilson
Look at different cachelines on each pass, otherwise each group of 16 flush the same cachline. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-02igt/gem_exec_flush: Add interruptible testingChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-02igt/gem_exec_flush: Add basic set-domain checksChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-01igt/gem_exec_flush: For good measure, check pread/pwriteChris Wilson
As a point of comparison, test the pread/pwrite interface as well. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-01lib: Enable clflush for 32bit x86 buildsChris Wilson
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-01igt/gem_exec_flush: Use subtest groups to tidy requirement checkingChris Wilson
We can make the requirement testing and reporting tidier by using igt_subtest_group. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-01igt/gem_exec_flush: Add a read-only variationChris Wilson
Alternate between two values written by the GPU so that we can look for stale cachelines without having to overwrite the value with the CPU. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-05-01lib: Apply magic clflush serialisationChris Wilson
On Baytrail, Braswell and Atoms beyond we see an issue where the mfence is insufficient to force the cacheline to be coherent (i.e. such that writes from the GPU are visible by the CPU after the call to clflush). A second clflush is ordered with an earlier clflush to the same address and this appears sufficient to give the coherency required for GPU/CPU interop. Testcase: igt/gem_exec_flush Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Akash Goel <akash.goel@intel.com>
2016-04-29tests: Add gem_exec_flushChris Wilson
A basic check that the execbuf flushes writes from the batch and that they are coherent afterwards. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-28igt_kms: Fix use after free in kmstest_get_pipe_from_crtc_idTvrtko Ursulin
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2016-04-27benchmark/gem_latency: sync startup correctlyChris Wilson
When waiting for the producers to start, use the cond/mutex of the Nth producer and not always the first. Spotted-by: "Goel, Akash" <akash.goel@intel.com> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-27igt: Add gem_exec_parallelChris Wilson
Attempt to fill buffers using many clients working in parallel. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-26tests/gem_close_race: Add igt_fixture block for igt_stop_hang_detector.marius vlad
igt_fork_hang_detector() was called from a igt_fixture block, while its counterpart (igt_stop_hang_detector) was called normally, causing SIGTERM to be sent when running under check target. Signed-off-by: Marius Vlad <marius.c.vlad@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-04-25igt/gem_busy: Avoid BSD emission on gen6Chris Wilson
Remember to skip using BSD on gen6, unless you want to kill the machine. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95134 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2016-04-25core_getversion: Only verify major version for i915.Robert Foss
This change mirrors the change in drm made by krh@redhat.com on "Mon Apr 6 17:18:17 2009" on the drm branch intel_on_all_hw. The assert(major < 1) is only needed for the legacy intel driver. Signed-off-by: Robert Foss <robert.foss@collabora.com> Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-04-25core_getversion: Switched from igt_assert to assert helpers.Robert Foss
Switched to assert helpers to enable better error output. Signed-off-by: Robert Foss <robert.foss@collabora.com> [tomeu: fix test of major version to be lte] Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2016-04-25tests/gem_busy: Fix passing invalid fd for basic-parallel-* tests.marius vlad
Caught by check target. Signed-off-by: Marius Vlad <marius.c.vlad@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
2016-04-25tests/kms_flip_event_leak: Open DRM device with DRIVER_ANYTomeu Vizoso
So that this test can be run in drivers other than i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> Series-version: 1 Series-to: intel-gfx Series-cc: padovan, daniels, marcheu, seanpaul, xexaxo, fedkem, mvlad, danvet Series-prefix: i-g-t Cover-letter: Make more tests generic Hi, these patches allow a few more tests to run on drivers other than i915, mainly by removing the last usage of DRM_IOCTL_I915_GET_PIPE_FROM_CRTC_ID and removing superfluous dependencies on bufmgr and tiled BOs. Thanks, Tomeu END
2016-04-25tests/kms_flip_event_leak: Use non-tiled formatsTomeu Vizoso
As the test doesn't actually need tiled BOs, drop the tiled formats so the test can run on drivers other than i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25tests/kms_flip: Open DRM device with DRIVER_ANYTomeu Vizoso
So that this test can be run in drivers other than i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25tests/kms_flip: Move bufmgr requirement into subtestsTomeu Vizoso
Because bufmgr is currently a i915-only thing and it's only needed in a subset of the subtests, require it only in the subtests that actually need it so that the other subtests aren't skipped without a reason. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25tests/kms_flip: Create tiled BOs only when neededTomeu Vizoso
Because attempts to create a tiled BO will cause a igt_require call to fail on drivers that don't support tiling, do so in the subtest that actually needs it so that other subtests aren't skipped without reason. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
2016-04-25tests/kms_render: Move dependency on i915 into subtestTomeu Vizoso
Batchbuffers are only needed in the subtest that does the blit on the GPU, so move that dependency into it so the other subtest can be ran on !i915. Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-28 14:20:40 +0000