* [Intel-gfx] [PATCH 0/2] hda/i915: split wait for component binding
@ 2022-02-23 19:52 Ramalingam C
2022-02-23 19:52 ` [Intel-gfx] [PATCH 1/2] kernel/hung_task: Exporting sysctl_hung_task_timeout_secs Ramalingam C
` (3 more replies)
0 siblings, 4 replies; 9+ messages in thread
From: Ramalingam C @ 2022-02-23 19:52 UTC (permalink / raw)
To: intel-gfx, dri-devel; +Cc: lucas.demarchi
To avoid the intended wait being reported as kworker hung, split the
wait for i915 component bind, in multiples of
sysctl_hung_task_timeout_secs.
To facilitate this we are exporting sysctl_hung_task_timeout_secs from
kernel/hung_task.c for the kernel modules' use.
Ramalingam C (2):
kernel/hung_task: Exporting sysctl_hung_task_timeout_secs
hda/i915: split the wait for the component binding
kernel/hung_task.c | 1 +
sound/hda/hdac_i915.c | 17 ++++++++++++++---
2 files changed, 15 insertions(+), 3 deletions(-)
--
2.20.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Intel-gfx] [PATCH 1/2] kernel/hung_task: Exporting sysctl_hung_task_timeout_secs
2022-02-23 19:52 [Intel-gfx] [PATCH 0/2] hda/i915: split wait for component binding Ramalingam C
@ 2022-02-23 19:52 ` Ramalingam C
2022-02-24 16:33 ` Lucas De Marchi
2022-02-23 19:52 ` [Intel-gfx] [PATCH 2/2] hda/i915: split the wait for the component binding Ramalingam C
` (2 subsequent siblings)
3 siblings, 1 reply; 9+ messages in thread
From: Ramalingam C @ 2022-02-23 19:52 UTC (permalink / raw)
To: intel-gfx, dri-devel; +Cc: lucas.demarchi
Exporting sysctl_hung_task_timeout_secs, to make it available for other
kernel modules.
Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
cc: Lucas De Marchi <lucas.demarchi@intel.com>
---
kernel/hung_task.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/kernel/hung_task.c b/kernel/hung_task.c
index db59b6d4f0e7..01120265395d 100644
--- a/kernel/hung_task.c
+++ b/kernel/hung_task.c
@@ -43,6 +43,7 @@ int __read_mostly sysctl_hung_task_check_count = PID_MAX_LIMIT;
* Zero means infinite timeout - no checking done:
*/
unsigned long __read_mostly sysctl_hung_task_timeout_secs = CONFIG_DEFAULT_HUNG_TASK_TIMEOUT;
+EXPORT_SYMBOL(sysctl_hung_task_timeout_secs);
/*
* Zero (default value) means use sysctl_hung_task_timeout_secs:
--
2.20.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [Intel-gfx] [PATCH 2/2] hda/i915: split the wait for the component binding
2022-02-23 19:52 [Intel-gfx] [PATCH 0/2] hda/i915: split wait for component binding Ramalingam C
2022-02-23 19:52 ` [Intel-gfx] [PATCH 1/2] kernel/hung_task: Exporting sysctl_hung_task_timeout_secs Ramalingam C
@ 2022-02-23 19:52 ` Ramalingam C
2022-02-24 16:34 ` Kai Vehmanen
2022-02-24 17:13 ` Lucas De Marchi
2022-02-24 14:52 ` [Intel-gfx] ✓ Fi.CI.BAT: success for hda/i915: split wait for " Patchwork
2022-02-25 4:21 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
3 siblings, 2 replies; 9+ messages in thread
From: Ramalingam C @ 2022-02-23 19:52 UTC (permalink / raw)
To: intel-gfx, dri-devel; +Cc: Kai Vehmanen, lucas.demarchi
Split the wait for component binding from i915 in multiples of
sysctl_hung_task_timeout_secs. This helps to avoid the possible kworker
thread hung detection given below.
<3>[ 60.946316] INFO: task kworker/11:1:104 blocked for more than 30
seconds.
<3>[ 60.946479] Tainted: G W
5.17.0-rc5-CI-CI_DRM_11265+ #1
<3>[ 60.946580] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs"
disables this message.
<6>[ 60.946688] task:kworker/11:1 state:D stack:14192 pid: 104
ppid: 2 flags:0x00004000
<6>[ 60.946713] Workqueue: events azx_probe_work [snd_hda_intel]
<6>[ 60.946740] Call Trace:
<6>[ 60.946745] <TASK>
<6>[ 60.946763] __schedule+0x42c/0xa80
<6>[ 60.946797] schedule+0x3f/0xc0
<6>[ 60.946811] schedule_timeout+0x1be/0x2e0
<6>[ 60.946829] ? del_timer_sync+0xb0/0xb0
<6>[ 60.946849] ? 0xffffffff81000000
<6>[ 60.946864] ? wait_for_completion_timeout+0x79/0x120
<6>[ 60.946879] wait_for_completion_timeout+0xab/0x120
<6>[ 60.946906] snd_hdac_i915_init+0xa5/0xb0 [snd_hda_core]
<6>[ 60.946943] azx_probe_work+0x71/0x84c [snd_hda_intel]
<6>[ 60.946974] process_one_work+0x275/0x5c0
<6>[ 60.947010] worker_thread+0x37/0x370
<6>[ 60.947028] ? process_one_work+0x5c0/0x5c0
<6>[ 60.947038] kthread+0xef/0x120
<6>[ 60.947047] ? kthread_complete_and_exit+0x20/0x20
<6>[ 60.947065] ret_from_fork+0x22/0x30
<6>[ 60.947110] </TASK>
Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
cc: Kai Vehmanen <kai.vehmanen@intel.com>
cc: Lucas De Marchi <lucas.demarchi@intel.com>
---
sound/hda/hdac_i915.c | 17 ++++++++++++++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/sound/hda/hdac_i915.c b/sound/hda/hdac_i915.c
index d20a450a9a15..daaeebc5099e 100644
--- a/sound/hda/hdac_i915.c
+++ b/sound/hda/hdac_i915.c
@@ -6,6 +6,7 @@
#include <linux/init.h>
#include <linux/module.h>
#include <linux/pci.h>
+#include <linux/sched/sysctl.h>
#include <sound/core.h>
#include <sound/hdaudio.h>
#include <sound/hda_i915.h>
@@ -163,7 +164,8 @@ static bool dg1_gfx_present(void)
int snd_hdac_i915_init(struct hdac_bus *bus)
{
struct drm_audio_component *acomp;
- int err;
+ unsigned long timeout, ret = 0;
+ int err, i, itr_cnt;
if (!i915_gfx_present())
return -ENODEV;
@@ -182,9 +184,18 @@ int snd_hdac_i915_init(struct hdac_bus *bus)
if (!acomp->ops) {
if (!IS_ENABLED(CONFIG_MODULES) ||
!request_module("i915")) {
+ if (!sysctl_hung_task_timeout_secs) {
+ itr_cnt = 1;
+ timeout = msecs_to_jiffies(60 * 1000);
+ } else {
+ itr_cnt = DIV_ROUND_UP(60, sysctl_hung_task_timeout_secs);
+ timeout = msecs_to_jiffies(sysctl_hung_task_timeout_secs * 1000);
+ }
+
/* 60s timeout */
- wait_for_completion_timeout(&acomp->master_bind_complete,
- msecs_to_jiffies(30 * 1000));
+ for (i = 0; i < itr_cnt || !ret; i++)
+ ret = wait_for_completion_timeout(&acomp->master_bind_complete,
+ timeout);
}
}
if (!acomp->ops) {
--
2.20.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for hda/i915: split wait for component binding
2022-02-23 19:52 [Intel-gfx] [PATCH 0/2] hda/i915: split wait for component binding Ramalingam C
2022-02-23 19:52 ` [Intel-gfx] [PATCH 1/2] kernel/hung_task: Exporting sysctl_hung_task_timeout_secs Ramalingam C
2022-02-23 19:52 ` [Intel-gfx] [PATCH 2/2] hda/i915: split the wait for the component binding Ramalingam C
@ 2022-02-24 14:52 ` Patchwork
2022-02-25 4:21 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
3 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2022-02-24 14:52 UTC (permalink / raw)
To: Ramalingam C; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 4667 bytes --]
== Series Details ==
Series: hda/i915: split wait for component binding
URL : https://patchwork.freedesktop.org/series/100661/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11277 -> Patchwork_22391
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/index.html
Participating hosts (46 -> 40)
------------------------------
Missing (6): fi-cml-u2 shard-tglu fi-bsw-cyan fi-icl-u2 fi-pnv-d510 fi-bdw-samus
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_22391:
### IGT changes ###
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@i915_module_load@reload:
- {bat-dg2-8}: [DMESG-FAIL][1] ([i915#5172]) -> [INCOMPLETE][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/bat-dg2-8/igt@i915_module_load@reload.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/bat-dg2-8/igt@i915_module_load@reload.html
Known issues
------------
Here are the changes found in Patchwork_22391 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_prime@amd-to-i915:
- fi-ivb-3770: NOTRUN -> [SKIP][3] ([fdo#109271]) +17 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/fi-ivb-3770/igt@amdgpu/amd_prime@amd-to-i915.html
* igt@gem_exec_suspend@basic-s3@smem:
- fi-skl-6600u: [PASS][4] -> [INCOMPLETE][5] ([i915#4547])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
* igt@i915_selftest@live@hangcheck:
- fi-snb-2600: [PASS][6] -> [INCOMPLETE][7] ([i915#3921])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
#### Possible fixes ####
* igt@i915_selftest@live@hangcheck:
- fi-ivb-3770: [INCOMPLETE][8] ([i915#3303]) -> [PASS][9]
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/fi-ivb-3770/igt@i915_selftest@live@hangcheck.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/fi-ivb-3770/igt@i915_selftest@live@hangcheck.html
- bat-dg1-5: [DMESG-FAIL][10] ([i915#4494] / [i915#4957]) -> [PASS][11]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/bat-dg1-5/igt@i915_selftest@live@hangcheck.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/bat-dg1-5/igt@i915_selftest@live@hangcheck.html
* igt@kms_busy@basic@modeset:
- {bat-adlp-6}: [DMESG-WARN][12] ([i915#3576]) -> [PASS][13]
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/bat-adlp-6/igt@kms_busy@basic@modeset.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/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#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303
[i915#3576]: https://gitlab.freedesktop.org/drm/intel/issues/3576
[i915#3921]: https://gitlab.freedesktop.org/drm/intel/issues/3921
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4494]: https://gitlab.freedesktop.org/drm/intel/issues/4494
[i915#4547]: https://gitlab.freedesktop.org/drm/intel/issues/4547
[i915#4957]: https://gitlab.freedesktop.org/drm/intel/issues/4957
[i915#5172]: https://gitlab.freedesktop.org/drm/intel/issues/5172
Build changes
-------------
* Linux: CI_DRM_11277 -> Patchwork_22391
CI-20190529: 20190529
CI_DRM_11277: a9d1ffee8dbe2c5506cccf9077eab8fe439eea46 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6355: 83ec34916bd8268bc331105cf77c4d3d3cd352be @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_22391: 25f3faec26be3820943854a42f7cbba3cb334c9d @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
25f3faec26be hda/i915: split the wait for the component binding
9fff1bdb8db0 kernel/hung_task: Exporting sysctl_hung_task_timeout_secs
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/index.html
[-- Attachment #2: Type: text/html, Size: 5378 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] kernel/hung_task: Exporting sysctl_hung_task_timeout_secs
2022-02-23 19:52 ` [Intel-gfx] [PATCH 1/2] kernel/hung_task: Exporting sysctl_hung_task_timeout_secs Ramalingam C
@ 2022-02-24 16:33 ` Lucas De Marchi
0 siblings, 0 replies; 9+ messages in thread
From: Lucas De Marchi @ 2022-02-24 16:33 UTC (permalink / raw)
To: Ramalingam C; +Cc: intel-gfx, dri-devel
On Thu, Feb 24, 2022 at 01:22:02AM +0530, Ramalingam C wrote:
>Exporting sysctl_hung_task_timeout_secs, to make it available for other
>kernel modules.
I guess this should only be done if second patch is accepted by sound
subsystem maintainers. If it is, then I'd do some changes in the commit
message.
Please use imperative style in commit messages. Here we also need to
give more details on why we are exporting this and Cc the maintainers.
Proposed new message:
Subject: kernel/hung_task: Export sysctl_hung_task_timeout_secs
Kernel modules may want to read sysctl_hung_task_timeout_secs so they
can do long waits by multiples of that value, avoiding the hung task
detector to trigger. This is already done in other places in the kernel
that are builtin-only, like:
block/bio.c:submit_bio_wait()
block/blk-mq.c:blk_execute_rq()
mm/kfence/core.c:toggle_allocation_gate()
Export it so it can also be used by modules.
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Luis Chamberlain <mcgrof@kernel.org>
Cc: Kai Vehmanen <kai.vehmanen@intel.com>
Cc: Jaroslav Kysela <perex@perex.cz>
Cc: Takashi Iwai <tiwai@suse.com>
>
>Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
>cc: Lucas De Marchi <lucas.demarchi@intel.com>
Acked-by: Lucas De Marchi <lucas.demarchi@intel.com>
Lucas De Marchi
>---
> kernel/hung_task.c | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/kernel/hung_task.c b/kernel/hung_task.c
>index db59b6d4f0e7..01120265395d 100644
>--- a/kernel/hung_task.c
>+++ b/kernel/hung_task.c
>@@ -43,6 +43,7 @@ int __read_mostly sysctl_hung_task_check_count = PID_MAX_LIMIT;
> * Zero means infinite timeout - no checking done:
> */
> unsigned long __read_mostly sysctl_hung_task_timeout_secs = CONFIG_DEFAULT_HUNG_TASK_TIMEOUT;
>+EXPORT_SYMBOL(sysctl_hung_task_timeout_secs);
>
> /*
> * Zero (default value) means use sysctl_hung_task_timeout_secs:
>--
>2.20.1
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Intel-gfx] [PATCH 2/2] hda/i915: split the wait for the component binding
2022-02-23 19:52 ` [Intel-gfx] [PATCH 2/2] hda/i915: split the wait for the component binding Ramalingam C
@ 2022-02-24 16:34 ` Kai Vehmanen
2022-02-25 11:42 ` Takashi Iwai
2022-02-24 17:13 ` Lucas De Marchi
1 sibling, 1 reply; 9+ messages in thread
From: Kai Vehmanen @ 2022-02-24 16:34 UTC (permalink / raw)
To: Ramalingam C
Cc: Kai Vehmanen, Takashi Iwai, intel-gfx, lucas.demarchi, dri-devel
Hi,
On Thu, 24 Feb 2022, Ramalingam C wrote:
> Split the wait for component binding from i915 in multiples of
> sysctl_hung_task_timeout_secs. This helps to avoid the possible kworker
> thread hung detection given below.
while I understand the problem, I'm not sure whether a simpler option
should be chosen. Maybe just split the wait_for_completion_timeout()
into small 5sec iterations, without consulting value of hung_task_timeout.
This would seem unligned with more mainstream use of
wait_for_completion_timeout() in kernel and still do the job.
I'll loop in Takashi here as well. Basicly the 60 sec timeout in
hda/hdac_i915.c is getting caught by hung_task_detection logic in builds
where the hung_task_timeout is below 60secs.
I have a patch that tries to avoid hitting the timeout in some of the more
common cases:
"ALSA: hda/i915 - skip acomp init if no matching display"
https://lists.freedesktop.org/archives/intel-gfx-trybot/2022-February/128278.html
... but we'll still be stuck with some configurations where the timeout
will be hit. And above needs careful testing.
One logic comment below as well, but I'll quote the whole patch to give
context to Takashi.
> <3>[ 60.946316] INFO: task kworker/11:1:104 blocked for more than 30
> seconds.
> <3>[ 60.946479] Tainted: G W
> 5.17.0-rc5-CI-CI_DRM_11265+ #1
> <3>[ 60.946580] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs"
> disables this message.
> <6>[ 60.946688] task:kworker/11:1 state:D stack:14192 pid: 104
> ppid: 2 flags:0x00004000
> <6>[ 60.946713] Workqueue: events azx_probe_work [snd_hda_intel]
> <6>[ 60.946740] Call Trace:
> <6>[ 60.946745] <TASK>
> <6>[ 60.946763] __schedule+0x42c/0xa80
> <6>[ 60.946797] schedule+0x3f/0xc0
> <6>[ 60.946811] schedule_timeout+0x1be/0x2e0
> <6>[ 60.946829] ? del_timer_sync+0xb0/0xb0
> <6>[ 60.946849] ? 0xffffffff81000000
> <6>[ 60.946864] ? wait_for_completion_timeout+0x79/0x120
> <6>[ 60.946879] wait_for_completion_timeout+0xab/0x120
> <6>[ 60.946906] snd_hdac_i915_init+0xa5/0xb0 [snd_hda_core]
> <6>[ 60.946943] azx_probe_work+0x71/0x84c [snd_hda_intel]
> <6>[ 60.946974] process_one_work+0x275/0x5c0
> <6>[ 60.947010] worker_thread+0x37/0x370
> <6>[ 60.947028] ? process_one_work+0x5c0/0x5c0
> <6>[ 60.947038] kthread+0xef/0x120
> <6>[ 60.947047] ? kthread_complete_and_exit+0x20/0x20
> <6>[ 60.947065] ret_from_fork+0x22/0x30
> <6>[ 60.947110] </TASK>
>
> Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
> cc: Kai Vehmanen <kai.vehmanen@intel.com>
> cc: Lucas De Marchi <lucas.demarchi@intel.com>
> ---
> sound/hda/hdac_i915.c | 17 ++++++++++++++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/sound/hda/hdac_i915.c b/sound/hda/hdac_i915.c
> index d20a450a9a15..daaeebc5099e 100644
> --- a/sound/hda/hdac_i915.c
> +++ b/sound/hda/hdac_i915.c
> @@ -6,6 +6,7 @@
> #include <linux/init.h>
> #include <linux/module.h>
> #include <linux/pci.h>
> +#include <linux/sched/sysctl.h>
> #include <sound/core.h>
> #include <sound/hdaudio.h>
> #include <sound/hda_i915.h>
> @@ -163,7 +164,8 @@ static bool dg1_gfx_present(void)
> int snd_hdac_i915_init(struct hdac_bus *bus)
> {
> struct drm_audio_component *acomp;
> - int err;
> + unsigned long timeout, ret = 0;
> + int err, i, itr_cnt;
>
> if (!i915_gfx_present())
> return -ENODEV;
> @@ -182,9 +184,18 @@ int snd_hdac_i915_init(struct hdac_bus *bus)
> if (!acomp->ops) {
> if (!IS_ENABLED(CONFIG_MODULES) ||
> !request_module("i915")) {
5~> + if (!sysctl_hung_task_timeout_secs) {
> + itr_cnt = 1;
> + timeout = msecs_to_jiffies(60 * 1000);
> + } else {
> + itr_cnt = DIV_ROUND_UP(60, sysctl_hung_task_timeout_secs);
> + timeout = msecs_to_jiffies(sysctl_hung_task_timeout_secs * 1000);
> + }
> +
> /* 60s timeout */
> - wait_for_completion_timeout(&acomp->master_bind_complete,
> - msecs_to_jiffies(30 * 1000));
> + for (i = 0; i < itr_cnt || !ret; i++)
> + ret = wait_for_completion_timeout(&acomp->master_bind_complete,
> + timeout);
I think that should be 'i < itr_cnt && !ret'. If wait_for_completion
returns with a positive value, we should stop waiting as the completion
has been signalled.
> }
> }
> if (!acomp->ops) {
> --
Br, Kai
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Intel-gfx] [PATCH 2/2] hda/i915: split the wait for the component binding
2022-02-23 19:52 ` [Intel-gfx] [PATCH 2/2] hda/i915: split the wait for the component binding Ramalingam C
2022-02-24 16:34 ` Kai Vehmanen
@ 2022-02-24 17:13 ` Lucas De Marchi
1 sibling, 0 replies; 9+ messages in thread
From: Lucas De Marchi @ 2022-02-24 17:13 UTC (permalink / raw)
To: Ramalingam C; +Cc: Kai Vehmanen, intel-gfx, dri-devel
On Thu, Feb 24, 2022 at 01:22:03AM +0530, Ramalingam C wrote:
>Split the wait for component binding from i915 in multiples of
>sysctl_hung_task_timeout_secs. This helps to avoid the possible kworker
>thread hung detection given below.
>
><3>[ 60.946316] INFO: task kworker/11:1:104 blocked for more than 30
>seconds.
><3>[ 60.946479] Tainted: G W
>5.17.0-rc5-CI-CI_DRM_11265+ #1
><3>[ 60.946580] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs"
>disables this message.
><6>[ 60.946688] task:kworker/11:1 state:D stack:14192 pid: 104
>ppid: 2 flags:0x00004000
><6>[ 60.946713] Workqueue: events azx_probe_work [snd_hda_intel]
><6>[ 60.946740] Call Trace:
><6>[ 60.946745] <TASK>
><6>[ 60.946763] __schedule+0x42c/0xa80
><6>[ 60.946797] schedule+0x3f/0xc0
><6>[ 60.946811] schedule_timeout+0x1be/0x2e0
><6>[ 60.946829] ? del_timer_sync+0xb0/0xb0
><6>[ 60.946849] ? 0xffffffff81000000
><6>[ 60.946864] ? wait_for_completion_timeout+0x79/0x120
><6>[ 60.946879] wait_for_completion_timeout+0xab/0x120
><6>[ 60.946906] snd_hdac_i915_init+0xa5/0xb0 [snd_hda_core]
><6>[ 60.946943] azx_probe_work+0x71/0x84c [snd_hda_intel]
><6>[ 60.946974] process_one_work+0x275/0x5c0
><6>[ 60.947010] worker_thread+0x37/0x370
><6>[ 60.947028] ? process_one_work+0x5c0/0x5c0
><6>[ 60.947038] kthread+0xef/0x120
><6>[ 60.947047] ? kthread_complete_and_exit+0x20/0x20
><6>[ 60.947065] ret_from_fork+0x22/0x30
><6>[ 60.947110] </TASK>
>
>Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
>cc: Kai Vehmanen <kai.vehmanen@intel.com>
>cc: Lucas De Marchi <lucas.demarchi@intel.com>
some more Cc needed?
$ ./scripts/get_maintainer.pl sound/hda/hdac_i915.c
Jaroslav Kysela <perex@perex.cz> (maintainer:SOUND)
Takashi Iwai <tiwai@suse.com> (maintainer:SOUND)
Lucas De Marchi <lucas.demarchi@intel.com> (commit_signer:2/2=100%)
Rodrigo Vivi <rodrigo.vivi@intel.com> (commit_signer:1/2=50%)
Ramalingam C <ramalingam.c@intel.com> (commit_signer:1/2=50%,authored:1/2=50%,removed_lines:1/1=100%)
Chris Wilson <chris@chris-wilson.co.uk> (authored:1/2=50%,added_lines:23/24=96%)
alsa-devel@alsa-project.org (moderated list:SOUND)
linux-kernel@vger.kernel.org (open list)
>---
> sound/hda/hdac_i915.c | 17 ++++++++++++++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
>diff --git a/sound/hda/hdac_i915.c b/sound/hda/hdac_i915.c
>index d20a450a9a15..daaeebc5099e 100644
>--- a/sound/hda/hdac_i915.c
>+++ b/sound/hda/hdac_i915.c
>@@ -6,6 +6,7 @@
> #include <linux/init.h>
> #include <linux/module.h>
> #include <linux/pci.h>
>+#include <linux/sched/sysctl.h>
> #include <sound/core.h>
> #include <sound/hdaudio.h>
> #include <sound/hda_i915.h>
>@@ -163,7 +164,8 @@ static bool dg1_gfx_present(void)
> int snd_hdac_i915_init(struct hdac_bus *bus)
> {
> struct drm_audio_component *acomp;
>- int err;
>+ unsigned long timeout, ret = 0;
>+ int err, i, itr_cnt;
>
> if (!i915_gfx_present())
> return -ENODEV;
>@@ -182,9 +184,18 @@ int snd_hdac_i915_init(struct hdac_bus *bus)
> if (!acomp->ops) {
> if (!IS_ENABLED(CONFIG_MODULES) ||
> !request_module("i915")) {
>+ if (!sysctl_hung_task_timeout_secs) {
>+ itr_cnt = 1;
>+ timeout = msecs_to_jiffies(60 * 1000);
>+ } else {
>+ itr_cnt = DIV_ROUND_UP(60, sysctl_hung_task_timeout_secs);
>+ timeout = msecs_to_jiffies(sysctl_hung_task_timeout_secs * 1000);
>+ }
>+
> /* 60s timeout */
>- wait_for_completion_timeout(&acomp->master_bind_complete,
>- msecs_to_jiffies(30 * 1000));
>+ for (i = 0; i < itr_cnt || !ret; i++)
it will return 0 if a timeout occurs so it's trapped forever here? Did
you mean &&?
Also maybe:
itr_cnt = DIV_ROUND_UP(60, sysctl_hung_task_timeout_secs ?: 60);
Lucas De Marchi
} while ();
>+ ret = wait_for_completion_timeout(&acomp->master_bind_complete,
>+ timeout);
> }
> }
> if (!acomp->ops) {
>--
>2.20.1
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for hda/i915: split wait for component binding
2022-02-23 19:52 [Intel-gfx] [PATCH 0/2] hda/i915: split wait for component binding Ramalingam C
` (2 preceding siblings ...)
2022-02-24 14:52 ` [Intel-gfx] ✓ Fi.CI.BAT: success for hda/i915: split wait for " Patchwork
@ 2022-02-25 4:21 ` Patchwork
3 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2022-02-25 4:21 UTC (permalink / raw)
To: Ramalingam C; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 30266 bytes --]
== Series Details ==
Series: hda/i915: split wait for component binding
URL : https://patchwork.freedesktop.org/series/100661/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11277_full -> Patchwork_22391_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Participating hosts (11 -> 11)
------------------------------
No changes in participating hosts
Known issues
------------
Here are the changes found in Patchwork_22391_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_create@create-massive:
- shard-apl: NOTRUN -> [DMESG-WARN][1] ([i915#4991])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl8/igt@gem_create@create-massive.html
* igt@gem_ctx_isolation@preservation-s3@rcs0:
- shard-kbl: [PASS][2] -> [DMESG-WARN][3] ([i915#180]) +2 similar issues
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-kbl1/igt@gem_ctx_isolation@preservation-s3@rcs0.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl1/igt@gem_ctx_isolation@preservation-s3@rcs0.html
* igt@gem_eio@kms:
- shard-tglb: [PASS][4] -> [FAIL][5] ([i915#232])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-tglb6/igt@gem_eio@kms.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-tglb1/igt@gem_eio@kms.html
* igt@gem_exec_balancer@parallel-contexts:
- shard-kbl: NOTRUN -> [DMESG-WARN][6] ([i915#5076])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl6/igt@gem_exec_balancer@parallel-contexts.html
* igt@gem_exec_fair@basic-deadline:
- shard-glk: [PASS][7] -> [FAIL][8] ([i915#2846])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-glk2/igt@gem_exec_fair@basic-deadline.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-glk8/igt@gem_exec_fair@basic-deadline.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-glk: [PASS][9] -> [FAIL][10] ([i915#2842])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-glk8/igt@gem_exec_fair@basic-none-share@rcs0.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-glk1/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-none@vcs0:
- shard-kbl: [PASS][11] -> [FAIL][12] ([i915#2842]) +3 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-kbl3/igt@gem_exec_fair@basic-none@vcs0.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl7/igt@gem_exec_fair@basic-none@vcs0.html
- shard-apl: [PASS][13] -> [FAIL][14] ([i915#2842])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-apl4/igt@gem_exec_fair@basic-none@vcs0.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl3/igt@gem_exec_fair@basic-none@vcs0.html
* igt@gem_lmem_swapping@heavy-random:
- shard-apl: NOTRUN -> [SKIP][15] ([fdo#109271] / [i915#4613])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl7/igt@gem_lmem_swapping@heavy-random.html
* igt@gem_lmem_swapping@heavy-verify-random:
- shard-kbl: NOTRUN -> [SKIP][16] ([fdo#109271] / [i915#4613]) +1 similar issue
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl3/igt@gem_lmem_swapping@heavy-verify-random.html
* igt@gem_lmem_swapping@parallel-random:
- shard-skl: NOTRUN -> [SKIP][17] ([fdo#109271] / [i915#4613]) +5 similar issues
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl5/igt@gem_lmem_swapping@parallel-random.html
* igt@gem_lmem_swapping@parallel-random-verify:
- shard-glk: NOTRUN -> [SKIP][18] ([fdo#109271] / [i915#4613])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-glk8/igt@gem_lmem_swapping@parallel-random-verify.html
* igt@gem_pread@exhaustion:
- shard-apl: NOTRUN -> [WARN][19] ([i915#2658])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl4/igt@gem_pread@exhaustion.html
* igt@gen9_exec_parse@allowed-all:
- shard-apl: [PASS][20] -> [DMESG-WARN][21] ([i915#1436] / [i915#716])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-apl8/igt@gen9_exec_parse@allowed-all.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl7/igt@gen9_exec_parse@allowed-all.html
* igt@gen9_exec_parse@basic-rejected:
- shard-iclb: NOTRUN -> [SKIP][22] ([i915#2856])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb7/igt@gen9_exec_parse@basic-rejected.html
* igt@kms_atomic_transition@plane-use-after-nonblocking-unbind@edp-1-pipe-a:
- shard-skl: [PASS][23] -> [DMESG-WARN][24] ([i915#1982])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-skl4/igt@kms_atomic_transition@plane-use-after-nonblocking-unbind@edp-1-pipe-a.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl9/igt@kms_atomic_transition@plane-use-after-nonblocking-unbind@edp-1-pipe-a.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-async-flip:
- shard-skl: NOTRUN -> [FAIL][25] ([i915#3743]) +2 similar issues
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl8/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-async-flip.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip:
- shard-kbl: NOTRUN -> [SKIP][26] ([fdo#109271] / [i915#3777]) +3 similar issues
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl7/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip.html
- shard-skl: NOTRUN -> [SKIP][27] ([fdo#109271] / [i915#3777]) +5 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl4/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-64bpp-rotate-0:
- shard-apl: NOTRUN -> [SKIP][28] ([fdo#109271]) +87 similar issues
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl7/igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-0.html
* igt@kms_ccs@pipe-a-bad-rotation-90-y_tiled_gen12_mc_ccs:
- shard-kbl: NOTRUN -> [SKIP][29] ([fdo#109271] / [i915#3886]) +8 similar issues
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl6/igt@kms_ccs@pipe-a-bad-rotation-90-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-a-crc-primary-basic-y_tiled_gen12_mc_ccs:
- shard-iclb: NOTRUN -> [SKIP][30] ([fdo#109278] / [i915#3886])
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb6/igt@kms_ccs@pipe-a-crc-primary-basic-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_mc_ccs:
- shard-apl: NOTRUN -> [SKIP][31] ([fdo#109271] / [i915#3886]) +1 similar issue
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl7/igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-b-ccs-on-another-bo-y_tiled_gen12_mc_ccs:
- shard-skl: NOTRUN -> [SKIP][32] ([fdo#109271] / [i915#3886]) +12 similar issues
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl10/igt@kms_ccs@pipe-b-ccs-on-another-bo-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-c-bad-aux-stride-y_tiled_gen12_rc_ccs:
- shard-glk: NOTRUN -> [SKIP][33] ([fdo#109271]) +9 similar issues
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-glk8/igt@kms_ccs@pipe-c-bad-aux-stride-y_tiled_gen12_rc_ccs.html
* igt@kms_chamelium@dp-crc-multiple:
- shard-skl: NOTRUN -> [SKIP][34] ([fdo#109271] / [fdo#111827]) +18 similar issues
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl9/igt@kms_chamelium@dp-crc-multiple.html
* igt@kms_color@pipe-d-gamma:
- shard-iclb: NOTRUN -> [SKIP][35] ([fdo#109278] / [i915#1149])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb6/igt@kms_color@pipe-d-gamma.html
* igt@kms_color_chamelium@pipe-a-ctm-blue-to-red:
- shard-kbl: NOTRUN -> [SKIP][36] ([fdo#109271] / [fdo#111827]) +10 similar issues
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl3/igt@kms_color_chamelium@pipe-a-ctm-blue-to-red.html
* igt@kms_color_chamelium@pipe-a-ctm-red-to-blue:
- shard-iclb: NOTRUN -> [SKIP][37] ([fdo#109284] / [fdo#111827]) +1 similar issue
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb6/igt@kms_color_chamelium@pipe-a-ctm-red-to-blue.html
* igt@kms_color_chamelium@pipe-c-ctm-0-25:
- shard-apl: NOTRUN -> [SKIP][38] ([fdo#109271] / [fdo#111827]) +5 similar issues
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl4/igt@kms_color_chamelium@pipe-c-ctm-0-25.html
* igt@kms_color_chamelium@pipe-d-ctm-max:
- shard-iclb: NOTRUN -> [SKIP][39] ([fdo#109278] / [fdo#109284] / [fdo#111827]) +1 similar issue
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb7/igt@kms_color_chamelium@pipe-d-ctm-max.html
* igt@kms_cursor_crc@pipe-b-cursor-32x32-random:
- shard-skl: NOTRUN -> [SKIP][40] ([fdo#109271] / [i915#1888]) +1 similar issue
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl10/igt@kms_cursor_crc@pipe-b-cursor-32x32-random.html
* igt@kms_cursor_crc@pipe-b-cursor-512x512-offscreen:
- shard-iclb: NOTRUN -> [SKIP][41] ([fdo#109278] / [fdo#109279])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb7/igt@kms_cursor_crc@pipe-b-cursor-512x512-offscreen.html
* igt@kms_cursor_crc@pipe-c-cursor-suspend:
- shard-apl: [PASS][42] -> [DMESG-WARN][43] ([i915#180]) +2 similar issues
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-apl2/igt@kms_cursor_crc@pipe-c-cursor-suspend.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl6/igt@kms_cursor_crc@pipe-c-cursor-suspend.html
* igt@kms_cursor_crc@pipe-d-cursor-256x256-onscreen:
- shard-kbl: NOTRUN -> [SKIP][44] ([fdo#109271]) +144 similar issues
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl4/igt@kms_cursor_crc@pipe-d-cursor-256x256-onscreen.html
* igt@kms_cursor_legacy@cursorb-vs-flipa-atomic-transitions:
- shard-tglb: NOTRUN -> [SKIP][45] ([fdo#109274] / [fdo#111825])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-tglb6/igt@kms_cursor_legacy@cursorb-vs-flipa-atomic-transitions.html
* igt@kms_cursor_legacy@cursorb-vs-flipb-atomic:
- shard-iclb: NOTRUN -> [SKIP][46] ([fdo#109274] / [fdo#109278])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb7/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size:
- shard-iclb: [PASS][47] -> [FAIL][48] ([i915#2346])
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb1/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb7/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
- shard-skl: NOTRUN -> [FAIL][49] ([i915#2346] / [i915#533])
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl6/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
* igt@kms_cursor_legacy@pipe-d-torture-bo:
- shard-skl: NOTRUN -> [SKIP][50] ([fdo#109271] / [i915#533]) +2 similar issues
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl5/igt@kms_cursor_legacy@pipe-d-torture-bo.html
* igt@kms_flip@2x-flip-vs-expired-vblank:
- shard-iclb: NOTRUN -> [SKIP][51] ([fdo#109274]) +1 similar issue
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb7/igt@kms_flip@2x-flip-vs-expired-vblank.html
* igt@kms_flip@flip-vs-expired-vblank@a-edp1:
- shard-skl: [PASS][52] -> [FAIL][53] ([i915#79])
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-skl4/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl9/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-indfb-draw-blt:
- shard-skl: NOTRUN -> [SKIP][54] ([fdo#109271]) +287 similar issues
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl8/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-cpu:
- shard-glk: [PASS][55] -> [FAIL][56] ([i915#2546])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-glk2/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-cpu.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-glk6/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-cpu.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-render:
- shard-tglb: NOTRUN -> [SKIP][57] ([fdo#109280] / [fdo#111825])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-tglb6/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-render.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-mmap-cpu:
- shard-iclb: NOTRUN -> [SKIP][58] ([fdo#109280]) +4 similar issues
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb6/igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-mmap-cpu.html
* igt@kms_hdr@bpc-switch-dpms:
- shard-skl: NOTRUN -> [FAIL][59] ([i915#1188])
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl4/igt@kms_hdr@bpc-switch-dpms.html
* igt@kms_pipe_b_c_ivb@enable-pipe-c-while-b-has-3-lanes:
- shard-iclb: NOTRUN -> [SKIP][60] ([fdo#109289])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb6/igt@kms_pipe_b_c_ivb@enable-pipe-c-while-b-has-3-lanes.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
- shard-glk: [PASS][61] -> [FAIL][62] ([i915#1888])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-glk2/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-glk6/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-7efc:
- shard-skl: NOTRUN -> [FAIL][63] ([fdo#108145] / [i915#265]) +6 similar issues
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl9/igt@kms_plane_alpha_blend@pipe-a-alpha-7efc.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb:
- shard-apl: NOTRUN -> [FAIL][64] ([i915#265])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl7/igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb.html
* igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
- shard-skl: [PASS][65] -> [FAIL][66] ([fdo#108145] / [i915#265])
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-skl7/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl10/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
* igt@kms_plane_alpha_blend@pipe-b-alpha-basic:
- shard-kbl: NOTRUN -> [FAIL][67] ([fdo#108145] / [i915#265]) +1 similar issue
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl7/igt@kms_plane_alpha_blend@pipe-b-alpha-basic.html
* igt@kms_plane_alpha_blend@pipe-c-alpha-7efc:
- shard-apl: NOTRUN -> [FAIL][68] ([fdo#108145] / [i915#265])
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl4/igt@kms_plane_alpha_blend@pipe-c-alpha-7efc.html
* igt@kms_plane_alpha_blend@pipe-c-alpha-transparent-fb:
- shard-skl: NOTRUN -> [FAIL][69] ([i915#265])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl10/igt@kms_plane_alpha_blend@pipe-c-alpha-transparent-fb.html
* igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area:
- shard-kbl: NOTRUN -> [SKIP][70] ([fdo#109271] / [i915#658]) +1 similar issue
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl6/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area.html
* igt@kms_psr2_sf@plane-move-sf-dmg-area:
- shard-skl: NOTRUN -> [SKIP][71] ([fdo#109271] / [i915#658]) +2 similar issues
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl10/igt@kms_psr2_sf@plane-move-sf-dmg-area.html
* igt@kms_psr@psr2_cursor_render:
- shard-iclb: [PASS][72] -> [SKIP][73] ([fdo#109441]) +2 similar issues
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb2/igt@kms_psr@psr2_cursor_render.html
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb8/igt@kms_psr@psr2_cursor_render.html
* igt@kms_setmode@basic:
- shard-apl: [PASS][74] -> [FAIL][75] ([i915#31])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-apl6/igt@kms_setmode@basic.html
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl1/igt@kms_setmode@basic.html
* igt@kms_vblank@pipe-d-ts-continuation-idle-hang:
- shard-iclb: NOTRUN -> [SKIP][76] ([fdo#109278]) +6 similar issues
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb7/igt@kms_vblank@pipe-d-ts-continuation-idle-hang.html
* igt@kms_writeback@writeback-check-output:
- shard-skl: NOTRUN -> [SKIP][77] ([fdo#109271] / [i915#2437])
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl6/igt@kms_writeback@writeback-check-output.html
* igt@perf@polling-small-buf:
- shard-skl: NOTRUN -> [FAIL][78] ([i915#1722])
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl3/igt@perf@polling-small-buf.html
* igt@prime_nv_test@nv_i915_sharing:
- shard-iclb: NOTRUN -> [SKIP][79] ([fdo#109291]) +1 similar issue
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb6/igt@prime_nv_test@nv_i915_sharing.html
* igt@sysfs_clients@busy:
- shard-skl: NOTRUN -> [SKIP][80] ([fdo#109271] / [i915#2994]) +8 similar issues
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl5/igt@sysfs_clients@busy.html
* igt@sysfs_clients@recycle:
- shard-apl: NOTRUN -> [SKIP][81] ([fdo#109271] / [i915#2994]) +1 similar issue
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl8/igt@sysfs_clients@recycle.html
* igt@sysfs_clients@sema-50:
- shard-kbl: NOTRUN -> [SKIP][82] ([fdo#109271] / [i915#2994]) +2 similar issues
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl3/igt@sysfs_clients@sema-50.html
* igt@sysfs_preempt_timeout@timeout@rcs0:
- shard-skl: NOTRUN -> [FAIL][83] ([i915#3259])
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl3/igt@sysfs_preempt_timeout@timeout@rcs0.html
#### Possible fixes ####
* igt@gem_ctx_isolation@preservation-s3@rcs0:
- shard-apl: [DMESG-WARN][84] ([i915#180]) -> [PASS][85] +4 similar issues
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-apl6/igt@gem_ctx_isolation@preservation-s3@rcs0.html
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl4/igt@gem_ctx_isolation@preservation-s3@rcs0.html
* igt@gem_exec_balancer@parallel-balancer:
- shard-iclb: [SKIP][86] ([i915#4525]) -> [PASS][87]
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb8/igt@gem_exec_balancer@parallel-balancer.html
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb2/igt@gem_exec_balancer@parallel-balancer.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-iclb: [FAIL][88] ([i915#2842]) -> [PASS][89]
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb6/igt@gem_exec_fair@basic-none-share@rcs0.html
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb8/igt@gem_exec_fair@basic-none-share@rcs0.html
- {shard-tglu}: [FAIL][90] ([i915#2842]) -> [PASS][91]
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-tglu-8/igt@gem_exec_fair@basic-none-share@rcs0.html
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-tglu-6/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-pace-share@rcs0:
- shard-glk: [FAIL][92] ([i915#2842]) -> [PASS][93]
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-glk1/igt@gem_exec_fair@basic-pace-share@rcs0.html
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-glk5/igt@gem_exec_fair@basic-pace-share@rcs0.html
* igt@gen9_exec_parse@allowed-single:
- shard-skl: [DMESG-WARN][94] ([i915#1436] / [i915#716]) -> [PASS][95]
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-skl2/igt@gen9_exec_parse@allowed-single.html
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl5/igt@gen9_exec_parse@allowed-single.html
* igt@i915_pm_dc@dc9-dpms:
- shard-iclb: [SKIP][96] ([i915#4281]) -> [PASS][97]
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb3/igt@i915_pm_dc@dc9-dpms.html
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb2/igt@i915_pm_dc@dc9-dpms.html
* igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a:
- {shard-tglu}: [FAIL][98] ([i915#3825]) -> [PASS][99]
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-tglu-6/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a.html
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-tglu-1/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a.html
* igt@kms_big_fb@y-tiled-32bpp-rotate-0:
- {shard-tglu}: [DMESG-WARN][100] ([i915#402]) -> [PASS][101]
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-tglu-4/igt@kms_big_fb@y-tiled-32bpp-rotate-0.html
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-tglu-5/igt@kms_big_fb@y-tiled-32bpp-rotate-0.html
* igt@kms_cursor_edge_walk@pipe-b-64x64-top-edge:
- shard-skl: [DMESG-WARN][102] ([i915#1982]) -> [PASS][103]
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-skl2/igt@kms_cursor_edge_walk@pipe-b-64x64-top-edge.html
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl5/igt@kms_cursor_edge_walk@pipe-b-64x64-top-edge.html
* igt@kms_cursor_legacy@flip-vs-cursor-busy-crc-atomic:
- shard-apl: [FAIL][104] ([i915#2346]) -> [PASS][105]
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-apl7/igt@kms_cursor_legacy@flip-vs-cursor-busy-crc-atomic.html
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-apl7/igt@kms_cursor_legacy@flip-vs-cursor-busy-crc-atomic.html
* igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2:
- shard-glk: [FAIL][106] ([i915#2122]) -> [PASS][107]
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-glk3/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2.html
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-glk9/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2.html
* igt@kms_flip@flip-vs-suspend-interruptible@d-edp1:
- shard-tglb: [DMESG-WARN][108] ([i915#2411] / [i915#2867]) -> [PASS][109]
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-tglb7/igt@kms_flip@flip-vs-suspend-interruptible@d-edp1.html
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-tglb1/igt@kms_flip@flip-vs-suspend-interruptible@d-edp1.html
* igt@kms_flip@flip-vs-suspend@c-dp1:
- shard-kbl: [DMESG-WARN][110] ([i915#180]) -> [PASS][111] +7 similar issues
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-kbl1/igt@kms_flip@flip-vs-suspend@c-dp1.html
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-kbl7/igt@kms_flip@flip-vs-suspend@c-dp1.html
* igt@kms_flip@plain-flip-ts-check-interruptible@c-edp1:
- shard-skl: [FAIL][112] ([i915#2122]) -> [PASS][113] +1 similar issue
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-skl8/igt@kms_flip@plain-flip-ts-check-interruptible@c-edp1.html
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl7/igt@kms_flip@plain-flip-ts-check-interruptible@c-edp1.html
* igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile-downscaling:
- shard-iclb: [SKIP][114] ([i915#3701]) -> [PASS][115]
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile-downscaling.html
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb8/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile-downscaling.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-shrfb-msflip-blt:
- shard-iclb: [FAIL][116] ([i915#1888] / [i915#2546]) -> [PASS][117]
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb7/igt@kms_frontbuffer_tracking@psr-1p-primscrn-shrfb-msflip-blt.html
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb6/igt@kms_frontbuffer_tracking@psr-1p-primscrn-shrfb-msflip-blt.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
- shard-iclb: [FAIL][118] ([i915#1888]) -> [PASS][119]
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb7/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb6/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html
* igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
- shard-skl: [FAIL][120] ([fdo#108145] / [i915#265]) -> [PASS][121]
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-skl2/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl5/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
* igt@kms_psr@psr2_cursor_plane_move:
- shard-iclb: [SKIP][122] ([fdo#109441]) -> [PASS][123] +1 similar issue
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb6/igt@kms_psr@psr2_cursor_plane_move.html
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb2/igt@kms_psr@psr2_cursor_plane_move.html
* igt@perf@polling:
- shard-skl: [FAIL][124] ([i915#1542]) -> [PASS][125]
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-skl8/igt@perf@polling.html
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl7/igt@perf@polling.html
* igt@perf@polling-parameterized:
- shard-glk: [FAIL][126] ([i915#1542]) -> [PASS][127]
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-glk4/igt@perf@polling-parameterized.html
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-glk4/igt@perf@polling-parameterized.html
#### Warnings ####
* igt@gem_exec_balancer@parallel-keep-submit-fence:
- shard-iclb: [SKIP][128] ([i915#4525]) -> [DMESG-WARN][129] ([i915#5076])
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb6/igt@gem_exec_balancer@parallel-keep-submit-fence.html
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb4/igt@gem_exec_balancer@parallel-keep-submit-fence.html
* igt@gem_exec_balancer@parallel-ordering:
- shard-iclb: [DMESG-FAIL][130] ([i915#5076]) -> [SKIP][131] ([i915#4525])
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb2/igt@gem_exec_balancer@parallel-ordering.html
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb7/igt@gem_exec_balancer@parallel-ordering.html
* igt@i915_pm_rc6_residency@rc6-fence:
- shard-iclb: [WARN][132] ([i915#2684]) -> [WARN][133] ([i915#1804] / [i915#2684]) +1 similar issue
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb2/igt@i915_pm_rc6_residency@rc6-fence.html
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb7/igt@i915_pm_rc6_residency@rc6-fence.html
* igt@kms_cursor_crc@pipe-c-cursor-512x170-offscreen:
- shard-skl: [SKIP][134] ([fdo#109271] / [i915#1888]) -> [SKIP][135] ([fdo#109271]) +2 similar issues
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-skl2/igt@kms_cursor_crc@pipe-c-cursor-512x170-offscreen.html
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-skl5/igt@kms_cursor_crc@pipe-c-cursor-512x170-offscreen.html
* igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area:
- shard-iclb: [SKIP][136] ([fdo#111068] / [i915#658]) -> [SKIP][137] ([i915#2920])
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb3/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area.html
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/shard-iclb2/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area.html
* igt@kms_psr2_su@page_flip-xrgb8888:
- shard-iclb: [FAIL][138] ([i915#4148]) -> [SKIP][139] ([fdo#109642] / [fdo#111068] / [i915#658])
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11277/shard-iclb2/igt@kms_psr2_su@page_flip-xrgb8888.html
[139]: https://intel-gfx-ci.01.o
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22391/index.html
[-- Attachment #2: Type: text/html, Size: 33429 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Intel-gfx] [PATCH 2/2] hda/i915: split the wait for the component binding
2022-02-24 16:34 ` Kai Vehmanen
@ 2022-02-25 11:42 ` Takashi Iwai
0 siblings, 0 replies; 9+ messages in thread
From: Takashi Iwai @ 2022-02-25 11:42 UTC (permalink / raw)
To: Kai Vehmanen
Cc: Kai Vehmanen, intel-gfx, lucas.demarchi, dri-devel, Takashi Iwai
On Thu, 24 Feb 2022 17:34:54 +0100,
Kai Vehmanen wrote:
>
> Hi,
>
> On Thu, 24 Feb 2022, Ramalingam C wrote:
>
> > Split the wait for component binding from i915 in multiples of
> > sysctl_hung_task_timeout_secs. This helps to avoid the possible kworker
> > thread hung detection given below.
>
> while I understand the problem, I'm not sure whether a simpler option
> should be chosen. Maybe just split the wait_for_completion_timeout()
> into small 5sec iterations, without consulting value of hung_task_timeout.
> This would seem unligned with more mainstream use of
> wait_for_completion_timeout() in kernel and still do the job.
>
> I'll loop in Takashi here as well. Basicly the 60 sec timeout in
> hda/hdac_i915.c is getting caught by hung_task_detection logic in builds
> where the hung_task_timeout is below 60secs.
>
> I have a patch that tries to avoid hitting the timeout in some of the more
> common cases:
> "ALSA: hda/i915 - skip acomp init if no matching display"
> https://lists.freedesktop.org/archives/intel-gfx-trybot/2022-February/128278.html
> ... but we'll still be stuck with some configurations where the timeout
> will be hit. And above needs careful testing.
>
> One logic comment below as well, but I'll quote the whole patch to give
> context to Takashi.
I agree with Kai, we can just make the wait_for_completion_timeout()
split in a loop independently from hung_task_timeout. It's simpler,
less error-prone.
thanks,
Takashi
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-02-25 11:42 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-23 19:52 [Intel-gfx] [PATCH 0/2] hda/i915: split wait for component binding Ramalingam C
2022-02-23 19:52 ` [Intel-gfx] [PATCH 1/2] kernel/hung_task: Exporting sysctl_hung_task_timeout_secs Ramalingam C
2022-02-24 16:33 ` Lucas De Marchi
2022-02-23 19:52 ` [Intel-gfx] [PATCH 2/2] hda/i915: split the wait for the component binding Ramalingam C
2022-02-24 16:34 ` Kai Vehmanen
2022-02-25 11:42 ` Takashi Iwai
2022-02-24 17:13 ` Lucas De Marchi
2022-02-24 14:52 ` [Intel-gfx] ✓ Fi.CI.BAT: success for hda/i915: split wait for " Patchwork
2022-02-25 4:21 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).