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 16449C4332F for ; Wed, 13 Dec 2023 17:24:25 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 6900110E7D8; Wed, 13 Dec 2023 17:24:24 +0000 (UTC) Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.151]) by gabe.freedesktop.org (Postfix) with ESMTPS id 51E0210E7DA for ; Wed, 13 Dec 2023 17:24:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1702488263; x=1734024263; h=message-id:date:mime-version:subject:to:references:from: in-reply-to:content-transfer-encoding; bh=ziTlhBI4jUn7pqeQhu5zCa0ywWhDN2PHXBkb5swk/KE=; b=gRRGqVE2JUbRQcirYxFuwd0t9x/UuHGyt5mdUQKbUZrBxSII/tnDILxg hDfFPb/WqlzUPw+Vx9J2CCqJvdhuyZGL9ifKeua+xGLkp9zmVvODO0MLK 3Bf+5QnWYQMkOpNPmgwQR/8972sNEp8CqUyurdbRRDR31qyar/JPxfvV2 3vN8HR5SQqfmK5ArLkKm5ndNlUaZozvfnxBlIrGoY5hYMS9Km4+k6OMuE T1lNpTOiSi7gXoWuzTVqbYEmh0+3cmqSLj5BdRSKYbxOHztK39hS6JMnd 8uQ7U+LM0gPFiIP1XZvhWfbaIPwlBf06YdhjAdAkBS9+z9aG71tfpu5mt g==; X-IronPort-AV: E=McAfee;i="6600,9927,10923"; a="375148833" X-IronPort-AV: E=Sophos;i="6.04,273,1695711600"; d="scan'208";a="375148833" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Dec 2023 09:24:22 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10923"; a="750201122" X-IronPort-AV: E=Sophos;i="6.04,273,1695711600"; d="scan'208";a="750201122" Received: from kryanx-mobl.ger.corp.intel.com (HELO [10.213.231.240]) ([10.213.231.240]) by orsmga006-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Dec 2023 09:24:21 -0800 Message-ID: <8fc913ae-814d-4236-9ca0-e4492c81b225@linux.intel.com> Date: Wed, 13 Dec 2023 17:24:19 +0000 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: =?UTF-8?B?UmU6IMOiIEZpLkNJLkJBVDogZmFpbHVyZSBmb3IgZHJtL2k5MTU6IChz?= =?UTF-8?Q?tolen=29_memory_region_related_fixes?= Content-Language: en-US To: Andrzej Hajda , intel-gfx@lists.freedesktop.org, Patchwork , =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= References: <20231213004237.20375-1-ville.syrjala@linux.intel.com> <170243147264.10260.8557231803342985702@emeril.freedesktop.org> <8be9ac53-0945-4371-91f2-40deed57fd11@intel.com> From: Tvrtko Ursulin Organization: Intel Corporation UK Plc In-Reply-To: <8be9ac53-0945-4371-91f2-40deed57fd11@intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" Hi Andrzej, On 13/12/2023 16:13, Andrzej Hajda wrote: > On 13.12.2023 02:37, Patchwork wrote: >> *Patch Details* >> *Series:*    drm/i915: (stolen) memory region related fixes >> *URL:*    https://patchwork.freedesktop.org/series/127721/ >> >> *State:*    failure >> *Details:* >> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_127721v1/index.html >> >> >> >>   CI Bug Log - changes from CI_DRM_14010 -> Patchwork_127721v1 >> >> >>     Summary >> >> *FAILURE* >> >> Serious unknown changes coming with Patchwork_127721v1 absolutely need >> to be >> verified manually. >> >> If you think the reported changes have nothing to do with the changes >> introduced in Patchwork_127721v1, please notify your bug team >> (I915-ci-infra@lists.freedesktop.org) to allow them >> to document this new failure mode, which will reduce false positives >> in CI. >> >> External URL: >> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_127721v1/index.html >> >> >>     Participating hosts (31 -> 33) >> >> Additional (4): bat-dg2-8 bat-dg2-9 bat-mtlp-8 fi-pnv-d510 >> Missing (2): bat-adlp-11 fi-snb-2520m >> >> >>     Possible new issues >> >> Here are the unknown changes that may have been introduced in >> Patchwork_127721v1: >> >> >>       IGT changes >> >> >>         Possible regressions >> >>   * >> >>     igt@i915_module_load@load: >> >>       o bat-mtlp-8: NOTRUN -> INCOMPLETE >> >> > > > This is due to overlapping initial fb's vma with GuC reserved area on > MTL, see ggtt_reserve_guc_top. > > My dirty quick_fix proposed: > @@ -143,6 +143,9 @@ initial_plane_vma(struct drm_i915_private *i915, >         if (IS_ERR(vma)) >                 goto err_obj; > > +       if (base + size > GUC_GGTT_TOP) > +               base = min(base, GUC_GGTT_TOP) - size; > + I saw you posting this before but forgot to comment. I couldn't quite figure out what the logic is supposed to do and how it wouldn't fail to inherit any firmware setup splash screen? Other than making the firmware aware and not use that range, I was thinking what else we could do in i915? Copy the fb to a non-conflicting location and re-program the display engine? If that is doable without visible glitching and can be fast enough not to slow the boot a lot. Unless I am missing something in your proposal? Regards, Tvrtko >         pinctl = PIN_GLOBAL | PIN_OFFSET_FIXED | base; >         if (HAS_GMCH(i915)) >                 pinctl |= PIN_MAPPABLE; > > > Regards > Andrzej > > >>   * >> >>     igt@i915_selftest@live@gem_contexts: >> >>       o bat-atsm-1: PASS >> >> -> INCOMPLETE >> >> >>     Known issues >> >> Here are the changes found in Patchwork_127721v1 that come from known >> issues: >> >> >>       IGT changes >> >> >>         Issues hit >> >>   * >> >>     igt@gem_exec_suspend@basic-s0@smem: >> >>       o bat-dg2-9: NOTRUN -> INCOMPLETE >> >> (i915#9275 ) >>   * >> >>     igt@gem_lmem_swapping@basic: >> >>       o fi-pnv-d510: NOTRUN -> SKIP >> >> (fdo#109271 ) +28 other tests skip >>   * >> >>     igt@gem_mmap@basic: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#4083 ) >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#4083 ) >> >>   * >> >>     igt@gem_mmap_gtt@basic: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#4077 ) +2 other tests skip >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#4077 ) +2 other tests skip >> >>   * >> >>     igt@gem_render_tiled_blits@basic: >> >>       o bat-dg2-9: NOTRUN -> SKIP >> >> (i915#4079 ) +1 other test skip >>   * >> >>     igt@gem_tiled_pread_basic: >> >>       o bat-dg2-8: NOTRUN -> SKIP >> >> (i915#4079 ) +1 other test skip >>   * >> >>     igt@i915_pm_rps@basic-api: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#6621 ) >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#6621 ) >> >>   * >> >>     igt@i915_suspend@basic-s3-without-i915: >> >>       o bat-dg2-8: NOTRUN -> SKIP >> >> (i915#6645 ) >>   * >> >>     igt@kms_addfb_basic@addfb25-y-tiled-small-legacy: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#5190 ) >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#5190 ) >> >>   * >> >>     igt@kms_addfb_basic@basic-y-tiled-legacy: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#4215 / i915#5190 ) >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#4215 / i915#5190 ) >> >>   * >> >>     igt@kms_addfb_basic@framebuffer-vs-set-tiling: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#4212 ) +6 other tests skip >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#4212 ) +6 other tests skip >> >>   * >> >>     igt@kms_addfb_basic@tile-pitch-mismatch: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#4212 / i915#5608 ) >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#4212 / i915#5608 ) >> >>   * >> >>     igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#4103 / i915#4213 / i915#5608 ) +1 other test skip >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#4103 / i915#4213 / i915#5608 ) +1 other test skip >> >>   * >> >>     igt@kms_force_connector_basic@force-load-detect: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (fdo#109285 ) >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (fdo#109285 ) >> >>   * >> >>     igt@kms_force_connector_basic@prune-stale-modes: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#5274 ) >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#5274 ) >> >>   * >> >>     igt@kms_pipe_crc_basic@suspend-read-crc: >> >>       o fi-ivb-3770: NOTRUN -> SKIP >> >> (fdo#109271 ) >>   * >> >>     igt@kms_pm_backlight@basic-brightness: >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#5354 ) >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#5354 ) >> >>   * >> >>     igt@kms_setmode@basic-clone-single-crtc: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#3555 ) >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#3555 ) >> >>   * >> >>     igt@prime_vgem@basic-fence-flip: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#3708 ) >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#3708 ) >> >>   * >> >>     igt@prime_vgem@basic-fence-mmap: >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#3708 / i915#4077 ) +1 other test skip >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#3708 / i915#4077 ) +1 other test skip >> >>   * >> >>     igt@prime_vgem@basic-write: >> >>       o >> >>         bat-dg2-9: NOTRUN -> SKIP >> >> (i915#3291 / i915#3708 ) +2 other tests skip >> >>       o >> >>         bat-dg2-8: NOTRUN -> SKIP >> >> (i915#3291 / i915#3708 ) +2 other tests skip >> >> >>         Possible fixes >> >>   * igt@i915_selftest@live@gt_heartbeat: >>       o fi-glk-j4005: INCOMPLETE >> >> -> PASS >> >> {name}: This element is suppressed. This means it is ignored when >> computing >> the status of the difference (SUCCESS, WARNING, or FAILURE). >> >> >>     Build changes >> >>   * Linux: CI_DRM_14010 -> Patchwork_127721v1 >> >> CI-20190529: 20190529 >> CI_DRM_14010: b4182ec1538e8cebf630083ec4296bed0061d594 @ >> git://anongit.freedesktop.org/gfx-ci/linux >> IGT_7635: 0b796be8ce05cb2070ce5136d248f438c962d11e @ >> https://gitlab.freedesktop.org/drm/igt-gpu-tools.git >> Patchwork_127721v1: b4182ec1538e8cebf630083ec4296bed0061d594 @ >> git://anongit.freedesktop.org/gfx-ci/linux >> >> >>       Linux commits >> >> 499717049180 drm/i915: Simplify intel_initial_plane_config() calling >> convention >> 3f12c223d707 drm/i915: Split the smem and lmem plane readout apart >> 2d21369e26a3 drm/i915: s/phys_base/dma_addr/ >> be622e53615d drm/i915: Fix MTL initial plane readout >> 8ad9926fb883 drm/i915: Fix region start during initial plane readout >> 55b4a59fce27 drm/i915: Fix PTE decode during initial plane readout >> db38b8a8227c drm/i915: Rename the DSM/GSM registers >> f7b2179e3b69 drm/i915: Disable the "binder" >> 77e7c3085969 drm/i915: Bypass LMEMBAR/GTTMMADR for MTL stolen memory >> access >> ef59e22db9f6 drm/i915: Remove ad-hoc lmem/stolen debugs >> b001bcb6a48f drm/i915: Print memory region info during probe >> 5b54e278890d drm/i915: Use struct resource for memory region IO as well >> >