- Jun 03, 2021
-
-
Andrey Grodzovsky authored
Unplug a device while an exported sync_obj and it's fence are is still around. Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com>
-
Andrey Grodzovsky authored
Unplug a device while an exported BO is still around. Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com>
-
Andrey Grodzovsky authored
Enables better testing of hot remove Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com>
-
- Feb 23, 2021
-
-
Ville Syrjälä authored
Restore the capability to run RGB pixel format tests when the driver doesn't advertize the atomic capability. Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Fixes: e96c0d8e ("tests/kms_plane: optimize pixel format tests") Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
-
Ville Syrjälä authored
Using COMMIT_LEGACY for the cursor plane makes the test fail because the cursor framebuffer is never handed over to the kernel, and instead only the BO gets passed in via the legacy cursor ioctl. Thus the rmfb ioctl will not turn off the cursor plane since the kernel is using an internal fb for the cursor. Let's avoid that by treating the cursor as just another universal plane. Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
-
Add subtests to verify content protection simultaneously on multiple outputs on the same MST topology. v3: -Remove the logging which are no longer required. (Anshuman) -Add logic to verify that CP is still enabled on other connectors while disabling it on one of the connectors. (Anshuman) v4: -Rename is_output_support_cp_capable(). (Anshuman, Ram) -Rephrase the comment in is_dp_mst_output. (Anshuman) -Remove the redundant HDCP check hunk before HDCP disable. (Anshuman) -Check the link on the HDCP enabled outputs when one of the outputs in the same DP MST topology is disabled. (Anshuman) -Revert the change in test_content_protection_cleanup(). (Anshuman) -Create fb's in a common function for both MST and SST. (Ram) -Rename is_dp_mst_output() to output_is_dp_mst(). (Ram) -Remove the redundant igt_debug() before HDCP enable. (Ram) -Rephrase the igt_assert statement during HDCP enable. (Ram) -Optimize the execution time by checking the link for all MST connectors together. (Ram) -No need of loop where we disable CP on one of the MST outputs. (Ram) -During verification by disabling one of the MST outputs, check for the negative case until the required timeout. (Ram) v5: -Modify skip message when HDCP support is not present. (Anshuman) -Remove the definition of color_t structure as it is no longer used. v6: -Optimize execution time in LIC test. (Anshuman) v7: -Check HDCP capability after modeset on MST. (Anshuman) v8: -Use try_commit for MST modeset and skip the test on failure. (Ankit) Signed-off-by: Anshuman Gupta <anshuman.gupta@intel.com> Signed-off-by: Karthik B S <karthik.b.s@intel.com> Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
-
- Feb 22, 2021
-
-
Allow amdgpu to run HW rotation IGT test case Added conditions to bypass all the requirements needed for intel when testing amdgpu. Additionally, freed unused frame buffers. Added swizzle 64kb tiling method for amdgpu-specific. Updated drm header for amdgpu tiling modifiers. v2: drm_fourcc.h copied from kernel header commit:8ba16d5993749c3f31fd2b49e16f0dc1e1770b9c from drm-next. removed igt_pipe_crc_collect_crc for intel gpu. Only on AMDGPU. v3: moved drm_fourcc.h to another patch. Removed creating redundant fb in prepare_crtc for amdgpu. Guarded display commit for amdgpu. Blocked cursor plane rotation for amdgpu. Added back tiling when creating reference fb. Signed-off-by: Sung Joon Kim <sungkim@amd.com> Reviewed by: Nikola Cornij <nikola.cornij@amd.com> Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
-
Aligns IGT drm_fourcc.h to the AMD's latest commit to this file in drm-next: 8ba16d5993749c3f31fd2b49e16f0dc1e1770b9c: drm/fourcc: Add AMD DRM modifiers. Signed-off-by: Sung Joon Kim <sungkim@amd.com> Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
-
Restrict execution of all subtests to two pipes(default) If you want to execute on all pipes need to pass extra argument(-e) Example: ./build/tests/kms_setmode -e --r basic V2: Handle when count_crtcs is less than 2 (petri) V3: Add helper function for limit_crtc_count (petri) Cc: Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com> Cc: Modem Bhanuprakash <bhanuprakash.modem@intel.com> Cc: Karthik B S <karthik.b.s@intel.com> Cc: Latvala Petri <petri.latvala@intel.com> Tested-by: Patnana Venkata Sai <venkata.sai.patnana@intel.com> Signed-off-by: Patnana Venkata Sai <venkata.sai.patnana@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
-
- Feb 21, 2021
-
-
KunalJoshi95 authored
disabled fsm handling for kms_color_chamelium which operates on single mode through out. Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
-
KunalJoshi95 authored
Added igt_chamelium_allow_fsm_handling. v2: changes for resolving gcc warnings (Petri) Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
-
- Feb 19, 2021
-
-
Added description for all subtests Signed-off-by: Arundhati Hagaragi <arundhati.hagaragi@intel.com> Cc: Gupta, Nidhi1 <nidhi1.gupta@intel.com> Cc: B S, Karthik <Karthik.B.S@intel.com> Reviewed-by: Jeevan B <jeevan.b@intel.com>
-
Added description for all subtests Signed-off-by: Arundhati Hagaragi <arundhati.hagaragi@intel.com> Cc: Modem, Bhanuprakash <Bhanuprakash.Modem@intel.com> v2: Modified description to be more specific. (Bhanu) Reviewed-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
-
- Feb 18, 2021
-
-
Martin Roukala authored
This test makes sure: * the clock is running at the expected rate * (potential) power gating has no effect on the clock v2: - use signed integer for the gpu timestamp diff (Bas) v3: - add test and subtest descriptions (Arek) - split the fast and long tests in different subtests (Martin) - use igt_stats to compute actual statistics (Chris) v4: - call igt_stats_fini() after finishing with the stats (Petri) v5: - add a human-readable text to asserts (Petri) - assert directly on the function call, not its return value (Petri) Signed-off-by: Martin Peres <martin.peres@mupuf.org> Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
-
Martin Roukala authored
Having a test fail with "r != 0" is not helpful, and forces developers to open the source code of IGT to figure out what this actually means. This commit changes these asserts so as the function call that failed is in the stack trace, and adds a human-readable explanation of what failed. Suggested-by: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Martin Peres <martin.peres@mupuf.org> Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
-
Martin Roukala authored
This will soon be followed by more amd_query_info tests, and the basic file is already big-enough. v2: - add test and subtest descriptions (Arek) - add the new file to autotools (Petri) v3: - fix compilation using autotools (Petri) Signed-off-by: Martin Peres <martin.peres@mupuf.org> Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
-
Extract the ioctl call logic in igt_wait_for_vblank_count() into a helper function, __igt_vblank_wait(). Adjust igt_assert() test in igt_wait_for_vblank_count() and igt_wait_for_vblank() accordingly. Signed-off-by: Sumera Priyadarsini <sylphrenadin@gmail.com>
-
Use igt_wait_for_vblank() instead of igt_wait_for_vblank_count() for callers requiring a single vblank interval. Signed-off-by: Sumera Priyadarsini <sylphrenadin@gmail.com>
-
Lionel Landwerlin authored
Yet another file. Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com> Acked-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
-
- Feb 17, 2021
-
-
Chris Wilson authored
In light of the VT-d workarounds, we may introduce padding around the scanout vma. This should not affect relocations referencing the scanout on !full-ppgtt where we leak the GGTT address of scanout to users. 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>
-
- Feb 16, 2021
-
-
With multidisplay set ups, the test ran only on first pipe with 2 outputs and it would skip on other displays connected. so added a variable to correct this behavior. Signed-off-by: Jeevan B <jeevan.b@intel.com> Reviewed-by: Karthik B S <karthik.b.s@intel.com>
-
- Feb 12, 2021
-
-
According to the CTA 861.G specification, table 44 in section 6.9 and also specifically section 6.9.1, the value of the constant must be 0, not 1, so fix this. An identical fix has been submitted for inclusion into the kernel headers (this is not exposed uapi). See: https://lists.freedesktop.org/archives/dri-devel/2021-February/297233.html Fixing this constant for Kodi also fixed some HDR TVs, so the wrong constant has real world consequences on some display hardware. See: https://github.com/xbmc/xbmc/pull/19129 Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com> Cc: Simon Ser <contact@emersion.fr> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
-
Adding Alder lake platform definitions Cc: Anusha Srivatsa <anusha.srivatsa@intel.com> Cc: Lucas De Marchi <lucas.demarchi@intel.com> Signed-off-by: Caz Yokoyama <caz.yokoyama@intel.com> Signed-off-by: Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
-
Taken from kernel commit: commit 0883d63b19bb ("drm/i915/adl_s: Add ADL-S platform info and PCI ids") commit 04057a1afc75 ("drm/i915: Sort EHL/JSL PCI IDs") commit 0e8e272f1368 ("drm/i915/ehl: Remove invalid PCI ID") Signed-off-by: Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
-
- Feb 10, 2021
-
-
Tvrtko Ursulin authored
Show a list of supported interactive commands when pressing 'h'. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-
Tvrtko Ursulin authored
Implement a default view where clients are aggregated by their PID. Toggled by pressing 'H' similar to top(1). v2: * Fix memory leak. v3: * Do not allow sort by client id in aggregated mode. * Tweak sort criteria and sorting decisions. (Chris) v4: * More tweaks to code flow. (Chris) Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-
Tvrtko Ursulin authored
Slight improvement with regards to wrapping header components to fit console width. If a single element is wider than max it can still overflow but it should now work better for practical console widths. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-
In multi display scenario, the subtest will get executed on the first pipe and exit,and does not execute on the second pipe as required. Added a change to take into account the number of the outputs connected, so that the subtest is executed on all the outputs, on the first 2 pipes. Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com> Reviewed-by: Karthik B S <karthik.b.s@intel.com>
-
- Feb 09, 2021
-
-
Petri Latvala authored
The SIGHUP handling was incorrectly done only when log level was at least 'normal'. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Arkadiusz Hiler <arek@hiler.eu> Cc: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-
- Feb 08, 2021
-
-
This patch includes below updates * For Flipline test: if refresh_rate <= Vrr_min then - Expected returned refresh rate would be vrr_max - At least 35% of the flips should be in threshold * Update "igt_display_commit_atomic" with "igt_display_commit2" * Calculate the target timestamp based on the delta between event timestamps & whatever the time left to reach the expected rate. * Add few debug prints V2: * Rebase V3: * Compute the target timestamp based on the delta (Ville) V4: * Add a comment to justify the result with refresh rate < 40Hz (Manasi) V5: * Align the starting point with flip completion event (Manasi) Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: Manasi Navare <manasi.d.navare@intel.com> Cc: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com> Reviewed-by: Manasi Navare <manasi.d.navare@intel.com>
-
- Feb 06, 2021
-
-
Chris Wilson authored
igt_require_gem() restores the default properties, undoing our override, so apply it again! Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
-
Chris Wilson authored
Since we write to the object and try to avoid relocations, we have to flag the object as being written to for the implicit fencing required by gem_read() Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Ramalingam C <ramalingam.c@intel.com>
-
Chris Wilson authored
On more recent kernels, we can set the heartbeat interval and so reduce how long it takes to declare the system wedged, speeding up the testing. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
-
Chris Wilson authored
Avoid skipping due to missing support after we have begun opening resources for the tests, we don't want to leak into the next. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
-
- Feb 05, 2021
-
-
Chris Wilson authored
We emit a store on each engine after loading the module to confirm the basic liveness of command submission. Trim away some of the chaff. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Ramalingam C <ramalingam.c@intel.com> Reviewed-by: Ramalingam C <ramalingam.c@intel.com>
-
Chris Wilson authored
As a new client does not have a delta for its first sample, treat as still as idle and hide it if the user requests not to be shown clients with no activity. 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>
-
Chris Wilson authored
Scale the bar_len into the fractional bar before truncating! 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>
-
Added Test description for test case Signed-off-by: Arundhati Hagaragi <arundhati.hagaragi@intel.com> Cc: Gupta, Nidhi1 <nidhi1.gupta@intel.com> Cc: B, Jeevan <Jeevan.B@intel.com> v2: modified description to be more specific. (Jeevan) Reviewed-by: Jeevan B <jeevan.b@intel.com>
-
- Feb 04, 2021
-
-
Juha-Pekka Heikkilä authored
Different resolutions with same content may have different crc hence generate buffer verification crcs for different modes if needed. Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Karthik B S <karthik.b.s@intel.com>
-
kms_atomic_transition test is taking minimum of 69.5s time to execute on CI. To reduce the execution time this patch will add the change which will run the test on 1 HDR plane, 1 SDR UV plane, 1 SDR Y plane and skip the rest of the planes. v4: -restricted execution of all the subtests to 2 pipes. (Uma) -Modified skip_plane() function. (Uma) v5: -added a extended flag, if it is set by the user test will be executed on all the pipes otherwise will be executed only on 2 pipes. (Karthik) v6: -in previous version extended flag and display passed in separate structures now added a data_t structure which includes igt_display_t, extended flag and drm_fd in single structure. (Uma) v7: -Added a pipe count variable instead of directly having a check of (pipe >= 2). This will be required if platforms are having 2 pipes as A+C. (Karthik) Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com> Reviewed-by: Uma Shankar <uma.shankar@intel.com> Acked-by: Karthik B S <karthik.b.s@intel.com>
-