Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [i-g-t 00/45] Add IGT display support for XE
@ 2023-04-18 16:44 Bhanuprakash Modem
  2023-04-18 16:44 ` [igt-dev] [i-g-t 01/45] lib/xe_ioctl: Add missing header for direct resolving Bhanuprakash Modem
                   ` (48 more replies)
  0 siblings, 49 replies; 72+ messages in thread
From: Bhanuprakash Modem @ 2023-04-18 16:44 UTC (permalink / raw)
  To: igt-dev

Add IGT support to work kms tests on XE driver.

TODO: Spin, BUSY & HANG related tests.

Bhanuprakash Modem (35):
  lib: Use i915 namespace for XE debugfs
  lib/igt_fb: Add copy engine support for XE
  lib/igt_draw: Add gpu draw routine support for XE
  tests/i915/kms_big_fb: Add XE support
  tests/i915/kms_big_joiner: Add XE support
  tests/i915/kms_cdclk: Add XE support
  tests/i915/kms_draw_crc: Add XE support
  tests/i915/kms_dsc: Add XE support
  tests/i915/kms_flip_scaled_crc: Add XE support
  tests/i915/kms_flip_tiling: Add XE support
  tests/i915/kms_mmap_write_crc: Add XE support
  tests/i915/kms_psr*: Add XE support
  tests/kms_3d: Add XE support
  tests/kms_async_flips: No XE support
  tests/kms_atomic_transition: Add XE support
  tests/color: Add XE support
  tests/kms_concurrent: Add XE support
  tests/kms_content_protection: Add XE support
  tests/kms_cursor_crc: Add XE support
  tests/kms_cursor_legacy: Add XE support
  tests/kms_flip: Add XE support
  tests/kms_flip_event_leak: Add XE support
  tests/kms_getfb: Add XE support
  tests/kms_hdmi_inject: Add XE support
  tests/kms_hdr: Add XE support
  tests/kms_panel_fitting: Add XE support
  tests/kms_plane: Add XE support
  tests/kms_plane_lowers: Add XE support
  tests/kms_plane_scaling: Add XE support
  tests/kms_prime: Add XE support
  tests/kms_rotation_crc: Add XE support
  tests/kms_setmode: Add XE support
  tests/kms_universal_plane: Add XE support
  tests/kms: Move intel specific kms tests to new dir
  xe-fast-feedback: Add kms tests to XE BAT

Zbigniew Kempczyński (10):
  lib/xe_ioctl: Add missing header for direct resolving
  lib/xe_query: Add region helpers and missing doc
  lib/xe_query: Remove commented out function prototype
  lib/intel_allocator: Add allocator support for Xe
  lib/intel_bufops: Add Xe support in bufops
  lib/intel_batchbuffer: Add Xe support in intel-bb
  tests/xe_intel_bb: Check if intel-bb Xe support correctness
  tests/xe-fast-feedback: Add xe_intel_bb test to BAT
  lib/gpgpu_fill: Use RENDER engine flag to work on Xe
  tests/xe_gpgpu_fill: Exercise gpgpu fill on Xe

 lib/gpgpu_fill.c                            |    4 +-
 lib/gpu_cmds.c                              |    2 +-
 lib/igt_draw.c                              |   82 +-
 lib/igt_draw.h                              |    2 +
 lib/igt_fb.c                                |   30 +-
 lib/igt_fb.h                                |    2 +
 lib/igt_kms.c                               |    4 +-
 lib/igt_psr.c                               |    4 +-
 lib/intel_allocator.c                       |   49 +-
 lib/intel_aux_pgtable.c                     |    2 +-
 lib/intel_batchbuffer.c                     |  430 +++++--
 lib/intel_batchbuffer.h                     |   28 +-
 lib/intel_bufops.c                          |   73 +-
 lib/intel_bufops.h                          |   13 +-
 lib/xe/xe_ioctl.h                           |    1 +
 lib/xe/xe_query.c                           |   45 +
 lib/xe/xe_query.h                           |    3 +-
 tests/chamelium/kms_chamelium_color.c       |    2 +-
 tests/i915/gem_caching.c                    |    4 +-
 tests/i915/gem_pxp.c                        |    2 +-
 tests/intel-ci/xe-fast-feedback.testlist    |   50 +
 tests/{i915 => intel}/kms_big_fb.c          |   75 +-
 tests/{i915 => intel}/kms_big_joiner.c      |    2 +-
 tests/{i915 => intel}/kms_cdclk.c           |    5 +-
 tests/{i915 => intel}/kms_draw_crc.c        |   24 +-
 tests/{i915 => intel}/kms_dsc.c             |    2 +-
 tests/{i915 => intel}/kms_dsc_helper.c      |    0
 tests/{i915 => intel}/kms_dsc_helper.h      |    0
 tests/{i915 => intel}/kms_flip_scaled_crc.c |    8 +-
 tests/{i915 => intel}/kms_flip_tiling.c     |    8 +-
 tests/{i915 => intel}/kms_mmap_write_crc.c  |    4 +-
 tests/{i915 => intel}/kms_psr.c             |   40 +-
 tests/{i915 => intel}/kms_psr2_sf.c         |    2 +-
 tests/{i915 => intel}/kms_psr2_su.c         |    2 +-
 tests/{i915 => intel}/kms_psr_stress_test.c |    2 +-
 tests/kms_3d.c                              |    8 +
 tests/kms_async_flips.c                     |    8 +-
 tests/kms_atomic_transition.c               |    8 +-
 tests/kms_color.c                           |   10 +-
 tests/kms_color_helper.c                    |    2 +-
 tests/kms_concurrent.c                      |    4 +-
 tests/kms_content_protection.c              |   12 +-
 tests/kms_cursor_crc.c                      |    4 +-
 tests/kms_cursor_legacy.c                   |    4 +-
 tests/kms_flip.c                            |   32 +-
 tests/kms_flip_event_leak.c                 |    5 +
 tests/kms_getfb.c                           |   22 +-
 tests/kms_hdmi_inject.c                     |   10 +-
 tests/kms_hdr.c                             |    8 +-
 tests/kms_panel_fitting.c                   |   12 +-
 tests/kms_plane.c                           |   19 +-
 tests/kms_plane_lowres.c                    |    9 +-
 tests/kms_plane_scaling.c                   |   41 +-
 tests/kms_prime.c                           |  241 +++-
 tests/kms_rotation_crc.c                    |   25 +-
 tests/kms_setmode.c                         |    7 +
 tests/kms_universal_plane.c                 |   15 +-
 tests/meson.build                           |   39 +-
 tests/xe/xe_gpgpu_fill.c                    |  133 +++
 tests/xe/xe_intel_bb.c                      | 1181 +++++++++++++++++++
 60 files changed, 2513 insertions(+), 352 deletions(-)
 rename tests/{i915 => intel}/kms_big_fb.c (92%)
 rename tests/{i915 => intel}/kms_big_joiner.c (99%)
 rename tests/{i915 => intel}/kms_cdclk.c (98%)
 rename tests/{i915 => intel}/kms_draw_crc.c (92%)
 rename tests/{i915 => intel}/kms_dsc.c (99%)
 rename tests/{i915 => intel}/kms_dsc_helper.c (100%)
 rename tests/{i915 => intel}/kms_dsc_helper.h (100%)
 rename tests/{i915 => intel}/kms_flip_scaled_crc.c (98%)
 rename tests/{i915 => intel}/kms_flip_tiling.c (96%)
 rename tests/{i915 => intel}/kms_mmap_write_crc.c (98%)
 rename tests/{i915 => intel}/kms_psr.c (93%)
 rename tests/{i915 => intel}/kms_psr2_sf.c (99%)
 rename tests/{i915 => intel}/kms_psr2_su.c (99%)
 rename tests/{i915 => intel}/kms_psr_stress_test.c (99%)
 create mode 100644 tests/xe/xe_gpgpu_fill.c
 create mode 100644 tests/xe/xe_intel_bb.c

--
2.40.0

^ permalink raw reply	[flat|nested] 72+ messages in thread
* Re: [igt-dev] [i-g-t, 17/45] tests/i915/kms_draw_crc: Add XE support
@ 2023-04-20  9:01 Gupta, Nidhi1
  0 siblings, 0 replies; 72+ messages in thread
From: Gupta, Nidhi1 @ 2023-04-20  9:01 UTC (permalink / raw)
  To: igt-dev@lists.freedesktop.org

[-- Attachment #1: Type: text/plain, Size: 5034 bytes --]

On 2023-04-18 at 09:10 PM, Bhanuprakash Modem wrote:



kms_draw_crc is failing with xe changes, initially to collect CRC the draw function igt_draw_rect()  is failing due to failure of

xe_vm_bind_array function in xe_ioctl file.

Test assertion failure function xe_vm_bind_array, file ../lib/xe/xe_ioctl.c:88

Are we missing something here?



>Add XE driver support for differnt draw methods.

Spelling of "different" is not correct.

>

>Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>

>---

>tests/i915/kms_draw_crc.c | 24 ++++++++++++++----------

> 1 file changed, 14 insertions(+), 10 deletions(-)

>

>diff --git a/tests/i915/kms_draw_crc.c b/tests/i915/kms_draw_crc.c

>index 3a3cd99a0..db6d71f2e 100644

>--- a/tests/i915/kms_draw_crc.c

>+++ b/tests/i915/kms_draw_crc.c

>@@ -149,8 +149,8 @@  static void draw_method_subtest(enum igt_draw_method method,

>              * comparison. Cache the value so we don't recompute it for every single

>             * subtest. */

>             if (!base_crcs[format_index].set) {

>-                          get_method_crc(gem_has_mappable_ggtt(drm_fd) ? IGT_DRAW_MMAP_GTT :

>-                                                                                                            IGT_DRAW_MMAP_WC,

>+                          get_method_crc(igt_draw_supports_method(drm_fd, IGT_DRAW_MMAP_GTT) ?

>+                                                IGT_DRAW_MMAP_GTT : IGT_DRAW_MMAP_WC,

>                                                  formats[format_index],

>                                                  DRM_FORMAT_MOD_LINEAR,

>                                                  &base_crcs[format_index].crc);

>@@ -195,8 +195,8 @@  static void fill_fb_subtest(void)

>             igt_plane_set_fb(primary, &fb);

>

>             igt_draw_rect_fb(drm_fd, bops, 0, &fb,

>-                                         gem_has_mappable_ggtt(drm_fd) ? IGT_DRAW_MMAP_GTT :

>-                                                                                                     IGT_DRAW_MMAP_WC,

>+                                         igt_draw_supports_method(drm_fd, IGT_DRAW_MMAP_GTT) ?

>+                                         IGT_DRAW_MMAP_GTT : IGT_DRAW_MMAP_WC,

>                                            0, 0, fb.width, fb.height, 0xFF);

Why igt_draw_supports_method is used here?



-Nidhi

>

>             rc = igt_display_commit2(&display, display.is_atomic ? COMMIT_ATOMIC : COMMIT_LEGACY);

>@@ -222,7 +222,7 @@  static void fill_fb_subtest(void)

>

>static void setup_environment(void)

> {

>-           drm_fd = drm_open_driver_master(DRIVER_INTEL);

>+           drm_fd = drm_open_driver_master(DRIVER_INTEL | DRIVER_XE);

>             igt_require(drm_fd >= 0);

>             igt_display_require(&display, drm_fd);

>             igt_display_require_output(&display);

>@@ -289,13 +289,17 @@  igt_main

>                                                  "method (%s) with different modifiers & DRM formats.",

>                                                  igt_draw_get_method_name(method));

>                            igt_subtest_with_dynamic_f("draw-method-%s", igt_draw_get_method_name(method)) {

>-                                         igt_skip_on(method == IGT_DRAW_MMAP_WC && !gem_mmap__has_wc(drm_fd));

>-                                         igt_skip_on(method == IGT_DRAW_MMAP_GTT && !gem_has_mappable_ggtt(drm_fd));

>+                                         if (!igt_draw_supports_method(drm_fd, method))

>+                                                        continue;

>

>-                                         for (format_idx = 0; format_idx < ARRAY_SIZE(formats); format_idx++) {

>-                                                        for (modifier_idx = 0; modifier_idx < ARRAY_SIZE(modifiers); modifier_idx++) {

>-                                                                       modifier = modifiers[modifier_idx];

>+                                         for (modifier_idx = 0; modifier_idx < ARRAY_SIZE(modifiers); modifier_idx++) {

>+                                                        modifier = modifiers[modifier_idx];

>

>+                                                        /* No tiling support in XE. */

>+                                                        if (is_xe_device(drm_fd) && modifier != DRM_FORMAT_MOD_LINEAR)

>+                                                                       continue;

>+

>+                                                        for (format_idx = 0; format_idx < ARRAY_SIZE(formats); format_idx++) {

>                                                                         if (!igt_display_has_format_mod(&display, formats[format_idx], modifier))

>                                                                                        continue;




[-- Attachment #2: Type: text/html, Size: 18582 bytes --]

^ permalink raw reply	[flat|nested] 72+ messages in thread

end of thread, other threads:[~2023-04-25  4:17 UTC | newest]

Thread overview: 72+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-04-18 16:44 [igt-dev] [i-g-t 00/45] Add IGT display support for XE Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 01/45] lib/xe_ioctl: Add missing header for direct resolving Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 02/45] lib/xe_query: Add region helpers and missing doc Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 03/45] lib/xe_query: Remove commented out function prototype Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 04/45] lib/intel_allocator: Add allocator support for Xe Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 05/45] lib/intel_bufops: Add Xe support in bufops Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 06/45] lib/intel_batchbuffer: Add Xe support in intel-bb Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 07/45] tests/xe_intel_bb: Check if intel-bb Xe support correctness Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 08/45] tests/xe-fast-feedback: Add xe_intel_bb test to BAT Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 09/45] lib/gpgpu_fill: Use RENDER engine flag to work on Xe Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 10/45] tests/xe_gpgpu_fill: Exercise gpgpu fill " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 11/45] lib: Use i915 namespace for XE debugfs Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 12/45] lib/igt_fb: Add copy engine support for XE Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 13/45] lib/igt_draw: Add gpu draw routine " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 14/45] tests/i915/kms_big_fb: Add XE support Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 15/45] tests/i915/kms_big_joiner: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 16/45] tests/i915/kms_cdclk: " Bhanuprakash Modem
2023-04-21  9:47   ` Nautiyal, Ankit K
2023-04-18 16:44 ` [igt-dev] [i-g-t 17/45] tests/i915/kms_draw_crc: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 18/45] tests/i915/kms_dsc: " Bhanuprakash Modem
2023-04-21  9:51   ` Nautiyal, Ankit K
2023-04-18 16:44 ` [igt-dev] [i-g-t 19/45] tests/i915/kms_flip_scaled_crc: " Bhanuprakash Modem
2023-04-21  8:07   ` Nautiyal, Ankit K
2023-04-18 16:44 ` [igt-dev] [i-g-t 20/45] tests/i915/kms_flip_tiling: " Bhanuprakash Modem
2023-04-21  9:57   ` Nautiyal, Ankit K
2023-04-18 16:44 ` [igt-dev] [i-g-t 21/45] tests/i915/kms_mmap_write_crc: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 22/45] tests/i915/kms_psr*: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 23/45] tests/kms_3d: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 24/45] tests/kms_async_flips: No " Bhanuprakash Modem
2023-04-21  9:15   ` Karthik B S
2023-04-18 16:44 ` [igt-dev] [i-g-t 25/45] tests/kms_atomic_transition: Add " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 26/45] tests/color: " Bhanuprakash Modem
2023-04-24  8:00   ` [igt-dev] [i-g-t,26/45] " Joshi, Kunal1
2023-04-24  8:13     ` Modem, Bhanuprakash
2023-04-18 16:44 ` [igt-dev] [i-g-t 27/45] tests/kms_concurrent: " Bhanuprakash Modem
2023-04-24  8:08   ` [igt-dev] [i-g-t,27/45] " Joshi, Kunal1
2023-04-18 16:44 ` [igt-dev] [i-g-t 28/45] tests/kms_content_protection: " Bhanuprakash Modem
2023-04-24  9:02   ` [igt-dev] [i-g-t, " Joshi, Kunal1
2023-04-18 16:44 ` [igt-dev] [i-g-t 29/45] tests/kms_cursor_crc: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 30/45] tests/kms_cursor_legacy: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 31/45] tests/kms_flip: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 32/45] tests/kms_flip_event_leak: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 33/45] tests/kms_getfb: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 34/45] tests/kms_hdmi_inject: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 35/45] tests/kms_hdr: " Bhanuprakash Modem
2023-04-21 10:09   ` Nautiyal, Ankit K
2023-04-18 16:44 ` [igt-dev] [i-g-t 36/45] tests/kms_panel_fitting: " Bhanuprakash Modem
2023-04-21 10:17   ` Nautiyal, Ankit K
2023-04-18 16:44 ` [igt-dev] [i-g-t 37/45] tests/kms_plane: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 38/45] tests/kms_plane_lowers: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 39/45] tests/kms_plane_scaling: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 40/45] tests/kms_prime: " Bhanuprakash Modem
2023-04-25  4:16   ` Karthik B S
2023-04-18 16:44 ` [igt-dev] [i-g-t 41/45] tests/kms_rotation_crc: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 42/45] tests/kms_setmode: " Bhanuprakash Modem
2023-04-18 16:44 ` [igt-dev] [i-g-t 43/45] tests/kms_universal_plane: " Bhanuprakash Modem
2023-04-18 16:45 ` [igt-dev] [i-g-t 44/45] tests/kms: Move intel specific kms tests to new dir Bhanuprakash Modem
2023-04-18 16:45 ` [igt-dev] [i-g-t 45/45] xe-fast-feedback: Add kms tests to XE BAT Bhanuprakash Modem
2023-04-18 16:58 ` [igt-dev] [i-g-t 00/45] Add IGT display support for XE Ville Syrjälä
2023-04-19  7:01   ` Zbigniew Kempczyński
2023-04-19 11:57     ` Modem, Bhanuprakash
2023-04-19 16:19       ` Zbigniew Kempczyński
2023-04-21  6:47       ` Nautiyal, Ankit K
2023-04-21  7:33         ` Modem, Bhanuprakash
2023-04-21  9:33           ` Nautiyal, Ankit K
2023-04-18 17:24 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2023-04-18 21:28 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2023-04-21 16:43 ` [igt-dev] [i-g-t 00/45] " Kamil Konieczny
2023-04-24  5:21   ` Modem, Bhanuprakash
2023-04-24  9:45     ` Kamil Konieczny
2023-04-24 10:05       ` Modem, Bhanuprakash
  -- strict thread matches above, loose matches on Subject: below --
2023-04-20  9:01 [igt-dev] [i-g-t, 17/45] tests/i915/kms_draw_crc: Add XE support Gupta, Nidhi1

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox