All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Menzel <pmenzel@molgen.mpg.de>
To: Daniel Stone <daniel@fooishbar.org>
Cc: "Dave Airlie" <airlied@linux.ie>,
	"Richard Gong" <richard.gong@amd.com>,
	"Xinhui Pan" <xinhui.pan@amd.com>,
	LKML <linux-kernel@vger.kernel.org>,
	amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
	"Alexander Deucher" <alexander.deucher@amd.com>,
	"Christian König" <christian.koenig@amd.com>,
	"Mario Limonciello" <mario.limonciello@amd.com>
Subject: Re: [PATCHv4] drm/amdgpu: disable ASPM on Intel Alder Lake based systems
Date: Tue, 3 May 2022 14:44:56 +0200	[thread overview]
Message-ID: <5a530210-2c22-d8e6-02e0-f321ba5e0e60@molgen.mpg.de> (raw)
In-Reply-To: <CAPj87rOERk-kNa6n-UdjQsDKXP9zzm8=an=FHcM+33yebW6ECw@mail.gmail.com>

Dear Daniel,


Am 03.05.22 um 14:25 schrieb Daniel Stone:
> On Sun, 1 May 2022 at 08:08, Paul Menzel <pmenzel@molgen.mpg.de> wrote:
>> Am 26.04.22 um 15:53 schrieb Gong, Richard:
>>> I think so. We captured dmesg log.
>>
>> Then the (whole) system did *not* freeze, if you could still log in
>> (maybe over network) and execute `dmesg`. Please also paste the
>> amdgpu(?) error logs in the commit message.
>>
>>> As mentioned early we need support from Intel on how to get ASPM working
>>> for VI generation on Intel Alder Lake, but we don't know where things
>>> currently stand.
>>
>> Who is working on this, and knows?
> 
> This has gone beyond the point of a reasonable request. The amount of
> detail you're demanding is completely unnecessary.

If a quirk is introduced possibly leading to higher power consumption, 
especially on systems nobody has access to yet, then the detail, where 
the system hangs/freezes is not unreasonable at all.

In the Linux logs from the issue there are messages like

     [   58.101385] Freezing of tasks failed after 20.003 seconds (4 
tasks refusing to freeze, wq_busy=0):

     [   78.278403] Freezing of tasks failed after 20.008 seconds (4 
tasks refusing to freeze, wq_busy=0):

and it looks like several suspend/resume cycles were done.

I see a lot of commit messages over the whole Linux kernel, where this 
level of detail is provided (by default), and

The second question was not for the commit message, but just for 
documentation purpose when the problem is going to be fixed properly. 
And it looks like (at least publicly) analyzing the root cause is not 
happening, and once the quirk lands, nobody is going to feel the 
pressure to work on it, as everyone’s plates are full.


Kind regards,

Paul

WARNING: multiple messages have this Message-ID (diff)
From: Paul Menzel <pmenzel@molgen.mpg.de>
To: Daniel Stone <daniel@fooishbar.org>
Cc: "Dave Airlie" <airlied@linux.ie>,
	"Richard Gong" <richard.gong@amd.com>,
	"Xinhui Pan" <xinhui.pan@amd.com>,
	LKML <linux-kernel@vger.kernel.org>,
	dri-devel@lists.freedesktop.org, amd-gfx@lists.freedesktop.org,
	"Alexander Deucher" <alexander.deucher@amd.com>,
	"Christian König" <christian.koenig@amd.com>,
	"Mario Limonciello" <mario.limonciello@amd.com>
Subject: Re: [PATCHv4] drm/amdgpu: disable ASPM on Intel Alder Lake based systems
Date: Tue, 3 May 2022 14:44:56 +0200	[thread overview]
Message-ID: <5a530210-2c22-d8e6-02e0-f321ba5e0e60@molgen.mpg.de> (raw)
In-Reply-To: <CAPj87rOERk-kNa6n-UdjQsDKXP9zzm8=an=FHcM+33yebW6ECw@mail.gmail.com>

Dear Daniel,


Am 03.05.22 um 14:25 schrieb Daniel Stone:
> On Sun, 1 May 2022 at 08:08, Paul Menzel <pmenzel@molgen.mpg.de> wrote:
>> Am 26.04.22 um 15:53 schrieb Gong, Richard:
>>> I think so. We captured dmesg log.
>>
>> Then the (whole) system did *not* freeze, if you could still log in
>> (maybe over network) and execute `dmesg`. Please also paste the
>> amdgpu(?) error logs in the commit message.
>>
>>> As mentioned early we need support from Intel on how to get ASPM working
>>> for VI generation on Intel Alder Lake, but we don't know where things
>>> currently stand.
>>
>> Who is working on this, and knows?
> 
> This has gone beyond the point of a reasonable request. The amount of
> detail you're demanding is completely unnecessary.

If a quirk is introduced possibly leading to higher power consumption, 
especially on systems nobody has access to yet, then the detail, where 
the system hangs/freezes is not unreasonable at all.

In the Linux logs from the issue there are messages like

     [   58.101385] Freezing of tasks failed after 20.003 seconds (4 
tasks refusing to freeze, wq_busy=0):

     [   78.278403] Freezing of tasks failed after 20.008 seconds (4 
tasks refusing to freeze, wq_busy=0):

and it looks like several suspend/resume cycles were done.

I see a lot of commit messages over the whole Linux kernel, where this 
level of detail is provided (by default), and

The second question was not for the commit message, but just for 
documentation purpose when the problem is going to be fixed properly. 
And it looks like (at least publicly) analyzing the root cause is not 
happening, and once the quirk lands, nobody is going to feel the 
pressure to work on it, as everyone’s plates are full.


Kind regards,

Paul

  reply	other threads:[~2022-05-03 12:45 UTC|newest]

Thread overview: 73+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-12 21:50 [PATCHv4] drm/amdgpu: disable ASPM on Intel Alder Lake based systems Richard Gong
2022-04-12 21:50 ` Richard Gong
2022-04-13  4:29 ` Lazar, Lijo
2022-04-13  4:29   ` Lazar, Lijo
2022-04-13  7:43 ` Paul Menzel
2022-04-13  7:43   ` Paul Menzel
2022-04-13  7:43   ` Paul Menzel
2022-04-13 13:00   ` Alex Deucher
2022-04-13 13:00     ` Alex Deucher
2022-04-13 13:00     ` Alex Deucher
2022-04-13 13:28     ` Limonciello, Mario
2022-04-13 13:28       ` Limonciello, Mario
2022-04-13 13:28       ` Limonciello, Mario
2022-04-14  7:52     ` Paul Menzel
2022-04-14  7:52       ` Paul Menzel
2022-04-14  7:52       ` Paul Menzel
2022-04-14 13:11       ` Alex Deucher
2022-04-14 13:11         ` Alex Deucher
2022-04-14 13:11         ` Alex Deucher
2022-04-19 21:46       ` Gong, Richard
2022-04-19 21:46         ` Gong, Richard
2022-04-20 20:29         ` Paul Menzel
2022-04-20 20:29           ` Paul Menzel
2022-04-20 20:29           ` Paul Menzel
2022-04-20 20:40           ` Alex Deucher
2022-04-20 20:40             ` Alex Deucher
2022-04-20 20:40             ` Alex Deucher
2022-04-20 20:48             ` Paul Menzel
2022-04-20 20:48               ` Paul Menzel
2022-04-20 20:48               ` Paul Menzel
2022-04-20 20:56               ` Gong, Richard
2022-04-20 20:56                 ` Gong, Richard
2022-04-20 20:56                 ` Gong, Richard
2022-04-20 21:02                 ` Paul Menzel
2022-04-20 21:02                   ` Paul Menzel
2022-04-20 21:02                   ` Paul Menzel
2022-04-20 21:12                   ` Gong, Richard
2022-04-20 21:12                     ` Gong, Richard
2022-04-20 21:12                     ` Gong, Richard
2022-04-20 21:15                     ` Alex Deucher
2022-04-20 21:15                       ` Alex Deucher
2022-04-20 21:15                       ` Alex Deucher
2022-04-20 21:13                   ` Alex Deucher
2022-04-20 21:13                     ` Alex Deucher
2022-04-20 21:13                     ` Alex Deucher
2022-04-20 21:16                     ` Limonciello, Mario
2022-04-20 21:16                       ` Limonciello, Mario
2022-04-20 21:16                       ` Limonciello, Mario
2022-04-21  1:12           ` Gong, Richard
2022-04-21  1:12             ` Gong, Richard
2022-04-21  1:12             ` Gong, Richard
2022-04-21  5:35             ` Paul Menzel
2022-04-21  5:35               ` Paul Menzel
2022-04-21  5:35               ` Paul Menzel
2022-04-26 13:53               ` Gong, Richard
2022-04-26 13:53                 ` Gong, Richard
2022-04-26 13:53                 ` Gong, Richard
2022-05-01  7:08                 ` Paul Menzel
2022-05-01  7:08                   ` Paul Menzel
2022-05-01  7:08                   ` Paul Menzel
2022-05-02 14:56                   ` Gong, Richard
2022-05-02 14:56                     ` Gong, Richard
2022-05-02 14:56                     ` Gong, Richard
2022-05-03 12:25                   ` Daniel Stone
2022-05-03 12:25                     ` Daniel Stone
2022-05-03 12:44                     ` Paul Menzel [this message]
2022-05-03 12:44                       ` Paul Menzel
2022-04-13 15:40 ` Nathan Chancellor
2022-04-13 15:40   ` Nathan Chancellor
2022-04-13 15:40   ` Nathan Chancellor
2022-04-19 21:08   ` Gong, Richard
2022-04-19 21:08     ` Gong, Richard
2022-04-19 21:08     ` Gong, Richard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5a530210-2c22-d8e6-02e0-f321ba5e0e60@molgen.mpg.de \
    --to=pmenzel@molgen.mpg.de \
    --cc=airlied@linux.ie \
    --cc=alexander.deucher@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=christian.koenig@amd.com \
    --cc=daniel@fooishbar.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mario.limonciello@amd.com \
    --cc=richard.gong@amd.com \
    --cc=xinhui.pan@amd.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.