From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C66C2C4345F for ; Thu, 25 Apr 2024 15:36:12 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 5E30511A56D; Thu, 25 Apr 2024 15:36:12 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="dlwHhhxr"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.12]) by gabe.freedesktop.org (Postfix) with ESMTPS id A0EDE11A56D for ; Thu, 25 Apr 2024 15:36:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1714059371; x=1745595371; h=date:from:to:cc:subject:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=0/M0apZY23d94S+AxDy7U5DvqmKY61dXFUwil5OC/X4=; b=dlwHhhxrrpYyl2O/++pLoLXc8uL2SME89fd0FO3rKs8xQ579g2MaevUR 7v/3Mnpdau4CcNpuEzrE66pRl8fXTCOddJEWhom+2xLuSfOD3eacGoq7e NSBFljIJpZ/qQh3OXyxFM7AGztbUppYDZm66D6PS/vtXs7eR+U0sWnxUI XFPdlyMTRW7rHl4R4RB22l2TdaSZT8DzvpVJx8GcaLDgKm87SmaOvqXZL X7dbLm1XRJpRqhWNqAlpGk+acNl4r3CqbJShEkh1kqQpWpHZInqKvqdA4 sOMZ4ETwRgcAA2o+uF7pKaqYBljFHaQczLWEG07ZicRmDxEd1SeFEELCX Q==; X-CSE-ConnectionGUID: KtU4MwatQ7m08gHoM4JqEg== X-CSE-MsgGUID: 3Cb6ur27R/KUxAwBU8Z3Rg== X-IronPort-AV: E=McAfee;i="6600,9927,11055"; a="21176241" X-IronPort-AV: E=Sophos;i="6.07,229,1708416000"; d="scan'208";a="21176241" Received: from fmviesa002.fm.intel.com ([10.60.135.142]) by orvoesa104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Apr 2024 08:36:11 -0700 X-CSE-ConnectionGUID: SfY2BWIYRIaV7PF2m0yoyw== X-CSE-MsgGUID: veBykH/6QnuoN2/V6BhoVg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,229,1708416000"; d="scan'208";a="48369781" Received: from linux.intel.com ([10.54.29.200]) by fmviesa002.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Apr 2024 08:36:10 -0700 Received: from maurocar-mobl2 (maurocar-mobl2.ger.corp.intel.com [10.245.244.170]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by linux.intel.com (Postfix) with ESMTPS id 1EB2B206D89D; Thu, 25 Apr 2024 08:36:07 -0700 (PDT) Date: Thu, 25 Apr 2024 17:36:05 +0200 From: Mauro Carvalho Chehab To: Katarzyna Piecielska Cc: igt-dev@lists.freedesktop.org, Kamil Konieczny , Aditya Chauhan , Girotra Himanshu , Sandeep Kumar Parupalli Subject: Re: [PATCH i-g-t v2 00/16] Introduce Mega feature to Xe and i915 tests Message-ID: <20240425173605.7a701d61@maurocar-mobl2> In-Reply-To: <20240423104913.3559587-1-katarzyna.piecielska@intel.com> References: <20240423104913.3559587-1-katarzyna.piecielska@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: igt-dev@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development mailing list for IGT GPU Tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" On Tue, 23 Apr 2024 10:48:57 +0000 Katarzyna Piecielska wrote: > After Xe and i915 test documentation alignment there is now time to > add Mega feature field. After live review of v1 we decided to reduce > number of Mega feature values: > > - General Core features will cover: > legacy and DRIVER_ANY tests > legacy and DRIVER_INTEL i915 tests > general Xe tests > Blitter tests > Command Submission tests > Memory management tests, VMA, > - Compute > Compute tests > GPGPU tests > - Compression > FlatCCS tests > - Media > Media tests > - Performance interface > Performance interface tests > - Power management > Power management tests > - RAS > RAS tests > - Render > Render copy, 3d > - SysMan > SysMan tests > > Also adding missing Mega feature to Display tests: General Display > Features. Those are the feature-mapping warnings after applying this series - for Xe: WARNING: feature Blitter has multiple parents: Hardware building block and Software building block WARNING: feature Power management has multiple parents: Infrastructure and Software building block WARNING: feature driver has multiple parents: Software building block and Hardware building block WARNING: feature RAS has multiple parents: Software Building Block and Hardware building block WARNING: feature CMD submission has multiple parents: Hardware building block and Software building block WARNING: feature MultiGPU has multiple parents: Hardware building block and Software building block WARNING: feature compute is duplicated at Functionality and Sub-category WARNING: feature compute has multiple parents: CMD submission and Hardware building block WARNING: feature compute has multiple hierarchical levels: 3 and 2 - for i915: WARNING: feature CMD submission has multiple parents: None and Infrastructure WARNING: feature CMD submission has multiple hierarchical levels: 1 and 2 WARNING: feature Memory management has multiple parents: Server and None WARNING: feature Memory management has multiple hierarchical levels: 2 and 1 WARNING: feature Power management has multiple parents: Firmware and None WARNING: feature Power management has multiple hierarchical levels: 2 and 1 WARNING: feature SysMan has multiple parents: None and Desktop \ Server WARNING: feature SysMan has multiple hierarchical levels: 1 and 2 WARNING: feature uapi has multiple parents: Infrastructure and None WARNING: feature uapi has multiple hierarchical levels: 2 and 1 WARNING: feature driver handler has multiple parents: CMD submission and driver WARNING: feature buffer management has multiple parents: Memory management and DRM WARNING: feature Blitter is duplicated at Functionality and Sub-category WARNING: feature Blitter has multiple parents: blitter and None WARNING: feature Blitter has multiple hierarchical levels: 2 and 1 WARNING: feature Context has multiple parents: Desktop client and Infrastructure WARNING: feature Performance has multiple parents: Server and None WARNING: feature Performance has multiple hierarchical levels: 2 and 1 WARNING: feature Display has multiple parents: None and Selftest WARNING: feature Display has multiple hierarchical levels: 1 and 2 WARNING: feature CMD Submission has multiple parents: None and Selftest WARNING: feature CMD Submission has multiple hierarchical levels: 1 and 2 WARNING: feature SR-IOV has multiple parents: Desktop \ Server and HW WARNING: feature driver has multiple parents: Infrastructure and None WARNING: feature driver has multiple hierarchical levels: 2 and 1 WARNING: feature DRM has multiple parents: Desktop client and Infrastructure > > Signed-off-by: Katarzyna Piecielska > Cc: Kamil Konieczny > Cc: Mauro Carvalho Chehab > Cc: Aditya Chauhan > Cc: Girotra Himanshu > CC: Sandeep Kumar Parupalli > > Katarzyna Piecielska (16): > tests/intel: Introduce General Core Mega feature > tests: Add Memory management tests to General Core Mega feature > tests/intel: Add Blitter tests to General Core Mega feature > tests/intel: Introduce Power management Mega feature > tests/intel: Introduce Compute Mega feature > tests: Add DRIVER_ANY tests to General Core Mega feature > tests/intel: Add legacy i915 tests to General Core Mega feature > tests/intel: Introduce SysMan Mega feature > tests/intel: Add general XE tests to General Core Mega feature > tests/intel: Introduce Performance Mega feature > tests/intel: Introduce Compression Mega feature > tests/intel: Introduce Render Mega feature > tests/intel: Add GPGPU tests to Compute Mega feature > tests/intel: Introduce Media Mega feature > tests/intel: Introduce RAS Mega feature > tests/intel: Add display tests to General Display Features > > tests/core_auth.c | 7 +- > tests/core_getclient.c | 7 +- > tests/core_getstats.c | 7 +- > tests/core_getversion.c | 7 +- > tests/core_hotunplug.c | 5 +- > tests/core_setmaster.c | 7 +- > tests/core_setmaster_vs_auth.c | 7 +- > tests/device_reset.c | 11 +- > tests/dmabuf.c | 4 +- > tests/drm_mm.c | 5 +- > tests/dumb_buffer.c | 7 +- > tests/intel/api_intel_allocator.c | 5 +- > tests/intel/api_intel_bb.c | 7 +- > tests/intel/debugfs_test.c | 5 +- > tests/intel/drm_fdinfo.c | 7 +- > tests/intel/gem_bad_reloc.c | 5 +- > tests/intel/gem_barrier_race.c | 1 + > tests/intel/gem_basic.c | 5 +- > tests/intel/gem_blits.c | 5 +- > tests/intel/gem_busy.c | 3 +- > tests/intel/gem_caching.c | 7 +- > tests/intel/gem_ccs.c | 5 +- > tests/intel/gem_close.c | 5 +- > tests/intel/gem_close_race.c | 5 +- > tests/intel/gem_compute.c | 5 +- > tests/intel/gem_create.c | 5 +- > tests/intel/gem_cs_tlb.c | 7 +- > tests/intel/gem_ctx_bad_destroy.c | 7 +- > tests/intel/gem_ctx_create.c | 5 +- > tests/intel/gem_ctx_engines.c | 5 +- > tests/intel/gem_ctx_exec.c | 5 +- > tests/intel/gem_ctx_freq.c | 5 +- > tests/intel/gem_ctx_isolation.c | 5 +- > tests/intel/gem_ctx_param.c | 5 +- > tests/intel/gem_ctx_persistence.c | 5 +- > tests/intel/gem_ctx_shared.c | 5 +- > tests/intel/gem_ctx_sseu.c | 5 +- > tests/intel/gem_ctx_switch.c | 1 + > tests/intel/gem_eio.c | 5 +- > tests/intel/gem_evict_alignment.c | 3 +- > tests/intel/gem_evict_everything.c | 3 +- > tests/intel/gem_exec_alignment.c | 7 +- > tests/intel/gem_exec_async.c | 5 +- > tests/intel/gem_exec_await.c | 5 +- > tests/intel/gem_exec_balancer.c | 5 +- > tests/intel/gem_exec_basic.c | 5 +- > tests/intel/gem_exec_big.c | 3 +- > tests/intel/gem_exec_capture.c | 5 +- > tests/intel/gem_exec_create.c | 5 +- > tests/intel/gem_exec_endless.c | 1 + > tests/intel/gem_exec_fair.c | 5 +- > tests/intel/gem_exec_fence.c | 5 +- > tests/intel/gem_exec_flush.c | 1 + > tests/intel/gem_exec_gttfill.c | 5 +- > tests/intel/gem_exec_latency.c | 3 +- > tests/intel/gem_exec_lut_handle.c | 3 +- > tests/intel/gem_exec_nop.c | 5 +- > tests/intel/gem_exec_parallel.c | 6 +- > tests/intel/gem_exec_params.c | 5 +- > tests/intel/gem_exec_reloc.c | 3 +- > tests/intel/gem_exec_schedule.c | 5 +- > tests/intel/gem_exec_store.c | 7 +- > tests/intel/gem_exec_suspend.c | 5 +- > tests/intel/gem_exec_whisper.c | 3 +- > tests/intel/gem_exercise_blt.c | 5 +- > tests/intel/gem_fd_exhaustion.c | 3 +- > tests/intel/gem_fence_thrash.c | 5 +- > tests/intel/gem_fence_upload.c | 3 +- > tests/intel/gem_fenced_exec_thrash.c | 1 + > tests/intel/gem_flink_basic.c | 7 +- > tests/intel/gem_flink_race.c | 7 +- > tests/intel/gem_gpgpu_fill.c | 3 +- > tests/intel/gem_gtt_cpu_tlb.c | 8 +- > tests/intel/gem_gtt_hog.c | 3 +- > tests/intel/gem_gtt_speed.c | 3 +- > tests/intel/gem_huc_copy.c | 7 +- > tests/intel/gem_linear_blits.c | 7 +- > tests/intel/gem_lmem_evict.c | 5 +- > tests/intel/gem_lmem_swapping.c | 7 +- > tests/intel/gem_lut_handle.c | 3 +- > tests/intel/gem_madvise.c | 7 +- > tests/intel/gem_media_fill.c | 7 +- > tests/intel/gem_media_vme.c | 5 +- > tests/intel/gem_mmap.c | 7 +- > tests/intel/gem_mmap_gtt.c | 5 +- > tests/intel/gem_mmap_offset.c | 5 +- > tests/intel/gem_mmap_wc.c | 5 +- > tests/intel/gem_partial_pwrite_pread.c | 5 +- > tests/intel/gem_pipe_control_store_loop.c | 5 +- > tests/intel/gem_ppgtt.c | 5 +- > tests/intel/gem_pread.c | 5 +- > tests/intel/gem_pread_after_blit.c | 3 +- > tests/intel/gem_pwrite.c | 3 +- > tests/intel/gem_pwrite_snooped.c | 1 + > tests/intel/gem_pxp.c | 7 +- > tests/intel/gem_read_read_speed.c | 3 +- > tests/intel/gem_readwrite.c | 5 +- > tests/intel/gem_reg_read.c | 5 +- > tests/intel/gem_render_copy.c | 5 +- > tests/intel/gem_render_copy_redux.c | 3 +- > tests/intel/gem_render_linear_blits.c | 5 +- > tests/intel/gem_render_tiled_blits.c | 5 +- > tests/intel/gem_request_retire.c | 7 +- > tests/intel/gem_reset_stats.c | 5 +- > tests/intel/gem_ringfill.c | 7 +- > tests/intel/gem_set_tiling_vs_blt.c | 7 +- > tests/intel/gem_set_tiling_vs_gtt.c | 5 +- > tests/intel/gem_set_tiling_vs_pwrite.c | 5 +- > tests/intel/gem_shrink.c | 3 +- > tests/intel/gem_softpin.c | 7 +- > tests/intel/gem_spin_batch.c | 7 +- > tests/intel/gem_streaming_writes.c | 3 +- > tests/intel/gem_stress.c | 1 + > tests/intel/gem_sync.c | 7 +- > tests/intel/gem_tiled_blits.c | 5 +- > tests/intel/gem_tiled_fence_blits.c | 5 +- > tests/intel/gem_tiled_partial_pwrite_pread.c | 5 +- > tests/intel/gem_tiled_pread_basic.c | 5 +- > tests/intel/gem_tiled_pread_pwrite.c | 5 +- > tests/intel/gem_tiled_swapping.c | 5 +- > tests/intel/gem_tiled_wb.c | 5 +- > tests/intel/gem_tiled_wc.c | 5 +- > tests/intel/gem_tiling_max_stride.c | 5 +- > tests/intel/gem_unfence_active_buffers.c | 5 +- > tests/intel/gem_unref_active_buffers.c | 7 +- > tests/intel/gem_userptr_blits.c | 5 +- > tests/intel/gem_vm_create.c | 3 +- > tests/intel/gem_wait.c | 9 +- > tests/intel/gem_watchdog.c | 5 +- > tests/intel/gem_workarounds.c | 5 +- > tests/intel/gen3_mixed_blits.c | 1 + > tests/intel/gen3_render_linear_blits.c | 1 + > tests/intel/gen3_render_mixed_blits.c | 1 + > tests/intel/gen3_render_tiledx_blits.c | 1 + > tests/intel/gen3_render_tiledy_blits.c | 1 + > tests/intel/gen7_exec_parse.c | 3 +- > tests/intel/gen9_exec_parse.c | 7 +- > tests/intel/i915_fb_tiling.c | 5 +- > tests/intel/i915_getparams_basic.c | 7 +- > tests/intel/i915_hangman.c | 7 +- > tests/intel/i915_module_load.c | 7 +- > tests/intel/i915_pciid.c | 7 +- > tests/intel/i915_pipe_stress.c | 4 +- > tests/intel/i915_pm_freq_api.c | 1 + > tests/intel/i915_pm_freq_mult.c | 7 +- > tests/intel/i915_pm_rc6_residency.c | 7 +- > tests/intel/i915_pm_rpm.c | 7 +- > tests/intel/i915_pm_rps.c | 7 +- > tests/intel/i915_pm_sseu.c | 5 +- > tests/intel/i915_power.c | 7 +- > tests/intel/i915_query.c | 7 +- > tests/intel/i915_selftest.c | 370 +++++++++++-------- > tests/intel/i915_suspend.c | 8 +- > tests/intel/intel_hwmon.c | 3 +- > tests/intel/perf.c | 7 +- > tests/intel/perf_pmu.c | 7 +- > tests/intel/prime_busy.c | 7 +- > tests/intel/prime_mmap.c | 7 +- > tests/intel/prime_mmap_coherency.c | 7 +- > tests/intel/prime_mmap_kms.c | 5 +- > tests/intel/prime_self_import.c | 7 +- > tests/intel/sysfs_defaults.c | 5 +- > tests/intel/sysfs_heartbeat_interval.c | 7 +- > tests/intel/sysfs_preempt_timeout.c | 7 +- > tests/intel/sysfs_timeslice_duration.c | 7 +- > tests/intel/xe_ccs.c | 3 +- > tests/intel/xe_compute.c | 3 +- > tests/intel/xe_compute_preempt.c | 3 +- > tests/intel/xe_copy_basic.c | 3 +- > tests/intel/xe_create.c | 1 + > tests/intel/xe_debugfs.c | 3 +- > tests/intel/xe_dma_buf_sync.c | 1 + > tests/intel/xe_drm_fdinfo.c | 3 +- > tests/intel/xe_evict.c | 2 + > tests/intel/xe_evict_ccs.c | 3 +- > tests/intel/xe_exec_atomic.c | 1 + > tests/intel/xe_exec_balancer.c | 2 +- > tests/intel/xe_exec_basic.c | 1 + > tests/intel/xe_exec_compute_mode.c | 1 + > tests/intel/xe_exec_fault_mode.c | 1 + > tests/intel/xe_exec_queue_property.c | 1 + > tests/intel/xe_exec_reset.c | 5 +- > tests/intel/xe_exec_store.c | 1 + > tests/intel/xe_exec_threads.c | 1 + > tests/intel/xe_exercise_blt.c | 3 +- > tests/intel/xe_gpgpu_fill.c | 3 +- > tests/intel/xe_gt_freq.c | 3 +- > tests/intel/xe_huc_copy.c | 1 + > tests/intel/xe_intel_bb.c | 1 + > tests/intel/xe_live_ktest.c | 1 + > tests/intel/xe_media_fill.c | 9 +- > tests/intel/xe_mmap.c | 3 +- > tests/intel/xe_module_load.c | 1 + > tests/intel/xe_noexec_ping_pong.c | 3 +- > tests/intel/xe_pat.c | 1 + > tests/intel/xe_peer2peer.c | 3 +- > tests/intel/xe_pm.c | 3 +- > tests/intel/xe_pm_residency.c | 3 +- > tests/intel/xe_prime_self_import.c | 1 + > tests/intel/xe_query.c | 66 ++-- > tests/intel/xe_render_copy.c | 1 + > tests/intel/xe_spin_batch.c | 1 + > tests/intel/xe_sysfs_defaults.c | 6 +- > tests/intel/xe_sysfs_scheduler.c | 3 +- > tests/intel/xe_vm.c | 1 + > tests/intel/xe_waitfence.c | 2 + > tests/prime_vgem.c | 7 +- > tests/sw_sync.c | 9 +- > tests/syncobj_basic.c | 7 +- > tests/syncobj_timeline.c | 9 +- > tests/syncobj_wait.c | 9 +- > tests/vgem_basic.c | 5 +- > tests/vgem_slow.c | 7 +- > 213 files changed, 859 insertions(+), 559 deletions(-) >