linux-kernel - Re: 6.9.0-rc2+ kernel hangs on boot (bisected, maybe LED related)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Wed, 24 Apr 2024 19:31:26 +0200
From: Johannes Berg <johannes@...solutions.net>
To: Ben Greear <greearb@...delatech.com>, Linux regressions mailing list
	 <regressions@...ts.linux.dev>, Lee Jones <lee@...nel.org>
Cc: LKML <linux-kernel@...r.kernel.org>, linux-leds@...r.kernel.org, Miri
	Korenblit <miriam.rachel.korenblit@...el.com>
Subject: Re: 6.9.0-rc2+ kernel hangs on boot (bisected, maybe LED related)

Hi Ben,

On Wed, 2024-04-24 at 10:26 -0700, Ben Greear wrote:
> 
> This patch makes the problem go away in my testbed with 24 Intel
> iwlwifi radios.  My guess is that it is just papering over the problem,

Agree, there seems to be some locking issue with LED stuff, but I'm not
sure where, and the driver doesn't even hold any locks here.

> but maybe good enough?

For all I care, yes. We explicitly do this last, from a worker that
holds no locks in the driver ... so it's odd. Looking at the history of
it, it seems that it was originally _nowait(), but then in 3.6 I changed
it because of some backport concerns. Though then I also moved it out of
the locked section.

> Would you like me to submit as official
> patch to linux-wireless?

Sure.

johannes