Thread (15 messages) 15 messages, 2 authors, 2026-06-05

回复:[PATCH v17 net-next 05/11] net/nebula-matrix: add channel layer

flat view

From: Illusion Wang <hidden>
Date: 2026-06-05 02:00:57
Also in: linux-doc, lkml

Sashiko reports a number of issues WRT the atomic status updates:

https://netdev-ai.bots.linux.dev/sashiko/#/patchset/20260601093149.25905-1-illusion.wang%40nebula-matrix.com
It's indeed quite complex to get things right this way. I'm wondering if
you should use a spin_lock instead to protect the relevant state?
Also a few process related notes:
- to make effective progresses you should proactively reply on the ML WRT
the sashiko comments discussion if and why they are (not) relevant.
- is 'illusion wang' (all lower case) your real name? we can't accept
pseudonims as SoB tag.
Thank you for your feedback.
Yes, I've reviewed the Sashiko report. The atomic status update race is real:
nbl_chan_recv_ack_msg writes status and need_waked while nbl_chan_send_msg
may read/clear them, with no ordering guarantee between the two atomics.
The spinlock fix addresses exactly what Sashiko flagged.
I'll update the patchset accordingly.
ps: "illusion wang" is indeed my real name.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help