[igt-dev] [PATCH i-g-t 1/2] tests: Add feature_discovery

[Lukasz Fiedorowicz]

Arkadiusz Hiler <arkadiusz.hiler at intel.com> writes:

> The main reason for adding this test is automatic tagging of the
> machines for the purposes of CI and other automation systems.
>
> The subtest are not doing much - they just check the presence of
> interfaces and available hardware to make sure certain "feature" is
> there and ready to be tested properly.
>
> The subtests should only pass or skip.
>
> Any failures or flip-floppers are reason for serious concern.
>
> Cc: Martin Peres <martin.peres at intel.com>
> Cc: Petri Latvala <petri.latvala at intel.com>
> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler at intel.com>

Reviewed-by: Lukasz Fiedorowicz <lukasz.fiedorowicz at intel.com>

> ---
>  tests/feature_discovery.c | 111 ++++++++++++++++++++++++++++++++++++++
>  tests/meson.build         |   1 +
>  2 files changed, 112 insertions(+)
>  create mode 100644 tests/feature_discovery.c
>
> diff --git a/tests/feature_discovery.c b/tests/feature_discovery.c
> new file mode 100644
> index 00000000..32ef9df7
> --- /dev/null
> +++ b/tests/feature_discovery.c
> @@ -0,0 +1,111 @@
> +/*
> + * Copyright © 2020 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + *
> + */
> +
> +#include "igt.h"
> +#include "igt_chamelium.h"
> +#include "igt_kms.h"
> +#include "igt_psr.h"
> +#include "igt_sysfs.h"
> +
> +static int fd;
> +static int debugfs_fd;
> +static igt_display_t display;
> +
> +IGT_TEST_DESCRIPTION("A metatest that checks for \"features\" presence. "
> +		     "The subtests here should only skip or pass, "
> +		     "anything else means we have a serious problem.");
> +igt_main {
> +
> +	igt_fixture {
> +		fd = drm_open_driver_master(DRIVER_ANY);
> +		debugfs_fd = igt_debugfs_dir(fd);
> +	}
> +
> +	igt_subtest_group {
> +
> +		igt_fixture {
> +			igt_display_require(&display, fd);
> +		}
> +
> +		igt_describe("Make sure that we have display support with some outputs connected.");
> +		igt_subtest("display") {
> +			/* will skip because of the fixture */
> +		}
> +
> +		igt_subtest_group {
> +			volatile int output_count = 0;
> +			igt_output_t *output;
> +			enum pipe pipe;
> +
> +			igt_fixture {
> +				/* this is what most of the 2x tests are doing */
> +				for_each_pipe(&display, pipe) {
> +					for_each_valid_output_on_pipe(&display, pipe, output) {
> +						if (output->pending_pipe == PIPE_NONE) {
> +							igt_output_set_pipe(output, pipe);
> +							output_count++;
> +							break;
> +						}
> +					}
> +				}
> +
> +				for (int i = 0; i < display.n_outputs; i++) {
> +					igt_output_set_pipe(&display.outputs[i], PIPE_NONE);
> +				}
> +			}
> +
> +			igt_describe("Make sure that we can use at least 2 outputs at the same time.");
> +			igt_subtest("display-2x") {
> +				igt_require(output_count >= 2);
> +			}
> +
> +			igt_describe("Make sure that we can use at least 3 outputs at the same time.");
> +			igt_subtest("display-3x") {
> +				igt_require(output_count >= 3);
> +			}
> +
> +			igt_describe("Make sure that we can use at least 4 outputs at the same time.");
> +			igt_subtest("display-4x") {
> +				igt_require(output_count >= 4);
> +			}
> +		}
> +
> +		igt_describe("Make sure that Chamelium is configured and reachable.");
> +		igt_subtest("chamelium") {
> +			struct chamelium *chamelium = chamelium_init(fd);
> +			igt_require(chamelium);
> +			chamelium_deinit(chamelium);
> +		}
> +
> +		igt_describe("Make sure that we have eDP panel with PSR1 support.");
> +		igt_subtest("psr1") {
> +			igt_require(psr_sink_support(fd, debugfs_fd, PSR_MODE_1));
> +		}
> +
> +		igt_describe("Make sure that we have eDP panel with PSR2 support.");
> +		igt_subtest("psr2") {
> +			igt_require(psr_sink_support(fd, debugfs_fd, PSR_MODE_2));
> +		}
> +	}
> +}
> diff --git a/tests/meson.build b/tests/meson.build
> index ca792ed8..d7f3b373 100644
> --- a/tests/meson.build
> +++ b/tests/meson.build
> @@ -13,6 +13,7 @@ test_progs = [
>  	'drm_mm',
>  	'drm_read',
>  	'fbdev',
> +	'feature_discovery',
>  	'kms_3d',
>  	'kms_addfb_basic',
>  	'kms_atomic',
> -- 
> 2.25.4
>
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev

-- 
Thanks,
Lukasz