- Oct 12, 2022
-
-
Bhanuprakash Modem authored
As we are intended to test few cases on internal panels separately, fix the logic to execute such tests on internal panels only. 'TRANSITION_MODESET_FAST' -> should be only for internal panels 'TRANSITION_MODESET' -> should be only for external panels Fixes: 2527ce52 (tests/kms_atomic_transition: Convert tests to dynamic) Cc: Karthik B S <karthik.b.s@intel.com> Signed-off-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com> Reviewed-by:
Karthik B S <karthik.b.s@intel.com>
-
- Oct 11, 2022
-
-
Matthew Auld authored
DG1, as expected only has a fixed sized bar, so attempting to force the BAR size should fail. Update the test to account for such devices, and perform a skip, instead of failing. Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/6458 Signed-off-by:
Matthew Auld <matthew.auld@intel.com> Cc: Nirmoy Das <nirmoy.das@intel.com> Tested-by:
Nirmoy Das <nirmoy.das@intel.com> Reviewed-by:
Nirmoy Das <nirmoy.das@intel.com>
-
Allow benchmark compilation on a FreeBSD system using the drop-in igt_freebsd header. Signed-off-by:
Jake Freeland <jfree@freebsd.org> Acked-by:
Petri Latvala <petri.latvala@intel.com>
-
FreeBSD uses iic instead of i2c for I/O management. This patch includes the FreeBSD iic header and uses a macro substitution to convert the i2c addr member into the iic slave member. Signed-off-by:
Jake Freeland <jfree@freebsd.org> Acked-by:
Petri Latvala <petri.latvala@intel.com>
-
Use sysctlbyname() to gather RAM and swap information for any FreeBSD system using igt. Signed-off-by:
Jake Freeland <jfree@freebsd.org> Acked-by:
Petri Latvala <petri.latvala@intel.com>
-
Both Linux and FreeBSD have a <sys/syscall.h>, but some operating systems do not. This patch opens up syscall.h access to FreeBSD through using #ifdef __FreeBSD__ Signed-off-by:
Jake Freeland <jfree@freebsd.org> Acked-by:
Petri Latvala <petri.latvala@intel.com>
-
Use #ifdef __linux__ to limit all Linux-specific headers to Linux systems Signed-off-by:
Jake Freeland <jfree@freebsd.org> Acked-by:
Petri Latvala <petri.latvala@intel.com>
-
This header attempts to be a simple drop-in to enable FreeBSD support in a given source file. It includes macro substitutions that replace Linux-specific phrases with their FreeBSD equivalents. Signed-off-by:
Jake Freeland <jfree@freebsd.org> Acked-by:
Petri Latvala <petri.latvala@intel.com>
-
- Oct 10, 2022
-
-
Sanitize the system state before starting the subtest. v2: Created a new functions for each subtest instead of managing the full logic within the igt_subtest(). v3: Minor changes. v4: Replaced igt_assert_f with igt_require_f in prune-stale-modes. v5: Added goto statement if connector_id found true. v6: Minor changes. v7: Added func pointer to subtests. Signed-off-by:
Mohammed Thasleem <mohammed.thasleem@intel.com>
-
In subtest_leak(), mmap() is called with the flag PROT_WRITE, but no PROT_READ. Later in the function, the mapped memory is read using `children[i]`. In FreeBSD, the lack of PROT_READ causes SIGSEGV. Adding the PROT_READ flag to the mmap() call fixes this. Signed-off-by:
Jake Freeland <jfree@freebsd.org> Reviewed-by:
Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com>
-
After each subtest gpu should be idle or else busy engine leaks into next subtest. References: https://gitlab.freedesktop.org/drm/intel/-/issues/5888 Suggested-by:
Chris Wilson <chris.p.wilson@intel.com> Signed-off-by:
Nirmoy Das <nirmoy.das@intel.com> Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com>
-
Add test description to all the available subtests. Signed-off-by:
Janga Rahul Kumar <janga.rahul.kumar@intel.com> Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com>
-
Converting the existing subtests to dynamic subtests. Adding support for each_pipe, currently PSR2 is supported on few pipes so breaking the loop for other pipes currently. v2: Add commit message and change break to continue. v3: Check for pipe supporting PSR2 or not and then add to dynamic subtest. v4: Check PSR2 support once in fixture then iterate pipes. v5: Increase array size for pipes and outputs. Signed-off-by:
Jeevan B <jeevan.b@intel.com> Reviewed-by:
Jouni Högander <jouni.hogander@intel.com>
-
Converting the existing subtests to dynamic subtests. Adding support for each_pipe, currently PSR2 is supported on few pipes so breaking the loop for other pipes currently. v2: Check for pipe supporting PSR2 or not and then add to dynamic subtest. v3: Check PSR2 support once in fixture then iterate pipes. v4: Add output part into dynamic name. v5: Add the missing code and increase pipes and outputs array size. Signed-off-by:
Jeevan B <jeevan.b@intel.com> Reviewed-by:
Jouni Högander <jouni.hogander@intel.com>
-
Karthik B S authored
Covert the existing subtests to dynamic subtests at pipe/output level. Signed-off-by:
Karthik B S <karthik.b.s@intel.com> Reviewed-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com>
-
Karthik B S authored
Reset display to clean the state before starting the subtest. Move the clean up part to a separate function so that cleanup code is run even if there is a failure in the subtest. This prepares the test for conversion to dynamic subtests. Remove redundant mode override as we're anyway overriding with the default mode. Check if platform is capable of atomic commit before choosing commit style in cleanup function. Signed-off-by:
Karthik B S <karthik.b.s@intel.com> Reviewed-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com>
-
Karthik B S authored
Add a modeset before checking output is HDCP capable. Test currently skips on multidisplay MST config telling no HDCP capable output is found because we're checking if output is HDCP capable before doing a modeset on that output. Try other pipe/output combo for MST if failure is seen on one, as the test is not dependent on the combination as long as each output has a valid pipe. Signed-off-by:
Karthik B S <karthik.b.s@intel.com> Reviewed-by:
Bhanuprakash Modem <bhanuprakash.modem@intel.com> Acked-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com>
-
- Oct 07, 2022
-
-
The mmap64() function exists to point 32-bit architectures to an mmap interface with the capability to address a 64-bit memory address space. As 32-bit machines are replaced by 64-bit, there is no longer a need to keep mmap64() around. This is especially apparent in operating systems like FreeBSD that exclude the mmap64() function altogether. This patch replaces all instances of mmap64() with mmap(). Signed-off-by:
Jake Freeland <jfree@freebsd.org> Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com>
-
- Oct 06, 2022
-
-
Mauro Carvalho Chehab authored
While analyzing the data, it is nice to have a quick summary about how much each set of the analyzed data contributes to the total. So, generate a function summary report with the interception of all the sets ("common function coverage"), plus the difference that an individual test group contributes. This is useful to provide a quick feedback on several situations: - When comparing FULL with BAT: - how many functions are added on FULL? - are there functions that are only on BAT? - When comparing different machines: - How many functions each individual machine has over the common group? - When comparing a group of tests: - How many functions an individual test adds for the coverage that weren't covered at the common group yet? Acked-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
When -O2/-O3 is used with GCC, IPA will produce function variants at the object code. So, for a function or macro foo(), it may produce several actual functions at the object code, like: foo.isra.3 foo.isra.4 For the purposes of code coverage, all of those are the same function, so use a regex with "\w+", which will exclude the .isra.<number>, considering all variants as the same function. Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
The indentation is not relevant for the output, but it makes easier when debugging issues at the produced file. Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
Don't override $record{source} due to BRF, or the script will crash. Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
The --only-drm filtering rules were filtering too much, ending to exclude functions on files with "h" on their names. Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Mauro Carvalho Chehab authored
Some fields (FNH, BRH, LH) produce warning messages when they're found at the info file. Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
- Oct 05, 2022
-
-
Mauro Carvalho Chehab authored
The logic which prints internal_assert expects a line break, as otherwise it would print things like: must only be called outside of a subtestplease refer to lib/igt_core documentation So, add a \n on a couple of places that it is missing it. Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Nirmoy <Das<nirmoy.das@intel.com> Acked-by:
Andrzej Hajda <andrzej.hajda@intel.com>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Andrzej Hajda <andrzej.hajda@intel.com> Reviewed-by:
Nirmoy <Das<nirmoy.das@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Add a missing close(master) to the subtest group fixture to prevent a leak during later tests. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Andrzej Hajda <andrzej.hajda@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Michael J. Ruhl <michael.j.ruhl@intel.com> Reviewed-by:
Nirmoy <Das<nirmoy.das@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Michael J. Ruhl <michael.j.ruhl@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Michael J. Ruhl <michael.j.ruhl@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Michael J. Ruhl <michael.j.ruhl@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Michael J. Ruhl <michael.j.ruhl@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Michael J. Ruhl <michael.j.ruhl@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Reviewed-by:
Andrzej Hajda <andrzej.hajda@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Close the device fd before we check for leaks during the atexit handlers. Signed-off-by:
Chris Wilson <chris.p.wilson@intel.com> Acked-by:
Andrzej Hajda <andrzej.hajda@intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-