* [Intel-gfx] [PATCH] drm/i915/gt: Avoid out-of-bounds access when loading HuC @ 2023-04-13 20:03 Lucas De Marchi 2023-04-13 21:50 ` Ceraolo Spurio, Daniele ` (3 more replies) 0 siblings, 4 replies; 5+ messages in thread From: Lucas De Marchi @ 2023-04-13 20:03 UTC (permalink / raw) To: intel-gfx; +Cc: Lucas De Marchi, Alan Previn When HuC is loaded by GSC, there is no header definition for the kernel to look at and firmware is just handed to GSC. However when reading the version, it should still check the size of the blob to guarantee it's not incurring into out-of-bounds array access. If firmware is smaller than expected, the following message is now printed: # echo boom > /lib/firmware/i915/dg2_huc_gsc.bin # dmesg | grep -i huc [drm] GT0: HuC firmware i915/dg2_huc_gsc.bin: invalid size: 5 < 184 [drm] *ERROR* GT0: HuC firmware i915/dg2_huc_gsc.bin: fetch failed -ENODATA ... Even without this change the size, header and signature are still checked by GSC when loading, so this only avoids the out-of-bounds array access. Fixes: a7b516bd981f ("drm/i915/huc: Add fetch support for gsc-loaded HuC binary") Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> Cc: Alan Previn <alan.previn.teres.alexis@intel.com> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com> --- drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 21 +++++++++++++++++---- 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c b/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c index 1ac6f9f340e3..a82a53dbbc86 100644 --- a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c +++ b/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c @@ -489,12 +489,25 @@ static void __force_fw_fetch_failures(struct intel_uc_fw *uc_fw, int e) } } -static int check_gsc_manifest(const struct firmware *fw, +static int check_gsc_manifest(struct intel_gt *gt, + const struct firmware *fw, struct intel_uc_fw *uc_fw) { u32 *dw = (u32 *)fw->data; - u32 version_hi = dw[HUC_GSC_VERSION_HI_DW]; - u32 version_lo = dw[HUC_GSC_VERSION_LO_DW]; + u32 version_hi, version_lo; + size_t min_size; + + /* Check the size of the blob before examining buffer contents */ + min_size = sizeof(u32) * (HUC_GSC_VERSION_LO_DW + 1); + if (unlikely(fw->size < min_size)) { + gt_warn(gt, "%s firmware %s: invalid size: %zu < %zu\n", + intel_uc_fw_type_repr(uc_fw->type), uc_fw->file_selected.path, + fw->size, min_size); + return -ENODATA; + } + + version_hi = dw[HUC_GSC_VERSION_HI_DW]; + version_lo = dw[HUC_GSC_VERSION_LO_DW]; uc_fw->file_selected.ver.major = FIELD_GET(HUC_GSC_MAJOR_VER_HI_MASK, version_hi); uc_fw->file_selected.ver.minor = FIELD_GET(HUC_GSC_MINOR_VER_HI_MASK, version_hi); @@ -665,7 +678,7 @@ static int check_fw_header(struct intel_gt *gt, return 0; if (uc_fw->loaded_via_gsc) - err = check_gsc_manifest(fw, uc_fw); + err = check_gsc_manifest(gt, fw, uc_fw); else err = check_ccs_header(gt, fw, uc_fw); if (err) -- 2.39.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/gt: Avoid out-of-bounds access when loading HuC 2023-04-13 20:03 [Intel-gfx] [PATCH] drm/i915/gt: Avoid out-of-bounds access when loading HuC Lucas De Marchi @ 2023-04-13 21:50 ` Ceraolo Spurio, Daniele 2023-04-13 22:25 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork ` (2 subsequent siblings) 3 siblings, 0 replies; 5+ messages in thread From: Ceraolo Spurio, Daniele @ 2023-04-13 21:50 UTC (permalink / raw) To: Lucas De Marchi, intel-gfx; +Cc: Alan Previn On 4/13/2023 1:03 PM, Lucas De Marchi wrote: > When HuC is loaded by GSC, there is no header definition for the kernel > to look at and firmware is just handed to GSC. However when reading the > version, it should still check the size of the blob to guarantee it's not > incurring into out-of-bounds array access. > > If firmware is smaller than expected, the following message is now > printed: > > # echo boom > /lib/firmware/i915/dg2_huc_gsc.bin > # dmesg | grep -i huc > [drm] GT0: HuC firmware i915/dg2_huc_gsc.bin: invalid size: 5 < 184 > [drm] *ERROR* GT0: HuC firmware i915/dg2_huc_gsc.bin: fetch failed -ENODATA > ... > > Even without this change the size, header and signature are still > checked by GSC when loading, so this only avoids the out-of-bounds array > access. > > Fixes: a7b516bd981f ("drm/i915/huc: Add fetch support for gsc-loaded HuC binary") > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> > Cc: Alan Previn <alan.previn.teres.alexis@intel.com> > Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com> Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> Daniele > --- > drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 21 +++++++++++++++++---- > 1 file changed, 17 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c b/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c > index 1ac6f9f340e3..a82a53dbbc86 100644 > --- a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c > +++ b/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c > @@ -489,12 +489,25 @@ static void __force_fw_fetch_failures(struct intel_uc_fw *uc_fw, int e) > } > } > > -static int check_gsc_manifest(const struct firmware *fw, > +static int check_gsc_manifest(struct intel_gt *gt, > + const struct firmware *fw, > struct intel_uc_fw *uc_fw) > { > u32 *dw = (u32 *)fw->data; > - u32 version_hi = dw[HUC_GSC_VERSION_HI_DW]; > - u32 version_lo = dw[HUC_GSC_VERSION_LO_DW]; > + u32 version_hi, version_lo; > + size_t min_size; > + > + /* Check the size of the blob before examining buffer contents */ > + min_size = sizeof(u32) * (HUC_GSC_VERSION_LO_DW + 1); > + if (unlikely(fw->size < min_size)) { > + gt_warn(gt, "%s firmware %s: invalid size: %zu < %zu\n", > + intel_uc_fw_type_repr(uc_fw->type), uc_fw->file_selected.path, > + fw->size, min_size); > + return -ENODATA; > + } > + > + version_hi = dw[HUC_GSC_VERSION_HI_DW]; > + version_lo = dw[HUC_GSC_VERSION_LO_DW]; > > uc_fw->file_selected.ver.major = FIELD_GET(HUC_GSC_MAJOR_VER_HI_MASK, version_hi); > uc_fw->file_selected.ver.minor = FIELD_GET(HUC_GSC_MINOR_VER_HI_MASK, version_hi); > @@ -665,7 +678,7 @@ static int check_fw_header(struct intel_gt *gt, > return 0; > > if (uc_fw->loaded_via_gsc) > - err = check_gsc_manifest(fw, uc_fw); > + err = check_gsc_manifest(gt, fw, uc_fw); > else > err = check_ccs_header(gt, fw, uc_fw); > if (err) ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gt: Avoid out-of-bounds access when loading HuC 2023-04-13 20:03 [Intel-gfx] [PATCH] drm/i915/gt: Avoid out-of-bounds access when loading HuC Lucas De Marchi 2023-04-13 21:50 ` Ceraolo Spurio, Daniele @ 2023-04-13 22:25 ` Patchwork 2023-04-13 22:36 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2023-04-14 4:59 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 3 siblings, 0 replies; 5+ messages in thread From: Patchwork @ 2023-04-13 22:25 UTC (permalink / raw) To: Lucas De Marchi; +Cc: intel-gfx == Series Details == Series: drm/i915/gt: Avoid out-of-bounds access when loading HuC URL : https://patchwork.freedesktop.org/series/116460/ State : warning == Summary == Error: dim checkpatch failed 3145aadb2876 drm/i915/gt: Avoid out-of-bounds access when loading HuC -:17: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #17: [drm] *ERROR* GT0: HuC firmware i915/dg2_huc_gsc.bin: fetch failed -ENODATA total: 0 errors, 1 warnings, 0 checks, 36 lines checked ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gt: Avoid out-of-bounds access when loading HuC 2023-04-13 20:03 [Intel-gfx] [PATCH] drm/i915/gt: Avoid out-of-bounds access when loading HuC Lucas De Marchi 2023-04-13 21:50 ` Ceraolo Spurio, Daniele 2023-04-13 22:25 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork @ 2023-04-13 22:36 ` Patchwork 2023-04-14 4:59 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 3 siblings, 0 replies; 5+ messages in thread From: Patchwork @ 2023-04-13 22:36 UTC (permalink / raw) To: Lucas De Marchi; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 10621 bytes --] == Series Details == Series: drm/i915/gt: Avoid out-of-bounds access when loading HuC URL : https://patchwork.freedesktop.org/series/116460/ State : success == Summary == CI Bug Log - changes from CI_DRM_13004 -> Patchwork_116460v1 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/index.html Participating hosts (35 -> 34) ------------------------------ Additional (1): bat-rpls-2 Missing (2): bat-mtlp-8 fi-snb-2520m Known issues ------------ Here are the changes found in Patchwork_116460v1 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@debugfs_test@basic-hwmon: - bat-rpls-2: NOTRUN -> [SKIP][1] ([i915#7456]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@debugfs_test@basic-hwmon.html * igt@fbdev@read: - bat-rpls-2: NOTRUN -> [SKIP][2] ([i915#2582]) +4 similar issues [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@fbdev@read.html * igt@gem_lmem_swapping@verify-random: - bat-rpls-2: NOTRUN -> [SKIP][3] ([i915#4613]) +3 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@gem_lmem_swapping@verify-random.html * igt@gem_tiled_pread_basic: - bat-rpls-2: NOTRUN -> [SKIP][4] ([i915#3282]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@gem_tiled_pread_basic.html * igt@i915_pm_backlight@basic-brightness: - bat-rpls-2: NOTRUN -> [SKIP][5] ([i915#7561]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@i915_pm_backlight@basic-brightness.html * igt@i915_selftest@live@slpc: - bat-adln-1: NOTRUN -> [DMESG-FAIL][6] ([i915#6997]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-adln-1/igt@i915_selftest@live@slpc.html * igt@i915_selftest@live@workarounds: - bat-rpls-2: NOTRUN -> [INCOMPLETE][7] ([i915#4983] / [i915#7913]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@i915_selftest@live@workarounds.html * igt@kms_busy@basic: - bat-rpls-2: NOTRUN -> [SKIP][8] ([i915#1845]) +14 similar issues [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@kms_busy@basic.html * igt@kms_chamelium_edid@hdmi-edid-read: - bat-rpls-2: NOTRUN -> [SKIP][9] ([i915#7828]) +7 similar issues [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@kms_chamelium_edid@hdmi-edid-read.html * igt@kms_chamelium_hpd@common-hpd-after-suspend: - bat-adln-1: NOTRUN -> [SKIP][10] ([i915#7828]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-adln-1/igt@kms_chamelium_hpd@common-hpd-after-suspend.html - fi-bsw-n3050: NOTRUN -> [SKIP][11] ([fdo#109271]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/fi-bsw-n3050/igt@kms_chamelium_hpd@common-hpd-after-suspend.html - bat-rpls-1: NOTRUN -> [SKIP][12] ([i915#7828]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-1/igt@kms_chamelium_hpd@common-hpd-after-suspend.html * igt@kms_flip@basic-flip-vs-dpms: - bat-rpls-2: NOTRUN -> [SKIP][13] ([i915#3637]) +3 similar issues [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@kms_flip@basic-flip-vs-dpms.html * igt@kms_force_connector_basic@force-load-detect: - bat-rpls-2: NOTRUN -> [SKIP][14] ([fdo#109285]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@kms_force_connector_basic@force-load-detect.html * igt@kms_frontbuffer_tracking@basic: - bat-rpls-2: NOTRUN -> [SKIP][15] ([i915#1849]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@kms_frontbuffer_tracking@basic.html * igt@kms_pipe_crc_basic@nonblocking-crc@pipe-d-dp-1: - bat-dg2-8: [PASS][16] -> [FAIL][17] ([i915#7932]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/bat-dg2-8/igt@kms_pipe_crc_basic@nonblocking-crc@pipe-d-dp-1.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-dg2-8/igt@kms_pipe_crc_basic@nonblocking-crc@pipe-d-dp-1.html * igt@kms_pipe_crc_basic@suspend-read-crc: - bat-rpls-1: NOTRUN -> [SKIP][18] ([i915#1845]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-1/igt@kms_pipe_crc_basic@suspend-read-crc.html * igt@kms_psr@sprite_plane_onoff: - bat-rpls-2: NOTRUN -> [SKIP][19] ([i915#1072]) +3 similar issues [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@kms_psr@sprite_plane_onoff.html * igt@kms_setmode@basic-clone-single-crtc: - bat-rpls-2: NOTRUN -> [SKIP][20] ([i915#3555] / [i915#4579]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@kms_setmode@basic-clone-single-crtc.html * igt@prime_vgem@basic-fence-flip: - bat-rpls-2: NOTRUN -> [SKIP][21] ([fdo#109295] / [i915#1845] / [i915#3708]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@prime_vgem@basic-fence-flip.html * igt@prime_vgem@basic-fence-read: - bat-rpls-2: NOTRUN -> [SKIP][22] ([fdo#109295] / [i915#3708]) +3 similar issues [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-2/igt@prime_vgem@basic-fence-read.html #### Possible fixes #### * igt@gem_exec_suspend@basic-s3@smem: - bat-rpls-1: [ABORT][23] ([i915#6687] / [i915#7978]) -> [PASS][24] [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/bat-rpls-1/igt@gem_exec_suspend@basic-s3@smem.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-1/igt@gem_exec_suspend@basic-s3@smem.html * igt@i915_pm_rps@basic-api: - bat-dg2-11: [FAIL][25] ([i915#8308]) -> [PASS][26] [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/bat-dg2-11/igt@i915_pm_rps@basic-api.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-dg2-11/igt@i915_pm_rps@basic-api.html * igt@i915_selftest@live@execlists: - fi-bsw-n3050: [ABORT][27] ([i915#7911] / [i915#7913]) -> [PASS][28] [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/fi-bsw-n3050/igt@i915_selftest@live@execlists.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/fi-bsw-n3050/igt@i915_selftest@live@execlists.html * igt@i915_selftest@live@gt_lrc: - bat-adln-1: [INCOMPLETE][29] ([i915#4983] / [i915#7609]) -> [PASS][30] [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/bat-adln-1/igt@i915_selftest@live@gt_lrc.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-adln-1/igt@i915_selftest@live@gt_lrc.html * igt@i915_selftest@live@slpc: - bat-rpls-1: [DMESG-FAIL][31] ([i915#6367]) -> [PASS][32] [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/bat-rpls-1/igt@i915_selftest@live@slpc.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-rpls-1/igt@i915_selftest@live@slpc.html * igt@kms_chamelium_hpd@common-hpd-after-suspend: - fi-kbl-7567u: [ABORT][33] ([i915#8059]) -> [PASS][34] [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/fi-kbl-7567u/igt@kms_chamelium_hpd@common-hpd-after-suspend.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/fi-kbl-7567u/igt@kms_chamelium_hpd@common-hpd-after-suspend.html * igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence@pipe-d-dp-1: - bat-dg2-8: [FAIL][35] ([i915#7932]) -> [PASS][36] [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/bat-dg2-8/igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence@pipe-d-dp-1.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/bat-dg2-8/igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence@pipe-d-dp-1.html [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845 [i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849 [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582 [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637 [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708 [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983 [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367 [i915#6687]: https://gitlab.freedesktop.org/drm/intel/issues/6687 [i915#6997]: https://gitlab.freedesktop.org/drm/intel/issues/6997 [i915#7456]: https://gitlab.freedesktop.org/drm/intel/issues/7456 [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561 [i915#7609]: https://gitlab.freedesktop.org/drm/intel/issues/7609 [i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828 [i915#7911]: https://gitlab.freedesktop.org/drm/intel/issues/7911 [i915#7913]: https://gitlab.freedesktop.org/drm/intel/issues/7913 [i915#7932]: https://gitlab.freedesktop.org/drm/intel/issues/7932 [i915#7978]: https://gitlab.freedesktop.org/drm/intel/issues/7978 [i915#8059]: https://gitlab.freedesktop.org/drm/intel/issues/8059 [i915#8308]: https://gitlab.freedesktop.org/drm/intel/issues/8308 Build changes ------------- * Linux: CI_DRM_13004 -> Patchwork_116460v1 CI-20190529: 20190529 CI_DRM_13004: 293fce331f7e09efa3b9d2046699f3e6af8a4ac9 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7254: 7fab01340a3f360abacd7914015be1ad485363d7 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_116460v1: 293fce331f7e09efa3b9d2046699f3e6af8a4ac9 @ git://anongit.freedesktop.org/gfx-ci/linux ### Linux commits 06fd5c43b010 drm/i915/gt: Avoid out-of-bounds access when loading HuC == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/index.html [-- Attachment #2: Type: text/html, Size: 12509 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/gt: Avoid out-of-bounds access when loading HuC 2023-04-13 20:03 [Intel-gfx] [PATCH] drm/i915/gt: Avoid out-of-bounds access when loading HuC Lucas De Marchi ` (2 preceding siblings ...) 2023-04-13 22:36 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork @ 2023-04-14 4:59 ` Patchwork 3 siblings, 0 replies; 5+ messages in thread From: Patchwork @ 2023-04-14 4:59 UTC (permalink / raw) To: Lucas De Marchi; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 13470 bytes --] == Series Details == Series: drm/i915/gt: Avoid out-of-bounds access when loading HuC URL : https://patchwork.freedesktop.org/series/116460/ State : success == Summary == CI Bug Log - changes from CI_DRM_13004_full -> Patchwork_116460v1_full ==================================================== Summary ------- **SUCCESS** No regressions found. Participating hosts (7 -> 8) ------------------------------ Additional (1): shard-rkl0 Known issues ------------ Here are the changes found in Patchwork_116460v1_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_fair@basic-pace-share@rcs0: - shard-glk: [PASS][1] -> [FAIL][2] ([i915#2842]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-glk6/igt@gem_exec_fair@basic-pace-share@rcs0.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-glk8/igt@gem_exec_fair@basic-pace-share@rcs0.html * igt@i915_pipe_stress@stress-xrgb8888-ytiled: - shard-apl: NOTRUN -> [FAIL][3] ([i915#7036]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-apl1/igt@i915_pipe_stress@stress-xrgb8888-ytiled.html * igt@i915_selftest@live@gt_heartbeat: - shard-apl: [PASS][4] -> [DMESG-FAIL][5] ([i915#5334]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-apl2/igt@i915_selftest@live@gt_heartbeat.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-apl3/igt@i915_selftest@live@gt_heartbeat.html * igt@kms_ccs@pipe-b-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc: - shard-apl: NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#3886]) +2 similar issues [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-apl1/igt@kms_ccs@pipe-b-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc.html * igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic: - shard-glk: [PASS][7] -> [FAIL][8] ([i915#72]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-glk3/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-glk3/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html * igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size: - shard-glk: [PASS][9] -> [FAIL][10] ([i915#2346]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-glk9/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-glk3/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-mmap-cpu: - shard-glk: NOTRUN -> [SKIP][11] ([fdo#109271]) +7 similar issues [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-glk4/igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-mmap-cpu.html * igt@kms_plane_scaling@plane-upscale-with-rotation-factor-0-25@pipe-a-dp-1: - shard-apl: NOTRUN -> [SKIP][12] ([fdo#109271]) +51 similar issues [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-apl1/igt@kms_plane_scaling@plane-upscale-with-rotation-factor-0-25@pipe-a-dp-1.html * igt@kms_psr2_su@page_flip-nv12: - shard-apl: NOTRUN -> [SKIP][13] ([fdo#109271] / [i915#658]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-apl1/igt@kms_psr2_su@page_flip-nv12.html #### Possible fixes #### * igt@gem_exec_fair@basic-deadline: - shard-glk: [FAIL][14] ([i915#2846]) -> [PASS][15] [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-glk6/igt@gem_exec_fair@basic-deadline.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-glk1/igt@gem_exec_fair@basic-deadline.html * igt@gem_exec_fair@basic-pace-share@rcs0: - {shard-rkl}: [FAIL][16] ([i915#2842]) -> [PASS][17] [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-rkl-2/igt@gem_exec_fair@basic-pace-share@rcs0.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-rkl-2/igt@gem_exec_fair@basic-pace-share@rcs0.html * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-apl: [FAIL][18] ([i915#2842]) -> [PASS][19] [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-apl7/igt@gem_exec_fair@basic-pace-solo@rcs0.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-apl1/igt@gem_exec_fair@basic-pace-solo@rcs0.html - shard-glk: [FAIL][20] ([i915#2842]) -> [PASS][21] [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-glk2/igt@gem_exec_fair@basic-pace-solo@rcs0.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-glk9/igt@gem_exec_fair@basic-pace-solo@rcs0.html * igt@i915_pm_rc6_residency@rc6-idle@rcs0: - shard-glk: [INCOMPLETE][22] -> [PASS][23] [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-glk5/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-glk4/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html * igt@i915_pm_rpm@dpms-non-lpsp: - {shard-rkl}: [SKIP][24] ([i915#1397]) -> [PASS][25] +2 similar issues [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-rkl-7/igt@i915_pm_rpm@dpms-non-lpsp.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-rkl-3/igt@i915_pm_rpm@dpms-non-lpsp.html * igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions: - shard-apl: [FAIL][26] ([i915#2346]) -> [PASS][27] [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-apl2/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-apl6/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html * igt@kms_flip@flip-vs-expired-vblank-interruptible@c-hdmi-a1: - shard-glk: [FAIL][28] ([i915#79]) -> [PASS][29] [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-glk1/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-hdmi-a1.html [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-glk5/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-hdmi-a1.html * igt@kms_flip@flip-vs-suspend-interruptible@b-dp1: - shard-apl: [ABORT][30] ([i915#180]) -> [PASS][31] [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-apl3/igt@kms_flip@flip-vs-suspend-interruptible@b-dp1.html [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-apl1/igt@kms_flip@flip-vs-suspend-interruptible@b-dp1.html * igt@kms_flip@plain-flip-fb-recreate@a-hdmi-a1: - shard-glk: [FAIL][32] ([i915#2122]) -> [PASS][33] +2 similar issues [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-glk8/igt@kms_flip@plain-flip-fb-recreate@a-hdmi-a1.html [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-glk2/igt@kms_flip@plain-flip-fb-recreate@a-hdmi-a1.html * igt@perf_pmu@most-busy-check-all@rcs0: - {shard-rkl}: [FAIL][34] ([i915#4349]) -> [PASS][35] [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13004/shard-rkl-4/igt@perf_pmu@most-busy-check-all@rcs0.html [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/shard-rkl-6/igt@perf_pmu@most-busy-check-all@rcs0.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289 [fdo#109302]: https://bugs.freedesktop.org/show_bug.cgi?id=109302 [fdo#109303]: https://bugs.freedesktop.org/show_bug.cgi?id=109303 [fdo#110189]: https://bugs.freedesktop.org/show_bug.cgi?id=110189 [fdo#111615]: https://bugs.freedesktop.org/show_bug.cgi?id=111615 [fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [fdo#112283]: https://bugs.freedesktop.org/show_bug.cgi?id=112283 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [i915#1397]: https://gitlab.freedesktop.org/drm/intel/issues/1397 [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180 [i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122 [i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346 [i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437 [i915#2527]: https://gitlab.freedesktop.org/drm/intel/issues/2527 [i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575 [i915#2587]: https://gitlab.freedesktop.org/drm/intel/issues/2587 [i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672 [i915#2681]: https://gitlab.freedesktop.org/drm/intel/issues/2681 [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842 [i915#2846]: https://gitlab.freedesktop.org/drm/intel/issues/2846 [i915#315]: https://gitlab.freedesktop.org/drm/intel/issues/315 [i915#3281]: https://gitlab.freedesktop.org/drm/intel/issues/3281 [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282 [i915#3299]: https://gitlab.freedesktop.org/drm/intel/issues/3299 [i915#3359]: https://gitlab.freedesktop.org/drm/intel/issues/3359 [i915#3458]: https://gitlab.freedesktop.org/drm/intel/issues/3458 [i915#3528]: https://gitlab.freedesktop.org/drm/intel/issues/3528 [i915#3539]: https://gitlab.freedesktop.org/drm/intel/issues/3539 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3591]: https://gitlab.freedesktop.org/drm/intel/issues/3591 [i915#3638]: https://gitlab.freedesktop.org/drm/intel/issues/3638 [i915#3689]: https://gitlab.freedesktop.org/drm/intel/issues/3689 [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708 [i915#3743]: https://gitlab.freedesktop.org/drm/intel/issues/3743 [i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840 [i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886 [i915#3938]: https://gitlab.freedesktop.org/drm/intel/issues/3938 [i915#3955]: https://gitlab.freedesktop.org/drm/intel/issues/3955 [i915#4070]: https://gitlab.freedesktop.org/drm/intel/issues/4070 [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077 [i915#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078 [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079 [i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083 [i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212 [i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270 [i915#4349]: https://gitlab.freedesktop.org/drm/intel/issues/4349 [i915#4538]: https://gitlab.freedesktop.org/drm/intel/issues/4538 [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579 [i915#4812]: https://gitlab.freedesktop.org/drm/intel/issues/4812 [i915#4816]: https://gitlab.freedesktop.org/drm/intel/issues/4816 [i915#4833]: https://gitlab.freedesktop.org/drm/intel/issues/4833 [i915#4852]: https://gitlab.freedesktop.org/drm/intel/issues/4852 [i915#4859]: https://gitlab.freedesktop.org/drm/intel/issues/4859 [i915#4860]: https://gitlab.freedesktop.org/drm/intel/issues/4860 [i915#4884]: https://gitlab.freedesktop.org/drm/intel/issues/4884 [i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176 [i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235 [i915#5286]: https://gitlab.freedesktop.org/drm/intel/issues/5286 [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334 [i915#5563]: https://gitlab.freedesktop.org/drm/intel/issues/5563 [i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095 [i915#6524]: https://gitlab.freedesktop.org/drm/intel/issues/6524 [i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658 [i915#6953]: https://gitlab.freedesktop.org/drm/intel/issues/6953 [i915#7036]: https://gitlab.freedesktop.org/drm/intel/issues/7036 [i915#7116]: https://gitlab.freedesktop.org/drm/intel/issues/7116 [i915#72]: https://gitlab.freedesktop.org/drm/intel/issues/72 [i915#7711]: https://gitlab.freedesktop.org/drm/intel/issues/7711 [i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828 [i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79 [i915#8011]: https://gitlab.freedesktop.org/drm/intel/issues/8011 [i915#8247]: https://gitlab.freedesktop.org/drm/intel/issues/8247 [i915#8292]: https://gitlab.freedesktop.org/drm/intel/issues/8292 [i915#8308]: https://gitlab.freedesktop.org/drm/intel/issues/8308 Build changes ------------- * Linux: CI_DRM_13004 -> Patchwork_116460v1 CI-20190529: 20190529 CI_DRM_13004: 293fce331f7e09efa3b9d2046699f3e6af8a4ac9 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7254: 7fab01340a3f360abacd7914015be1ad485363d7 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_116460v1: 293fce331f7e09efa3b9d2046699f3e6af8a4ac9 @ git://anongit.freedesktop.org/gfx-ci/linux piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_116460v1/index.html [-- Attachment #2: Type: text/html, Size: 10828 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-04-14 4:59 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-04-13 20:03 [Intel-gfx] [PATCH] drm/i915/gt: Avoid out-of-bounds access when loading HuC Lucas De Marchi 2023-04-13 21:50 ` Ceraolo Spurio, Daniele 2023-04-13 22:25 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork 2023-04-13 22:36 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2023-04-14 4:59 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.