[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <willemdebruijn.kernel.16c9d77b1d2c@gmail.com>
Date: Tue, 02 Sep 2025 18:41:15 -0400
From: Willem de Bruijn <willemdebruijn.kernel@...il.com>
To: Breno Leitao <leitao@...ian.org>,
Andrew Lunn <andrew+netdev@...n.ch>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>,
Simon Horman <horms@...nel.org>,
Sebastian Andrzej Siewior <bigeasy@...utronix.de>,
Clark Williams <clrkwllms@...nel.org>,
Steven Rostedt <rostedt@...dmis.org>
Cc: netdev@...r.kernel.org,
linux-kernel@...r.kernel.org,
linux-rt-devel@...ts.linux.dev,
kernel-team@...a.com,
efault@....de,
calvin@...nvd.org,
Breno Leitao <leitao@...ian.org>
Subject: Re: [PATCH 1/7] netconsole: Split UDP message building and sending
operations
Remember to label the target branch: [PATCH net-next 1/7]
Breno Leitao wrote:
> Split the netpoll_send_udp() function into two separate operations:
> netpoll_prepare_skb() for message preparation and netpoll_send_skb()
> for transmission.
>
> This improves separation of concerns. SKB building logic is now isolated
> from the actual network transmission, improving code modularity and
> testability.
>
> Why?
>
> The separation of SKB preparation and transmission operations enables
> more granular locking strategies. The netconsole buffer requires lock
> protection during packet construction, but the transmission phase can
> proceed without holding the same lock.
>
> Also, this makes netpoll only reponsible for handling SKB.
>
> netpoll_prepare_skb() is now exported, but, in the upcoming change, it
> will be moved to netconsole, and become static.
>
> Signed-off-by: Breno Leitao <leitao@...ian.org>
aside from the above,
Reviewed-by: Willem de Bruijn <willemb@...gle.com>
Powered by blists - more mailing lists