* [PATCH v2 0/1] Fix i915 error_state_read ptr use
@ 2022-02-26 6:27 ` Alan Previn
0 siblings, 0 replies; 11+ messages in thread
From: Alan Previn @ 2022-02-26 6:27 UTC (permalink / raw)
To: intel-gfx; +Cc: Stuart Summers, dri-devel, Alan Previn
Fix pointer offset usage in error_state_read
when there is no i915_gpu_coredump but buf offset
is non-zero.
This is the 2nd rev of this series.
Changes from prior revs:
v2: - Fix build issue: uninitialized var
Reported-by: kernel test robot <lkp@intel.com>
Alan Previn (1):
drm/i915/reset: Fix error_state_read ptr + offset use
drivers/gpu/drm/i915/i915_sysfs.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Intel-gfx] [PATCH v2 0/1] Fix i915 error_state_read ptr use
@ 2022-02-26 6:27 ` Alan Previn
0 siblings, 0 replies; 11+ messages in thread
From: Alan Previn @ 2022-02-26 6:27 UTC (permalink / raw)
To: intel-gfx; +Cc: dri-devel, Alan Previn
Fix pointer offset usage in error_state_read
when there is no i915_gpu_coredump but buf offset
is non-zero.
This is the 2nd rev of this series.
Changes from prior revs:
v2: - Fix build issue: uninitialized var
Reported-by: kernel test robot <lkp@intel.com>
Alan Previn (1):
drm/i915/reset: Fix error_state_read ptr + offset use
drivers/gpu/drm/i915/i915_sysfs.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Intel-gfx] [PATCH v2 1/1] drm/i915/reset: Fix error_state_read ptr + offset use
2022-02-26 6:27 ` [Intel-gfx] " Alan Previn
(?)
@ 2022-02-26 6:27 ` Alan Previn
2022-03-01 21:37 ` John Harrison
-1 siblings, 1 reply; 11+ messages in thread
From: Alan Previn @ 2022-02-26 6:27 UTC (permalink / raw)
To: intel-gfx; +Cc: Alan Previn
Fix our pointer offset usage in error_state_read
when there is no i915_gpu_coredump but buf offset
is non-zero.
This fixes a kernel page fault can happen when
multiple tests are running concurrently in a loop
and one is producing engine resets and consuming
the i915 error_state dump while the other is
forcing full GT resets. (takes a while to trigger).
The dmesg call trace:
5014 [ 5590.803000] BUG: unable to handle page fault for address: ffffffffa0b0e000
5015 [ 5590.803009] #PF: supervisor read access in kernel mode
5016 [ 5590.803013] #PF: error_code(0x0000) - not-present page
5017 [ 5590.803016] PGD 5814067 P4D 5814067 PUD 5815063 PMD 109de4067 PTE 0
5018 [ 5590.803022] Oops: 0000 [#1] PREEMPT SMP NOPTI
5019 [ 5590.803026] CPU: 5 PID: 13656 Comm: i915_hangman Tainted: G U 5.17.0-rc5-
ups69-guc-err-capt-rev6+ #136
5020 [ 5590.803033] Hardware name: Intel Corporation Alder Lake Client Platform/
AlderLake-M LP4x RVP, BIOS ADLPFWI1.R00.3031.A02.2201171222
01/17/2022
5021 [ 5590.803039] RIP: 0010:memcpy_erms+0x6/0x10
5022 [ 5590.803045] Code: fe ff ff cc eb 1e 0f 1f 00 48 89 f8 48 89 d1 48 c1 e9 03 83
e2 07 f3 48 a5 89 d1 f3 a4 c3 66 0f 1f 44 00 00 48 89 f8 48 89
d1 <f3> a4 c3 0f 1f 80 00 00 00 00 48 89 f8 48 83 fa 20 72 7e 40 38 fe
5023 [ 5590.803054] RSP: 0018:ffffc90003a8fdf0 EFLAGS: 00010282
5024 [ 5590.803057] RAX: ffff888107ee9000 RBX: ffff888108cb1a00 RCX: 0000000000000f8f
5025 [ 5590.803061] RDX: 0000000000001000 RSI: ffffffffa0b0e000 RDI: ffff888107ee9071
5026 [ 5590.803065] RBP: 0000000000000000 R08: 0000000000000001 R09: 0000000000000001
5027 [ 5590.803069] R10: 0000000000000001 R11: 0000000000000002 R12: 0000000000000019
5028 [ 5590.803073] R13: 0000000000174fff R14: 0000000000001000 R15: ffff888107ee9000
5029 [ 5590.803077] FS: 00007f62a99bee80(0000) GS:ffff88849f880000(0000) knlGS:0000000000000000
5030 [ 5590.803082] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
5031 [ 5590.803085] CR2: ffffffffa0b0e000 CR3: 000000010a1a8004 CR4: 0000000000770ee0
5032 [ 5590.803089] PKRU: 55555554
5033 [ 5590.803091] Call Trace:
5034 [ 5590.803093] <TASK>
5035 [ 5590.803096] error_state_read+0xa1/0xd0 [i915]
5036 [ 5590.803175] kernfs_fop_read_iter+0xb2/0x1b0
5037 [ 5590.803180] new_sync_read+0x116/0x1a0
5038 [ 5590.803185] vfs_read+0x114/0x1b0
5039 [ 5590.803189] ksys_read+0x63/0xe0
5040 [ 5590.803193] do_syscall_64+0x38/0xc0
5041 [ 5590.803197] entry_SYSCALL_64_after_hwframe+0x44/0xae
5042 [ 5590.803201] RIP: 0033:0x7f62aaea5912
5043 [ 5590.803204] Code: c0 e9 b2 fe ff ff 50 48 8d 3d 5a b9 0c 00 e8 05 19 02 00 0f 1f 44 00
00 f3 0f 1e fa 64 8b 04 25 18 00 00 00 85 c0 75 10 0f 05 <48> 3d 00 f0
ff ff 77 56 c3 0f 1f 44 00 00 48 83 ec 28 48 89 54 24
5044 [ 5590.803213] RSP: 002b:00007fff5b659ae8 EFLAGS: 00000246 ORIG_RAX: 0000000000000000
5045 [ 5590.803218] RAX: ffffffffffffffda RBX: 0000000000100000 RCX: 00007f62aaea5912
5046 [ 5590.803221] RDX: 000000000008b000 RSI: 00007f62a8c4000f RDI: 0000000000000006
5047 [ 5590.803225] RBP: 00007f62a8bcb00f R08: 0000000000200010 R09: 0000000000101000
5048 [ 5590.803229] R10: 0000000000000001 R11: 0000000000000246 R12: 0000000000000006
5049 [ 5590.803233] R13: 0000000000075000 R14: 00007f62a8acb010 R15: 0000000000200000
5050 [ 5590.803238] </TASK>
5051 [ 5590.803240] Modules linked in: i915 ttm drm_buddy drm_dp_helper drm_kms_helper syscopyarea
sysfillrect sysimgblt fb_sys_fops prime_numbers nfnetlink br_netfilter overlay
mei_pxp mei_hdcp x86_pkg_temp_thermal coretemp kvm_intel snd_hda_codec_hdmi
snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep snd_hda_core snd_pcm mei_me
mei fuse ip_tables x_tables crct10dif_pclmul e1000e crc32_pclmul ptp i2c_i801
ghash_clmulni_intel i2c_smbus pps_core [last unloa ded: ttm]
5052 [ 5590.803277] CR2: ffffffffa0b0e000
5053 [ 5590.803280] ---[ end trace 0000000000000000 ]---
Signed-off-by: Alan Previn <alan.previn.teres.alexis@intel.com>
---
drivers/gpu/drm/i915/i915_sysfs.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_sysfs.c b/drivers/gpu/drm/i915/i915_sysfs.c
index a4d1759375b9..c40e51298066 100644
--- a/drivers/gpu/drm/i915/i915_sysfs.c
+++ b/drivers/gpu/drm/i915/i915_sysfs.c
@@ -432,7 +432,7 @@ static ssize_t error_state_read(struct file *filp, struct kobject *kobj,
struct device *kdev = kobj_to_dev(kobj);
struct drm_i915_private *i915 = kdev_minor_to_i915(kdev);
struct i915_gpu_coredump *gpu;
- ssize_t ret;
+ ssize_t ret = 0;
gpu = i915_first_error_state(i915);
if (IS_ERR(gpu)) {
@@ -444,8 +444,10 @@ static ssize_t error_state_read(struct file *filp, struct kobject *kobj,
const char *str = "No error state collected\n";
size_t len = strlen(str);
- ret = min_t(size_t, count, len - off);
- memcpy(buf, str + off, ret);
+ if (off < len) {
+ ret = min_t(size_t, count, len - off);
+ memcpy(buf, str + off, ret);
+ }
}
return ret;
--
2.25.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Fix i915 error_state_read ptr use (rev2)
2022-02-26 6:27 ` [Intel-gfx] " Alan Previn
(?)
(?)
@ 2022-02-26 8:00 ` Patchwork
-1 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2022-02-26 8:00 UTC (permalink / raw)
To: Alan Previn; +Cc: intel-gfx
== Series Details ==
Series: Fix i915 error_state_read ptr use (rev2)
URL : https://patchwork.freedesktop.org/series/100768/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
729b7794c19d drm/i915/reset: Fix error_state_read ptr + offset use
-:18: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#18:
5014 [ 5590.803000] BUG: unable to handle page fault for address: ffffffffa0b0e000
total: 0 errors, 1 warnings, 0 checks, 20 lines checked
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for Fix i915 error_state_read ptr use (rev2)
2022-02-26 6:27 ` [Intel-gfx] " Alan Previn
` (2 preceding siblings ...)
(?)
@ 2022-02-26 8:32 ` Patchwork
-1 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2022-02-26 8:32 UTC (permalink / raw)
To: Alan Previn; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 4822 bytes --]
== Series Details ==
Series: Fix i915 error_state_read ptr use (rev2)
URL : https://patchwork.freedesktop.org/series/100768/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11291 -> Patchwork_22425
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/index.html
Participating hosts (40 -> 40)
------------------------------
Additional (1): fi-pnv-d510
Missing (1): fi-bsw-cyan
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_22425:
### IGT changes ###
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@gem_ringfill@basic-all:
- {bat-dg2-9}: [PASS][1] -> [DMESG-WARN][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/bat-dg2-9/igt@gem_ringfill@basic-all.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/bat-dg2-9/igt@gem_ringfill@basic-all.html
* igt@i915_selftest@live@objects:
- {fi-jsl-1}: [PASS][3] -> [INCOMPLETE][4] +1 similar issue
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/fi-jsl-1/igt@i915_selftest@live@objects.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/fi-jsl-1/igt@i915_selftest@live@objects.html
Known issues
------------
Here are the changes found in Patchwork_22425 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_cs_nop@sync-fork-compute0:
- fi-snb-2600: NOTRUN -> [SKIP][5] ([fdo#109271]) +17 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/fi-snb-2600/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html
* igt@gem_flink_basic@bad-flink:
- fi-skl-6600u: NOTRUN -> [FAIL][6] ([i915#4547])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/fi-skl-6600u/igt@gem_flink_basic@bad-flink.html
* igt@prime_vgem@basic-userptr:
- fi-pnv-d510: NOTRUN -> [SKIP][7] ([fdo#109271]) +57 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/fi-pnv-d510/igt@prime_vgem@basic-userptr.html
#### Possible fixes ####
* igt@gem_exec_suspend@basic-s3@smem:
- fi-skl-6600u: [INCOMPLETE][8] ([i915#4547]) -> [PASS][9]
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
* igt@i915_selftest@live@hangcheck:
- fi-snb-2600: [INCOMPLETE][10] ([i915#3921]) -> [PASS][11]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
* igt@i915_selftest@live@perf:
- {fi-tgl-dsi}: [DMESG-WARN][12] ([i915#2867]) -> [PASS][13] +17 similar issues
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/fi-tgl-dsi/igt@i915_selftest@live@perf.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/fi-tgl-dsi/igt@i915_selftest@live@perf.html
* igt@kms_busy@basic@modeset:
- {bat-adlp-6}: [DMESG-WARN][14] ([i915#3576]) -> [PASS][15]
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/bat-adlp-6/igt@kms_busy@basic@modeset.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/bat-adlp-6/igt@kms_busy@basic@modeset.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
[i915#2867]: https://gitlab.freedesktop.org/drm/intel/issues/2867
[i915#3576]: https://gitlab.freedesktop.org/drm/intel/issues/3576
[i915#3921]: https://gitlab.freedesktop.org/drm/intel/issues/3921
[i915#4547]: https://gitlab.freedesktop.org/drm/intel/issues/4547
[i915#5127]: https://gitlab.freedesktop.org/drm/intel/issues/5127
Build changes
-------------
* Linux: CI_DRM_11291 -> Patchwork_22425
CI-20190529: 20190529
CI_DRM_11291: 1e3f898ee67cb2fd8c58799ae8a094f73054c8b9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6359: 57049558c452272b27eeb099fac07e55a924bbf9 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_22425: 729b7794c19dc6323062363321f9a38673aba221 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
729b7794c19d drm/i915/reset: Fix error_state_read ptr + offset use
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/index.html
[-- Attachment #2: Type: text/html, Size: 5744 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for Fix i915 error_state_read ptr use (rev2)
2022-02-26 6:27 ` [Intel-gfx] " Alan Previn
` (3 preceding siblings ...)
(?)
@ 2022-02-27 4:41 ` Patchwork
-1 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2022-02-27 4:41 UTC (permalink / raw)
To: Teres Alexis, Alan Previn; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 30264 bytes --]
== Series Details ==
Series: Fix i915 error_state_read ptr use (rev2)
URL : https://patchwork.freedesktop.org/series/100768/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11291_full -> Patchwork_22425_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Participating hosts (11 -> 11)
------------------------------
No changes in participating hosts
Known issues
------------
Here are the changes found in Patchwork_22425_full that come from known issues:
### CI changes ###
#### Possible fixes ####
* boot:
- shard-glk: ([FAIL][1], [PASS][2], [PASS][3], [PASS][4], [PASS][5], [PASS][6], [PASS][7], [PASS][8], [PASS][9], [PASS][10], [PASS][11], [PASS][12], [PASS][13], [PASS][14], [PASS][15], [PASS][16], [PASS][17], [PASS][18], [PASS][19], [PASS][20], [PASS][21], [PASS][22], [PASS][23], [PASS][24], [PASS][25]) ([i915#4392]) -> ([PASS][26], [PASS][27], [PASS][28], [PASS][29], [PASS][30], [PASS][31], [PASS][32], [PASS][33], [PASS][34], [PASS][35], [PASS][36], [PASS][37], [PASS][38], [PASS][39], [PASS][40], [PASS][41], [PASS][42], [PASS][43], [PASS][44], [PASS][45], [PASS][46], [PASS][47], [PASS][48], [PASS][49], [PASS][50])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk1/boot.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk1/boot.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk1/boot.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk1/boot.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk2/boot.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk2/boot.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk3/boot.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk3/boot.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk3/boot.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk5/boot.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk5/boot.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk5/boot.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk6/boot.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk6/boot.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk6/boot.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk7/boot.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk7/boot.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk7/boot.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk7/boot.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk8/boot.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk8/boot.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk8/boot.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk9/boot.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk9/boot.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk9/boot.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk5/boot.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk9/boot.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk9/boot.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk9/boot.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk8/boot.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk8/boot.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk8/boot.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk8/boot.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk7/boot.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk7/boot.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk7/boot.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk6/boot.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk6/boot.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk6/boot.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk5/boot.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk5/boot.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk5/boot.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk3/boot.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk3/boot.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk3/boot.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/boot.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/boot.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/boot.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk1/boot.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk1/boot.html
### IGT changes ###
#### Issues hit ####
* igt@feature_discovery@display-3x:
- shard-iclb: NOTRUN -> [SKIP][51] ([i915#1839])
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@feature_discovery@display-3x.html
* igt@gem_create@create-massive:
- shard-iclb: NOTRUN -> [DMESG-WARN][52] ([i915#4991])
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@gem_create@create-massive.html
- shard-skl: NOTRUN -> [DMESG-WARN][53] ([i915#4991])
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl10/igt@gem_create@create-massive.html
* igt@gem_exec_balancer@parallel-bb-first:
- shard-kbl: NOTRUN -> [DMESG-WARN][54] ([i915#5076])
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-kbl3/igt@gem_exec_balancer@parallel-bb-first.html
* igt@gem_exec_fair@basic-flow@rcs0:
- shard-tglb: [PASS][55] -> [FAIL][56] ([i915#2842])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-tglb7/igt@gem_exec_fair@basic-flow@rcs0.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglb3/igt@gem_exec_fair@basic-flow@rcs0.html
* igt@gem_exec_fair@basic-none-solo@rcs0:
- shard-iclb: NOTRUN -> [FAIL][57] ([i915#2842])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@gem_exec_fair@basic-none-solo@rcs0.html
* igt@gem_exec_fair@basic-none@vcs0:
- shard-kbl: [PASS][58] -> [FAIL][59] ([i915#2842]) +2 similar issues
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-kbl3/igt@gem_exec_fair@basic-none@vcs0.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-kbl3/igt@gem_exec_fair@basic-none@vcs0.html
* igt@gem_exec_params@rsvd2-dirt:
- shard-iclb: NOTRUN -> [SKIP][60] ([fdo#109283])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@gem_exec_params@rsvd2-dirt.html
* igt@gem_exec_suspend@basic-s3@smem:
- shard-kbl: [PASS][61] -> [DMESG-WARN][62] ([i915#180]) +1 similar issue
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-kbl4/igt@gem_exec_suspend@basic-s3@smem.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-kbl1/igt@gem_exec_suspend@basic-s3@smem.html
* igt@gem_huc_copy@huc-copy:
- shard-tglb: NOTRUN -> [SKIP][63] ([i915#2190])
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglb7/igt@gem_huc_copy@huc-copy.html
* igt@gem_lmem_swapping@random:
- shard-skl: NOTRUN -> [SKIP][64] ([fdo#109271] / [i915#4613])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl4/igt@gem_lmem_swapping@random.html
* igt@gem_pwrite@basic-exhaustion:
- shard-tglb: NOTRUN -> [WARN][65] ([i915#2658])
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglb7/igt@gem_pwrite@basic-exhaustion.html
* igt@gem_pxp@create-protected-buffer:
- shard-iclb: NOTRUN -> [SKIP][66] ([i915#4270]) +2 similar issues
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@gem_pxp@create-protected-buffer.html
* igt@gem_render_copy@y-tiled-mc-ccs-to-y-tiled-ccs:
- shard-iclb: NOTRUN -> [SKIP][67] ([i915#768]) +1 similar issue
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@gem_render_copy@y-tiled-mc-ccs-to-y-tiled-ccs.html
* igt@gen9_exec_parse@batch-zero-length:
- shard-iclb: NOTRUN -> [SKIP][68] ([i915#2856]) +2 similar issues
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@gen9_exec_parse@batch-zero-length.html
* igt@i915_pm_rc6_residency@rc6-idle:
- shard-iclb: NOTRUN -> [WARN][69] ([i915#1804] / [i915#2684])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@i915_pm_rc6_residency@rc6-idle.html
* igt@i915_pm_rpm@gem-execbuf-stress-pc8:
- shard-iclb: NOTRUN -> [SKIP][70] ([fdo#109293] / [fdo#109506])
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@i915_pm_rpm@gem-execbuf-stress-pc8.html
* igt@i915_pm_rpm@system-suspend:
- shard-skl: [PASS][71] -> [INCOMPLETE][72] ([i915#151])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-skl7/igt@i915_pm_rpm@system-suspend.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl6/igt@i915_pm_rpm@system-suspend.html
* igt@kms_big_fb@linear-32bpp-rotate-180:
- shard-glk: [PASS][73] -> [DMESG-WARN][74] ([i915#118])
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk1/igt@kms_big_fb@linear-32bpp-rotate-180.html
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk8/igt@kms_big_fb@linear-32bpp-rotate-180.html
* igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-180-hflip:
- shard-glk: NOTRUN -> [SKIP][75] ([fdo#109271] / [i915#3777])
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-180-hflip.html
* igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-hflip:
- shard-skl: NOTRUN -> [SKIP][76] ([fdo#109271] / [i915#3777]) +2 similar issues
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl4/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-hflip.html
* igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-hflip-async-flip:
- shard-apl: NOTRUN -> [SKIP][77] ([fdo#109271] / [i915#3777]) +1 similar issue
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-apl2/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-hflip-async-flip.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip:
- shard-kbl: NOTRUN -> [SKIP][78] ([fdo#109271] / [i915#3777]) +2 similar issues
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-kbl3/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-async-flip:
- shard-skl: NOTRUN -> [FAIL][79] ([i915#3743])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl10/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html
* igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc:
- shard-apl: NOTRUN -> [SKIP][80] ([fdo#109271] / [i915#3886])
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-apl2/igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc.html
- shard-kbl: NOTRUN -> [SKIP][81] ([fdo#109271] / [i915#3886])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-kbl3/igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-c-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc:
- shard-iclb: NOTRUN -> [SKIP][82] ([fdo#109278] / [i915#3886]) +5 similar issues
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@kms_ccs@pipe-c-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc.html
- shard-skl: NOTRUN -> [SKIP][83] ([fdo#109271] / [i915#3886]) +8 similar issues
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl10/igt@kms_ccs@pipe-c-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-c-random-ccs-data-y_tiled_gen12_rc_ccs_cc:
- shard-glk: NOTRUN -> [SKIP][84] ([fdo#109271] / [i915#3886]) +2 similar issues
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/igt@kms_ccs@pipe-c-random-ccs-data-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_chamelium@dp-hpd-storm:
- shard-iclb: NOTRUN -> [SKIP][85] ([fdo#109284] / [fdo#111827]) +6 similar issues
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@kms_chamelium@dp-hpd-storm.html
* igt@kms_chamelium@hdmi-edid-change-during-suspend:
- shard-apl: NOTRUN -> [SKIP][86] ([fdo#109271] / [fdo#111827]) +2 similar issues
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-apl2/igt@kms_chamelium@hdmi-edid-change-during-suspend.html
* igt@kms_chamelium@hdmi-hpd-storm-disable:
- shard-skl: NOTRUN -> [SKIP][87] ([fdo#109271] / [fdo#111827]) +15 similar issues
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl10/igt@kms_chamelium@hdmi-hpd-storm-disable.html
* igt@kms_color_chamelium@pipe-b-ctm-0-75:
- shard-tglb: NOTRUN -> [SKIP][88] ([fdo#109284] / [fdo#111827])
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglb7/igt@kms_color_chamelium@pipe-b-ctm-0-75.html
* igt@kms_color_chamelium@pipe-b-degamma:
- shard-kbl: NOTRUN -> [SKIP][89] ([fdo#109271] / [fdo#111827]) +6 similar issues
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-kbl3/igt@kms_color_chamelium@pipe-b-degamma.html
* igt@kms_color_chamelium@pipe-d-ctm-0-25:
- shard-glk: NOTRUN -> [SKIP][90] ([fdo#109271] / [fdo#111827]) +2 similar issues
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/igt@kms_color_chamelium@pipe-d-ctm-0-25.html
- shard-iclb: NOTRUN -> [SKIP][91] ([fdo#109278] / [fdo#109284] / [fdo#111827])
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@kms_color_chamelium@pipe-d-ctm-0-25.html
* igt@kms_content_protection@dp-mst-type-0:
- shard-iclb: NOTRUN -> [SKIP][92] ([i915#3116])
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@kms_content_protection@dp-mst-type-0.html
* igt@kms_cursor_crc@pipe-b-cursor-32x32-onscreen:
- shard-skl: NOTRUN -> [SKIP][93] ([fdo#109271]) +202 similar issues
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl10/igt@kms_cursor_crc@pipe-b-cursor-32x32-onscreen.html
* igt@kms_cursor_crc@pipe-b-cursor-512x512-sliding:
- shard-iclb: NOTRUN -> [SKIP][94] ([fdo#109278] / [fdo#109279]) +1 similar issue
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@kms_cursor_crc@pipe-b-cursor-512x512-sliding.html
* igt@kms_cursor_crc@pipe-d-cursor-256x256-random:
- shard-glk: NOTRUN -> [SKIP][95] ([fdo#109271]) +31 similar issues
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/igt@kms_cursor_crc@pipe-d-cursor-256x256-random.html
* igt@kms_cursor_crc@pipe-d-cursor-512x512-offscreen:
- shard-tglb: NOTRUN -> [SKIP][96] ([fdo#109279] / [i915#3359])
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglb7/igt@kms_cursor_crc@pipe-d-cursor-512x512-offscreen.html
* igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size:
- shard-iclb: NOTRUN -> [SKIP][97] ([fdo#109274] / [fdo#109278]) +2 similar issues
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size:
- shard-skl: NOTRUN -> [FAIL][98] ([i915#2346] / [i915#533])
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl8/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
* igt@kms_cursor_legacy@pipe-d-single-move:
- shard-iclb: NOTRUN -> [SKIP][99] ([fdo#109278]) +28 similar issues
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@kms_cursor_legacy@pipe-d-single-move.html
* igt@kms_flip@2x-plain-flip-fb-recreate-interruptible:
- shard-iclb: NOTRUN -> [SKIP][100] ([fdo#109274]) +5 similar issues
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible.html
* igt@kms_flip@2x-wf_vblank-ts-check-interruptible@ab-hdmi-a1-hdmi-a2:
- shard-glk: [PASS][101] -> [FAIL][102] ([i915#2122])
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk5/igt@kms_flip@2x-wf_vblank-ts-check-interruptible@ab-hdmi-a1-hdmi-a2.html
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk8/igt@kms_flip@2x-wf_vblank-ts-check-interruptible@ab-hdmi-a1-hdmi-a2.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1:
- shard-skl: [PASS][103] -> [FAIL][104] ([i915#79]) +1 similar issue
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-skl4/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1.html
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl6/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1.html
* igt@kms_flip@flip-vs-suspend-interruptible@a-edp1:
- shard-skl: NOTRUN -> [INCOMPLETE][105] ([i915#4839])
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl8/igt@kms_flip@flip-vs-suspend-interruptible@a-edp1.html
* igt@kms_flip@flip-vs-suspend@a-dp1:
- shard-apl: [PASS][106] -> [DMESG-WARN][107] ([i915#180]) +2 similar issues
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-apl6/igt@kms_flip@flip-vs-suspend@a-dp1.html
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-apl3/igt@kms_flip@flip-vs-suspend@a-dp1.html
* igt@kms_flip@plain-flip-ts-check-interruptible@a-edp1:
- shard-skl: [PASS][108] -> [FAIL][109] ([i915#2122])
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-skl10/igt@kms_flip@plain-flip-ts-check-interruptible@a-edp1.html
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl9/igt@kms_flip@plain-flip-ts-check-interruptible@a-edp1.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytileccs-to-64bpp-ytile-downscaling:
- shard-iclb: [PASS][110] -> [SKIP][111] ([i915#3701])
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-iclb8/igt@kms_flip_scaled_crc@flip-32bpp-ytileccs-to-64bpp-ytile-downscaling.html
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb2/igt@kms_flip_scaled_crc@flip-32bpp-ytileccs-to-64bpp-ytile-downscaling.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-cur-indfb-move:
- shard-iclb: NOTRUN -> [SKIP][112] ([fdo#109280]) +18 similar issues
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-cur-indfb-move.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-mmap-wc:
- shard-kbl: NOTRUN -> [SKIP][113] ([fdo#109271]) +62 similar issues
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-kbl3/igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-render:
- shard-tglb: NOTRUN -> [SKIP][114] ([fdo#109280] / [fdo#111825])
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglb7/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-render.html
* igt@kms_hdr@static-toggle:
- shard-iclb: NOTRUN -> [SKIP][115] ([i915#1187])
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@kms_hdr@static-toggle.html
* igt@kms_pipe_crc_basic@disable-crc-after-crtc-pipe-d:
- shard-glk: NOTRUN -> [SKIP][116] ([fdo#109271] / [i915#533])
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/igt@kms_pipe_crc_basic@disable-crc-after-crtc-pipe-d.html
* igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence:
- shard-apl: NOTRUN -> [SKIP][117] ([fdo#109271] / [i915#533])
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-apl8/igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence.html
- shard-skl: NOTRUN -> [SKIP][118] ([fdo#109271] / [i915#533])
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl4/igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence.html
- shard-kbl: NOTRUN -> [SKIP][119] ([fdo#109271] / [i915#533])
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-kbl3/igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence.html
* igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
- shard-skl: NOTRUN -> [FAIL][120] ([fdo#108145] / [i915#265]) +1 similar issue
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl4/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
* igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb:
- shard-glk: NOTRUN -> [FAIL][121] ([fdo#108145] / [i915#265])
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb.html
* igt@kms_plane_lowres@pipe-a-tiling-none:
- shard-iclb: NOTRUN -> [SKIP][122] ([i915#3536]) +1 similar issue
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@kms_plane_lowres@pipe-a-tiling-none.html
* igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping:
- shard-skl: NOTRUN -> [SKIP][123] ([fdo#109271] / [i915#2733])
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl10/igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping.html
* igt@kms_psr2_sf@cursor-plane-update-sf:
- shard-skl: NOTRUN -> [SKIP][124] ([fdo#109271] / [i915#658])
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl10/igt@kms_psr2_sf@cursor-plane-update-sf.html
- shard-iclb: NOTRUN -> [SKIP][125] ([fdo#111068] / [i915#658])
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@kms_psr2_sf@cursor-plane-update-sf.html
* igt@kms_psr@psr2_cursor_plane_onoff:
- shard-iclb: [PASS][126] -> [SKIP][127] ([fdo#109441]) +1 similar issue
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@kms_psr@psr2_cursor_plane_onoff.html
* igt@kms_psr@psr2_cursor_render:
- shard-iclb: NOTRUN -> [SKIP][128] ([fdo#109441]) +1 similar issue
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@kms_psr@psr2_cursor_render.html
* igt@kms_psr@psr2_sprite_plane_move:
- shard-tglb: NOTRUN -> [FAIL][129] ([i915#132] / [i915#3467])
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglb7/igt@kms_psr@psr2_sprite_plane_move.html
* igt@kms_setmode@basic:
- shard-glk: [PASS][130] -> [FAIL][131] ([i915#31])
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk1/igt@kms_setmode@basic.html
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk8/igt@kms_setmode@basic.html
* igt@kms_writeback@writeback-fb-id:
- shard-skl: NOTRUN -> [SKIP][132] ([fdo#109271] / [i915#2437])
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl10/igt@kms_writeback@writeback-fb-id.html
- shard-iclb: NOTRUN -> [SKIP][133] ([i915#2437])
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@kms_writeback@writeback-fb-id.html
* igt@nouveau_crc@pipe-b-source-outp-complete:
- shard-iclb: NOTRUN -> [SKIP][134] ([i915#2530])
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@nouveau_crc@pipe-b-source-outp-complete.html
* igt@nouveau_crc@pipe-d-ctx-flip-skip-current-frame:
- shard-iclb: NOTRUN -> [SKIP][135] ([fdo#109278] / [i915#2530])
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@nouveau_crc@pipe-d-ctx-flip-skip-current-frame.html
* igt@nouveau_crc@pipe-d-source-rg:
- shard-tglb: NOTRUN -> [SKIP][136] ([i915#2530])
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglb7/igt@nouveau_crc@pipe-d-source-rg.html
* igt@perf@gen12-oa-tlb-invalidate:
- shard-iclb: NOTRUN -> [SKIP][137] ([fdo#109289]) +2 similar issues
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@perf@gen12-oa-tlb-invalidate.html
* igt@prime_nv_api@i915_nv_import_twice_check_flink_name:
- shard-iclb: NOTRUN -> [SKIP][138] ([fdo#109291]) +2 similar issues
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb3/igt@prime_nv_api@i915_nv_import_twice_check_flink_name.html
* igt@prime_nv_api@i915_nv_reimport_twice_check_flink_name:
- shard-apl: NOTRUN -> [SKIP][139] ([fdo#109271]) +30 similar issues
[139]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-apl2/igt@prime_nv_api@i915_nv_reimport_twice_check_flink_name.html
* igt@syncobj_timeline@transfer-timeline-point:
- shard-kbl: NOTRUN -> [DMESG-FAIL][140] ([i915#5098])
[140]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-kbl4/igt@syncobj_timeline@transfer-timeline-point.html
* igt@sysfs_clients@busy:
- shard-glk: NOTRUN -> [SKIP][141] ([fdo#109271] / [i915#2994])
[141]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/igt@sysfs_clients@busy.html
* igt@sysfs_clients@fair-7:
- shard-iclb: NOTRUN -> [SKIP][142] ([i915#2994]) +1 similar issue
[142]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb5/igt@sysfs_clients@fair-7.html
* igt@sysfs_clients@sema-10:
- shard-skl: NOTRUN -> [SKIP][143] ([fdo#109271] / [i915#2994]) +3 similar issues
[143]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-skl10/igt@sysfs_clients@sema-10.html
#### Possible fixes ####
* igt@feature_discovery@psr2:
- shard-iclb: [SKIP][144] ([i915#658]) -> [PASS][145]
[144]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-iclb8/igt@feature_discovery@psr2.html
[145]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb2/igt@feature_discovery@psr2.html
* igt@gem_ctx_isolation@preservation-s3@rcs0:
- shard-apl: [DMESG-WARN][146] ([i915#180]) -> [PASS][147] +3 similar issues
[146]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-apl4/igt@gem_ctx_isolation@preservation-s3@rcs0.html
[147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-apl8/igt@gem_ctx_isolation@preservation-s3@rcs0.html
* igt@gem_eio@kms:
- {shard-tglu}: [INCOMPLETE][148] ([i915#5182]) -> [PASS][149]
[148]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-tglu-5/igt@gem_eio@kms.html
[149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglu-1/igt@gem_eio@kms.html
* igt@gem_exec_capture@pi@bcs0:
- {shard-tglu}: [INCOMPLETE][150] ([i915#3371] / [i915#3731]) -> [PASS][151]
[150]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-tglu-5/igt@gem_exec_capture@pi@bcs0.html
[151]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-tglu-3/igt@gem_exec_capture@pi@bcs0.html
* igt@gem_exec_fair@basic-deadline:
- shard-glk: [FAIL][152] ([i915#2846]) -> [PASS][153]
[152]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk9/igt@gem_exec_fair@basic-deadline.html
[153]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-glk2/igt@gem_exec_fair@basic-deadline.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-iclb: [FAIL][154] ([i915#2842]) -> [PASS][155] +1 similar issue
[154]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-iclb6/igt@gem_exec_fair@basic-none-share@rcs0.html
[155]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-iclb4/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-none@vcs0:
- shard-apl: [FAIL][156] ([i915#2842]) -> [PASS][157]
[156]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-apl2/igt@gem_exec_fair@basic-none@vcs0.html
[157]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/shard-apl6/igt@gem_exec_fair@basic-none@vcs0.html
* igt@gem_exec_fair@basic-pace-share@rcs0:
- shard-glk: [FAIL][158] ([i915#2842]) -> [PASS][159]
[158]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11291/shard-glk2/igt@gem_exec_fair@ba
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22425/index.html
[-- Attachment #2: Type: text/html, Size: 33751 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Intel-gfx] [PATCH v2 1/1] drm/i915/reset: Fix error_state_read ptr + offset use
2022-02-26 6:27 ` [Intel-gfx] [PATCH v2 1/1] drm/i915/reset: Fix error_state_read ptr + offset use Alan Previn
@ 2022-03-01 21:37 ` John Harrison
2022-03-08 19:47 ` Teres Alexis, Alan Previn
0 siblings, 1 reply; 11+ messages in thread
From: John Harrison @ 2022-03-01 21:37 UTC (permalink / raw)
To: Alan Previn, intel-gfx
On 2/25/2022 22:27, Alan Previn wrote:
> Fix our pointer offset usage in error_state_read
> when there is no i915_gpu_coredump but buf offset
> is non-zero.
>
> This fixes a kernel page fault can happen when
> multiple tests are running concurrently in a loop
> and one is producing engine resets and consuming
> the i915 error_state dump while the other is
> forcing full GT resets. (takes a while to trigger).
Does need a fixes tag given that it is fixing a bug in an earlier patch.
Especially when it is a kernel BUG.
E.g.:
13:23> dim fixes 0e39037b31655
Fixes: 0e39037b3165 ("drm/i915: Cache the error string")
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Cc: <stable@vger.kernel.org> # v5.0+
> The dmesg call trace:
>
> 5014 [ 5590.803000] BUG: unable to handle page fault for address: ffffffffa0b0e000
> 5015 [ 5590.803009] #PF: supervisor read access in kernel mode
> 5016 [ 5590.803013] #PF: error_code(0x0000) - not-present page
> 5017 [ 5590.803016] PGD 5814067 P4D 5814067 PUD 5815063 PMD 109de4067 PTE 0
> 5018 [ 5590.803022] Oops: 0000 [#1] PREEMPT SMP NOPTI
> 5019 [ 5590.803026] CPU: 5 PID: 13656 Comm: i915_hangman Tainted: G U 5.17.0-rc5-
> ups69-guc-err-capt-rev6+ #136
> 5020 [ 5590.803033] Hardware name: Intel Corporation Alder Lake Client Platform/
> AlderLake-M LP4x RVP, BIOS ADLPFWI1.R00.3031.A02.2201171222
> 01/17/2022
> 5021 [ 5590.803039] RIP: 0010:memcpy_erms+0x6/0x10
> 5022 [ 5590.803045] Code: fe ff ff cc eb 1e 0f 1f 00 48 89 f8 48 89 d1 48 c1 e9 03 83
> e2 07 f3 48 a5 89 d1 f3 a4 c3 66 0f 1f 44 00 00 48 89 f8 48 89
> d1 <f3> a4 c3 0f 1f 80 00 00 00 00 48 89 f8 48 83 fa 20 72 7e 40 38 fe
> 5023 [ 5590.803054] RSP: 0018:ffffc90003a8fdf0 EFLAGS: 00010282
> 5024 [ 5590.803057] RAX: ffff888107ee9000 RBX: ffff888108cb1a00 RCX: 0000000000000f8f
> 5025 [ 5590.803061] RDX: 0000000000001000 RSI: ffffffffa0b0e000 RDI: ffff888107ee9071
> 5026 [ 5590.803065] RBP: 0000000000000000 R08: 0000000000000001 R09: 0000000000000001
> 5027 [ 5590.803069] R10: 0000000000000001 R11: 0000000000000002 R12: 0000000000000019
> 5028 [ 5590.803073] R13: 0000000000174fff R14: 0000000000001000 R15: ffff888107ee9000
> 5029 [ 5590.803077] FS: 00007f62a99bee80(0000) GS:ffff88849f880000(0000) knlGS:0000000000000000
> 5030 [ 5590.803082] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> 5031 [ 5590.803085] CR2: ffffffffa0b0e000 CR3: 000000010a1a8004 CR4: 0000000000770ee0
> 5032 [ 5590.803089] PKRU: 55555554
> 5033 [ 5590.803091] Call Trace:
> 5034 [ 5590.803093] <TASK>
> 5035 [ 5590.803096] error_state_read+0xa1/0xd0 [i915]
> 5036 [ 5590.803175] kernfs_fop_read_iter+0xb2/0x1b0
> 5037 [ 5590.803180] new_sync_read+0x116/0x1a0
> 5038 [ 5590.803185] vfs_read+0x114/0x1b0
> 5039 [ 5590.803189] ksys_read+0x63/0xe0
> 5040 [ 5590.803193] do_syscall_64+0x38/0xc0
> 5041 [ 5590.803197] entry_SYSCALL_64_after_hwframe+0x44/0xae
> 5042 [ 5590.803201] RIP: 0033:0x7f62aaea5912
> 5043 [ 5590.803204] Code: c0 e9 b2 fe ff ff 50 48 8d 3d 5a b9 0c 00 e8 05 19 02 00 0f 1f 44 00
> 00 f3 0f 1e fa 64 8b 04 25 18 00 00 00 85 c0 75 10 0f 05 <48> 3d 00 f0
> ff ff 77 56 c3 0f 1f 44 00 00 48 83 ec 28 48 89 54 24
> 5044 [ 5590.803213] RSP: 002b:00007fff5b659ae8 EFLAGS: 00000246 ORIG_RAX: 0000000000000000
> 5045 [ 5590.803218] RAX: ffffffffffffffda RBX: 0000000000100000 RCX: 00007f62aaea5912
> 5046 [ 5590.803221] RDX: 000000000008b000 RSI: 00007f62a8c4000f RDI: 0000000000000006
> 5047 [ 5590.803225] RBP: 00007f62a8bcb00f R08: 0000000000200010 R09: 0000000000101000
> 5048 [ 5590.803229] R10: 0000000000000001 R11: 0000000000000246 R12: 0000000000000006
> 5049 [ 5590.803233] R13: 0000000000075000 R14: 00007f62a8acb010 R15: 0000000000200000
> 5050 [ 5590.803238] </TASK>
> 5051 [ 5590.803240] Modules linked in: i915 ttm drm_buddy drm_dp_helper drm_kms_helper syscopyarea
> sysfillrect sysimgblt fb_sys_fops prime_numbers nfnetlink br_netfilter overlay
> mei_pxp mei_hdcp x86_pkg_temp_thermal coretemp kvm_intel snd_hda_codec_hdmi
> snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep snd_hda_core snd_pcm mei_me
> mei fuse ip_tables x_tables crct10dif_pclmul e1000e crc32_pclmul ptp i2c_i801
> ghash_clmulni_intel i2c_smbus pps_core [last unloa ded: ttm]
> 5052 [ 5590.803277] CR2: ffffffffa0b0e000
> 5053 [ 5590.803280] ---[ end trace 0000000000000000 ]---
>
> Signed-off-by: Alan Previn <alan.previn.teres.alexis@intel.com>
> ---
> drivers/gpu/drm/i915/i915_sysfs.c | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_sysfs.c b/drivers/gpu/drm/i915/i915_sysfs.c
> index a4d1759375b9..c40e51298066 100644
> --- a/drivers/gpu/drm/i915/i915_sysfs.c
> +++ b/drivers/gpu/drm/i915/i915_sysfs.c
> @@ -432,7 +432,7 @@ static ssize_t error_state_read(struct file *filp, struct kobject *kobj,
> struct device *kdev = kobj_to_dev(kobj);
> struct drm_i915_private *i915 = kdev_minor_to_i915(kdev);
> struct i915_gpu_coredump *gpu;
> - ssize_t ret;
> + ssize_t ret = 0;
>
> gpu = i915_first_error_state(i915);
> if (IS_ERR(gpu)) {
> @@ -444,8 +444,10 @@ static ssize_t error_state_read(struct file *filp, struct kobject *kobj,
> const char *str = "No error state collected\n";
> size_t len = strlen(str);
>
> - ret = min_t(size_t, count, len - off);
> - memcpy(buf, str + off, ret);
> + if (off < len) {
> + ret = min_t(size_t, count, len - off);
> + memcpy(buf, str + off, ret);
> + }
So the problem is that the error dump disappeared while it was being
read? That seems like it cause more problems than just this out-of-range
access. E.g. what if the dump was freed and a new one created? The
entity doing the partial reads would end up with half of one dump and
half of the next.
I think we should at least add a FIXME comment to the code that fast
recycling dumps could cause inconsistent sysfs reads.
I guess you could do something like add a unique id to the gpu coredump
structure. Then, when a partial sysfs read occurs starting at zero (i.e.
a new read), take a note of the id somewhere (e.g. inside the i915
structure). When the next non-zero read comes in, compare the save id
with the current coredump's id and return an error if there is a mis-match.
Not sure if this would be viewed as an important enough problem to be
worth fixing. I'd be happy with just a FIXME comment for now.
I would also change the test to be 'if (off)' rather than 'if (off <
len)'. Technically, the user could have read the first 10 bytes of a
coredump and then gets "tate collected\n" as the remainder, for example.
If we ensure that 'off' is zero then we know that this is a fresh read
from scratch.
John.
> }
>
> return ret;
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Intel-gfx] [PATCH v2 1/1] drm/i915/reset: Fix error_state_read ptr + offset use
2022-03-01 21:37 ` John Harrison
@ 2022-03-08 19:47 ` Teres Alexis, Alan Previn
2022-03-10 1:18 ` John Harrison
0 siblings, 1 reply; 11+ messages in thread
From: Teres Alexis, Alan Previn @ 2022-03-08 19:47 UTC (permalink / raw)
To: John Harrison, intel-gfx
On 3/1/2022 1:37 PM, John Harrison wrote:
> On 2/25/2022 22:27, Alan Previn wrote:
>> ...
>> This fixes a kernel page fault can happen when
>> multiple tests are running concurrently in a loop
>> and one is producing engine resets and consuming
>> the i915 error_state dump while the other is
>> forcing full GT resets. (takes a while to trigger).
> Does need a fixes tag given that it is fixing a bug in an earlier
> patch. Especially when it is a kernel BUG.
> E.g.:
> 13:23> dim fixes 0e39037b31655
> Fixes: 0e39037b3165 ("drm/i915: Cache the error string")
>
okay, will add that.
diff --git a/drivers/gpu/drm/i915/i915_sysfs.c
b/drivers/gpu/drm/i915/i915_sysfs.c
>> index a4d1759375b9..c40e51298066 100644
>> --- a/drivers/gpu/drm/i915/i915_sysfs.c
>> +++ b/drivers/gpu/drm/i915/i915_sysfs.c
>> @@ -432,7 +432,7 @@ static ssize_t error_state_read(struct file
>> *filp, struct kobject *kobj,
>> struct device *kdev = kobj_to_dev(kobj);
>> struct drm_i915_private *i915 = kdev_minor_to_i915(kdev);
>> struct i915_gpu_coredump *gpu;
>> - ssize_t ret;
>> + ssize_t ret = 0;
>> gpu = i915_first_error_state(i915);
>> if (IS_ERR(gpu)) {
>> @@ -444,8 +444,10 @@ static ssize_t error_state_read(struct file
>> *filp, struct kobject *kobj,
>> const char *str = "No error state collected\n";
>> size_t len = strlen(str);
>> - ret = min_t(size_t, count, len - off);
>> - memcpy(buf, str + off, ret);
>> + if (off < len) {
>> + ret = min_t(size_t, count, len - off);
>> + memcpy(buf, str + off, ret);
>> + }
> So the problem is that the error dump disappeared while it was being
> read? That seems like it cause more problems than just this
> out-of-range access. E.g. what if the dump was freed and a new one
> created? The entity doing the partial reads would end up with half of
> one dump and half of the next.
>
> I think we should at least add a FIXME comment to the code that fast
> recycling dumps could cause inconsistent sysfs reads.
>
> I guess you could do something like add a unique id to the gpu
> coredump structure. Then, when a partial sysfs read occurs starting at
> zero (i.e. a new read), take a note of the id somewhere (e.g. inside
> the i915 structure). When the next non-zero read comes in, compare the
> save id with the current coredump's id and return an error if there is
> a mis-match.
>
> Not sure if this would be viewed as an important enough problem to be
> worth fixing. I'd be happy with just a FIXME comment for now.
For now I shall add a FIXME additional checks might impact IGT's that
currently dump and check the error state. I would assume with that
additional check in kernel, we would need to return a specific error
value like -ENODATA or -ENOLINK and handle it accordingly in the igt.
>
> I would also change the test to be 'if (off)' rather than 'if (off <
> len)'. Technically, the user could have read the first 10 bytes of a
> coredump and then gets "tate collected\n" as the remainder, for
> example. If we ensure that 'off' is zero then we know that this is a
> fresh read from scratch.
>
> John.
>
I'm a little confused, did u mean: in the case we dont have a gpu-state
to report, and then the user offset is zero (i.e. "if (!off)" ) then we
copy the string vs if we dont have a gpu-state to report and the
user-offset is non-zero, then we return an error (like the -ENOLINK?).
If thats what you meant, then it does mean we are assuming that (in the
case we dont have a gpu-state) the user will never come in with a
first-time-read where the user's buffer size of less than the string
length and be expected continue to read the rest of the string-length.
This i guess is okay since even 6 chars are enough to say "No err". :)
>> }
>> return ret;
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Intel-gfx] [PATCH v2 1/1] drm/i915/reset: Fix error_state_read ptr + offset use
2022-03-08 19:47 ` Teres Alexis, Alan Previn
@ 2022-03-10 1:18 ` John Harrison
2022-03-10 1:45 ` Teres Alexis, Alan Previn
0 siblings, 1 reply; 11+ messages in thread
From: John Harrison @ 2022-03-10 1:18 UTC (permalink / raw)
To: Teres Alexis, Alan Previn, intel-gfx
On 3/8/2022 11:47, Teres Alexis, Alan Previn wrote:
> On 3/1/2022 1:37 PM, John Harrison wrote:
>> On 2/25/2022 22:27, Alan Previn wrote:
>>> ...
>>> This fixes a kernel page fault can happen when
>>> multiple tests are running concurrently in a loop
>>> and one is producing engine resets and consuming
>>> the i915 error_state dump while the other is
>>> forcing full GT resets. (takes a while to trigger).
>> Does need a fixes tag given that it is fixing a bug in an earlier
>> patch. Especially when it is a kernel BUG.
>> E.g.:
>> 13:23> dim fixes 0e39037b31655
>> Fixes: 0e39037b3165 ("drm/i915: Cache the error string")
>>
> okay, will add that.
>
> diff --git a/drivers/gpu/drm/i915/i915_sysfs.c
> b/drivers/gpu/drm/i915/i915_sysfs.c
>>> index a4d1759375b9..c40e51298066 100644
>>> --- a/drivers/gpu/drm/i915/i915_sysfs.c
>>> +++ b/drivers/gpu/drm/i915/i915_sysfs.c
>>> @@ -432,7 +432,7 @@ static ssize_t error_state_read(struct file
>>> *filp, struct kobject *kobj,
>>> struct device *kdev = kobj_to_dev(kobj);
>>> struct drm_i915_private *i915 = kdev_minor_to_i915(kdev);
>>> struct i915_gpu_coredump *gpu;
>>> - ssize_t ret;
>>> + ssize_t ret = 0;
>>> gpu = i915_first_error_state(i915);
>>> if (IS_ERR(gpu)) {
>>> @@ -444,8 +444,10 @@ static ssize_t error_state_read(struct file
>>> *filp, struct kobject *kobj,
>>> const char *str = "No error state collected\n";
>>> size_t len = strlen(str);
>>> - ret = min_t(size_t, count, len - off);
>>> - memcpy(buf, str + off, ret);
>>> + if (off < len) {
>>> + ret = min_t(size_t, count, len - off);
>>> + memcpy(buf, str + off, ret);
>>> + }
>> So the problem is that the error dump disappeared while it was being
>> read? That seems like it cause more problems than just this
>> out-of-range access. E.g. what if the dump was freed and a new one
>> created? The entity doing the partial reads would end up with half of
>> one dump and half of the next.
>>
>> I think we should at least add a FIXME comment to the code that fast
>> recycling dumps could cause inconsistent sysfs reads.
>>
>> I guess you could do something like add a unique id to the gpu
>> coredump structure. Then, when a partial sysfs read occurs starting
>> at zero (i.e. a new read), take a note of the id somewhere (e.g.
>> inside the i915 structure). When the next non-zero read comes in,
>> compare the save id with the current coredump's id and return an
>> error if there is a mis-match.
>>
>> Not sure if this would be viewed as an important enough problem to be
>> worth fixing. I'd be happy with just a FIXME comment for now.
> For now I shall add a FIXME additional checks might impact IGT's that
> currently dump and check the error state. I would assume with that
> additional check in kernel, we would need to return a specific error
> value like -ENODATA or -ENOLINK and handle it accordingly in the igt.
If an an extra check against returning invalid data affects an existing
IGT then that IGT is already broken. The check would to prevent userland
reading the first half of one capture and then getting the second half
of a completely different one. If that is already happening then the
returned data is meaningless gibberish and even if the IGT says it is
passing, it really isn't.
>>
>> I would also change the test to be 'if (off)' rather than 'if (off <
>> len)'. Technically, the user could have read the first 10 bytes of a
>> coredump and then gets "tate collected\n" as the remainder, for
>> example. If we ensure that 'off' is zero then we know that this is a
>> fresh read from scratch.
>>
>> John.
>>
> I'm a little confused, did u mean: in the case we dont have a
> gpu-state to report, and then the user offset is zero (i.e. "if
> (!off)" ) then we copy the string vs if we dont have a gpu-state to
> report and the user-offset is non-zero, then we return an error (like
> the -ENOLINK?). If thats what you meant, then it does mean we are
> assuming that (in the case we dont have a gpu-state) the user will
> never come in with a first-time-read where the user's buffer size of
> less than the string length and be expected continue to read the rest
> of the string-length. This i guess is okay since even 6 chars are
> enough to say "No err". :)
Sorry, yes. That was meant to say 'if (!off)'.
Hmm, I guess technically the user could be issuing single byte reads.
User's can be evil.
Okay. Maybe just add a FIXME about needing to check for
changed/missing/new error state since last read if the offset is
non-zero and otherwise leave it as is.
John.
>>> }
>>> return ret;
>>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Intel-gfx] [PATCH v2 1/1] drm/i915/reset: Fix error_state_read ptr + offset use
2022-03-10 1:18 ` John Harrison
@ 2022-03-10 1:45 ` Teres Alexis, Alan Previn
2022-03-11 4:52 ` Teres Alexis, Alan Previn
0 siblings, 1 reply; 11+ messages in thread
From: Teres Alexis, Alan Previn @ 2022-03-10 1:45 UTC (permalink / raw)
To: John Harrison, intel-gfx
On 3/9/2022 5:18 PM, John Harrison wrote:
> On 3/8/2022 11:47, Teres Alexis, Alan Previn wrote:
>> On 3/1/2022 1:37 PM, John Harrison wrote:
>>> On 2/25/2022 22:27, Alan Previn wrote:
>>>> ...
>>>> This fixes a kernel page fault can happen when
>>>> multiple tests are running concurrently in a loop
>>>> and one is producing engine resets and consuming
>>>> the i915 error_state dump while the other is
>>>> forcing full GT resets. (takes a while to trigger).
>>> Does need a fixes tag given that it is fixing a bug in an earlier
>>> patch. Especially when it is a kernel BUG.
>>> E.g.:
>>> 13:23> dim fixes 0e39037b31655
>>> Fixes: 0e39037b3165 ("drm/i915: Cache the error string")
>>>
>> okay, will add that.
>>
>> diff --git a/drivers/gpu/drm/i915/i915_sysfs.c
>> b/drivers/gpu/drm/i915/i915_sysfs.c
>>>> index a4d1759375b9..c40e51298066 100644
>>>> --- a/drivers/gpu/drm/i915/i915_sysfs.c
>>>> +++ b/drivers/gpu/drm/i915/i915_sysfs.c
>>>> @@ -432,7 +432,7 @@ static ssize_t error_state_read(struct file
>>>> *filp, struct kobject *kobj,
>>>> struct device *kdev = kobj_to_dev(kobj);
>>>> struct drm_i915_private *i915 = kdev_minor_to_i915(kdev);
>>>> struct i915_gpu_coredump *gpu;
>>>> - ssize_t ret;
>>>> + ssize_t ret = 0;
>>>> gpu = i915_first_error_state(i915);
>>>> if (IS_ERR(gpu)) {
>>>> @@ -444,8 +444,10 @@ static ssize_t error_state_read(struct file
>>>> *filp, struct kobject *kobj,
>>>> const char *str = "No error state collected\n";
>>>> size_t len = strlen(str);
>>>> - ret = min_t(size_t, count, len - off);
>>>> - memcpy(buf, str + off, ret);
>>>> + if (off < len) {
>>>> + ret = min_t(size_t, count, len - off);
>>>> + memcpy(buf, str + off, ret);
>>>> + }
>>> So the problem is that the error dump disappeared while it was being
>>> read? That seems like it cause more problems than just this
>>> out-of-range access. E.g. what if the dump was freed and a new one
>>> created? The entity doing the partial reads would end up with half
>>> of one dump and half of the next.
>>>
>>> I think we should at least add a FIXME comment to the code that fast
>>> recycling dumps could cause inconsistent sysfs reads.
>>>
>>> I guess you could do something like add a unique id to the gpu
>>> coredump structure. Then, when a partial sysfs read occurs starting
>>> at zero (i.e. a new read), take a note of the id somewhere (e.g.
>>> inside the i915 structure). When the next non-zero read comes in,
>>> compare the save id with the current coredump's id and return an
>>> error if there is a mis-match.
>>>
>>> Not sure if this would be viewed as an important enough problem to
>>> be worth fixing. I'd be happy with just a FIXME comment for now.
>> For now I shall add a FIXME additional checks might impact IGT's that
>> currently dump and check the error state. I would assume with that
>> additional check in kernel, we would need to return a specific error
>> value like -ENODATA or -ENOLINK and handle it accordingly in the igt.
> If an an extra check against returning invalid data affects an
> existing IGT then that IGT is already broken. The check would to
> prevent userland reading the first half of one capture and then
> getting the second half of a completely different one. If that is
> already happening then the returned data is meaningless gibberish and
> even if the IGT says it is passing, it really isn't.
>
>
>>>
>>> I would also change the test to be 'if (off)' rather than 'if (off <
>>> len)'. Technically, the user could have read the first 10 bytes of a
>>> coredump and then gets "tate collected\n" as the remainder, for
>>> example. If we ensure that 'off' is zero then we know that this is a
>>> fresh read from scratch.
>>>
>>> John.
>>>
>> I'm a little confused, did u mean: in the case we dont have a
>> gpu-state to report, and then the user offset is zero (i.e. "if
>> (!off)" ) then we copy the string vs if we dont have a gpu-state to
>> report and the user-offset is non-zero, then we return an error (like
>> the -ENOLINK?). If thats what you meant, then it does mean we are
>> assuming that (in the case we dont have a gpu-state) the user will
>> never come in with a first-time-read where the user's buffer size of
>> less than the string length and be expected continue to read the rest
>> of the string-length. This i guess is okay since even 6 chars are
>> enough to say "No err". :)
> Sorry, yes. That was meant to say 'if (!off)'.
>
> Hmm, I guess technically the user could be issuing single byte reads.
> User's can be evil.
>
> Okay. Maybe just add a FIXME about needing to check for
> changed/missing/new error state since last read if the offset is
> non-zero and otherwise leave it as is.
>
> John.
>
Sounds good - will do. Apologies on the tardiness with the responses.
>
>>>> }
>>>> return ret;
>>>
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Intel-gfx] [PATCH v2 1/1] drm/i915/reset: Fix error_state_read ptr + offset use
2022-03-10 1:45 ` Teres Alexis, Alan Previn
@ 2022-03-11 4:52 ` Teres Alexis, Alan Previn
0 siblings, 0 replies; 11+ messages in thread
From: Teres Alexis, Alan Previn @ 2022-03-11 4:52 UTC (permalink / raw)
To: John Harrison, intel-gfx
i missed something in rev3, but rev4 ended up as a different series.
Please review this new URL for this patch. Apologies for the confusion:
https://patchwork.freedesktop.org/series/101256/
...alan
On 3/9/2022 5:45 PM, Teres Alexis, Alan Previn wrote:
>
> On 3/9/2022 5:18 PM, John Harrison wrote:
>> On 3/8/2022 11:47, Teres Alexis, Alan Previn wrote:
>>> On 3/1/2022 1:37 PM, John Harrison wrote:
>>>> On 2/25/2022 22:27, Alan Previn wrote:
>>>>> ...
>>>>> This fixes a kernel page fault can happen when
>>>>> multiple tests are running concurrently in a loop
>>>>> and one is producing engine resets and consuming
>>>>> the i915 error_state dump while the other is
>>>>> forcing full GT resets. (takes a while to trigger).
>>>> Does need a fixes tag given that it is fixing a bug in an earlier
>>>> patch. Especially when it is a kernel BUG.
>>>> E.g.:
>>>> 13:23> dim fixes 0e39037b31655
>>>> Fixes: 0e39037b3165 ("drm/i915: Cache the error string")
>>>>
>>> okay, will add that.
>>>
>>> diff --git a/drivers/gpu/drm/i915/i915_sysfs.c
>>> b/drivers/gpu/drm/i915/i915_sysfs.c
>>>>> index a4d1759375b9..c40e51298066 100644
>>>>> --- a/drivers/gpu/drm/i915/i915_sysfs.c
>>>>> +++ b/drivers/gpu/drm/i915/i915_sysfs.c
>>>>> @@ -432,7 +432,7 @@ static ssize_t error_state_read(struct file
>>>>> *filp, struct kobject *kobj,
>>>>> struct device *kdev = kobj_to_dev(kobj);
>>>>> struct drm_i915_private *i915 = kdev_minor_to_i915(kdev);
>>>>> struct i915_gpu_coredump *gpu;
>>>>> - ssize_t ret;
>>>>> + ssize_t ret = 0;
>>>>> gpu = i915_first_error_state(i915);
>>>>> if (IS_ERR(gpu)) {
>>>>> @@ -444,8 +444,10 @@ static ssize_t error_state_read(struct file
>>>>> *filp, struct kobject *kobj,
>>>>> const char *str = "No error state collected\n";
>>>>> size_t len = strlen(str);
>>>>> - ret = min_t(size_t, count, len - off);
>>>>> - memcpy(buf, str + off, ret);
>>>>> + if (off < len) {
>>>>> + ret = min_t(size_t, count, len - off);
>>>>> + memcpy(buf, str + off, ret);
>>>>> + }
>>>> So the problem is that the error dump disappeared while it was
>>>> being read? That seems like it cause more problems than just this
>>>> out-of-range access. E.g. what if the dump was freed and a new one
>>>> created? The entity doing the partial reads would end up with half
>>>> of one dump and half of the next.
>>>>
>>>> I think we should at least add a FIXME comment to the code that
>>>> fast recycling dumps could cause inconsistent sysfs reads.
>>>>
>>>> I guess you could do something like add a unique id to the gpu
>>>> coredump structure. Then, when a partial sysfs read occurs starting
>>>> at zero (i.e. a new read), take a note of the id somewhere (e.g.
>>>> inside the i915 structure). When the next non-zero read comes in,
>>>> compare the save id with the current coredump's id and return an
>>>> error if there is a mis-match.
>>>>
>>>> Not sure if this would be viewed as an important enough problem to
>>>> be worth fixing. I'd be happy with just a FIXME comment for now.
>>> For now I shall add a FIXME additional checks might impact IGT's
>>> that currently dump and check the error state. I would assume with
>>> that additional check in kernel, we would need to return a specific
>>> error value like -ENODATA or -ENOLINK and handle it accordingly in
>>> the igt.
>> If an an extra check against returning invalid data affects an
>> existing IGT then that IGT is already broken. The check would to
>> prevent userland reading the first half of one capture and then
>> getting the second half of a completely different one. If that is
>> already happening then the returned data is meaningless gibberish and
>> even if the IGT says it is passing, it really isn't.
>>
>>
>>>>
>>>> I would also change the test to be 'if (off)' rather than 'if (off
>>>> < len)'. Technically, the user could have read the first 10 bytes
>>>> of a coredump and then gets "tate collected\n" as the remainder,
>>>> for example. If we ensure that 'off' is zero then we know that this
>>>> is a fresh read from scratch.
>>>>
>>>> John.
>>>>
>>> I'm a little confused, did u mean: in the case we dont have a
>>> gpu-state to report, and then the user offset is zero (i.e. "if
>>> (!off)" ) then we copy the string vs if we dont have a gpu-state to
>>> report and the user-offset is non-zero, then we return an error
>>> (like the -ENOLINK?). If thats what you meant, then it does mean we
>>> are assuming that (in the case we dont have a gpu-state) the user
>>> will never come in with a first-time-read where the user's buffer
>>> size of less than the string length and be expected continue to read
>>> the rest of the string-length. This i guess is okay since even 6
>>> chars are enough to say "No err". :)
>> Sorry, yes. That was meant to say 'if (!off)'.
>>
>> Hmm, I guess technically the user could be issuing single byte reads.
>> User's can be evil.
>>
>> Okay. Maybe just add a FIXME about needing to check for
>> changed/missing/new error state since last read if the offset is
>> non-zero and otherwise leave it as is.
>>
>> John.
>>
> Sounds good - will do. Apologies on the tardiness with the responses.
>>
>>>>> }
>>>>> return ret;
>>>>
>>
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2022-03-11 4:52 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-26 6:27 [PATCH v2 0/1] Fix i915 error_state_read ptr use Alan Previn
2022-02-26 6:27 ` [Intel-gfx] " Alan Previn
2022-02-26 6:27 ` [Intel-gfx] [PATCH v2 1/1] drm/i915/reset: Fix error_state_read ptr + offset use Alan Previn
2022-03-01 21:37 ` John Harrison
2022-03-08 19:47 ` Teres Alexis, Alan Previn
2022-03-10 1:18 ` John Harrison
2022-03-10 1:45 ` Teres Alexis, Alan Previn
2022-03-11 4:52 ` Teres Alexis, Alan Previn
2022-02-26 8:00 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Fix i915 error_state_read ptr use (rev2) Patchwork
2022-02-26 8:32 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2022-02-27 4:41 ` [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.