- May 25, 2022
-
-
Dmitry Osipenko authored
Add test for the madvise ioctl of Panfrost driver. Signed-off-by:
Dmitry Osipenko <dmitry.osipenko@collabora.com>
-
Dmitry Osipenko authored
Allow igt_panfrost_gem_new() to specify BO flags by adding the flags argument to it. This is a preparatory patch for addition of the new test for madvise ioctl, which will use a non-zero flags. Signed-off-by:
Dmitry Osipenko <dmitry.osipenko@collabora.com>
-
Swati2 Sharma authored
In this patch, removed redundant code. Also, before starting the subtest, cleaned up the states to default by igt_display_reset(). Few minor fixes in indentation. Also, added subtests description. v2: -minor mistake in subtest name -commit was missing after reset, added v3: -fixed styling error -replaced drm calls with igt wrappers v4: -added igt_display_require_output() in igt_fixture -modularized code, added func() for checks -added func() to get highest mode v5: -minor fixes v6: -made modifications based on lib changes v7: -added igt_fixture Signed-off-by:
Swati Sharma <swati2.sharma@intel.com> Signed-off-by:
Patnana Venkata Sai <venkata.sai.patnana@intel.com> Reviewed-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com>
-
Swati2 Sharma authored
Made corresponding changes in tests wrt lib changes in [1/3]. Basically, we have replaced output->config.connector with output->name. Suggested-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com> Signed-off-by:
Swati Sharma <swati2.sharma@intel.com> Reviewed-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com>
-
Swati2 Sharma authored
Use libdrm connector name from output->name, instead of constructing it from drm resources. Suggested-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com> Signed-off-by:
Swati Sharma <swati2.sharma@intel.com> Reviewed-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com>
-
Mauro Carvalho Chehab authored
Most of the time, i915 module unload happens inside a subtest. So, using igt_fork() is OK. However, on i915_selftest, this is called earlier, at igt_kselftest_begin. This causes an error on IGT: forking is only allowed in subtests or igt_simple_main please refer to lib/igt_core documentation As igt_fork() can only be used inside subtests. So, use igt_fork_helper() instead. Reviewed-by:
Petri Latvala <petri.latvala@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
When internal_assert() hits, it is desired to know what part of the code actually hit the issue. Reviewed-by:
Petri Latvala <petri.latvala@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Zbigniew Kempczyński authored
On some platforms (like on ATS) 0x0 may not be available so allocator should be instantiated with safe start offset to avoid getting -ENOSPC. Change require also relaxation of automatic range selection in the allocator. Signed-off-by:
Zbigniew Kempczyński <zbigniew.kempczynski@intel.com> Cc: Ashutosh Dixit <ashutosh.dixit@intel.com> Cc: Dominik Grzegorzek <dominik.grzegorzek@intel.com> Reviewed-by:
Ashutosh Dixit <ashutosh.dixit@intel.com>
-
If a DUT has Chamelium ports connected via an adapter (for example, DP on the Chamelium side -> DP-HDMI adapter -> HDMI on the DUT), this will usually cause many tests to fail. If mismatching port types are found on both sides, the tests will now be aborted with a warning. This behavior can be overridden with a new AdapterAllowed config value, which must be set in [Chamelium:PORT] blocks in .igtrc. Signed-off-by:
Ryszard Knop <ryszard.knop@intel.com> Reviewed-by:
Petri Latvala <petri.latvala@intel.com>
-
Add support for testing/skipping flat ccs modifiers Signed-off-by:
Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com> Signed-off-by:
Jeevan B <jeevan.b@intel.com> Reviewed-by:
Mika Kahola <mika.kahola@intel.com>
-
I915_FORMAT_MOD_4_TILED_DG2_RC_CCS I915_FORMAT_MOD_4_TILED_DG2_MC_CCS I915_FORMAT_MOD_4_TILED_DG2_RC_CCS_CC modifier added for testing and prevent impossible ccs test from flat ccs, fb generation for FB_HAS_PLANE will use linear modifier instead if Y-modifier as its no longer supported for latest platform. v2: Modified commit message. (Petri) Signed-off-by:
Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com> Signed-off-by:
Jeevan B <jeevan.b@intel.com> Reviewed-by:
Mika Kahola <mika.kahola@intel.com>
-
Add support for DG2 flat ccs framebuffers with tile-4. Signed-off-by:
Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com> Signed-off-by:
Jeevan B <jeevan.b@intel.com> Reviewed-by:
Mika Kahola <mika.kahola@intel.com>
-
commit 9035039e1ed691cd893777a42e048003a2f349d6 Author: Mika Kahola <mika.kahola@intel.com> Date: Mon Apr 11 17:34:04 2022 +0300 drm/fourcc: Introduce format modifier for DG2 clear color Signed-off-by:
Jeevan B <jeevan.b@intel.com> Reviewed-by:
Petri Latvala <petri.latvala@intel.com>
-
- May 24, 2022
-
-
Swati2 Sharma authored
Added subtests as a list. Also, renamed tests to distinguish between plane vs planes tests. Lastly, for clipping/clamping tests added new tests as modifiers, pixel-formats and rotation. v2: fixed indentation (JP) Signed-off-by:
Swati Sharma <swati2.sharma@intel.com> Reviewed-by:
Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
-
Swati2 Sharma authored
Signed-off-by:
Swati Sharma <swati2.sharma@intel.com>
-
André Almeida authored
kms_async_flips uses igt_draw_ and i915 modifiers, which make this test fails for other vendors that are not Intel. Use more generic functions and modifiers so other devices can be tested. Since i915's "Linear memory does not support async flips", we can't use DRM_FORMAT_MOD_LINEAR and need to keep using I915_FORMAT_MOD_X_TILED. Signed-off-by:
André Almeida <andrealmeid@igalia.com> Reviewed-by:
Melissa Wen <mwen@igalia.com> Reviewed-by:
Alex Hung <alex.hung@amd.com>
-
André Almeida authored
Add support for drawing rectangles with random colors, useful for tests that you don't mind which color will be used. Signed-off-by:
André Almeida <andrealmeid@igalia.com> Reviewed-by:
Melissa Wen <mwen@igalia.com> Reviewed-by:
Alex Hung <alex.hung@amd.com>
-
Bhanuprakash Modem authored
This patch will find the connector/mode combination that fits into the bandwidth when more than one monitor is connected. Example: When two monitors connected through MST, the second monitor also tries to use the same mode. So two such modes may not fit into the link bandwidth. So, iterate through connected outputs & modes and find a combination of modes those fit into the link BW. V2: * Exit if suitable modes not found in retry. * New function to re-use the code. Cc: Karthik B S <karthik.b.s@intel.com> Cc: Ankit Nautiyal <ankit.k.nautiyal@intel.com> Signed-off-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com> Reviewed-by:
Ankit Nautiyal <ankit.k.nautiyal@intel.com>
-
- May 23, 2022
-
-
Bhanuprakash Modem authored
If the given MST connector is disconnected the DPCD read out request will throw an MST NAK reply error. This patch will handle this in the test that skips to read the DPCD of disconnected MST connectors. <3> [793.017171] i915 0000:03:00.0: [drm] *ERROR* mstb 00000000e5e57ca0 port 1: DPCD read on addr 0x0 for 16 bytes NAKed Cc: Imre Deak <imre.deak@intel.com> Signed-off-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com> Reviewed-by:
Swati Sharma <swati2.sharma@intel.com>
-
- May 20, 2022
-
-
The kms_concurrent is about testing mode setting with reducing the resolution and then again increasing it, for this the test will take the high resolution supported by the connector and then calculate the lowest resolution, if the calculated resolution is not supported by the connector it will assign the default resolution of 1024x 768 without checking. Added check to skip the subtest if the current mode is smaller than 1024 mode (igt_skip_on_f(mode_default->vdisplay < 1024)) becuase if the default resolution is greater then the highest resolution supported by the connector then the test should skip. Signed-off-by:
Nidhi Gupta <nidhi1.gupta@intel.com> Reviewed-by:
Karthik B S <karthik.b.s@intel.com>
-
Zbigniew Kempczyński authored
Test calls igt_fork() so for i915 requires offset allocation arbitration (allocator in multiprocess mode) especially when same drm fd is used in children. Dedicated thread (intel_allocator_multiprocess_start()) is required to be started on the very beginning to handle offset allocations as well as stopping it (intel_allocator_multiprocess_stop()) before test exits. Signed-off-by:
Zbigniew Kempczyński <zbigniew.kempczynski@intel.com> Cc: Luciano Coelho <luciano.coelho@intel.com> Cc: Swati Sharma <swati2.sharma@intel.com> Acked-by:
Petri Latvala <petri.latvala@intel.com> Tested-by:
Luciano Coelho <luciano.coelho@intel.com>
-
- May 19, 2022
-
-
Few gem rpm tests relies on enabling kms crtc in order to trigger rpm resume but on headless platforms these tests skip. Let it trigger the rpm resume by taking user forcewake. v2: - removed has_runtime_pm cond from enable_one_screen_or_forcewake_and_wait(). [Ashutosh] - removed if (ms_data.res) cond from basic_subtest(). [Ashutosh] - clear forcewake in both only for headless. [Ashutosh] v3: - s/disable_all_screens_or_clr_forcewake_and_wait/ disable_all_screens_or_forcewake_put_and_wait. [Ashutosh] - s/enable_one_screen_or_forcewake_and_wait/ enable_one_screen_or_forcewake_get_and_wait. [Ashutosh] - Removed if (!default_mode_params) cond. [Ashutosh] - s/clear_forcewake/forcewake_put/ Cc: Chris Wilson <chris.p.wilson@intel.com> Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com> Reviewed-by:
Ashutosh Dixit <ashutosh.dixit@intel.com>
-
Initialize igt_display_t display only in case of there are valid drm kms resources. v2: - Move igt_display_fini under the 'if (data->res)' cond. [Ashutosh] Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com> Reviewed-by:
Ashutosh Dixit <ashutosh.dixit@intel.com>
-
Change var display_disabled to display_enabled, also modify the passed argument and condition accordingly. Cc: Ashutosh Dixit <ashutosh.dixit@intel.com> Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com> Reviewed-by:
Ashutosh Dixit <ashutosh.dixit@intel.com>
-
Refactor out devid initialization out of init_mode_set_data(). Initialize it in setup_environment(). Reviewed-by:
Ashutosh Dixit <ashutosh.dixit@intel.com> Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com>
-
Jouni Högander authored
Add new testcase for testing selective fetch using big framebuffer. New testcase is using much bigger fb than what is the visible area and setting visible offset to half of the width and the height. This reveals issues in selective fetch area calculation: https://gitlab.freedesktop.org/drm/intel/-/issues/5440 Signed-off-by:
Jouni Högander <jouni.hogander@intel.com> Reviewed-by:
Mika Kahola <mika.kahola@intel.com>
-
Jouni Högander authored
Adding new continuous move testcases for cursor and overlay planes. These new testcases are testing selective fetch when updated plane is fully or partially out of visible area. This was found to reveal bug in selective fetch area calculation: https://gitlab.freedesktop.org/drm/intel/-/issues/5440 Signed-off-by:
Jouni Högander <jouni.hogander@intel.com> Reviewed-by:
Mika Kahola <mika.kahola@intel.com>
-
Jouni Högander authored
Modify current move continuous testcase to ease up adding new testcases. Remove exceeding visible area as testcases to perform this are about to be added as separate testcases. v2: Fix compilation error and warning Signed-off-by:
Jouni Högander <jouni.hogander@intel.com> Reviewed-by:
Mika Kahola <mika.kahola@intel.com>
-
- May 18, 2022
-
-
Few PCI devices under DGFX card tree don't have any kernel driver. These devices will block D3cold state of gfx root port. Adding support to setup d3cold by enabling runtime PM for all PCI devices under the gfx root port. It will not save/restore pci devices power attributes. It will be useful to tune D3Cold after boot. v2: - Change naming convention from configure_autosuspend to setup-d3cold. [Rodrigo]. v3: - Kept setting auto-suspend delay as optional. [Rodrigo] Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com> Reviewed-by:
Rodrigo Vivi <rodrigo.vivi@intel.com>
-
intel_pm_rpm tool is a debug tool. It can be use to setup and prepare the gfx card to go to D3Cold. It also provide the debug option to disable all display and prepare device to enter to runtime suspend. v2: - Removed IS_DGFX() condition. v3: - Change naming convention from setup-d3cold to force-d3cold-wait disable-display to disable-display-wait. [Rodrigo] v4: - Use igt_setup_runtime_pm() only for --disable-display-wait. Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com> Reviewed-by:
Rodrigo Vivi <rodrigo.vivi@intel.com>
-
Add an option to configure autosuspend_delay_ms as i915 read autosuspend_delay_ms to all PCI devices under GFX root port. configuring autosuspend_delay_ms as optional. Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com> Reviewed-by:
Badal Nilawar <badal.nilawar@intel.com>
-
Enable gfx card pci devices runtime pm for all pci devices and bridge under the topology of Gfx Card root port. Added a library function to get the PCI root port ACPI D state and to print the pci card devices runtime pm status. v2: - Save pci dev power attrs to dynamically allocated array. [Rodrigo] - Set autosuspend delay to 0 on supported devices. [Rodrigo] - %s/else if/else in igt_pm_get_acpi_real_d_state. [Kamil] v3: - Add comment for MAX_PCI_DEVICES. [Badal] - Use static global array __pci_dev_pwrattr[]. [Rodrigo] - Use pci_slot_match iter. [Badal] - Destroy the iter. [Badal] v4: - Added igt_pm_enable_pci_card_autosuspend() to avoid any control attr save/restore by exit handler. [Rodrigo] v5: - Code refactoring to avoid code duplication. - Few function name changed. [Rodrigo] %s/igt_pm_enable_pci_card_autosuspend/igt_pm_enable_runtime_pm. %s/__igt_pm_setup_pci_card_runtime_pm/igt_pm_setup_pci_dev_power_attrs. %s/igt_pm_setup_power_attr/igt_pm_write_power_attr. v5: - %s/igt_pm_save_power_attr/igt_pm_read_power_attr/ - Added len argument to igt_pm_write_power_attr(). Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com> Reviewed-by:
Rodrigo Vivi <rodrigo.vivi@intel.com>
-
Utility library function to get the Gfx Card PCI topology root port pci device using libpaciaccess. root port will be used to get the real ACPI D state. Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com> Reviewed-by:
Rodrigo Vivi <rodrigo.vivi@intel.com>
-
Add memory region placement to gem-{mmap-type, execbuf} subtest. This will be useful to extend these sub-test for D3Cold. Cc: Chris Wilson <chris.p.wilson@intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by:
Anshuman Gupta <anshuman.gupta@intel.com> Reviewed-by:
Rodrigo Vivi <rodrigo.vivi@intel.com>
-
Mauro Carvalho Chehab authored
The logic already stores pw-reserve PID on a static var. Store also the pipewire-pulse pid, in order to keep passing the arguments on all functions. Suggested-by:
Andi Shyti <andi.shyti@linux.intel.com> Reviewed-by:
Andi Shyti <andi.shyti@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
Newer distributions like Fedora and openSUSE thumbweed are now coming with pipewire-pulse instead of pulseaudio - either as default or as an optional audio stack. That adds a new requirement when unloading sound drivers, which, in turn, is needed when the DRM driver is bound into it. Add the needed logic to work properly in case pipewire-pulse is detected. Tested on ADL-N with Fedora 35 and wireplumber: IGT-Version: 1.26-g982672f3 (x86_64) (Linux: 5.18.0-rc7-drm-ad75b5b819c9+ x86_64) Starting subtest: unbind-rebind process 585 (alsactl) is using audio device. Should be terminated. process 11932 (pipewire) is using audio device. Should be terminated. process 11937 (pipewire-pulse) is using audio device. Should be requested to stop using them. Preventing pipewire-pulse to use the audio drivers Device Audio0 can not be acquired: Success reserve acquired Unloaded audio driver snd_hda_intel Realoading snd_hda_intel Subtest unbind-rebind: SUCCESS (2.603s) Reviewed-by:
Jonathan Cavitt <jonathan.cavitt@intel.com> Reviewed-by:
Andi Shyti <andi.shyti@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
This function is needed outside igt_core. So, make it exportable, and allow passing different signs to the children. Reviewed-by:
Andi Shyti <andi.shyti@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
Current Linux Kernel don't report if the audio driver binds into the DRM driver. As this is CPU specific, allow audio driver unload fail without skipping the IGT tests on legacy Kernels, as this may not be mandatory. On new kernels where lsmod will properly display the dependency between the audio and DRM drivers, skip the core hotunplug test if it fails to unload the audio driver, as this is unrelated to the DRM driver - and it could simply because there are some userspace code using the audio device while the IGT test is running. Reviewed-by:
Jonathan Cavitt <jonathan.cavitt@intel.com> Reviewed-by:
Andi Shyti <andi.shyti@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
The current audio unbind logic is wrong: it expects the audio driver to depend on i915 only on Haswell, Broadwell and DG1. That doesn't match the Kernel driver, where snd-hda-audio binds on i915 also on Skylake, Braswell and whenever it needs to use pm runtime from the DRM driver. Now that lib/igt-kmod has gained improved support for audio unbind, update core_hotunplug to benefit from the newer logic. Reviewed-by:
Andi Shyti <andi.shyti@linux.intel.com> Reviewed-by:
Jonathan Cavitt <jonathan.cavitt@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
Add code to free allocated strings at the module unload routines from igt_kmod. Reviewed-by:
Andi Shyti <andi.shyti@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-