On Mon, Apr 22, 2024 at 11:01:03AM +0800, Jason Xing wrote: ...
> diff --git a/include/net/rstreason.h b/include/net/rstreason.h ... > +/** > + * There are three parts in order: > + * 1) reset reason in MPTCP: only for MPTCP use > + * 2) skb drop reason: relying on drop reasons for such as passive reset > + * 3) independent reset reason: such as active reset reasons > + */ Hi Jason, A minor nit from my side. '/**' denotes the beginning of a Kernel doc, but other than that, this comment is not a Kernel doc. FWIIW, I would suggest providing a proper Kernel doc for enum sk_rst_reason. But another option would be to simply make this a normal comment, starting with "/* There are" > +enum sk_rst_reason { ...