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

From: Illusion Wang

Date: Thu Jun 04 2026 - 22:02:50 EST


> 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.