Re: [PATCH v3 net-next 1/7] net: skb_drop_reason: add document for drop reasons

From: Jakub Kicinski
Date: Thu Feb 10 2022 - 11:13:34 EST


On Thu, 10 Feb 2022 21:42:14 +0800 Menglong Dong wrote:
> How about introducing a field to 'struct sock' for drop reasons? As sk is
> locked during the packet process in tcp_v4_do_rcv(), this seems to work.

I find adding temporary storage to persistent data structures awkward.
You can put a structure on the stack and pass it thru the call chain,
that's just my subjective preference, tho, others may have better ideas.