Thread (4 messages) 4 messages, 2 authors, 24d ago

Re: [PATCH net-next 1/2] pcnet32: stop holding device spin lock during napi_complete_done

From: Andrew Lunn <andrew@lunn.ch>
Date: 2026-05-20 12:12:40

On Wed, May 20, 2026 at 08:33:28AM +0200, Oscar Maes wrote:
napi_complete_done may call gro_flush_normal (though not currently, as GRO
is unsupported at the moment), which may result in packet TX. This will
eventually result in calling pcnet32_start_xmit - resulting in a deadlock
while trying to re-acquire the already locked spin lock.
Please continue to commit message with something like:

  It is safe to release the spinlock, because it is protecting X,
  which is not modified during the time the spinlock is released.

You need to convince us this is safe.


    Andrew

---
pw-bot: cr
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help