Received-SPF: pass (google.com: domain of linux-kernel+bounces-129649-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <21a230d8-92cf-4ae9-ae4c-34b3a2fe2670@leemhuis.info>
Date: Wed, 3 Apr 2024 13:14:07 +0200
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Reply-To: Linux regressions mailing list <regressions@lists.linux.dev>
Subject: Re: [PATCH 6.8 000/715] 6.8.2-rc1 review
To: Jon Hunter <jonathanh@nvidia.com>, Sasha Levin <sashal@kernel.org>,
 linux-kernel@vger.kernel.org, stable@vger.kernel.org
Cc: torvalds@linux-foundation.org, akpm@linux-foundation.org,
 linux@roeck-us.net, shuah@kernel.org, patches@kernelci.org,
 lkft-triage@lists.linaro.org, florian.fainelli@broadcom.com, pavel@denx.de,
 "linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>
References: <20240324223455.1342824-1-sashal@kernel.org>
 <0cdbf4d6-3dce-42ee-8029-c486ec999fb5@nvidia.com>
From: "Linux regression tracking (Thorsten Leemhuis)"
 <regressions@leemhuis.info>
Content-Language: en-US, de-DE
In-Reply-To: <0cdbf4d6-3dce-42ee-8029-c486ec999fb5@nvidia.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

On 03.04.24 12:02, Jon Hunter wrote:
> On 24/03/2024 22:22, Sasha Levin wrote:
>> Tejun Heo (9):
>>    workqueue: Move pwq->max_active to wq->max_active
>>    workqueue: Factor out pwq_is_empty()
>>    workqueue: Replace pwq_activate_inactive_work() with
>>      [__]pwq_activate_work()
>>    workqueue: Move nr_active handling into helpers
>>    workqueue: Make wq_adjust_max_active() round-robin pwqs while
>>      activating
>>    workqueue: RCU protect wq->dfl_pwq and implement accessors for it
>>    workqueue: Introduce struct wq_node_nr_active
>>    workqueue: Implement system-wide nr_active enforcement for unbound
>>      workqueues
>>    workqueue: Don't call cpumask_test_cpu() with -1 CPU in
>>      wq_update_node_max_active()
> 
> 
> Sorry I am late on this, but I was not copied and so did not see this. I
> noticed a suspend regression on Tegra186 with v6.8.y and bisect is point
> to commit "workqueue: Implement system-wide nr_active enforcement for
> unbound workqueues".
> 
> Reverting commits "workqueue: Don't call cpumask_test_cpu() with -1 CPU
> in wq_update_node_max_active()" and "workqueue: Implement system-wide
> nr_active enforcement for unbound workqueues" does fix the problem.
> 
> Note that I am not seeing this regression on the mainline with v6.9-rc2
> and so I am not sure if there is something else missing? I am also still
> seeing the problem with v6.8.3-rc1.

FWIW, there is a ongoing discussion about the workqueue backports (and
if those should be removed) due to a earlier regression report (about
hibernate, not suspend) here:

https://lore.kernel.org/all/ce4c2f67-c298-48a0-87a3-f933d646c73b@leemhuis.info/

Ciao, Thorsten