[PATCH bpf-next] net: tcp: replace the document for "lsndtime" in tcp_sock

Menglong Dong posted 1 patch 1 month ago
There is a newer version of this series
include/linux/tcp.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
[PATCH bpf-next] net: tcp: replace the document for "lsndtime" in tcp_sock
Posted by Menglong Dong 1 month ago
The document for "lsndtime" in struct tcp_sock is placed in the wrong
place, so let's replace it in the proper place.

Fixes: d5fed5addb2b ("tcp: reorganize tcp_sock fast path variables")
Signed-off-by: Menglong Dong <dongml2@chinatelecom.cn>
---
 include/linux/tcp.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/include/linux/tcp.h b/include/linux/tcp.h
index 6a5e08b937b3..f88daaa76d83 100644
--- a/include/linux/tcp.h
+++ b/include/linux/tcp.h
@@ -200,7 +200,6 @@ struct tcp_sock {
 
 	/* TX read-mostly hotpath cache lines */
 	__cacheline_group_begin(tcp_sock_read_tx);
-	/* timestamp of last sent data packet (for restart window) */
 	u32	max_window;	/* Maximal window ever seen from peer	*/
 	u32	rcv_ssthresh;	/* Current window clamp			*/
 	u32	reordering;	/* Packet reordering metric.		*/
@@ -263,7 +262,7 @@ struct tcp_sock {
 	u32	chrono_stat[3];	/* Time in jiffies for chrono_stat stats */
 	u32	write_seq;	/* Tail(+1) of data held in tcp send buffer */
 	u32	pushed_seq;	/* Last pushed seq, required to talk to windows */
-	u32	lsndtime;
+	u32	lsndtime;	/* timestamp of last sent data packet (for restart window) */
 	u32	mdev_us;	/* medium deviation			*/
 	u32	rtt_seq;	/* sequence number to update rttvar	*/
 	u64	tcp_wstamp_ns;	/* departure time for next sent data packet */
-- 
2.39.5
Re: [PATCH bpf-next] net: tcp: replace the document for "lsndtime" in tcp_sock
Posted by Martin KaFai Lau 3 weeks, 5 days ago
On 10/25/24 11:54 PM, Menglong Dong wrote:
> The document for "lsndtime" in struct tcp_sock is placed in the wrong
> place, so let's replace it in the proper place.

The patch lgtm but this should be tagged for net-next material.
Re: [PATCH bpf-next] net: tcp: replace the document for "lsndtime" in tcp_sock
Posted by Menglong Dong 3 weeks, 5 days ago
On Wed, Oct 30, 2024 at 2:42 AM Martin KaFai Lau <martin.lau@linux.dev> wrote:
>
> On 10/25/24 11:54 PM, Menglong Dong wrote:
> > The document for "lsndtime" in struct tcp_sock is placed in the wrong
> > place, so let's replace it in the proper place.
>
> The patch lgtm but this should be tagged for net-next material.

Oops......Sorry that I tagged the target branch wrong :/
I'll resend it to the net-next branch.

Thanks!
Menglong Dong

>