net/tcp: rename ack_edge and seq_init to more common rcv_nxt and irs
Use the names from RFC 9293 Signed-off-by: Mikhail Kshevetskiy <mikhail.kshevetskiy@iopsys.eu> Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:

committed by
Tom Rini

parent
2b17749870
commit
82bf3aafa6
@@ -285,8 +285,8 @@ enum tcp_state {
|
|||||||
*
|
*
|
||||||
* @state: TCP connection state
|
* @state: TCP connection state
|
||||||
*
|
*
|
||||||
* @seq_init: Initial receive sequence number
|
* @irs: Initial receive sequence number
|
||||||
* @ack_edge: Receive next
|
* @rcv_nxt: Receive next
|
||||||
*
|
*
|
||||||
* @loc_timestamp: Local timestamp
|
* @loc_timestamp: Local timestamp
|
||||||
* @rmt_timestamp: Remote timestamp
|
* @rmt_timestamp: Remote timestamp
|
||||||
@@ -301,8 +301,8 @@ struct tcp_stream {
|
|||||||
/* TCP connection state */
|
/* TCP connection state */
|
||||||
enum tcp_state state;
|
enum tcp_state state;
|
||||||
|
|
||||||
u32 seq_init;
|
u32 irs;
|
||||||
u32 ack_edge;
|
u32 rcv_nxt;
|
||||||
|
|
||||||
/* TCP option timestamp */
|
/* TCP option timestamp */
|
||||||
u32 loc_timestamp;
|
u32 loc_timestamp;
|
||||||
|
32
net/tcp.c
32
net/tcp.c
@@ -360,9 +360,9 @@ int tcp_set_tcp_header(struct tcp_stream *tcp, uchar *pkt, int payload_len,
|
|||||||
pkt_len = pkt_hdr_len + payload_len;
|
pkt_len = pkt_hdr_len + payload_len;
|
||||||
tcp_len = pkt_len - IP_HDR_SIZE;
|
tcp_len = pkt_len - IP_HDR_SIZE;
|
||||||
|
|
||||||
tcp->ack_edge = tcp_ack_num;
|
tcp->rcv_nxt = tcp_ack_num;
|
||||||
/* TCP Header */
|
/* TCP Header */
|
||||||
b->ip.hdr.tcp_ack = htonl(tcp->ack_edge);
|
b->ip.hdr.tcp_ack = htonl(tcp->rcv_nxt);
|
||||||
b->ip.hdr.tcp_src = htons(tcp->lport);
|
b->ip.hdr.tcp_src = htons(tcp->lport);
|
||||||
b->ip.hdr.tcp_dst = htons(tcp->rport);
|
b->ip.hdr.tcp_dst = htons(tcp->rport);
|
||||||
b->ip.hdr.tcp_seq = htonl(tcp_seq_num);
|
b->ip.hdr.tcp_seq = htonl(tcp_seq_num);
|
||||||
@@ -396,10 +396,10 @@ int tcp_set_tcp_header(struct tcp_stream *tcp, uchar *pkt, int payload_len,
|
|||||||
return pkt_hdr_len;
|
return pkt_hdr_len;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void tcp_update_ack_edge(struct tcp_stream *tcp)
|
static void tcp_update_rcv_nxt(struct tcp_stream *tcp)
|
||||||
{
|
{
|
||||||
if (tcp_seq_cmp(tcp->ack_edge, tcp->lost.hill[0].l) >= 0) {
|
if (tcp_seq_cmp(tcp->rcv_nxt, tcp->lost.hill[0].l) >= 0) {
|
||||||
tcp->ack_edge = tcp->lost.hill[0].r;
|
tcp->rcv_nxt = tcp->lost.hill[0].r;
|
||||||
|
|
||||||
memmove(&tcp->lost.hill[0], &tcp->lost.hill[1],
|
memmove(&tcp->lost.hill[0], &tcp->lost.hill[1],
|
||||||
(TCP_SACK_HILLS - 1) * sizeof(struct sack_edges));
|
(TCP_SACK_HILLS - 1) * sizeof(struct sack_edges));
|
||||||
@@ -434,7 +434,7 @@ void tcp_hole(struct tcp_stream *tcp, u32 tcp_seq_num, u32 len)
|
|||||||
tcp_seq_num = tcp->lost.hill[i].l;
|
tcp_seq_num = tcp->lost.hill[i].l;
|
||||||
}
|
}
|
||||||
if (tcp_seq_cmp(tcp->lost.hill[i].r, tcp_seq_num + len) >= 0) {
|
if (tcp_seq_cmp(tcp->lost.hill[i].r, tcp_seq_num + len) >= 0) {
|
||||||
tcp_update_ack_edge(tcp);
|
tcp_update_rcv_nxt(tcp);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -463,12 +463,12 @@ void tcp_hole(struct tcp_stream *tcp, u32 tcp_seq_num, u32 len)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
tcp_update_ack_edge(tcp);
|
tcp_update_rcv_nxt(tcp);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (i == TCP_SACK_HILLS) {
|
if (i == TCP_SACK_HILLS) {
|
||||||
tcp_update_ack_edge(tcp);
|
tcp_update_rcv_nxt(tcp);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -489,7 +489,7 @@ void tcp_hole(struct tcp_stream *tcp, u32 tcp_seq_num, u32 len)
|
|||||||
tcp->lost.hill[i].r = tcp_seq_num + len;
|
tcp->lost.hill[i].r = tcp_seq_num + len;
|
||||||
tcp->lost.len = TCP_OPT_LEN_2 + cnt * TCP_OPT_LEN_8;
|
tcp->lost.len = TCP_OPT_LEN_2 + cnt * TCP_OPT_LEN_8;
|
||||||
|
|
||||||
tcp_update_ack_edge(tcp);
|
tcp_update_rcv_nxt(tcp);
|
||||||
};
|
};
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -566,8 +566,8 @@ static u8 tcp_state_machine(struct tcp_stream *tcp, u8 tcp_flags,
|
|||||||
debug_cond(DEBUG_INT_STATE, "TCP CLOSED %x\n", tcp_flags);
|
debug_cond(DEBUG_INT_STATE, "TCP CLOSED %x\n", tcp_flags);
|
||||||
if (tcp_syn) {
|
if (tcp_syn) {
|
||||||
action = TCP_SYN | TCP_ACK;
|
action = TCP_SYN | TCP_ACK;
|
||||||
tcp->seq_init = tcp_seq_num;
|
tcp->irs = tcp_seq_num;
|
||||||
tcp->ack_edge = tcp_seq_num + 1;
|
tcp->rcv_nxt = tcp_seq_num + 1;
|
||||||
tcp->lost.len = TCP_OPT_LEN_2;
|
tcp->lost.len = TCP_OPT_LEN_2;
|
||||||
tcp->state = TCP_SYN_RECEIVED;
|
tcp->state = TCP_SYN_RECEIVED;
|
||||||
} else if (tcp_ack || tcp_fin) {
|
} else if (tcp_ack || tcp_fin) {
|
||||||
@@ -583,8 +583,8 @@ static u8 tcp_state_machine(struct tcp_stream *tcp, u8 tcp_flags,
|
|||||||
tcp->state = TCP_CLOSE_WAIT;
|
tcp->state = TCP_CLOSE_WAIT;
|
||||||
} else if (tcp_ack || (tcp_syn && tcp_ack)) {
|
} else if (tcp_ack || (tcp_syn && tcp_ack)) {
|
||||||
action |= TCP_ACK;
|
action |= TCP_ACK;
|
||||||
tcp->seq_init = tcp_seq_num;
|
tcp->irs = tcp_seq_num;
|
||||||
tcp->ack_edge = tcp_seq_num + 1;
|
tcp->rcv_nxt = tcp_seq_num + 1;
|
||||||
tcp->state = TCP_ESTABLISHED;
|
tcp->state = TCP_ESTABLISHED;
|
||||||
|
|
||||||
if (tcp_syn && tcp_ack)
|
if (tcp_syn && tcp_ack)
|
||||||
@@ -633,7 +633,7 @@ static u8 tcp_state_machine(struct tcp_stream *tcp, u8 tcp_flags,
|
|||||||
case TCP_FIN_WAIT_1:
|
case TCP_FIN_WAIT_1:
|
||||||
debug_cond(DEBUG_INT_STATE, "TCP_FIN_WAIT_1 (%x)\n", tcp_flags);
|
debug_cond(DEBUG_INT_STATE, "TCP_FIN_WAIT_1 (%x)\n", tcp_flags);
|
||||||
if (tcp_fin) {
|
if (tcp_fin) {
|
||||||
tcp->ack_edge++;
|
tcp->rcv_nxt++;
|
||||||
action = TCP_ACK | TCP_FIN;
|
action = TCP_ACK | TCP_FIN;
|
||||||
tcp->state = TCP_FIN_WAIT_2;
|
tcp->state = TCP_FIN_WAIT_2;
|
||||||
}
|
}
|
||||||
@@ -748,7 +748,7 @@ void rxhand_tcp_f(union tcp_build_pkt *b, unsigned int pkt_len)
|
|||||||
} else if (tcp_action != TCP_DATA) {
|
} else if (tcp_action != TCP_DATA) {
|
||||||
debug_cond(DEBUG_DEV_PKT,
|
debug_cond(DEBUG_DEV_PKT,
|
||||||
"TCP Action (action=%x,Seq=%u,Ack=%u,Pay=%d)\n",
|
"TCP Action (action=%x,Seq=%u,Ack=%u,Pay=%d)\n",
|
||||||
tcp_action, tcp_ack_num, tcp->ack_edge, payload_len);
|
tcp_action, tcp_ack_num, tcp->rcv_nxt, payload_len);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Warning: Incoming Ack & Seq sequence numbers are transposed
|
* Warning: Incoming Ack & Seq sequence numbers are transposed
|
||||||
@@ -756,7 +756,7 @@ void rxhand_tcp_f(union tcp_build_pkt *b, unsigned int pkt_len)
|
|||||||
*/
|
*/
|
||||||
net_send_tcp_packet(0, tcp->rhost, tcp->rport, tcp->lport,
|
net_send_tcp_packet(0, tcp->rhost, tcp->rport, tcp->lport,
|
||||||
(tcp_action & (~TCP_PUSH)),
|
(tcp_action & (~TCP_PUSH)),
|
||||||
tcp_ack_num, tcp->ack_edge);
|
tcp_ack_num, tcp->rcv_nxt);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user