[PATCH i-g-t v4 11/11] tests/xe_intel_bb: Use supported tilings instead hardcoded ones
Karolina Stolarek
karolina.stolarek at intel.com
Thu May 9 05:18:18 UTC 2024
On 8.05.2024 14:54, Zbigniew Kempczyński wrote:
> Use introduced render tilings in cmds-info to select appropriate one
> on which render subtest is executed.
>
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
Reviewed-by: Karolina Stolarek <karolina.stolarek at intel.com>
> ---
> tests/intel/xe_intel_bb.c | 25 +++++++++++--------------
> 1 file changed, 11 insertions(+), 14 deletions(-)
>
> diff --git a/tests/intel/xe_intel_bb.c b/tests/intel/xe_intel_bb.c
> index 09164c41fc..ad6a2d22ca 100644
> --- a/tests/intel/xe_intel_bb.c
> +++ b/tests/intel/xe_intel_bb.c
> @@ -18,6 +18,7 @@
>
> #include "igt.h"
> #include "igt_crc.h"
> +#include "intel_blt.h"
> #include "intel_bufops.h"
> #include "intel_mocs.h"
> #include "intel_pat.h"
> @@ -978,19 +979,10 @@ const char *help_str =
>
> igt_main_args("dpib", NULL, help_str, opt_handler, NULL)
> {
> - int xe, i;
> + int xe;
> struct buf_ops *bops;
> uint32_t width;
>
> - struct test {
> - uint32_t tiling;
> - const char *tiling_name;
> - } tests[] = {
> - { I915_TILING_NONE, "none" },
> - { I915_TILING_X, "x" },
> - { I915_TILING_4, "4" },
> - };
> -
> igt_fixture {
> xe = drm_open_driver(DRIVER_XE);
> bops = buf_ops_create(xe);
> @@ -1053,14 +1045,19 @@ igt_main_args("dpib", NULL, help_str, opt_handler, NULL)
> delta_check(bops);
>
> igt_subtest_with_dynamic("render") {
> + int tiling;
> +
> igt_require(xe_has_engine_class(xe, DRM_XE_ENGINE_CLASS_RENDER));
>
> - for (i = 0; i < ARRAY_SIZE(tests); i++) {
> - const struct test *t = &tests[i];
> + for_each_tiling(tiling) {
> + if (!render_supports_tiling(xe, tiling, false))
> + continue;
>
> for (width = 512; width <= 1024; width += 512)
> - igt_dynamic_f("render-%s-%u", t->tiling_name, width)
> - render(bops, t->tiling, width, width);
> + igt_dynamic_f("render-%s-%u",
> + blt_tiling_name(tiling), width)
> + render(bops, blt_tile_to_i915_tile(tiling),
> + width, width);
> }
> }
>
More information about the igt-dev
mailing list