From: John Ogness <john.ogness@linutronix.de>
To: Pavel Machek <pavel@ucw.cz>, Johannes Berg <johannes@sipsolutions.net>
Cc: kernel list <linux-kernel@vger.kernel.org>,
tglx@linutronix.de, mingo@redhat.com, bp@alien8.de,
dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com,
peterz@infradead.org, will@kernel.org, longman@redhat.com,
miriam.rachel.korenblit@intel.com,
linux-wireless@vger.kernel.org, pmladek@suse.com,
rostedt@goodmis.org, senozhatsky@chromium.org
Subject: Re: locking problems in iwlwifi? was Re: 6.16-rcX: crashing way too often on thinkpad X220
Date: Thu, 24 Jul 2025 10:56:22 +0206 [thread overview]
Message-ID: <84h5z2vupd.fsf@jogness.linutronix.de> (raw)
In-Reply-To: <aIFH9I6b/tebYcEL@duo.ucw.cz>
On 2025-07-23, Pavel Machek <pavel@ucw.cz> wrote:
>> On Wed, 2025-07-23 at 17:42 +0200, Pavel Machek wrote:
>> > [ 402.125635] ------------[ cut here ]------------
>> > [ 402.125638] raw_local_irq_restore() called with IRQs enabled
>> > [ 402.125645] WARNING: CPU: 3 PID: 387 at kernel/locking/irqflag-debug.c:10 warn_bogus_irq_restore+0x25/0x30
>> > [ 402.125654] Modules linked in:
>> > [ 402.125661] CPU: 3 UID: 0 PID: 387 Comm: kworker/u16:5 Tainted: G S 6.16.0-rc7+ #303 PREEMPT(voluntary)
>> > [ 402.125667] Tainted: [S]=CPU_OUT_OF_SPEC
>> > [ 402.125668] Hardware name: LENOVO 4291W3B/4291W3B, BIOS 8DET73WW (1.43 ) 10/12/2016
>> > [ 402.125671] Workqueue: events_unbound cfg80211_wiphy_work
>> > [ 402.125678] RIP: 0010:warn_bogus_irq_restore+0x25/0x30
>> > [ 402.125683] Code: 90 90 90 90 90 80 3d 51 3d dc 00 00 74 05 c3 cc cc cc cc 55 48 c7 c7 c0 4f c9 85 48 89 e5 c6 05 38 3d dc 00 01 e8 9b d8 e6 fe <0f> 0b 5d c3 cc cc cc cc cc cc cc 90 90 90 90 90 90 90 90 90 90 90
>> > [ 402.125686] RSP: 0018:ffffc9000173fb30 EFLAGS: 00010282
>> > [ 402.125691] RAX: 0000000000000000 RBX: ffffffff8616b460 RCX: 0000000000000000
>> > [ 402.125694] RDX: 0000000000000003 RSI: 0000000000000027 RDI: 00000000ffffffff
>> > [ 402.125696] RBP: ffffc9000173fb30 R08: 0000000028935f32 R09: 0000000000000001
>> > [ 402.125699] R10: 0000000000000044 R11: ffff888100ba52c8 R12: 0000000000000001
>> > [ 402.125702] R13: ffffc9000173fbcb R14: ffffffff84301224 R15: 0000000000000000
>> > [ 402.125704] FS: 0000000000000000(0000) GS:ffff88829007f000(0000) knlGS:0000000000000000
>> > [ 402.125707] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
>> > [ 402.125710] CR2: 000055967d471ee0 CR3: 0000000006046001 CR4: 00000000000606b0
>> > [ 402.125713] Call Trace:
>> > [ 402.125716] <TASK>
>> > [ 402.125719] console_flush_all+0x41e/0x460
>> > [ 402.125725] ? console_flush_all+0x43/0x460
>> > [ 402.125735] console_unlock+0x55/0x100
>> > [ 402.125741] vprintk_emit+0x157/0x320
>> > [ 402.125748] vprintk_default+0x18/0x20
>> > [ 402.125752] vprintk+0x9/0x10
>> > [ 402.125756] _printk+0x52/0x70
>> > [ 402.125766] ieee80211_sta_rx_queued_mgmt+0x4c8/0xd30
>> > [ 402.125775] ? __this_cpu_preempt_check+0x13/0x20
>> > [ 402.125784] ieee80211_iface_work+0x3ad/0x500
>>
>> That's not great, but I don't see how the driver or wifi subsystem is
>> involved ... ieee80211_sta_rx_queued_mgmt() doesn't even use spinlocks
>> let alone disable IRQs or use raw_ APIs, and it's in the middle of
>> printk anyway.
printk() does perform local_irq_save()/_restore(). I rechecked all the
sites and they are properly balanced in all paths.
It looks like you may have found other issues related to corruption, so
I will assume printk is not the culprit. But please keep us CC or ping
us again if printk comes under suspicion again.
John Ogness
next prev parent reply other threads:[~2025-07-24 8:50 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <aH/L1PCwtwe8Y1+a@duo.ucw.cz>
[not found] ` <aID6XPLXuGo+ViTm@duo.ucw.cz>
2025-07-23 15:42 ` locking problems in iwlwifi? was Re: 6.16-rcX: crashing way too often on thinkpad X220 Pavel Machek
2025-07-23 17:13 ` Johannes Berg
2025-07-23 20:37 ` Pavel Machek
2025-07-24 8:50 ` John Ogness [this message]
2025-07-23 21:42 ` Pavel Machek
2025-09-15 9:32 ` Pavel Machek
2025-07-23 17:31 ` Thomas Gleixner
2025-07-23 17:32 ` Thomas Gleixner
2025-07-24 17:15 ` Pavel Machek
2025-07-24 17:51 ` Waiman Long
2025-07-24 17:55 ` Waiman Long
2025-07-24 21:24 ` Thomas Gleixner
2025-07-25 15:23 ` Thomas Gleixner
2025-07-25 19:05 ` Pavel Machek
2025-07-24 19:07 ` Pavel Machek
2025-07-24 19:22 ` Waiman Long
2025-07-24 22:21 ` Thomas Gleixner
2025-07-25 1:56 ` Waiman Long
2025-07-24 19:45 ` John Ogness
2025-07-25 0:16 ` Waiman Long
2025-07-25 13:44 ` Thomas Gleixner
2025-07-25 19:38 ` Waiman Long
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=84h5z2vupd.fsf@jogness.linutronix.de \
--to=john.ogness@linutronix.de \
--cc=bp@alien8.de \
--cc=dave.hansen@linux.intel.com \
--cc=hpa@zytor.com \
--cc=johannes@sipsolutions.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=longman@redhat.com \
--cc=mingo@redhat.com \
--cc=miriam.rachel.korenblit@intel.com \
--cc=pavel@ucw.cz \
--cc=peterz@infradead.org \
--cc=pmladek@suse.com \
--cc=rostedt@goodmis.org \
--cc=senozhatsky@chromium.org \
--cc=tglx@linutronix.de \
--cc=will@kernel.org \
--cc=x86@kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox