All of lore.kernel.org
 help / color / mirror / Atom feed
* netpoll:  raspberrypi [4 5]  driver locking woes
@ 2025-08-25  5:53 Mike Galbraith
  2025-08-25 10:10 ` Mike Galbraith
  2025-08-26  9:49 ` Mike Galbraith
  0 siblings, 2 replies; 7+ messages in thread
From: Mike Galbraith @ 2025-08-25  5:53 UTC (permalink / raw)
  To: lkml; +Cc: Robert Hancock, Doug Berger, Florian Fainelli, Breno Leitao

Greetings,

After meeting some netconsole+ieee80211 woes recently, I decided to
build a lockdep enabled 6.17-rc2 kernel for my rpi[45] and give it a go
on their wired nics just to make sure there were no surprises lurking..
but damn the bad luck, both cute/dinky boxen whimpered.

The pi5 met locking woes due to...

   138badbc21a0 net: macb: use NAPI for TX completion path

...injecting spin_lock/unlock_irq() into polling, which inspired
netpoll.c to point a finger directly at the driver.  A quick poke
didn't produce instant gratification, but a wedged in revert did.

On the pi4, lockdep muttered about ring->lock in bcmgenet_xmit() and
bcmgenet_tx_poll() - switching to irqsave/restore spinlock variants
there silenced it.

Note to readers: before turning on lockdep, perhaps consider whacking
lockdep_assert_held() instances in gpiolib.c.. ymmv, but here it would
have saved time and naughty language expenditure.

	-Mike

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2025-08-29  2:55 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-25  5:53 netpoll: raspberrypi [4 5] driver locking woes Mike Galbraith
2025-08-25 10:10 ` Mike Galbraith
2025-08-26  9:49 ` Mike Galbraith
     [not found]   ` <f4fa3fcc637ffb6531982a90dbd9c27114e93036.camel@gmx.de>
2025-08-28 14:57     ` Sean Anderson
2025-08-28 17:26       ` Mike Galbraith
2025-08-28 18:56         ` Sean Anderson
2025-08-29  2:55           ` Mike Galbraith

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.