mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-07-28 11:49:44 +00:00
Kernel: Don't use delayed ACKs when establishing the connection
When establishing the connection we should send ACKs right away so as to not delay the connection process. This didn't previously matter because we'd flush all delayed ACKs when NetworkTask waits for incoming packets.
This commit is contained in:
parent
aea709de31
commit
990b2d0989
Notes:
sideshowbarker
2024-07-18 18:09:48 +09:00
Author: https://github.com/gunnarbeutner
Commit: 990b2d0989
Pull-request: https://github.com/SerenityOS/serenity/pull/7078
Reviewed-by: https://github.com/awesomekling
1 changed files with 2 additions and 2 deletions
|
@ -435,12 +435,12 @@ void handle_tcp(const IPv4Packet& ipv4_packet, const Time& packet_timestamp)
|
|||
switch (tcp_packet.flags()) {
|
||||
case TCPFlags::SYN:
|
||||
socket->set_ack_number(tcp_packet.sequence_number() + payload_size + 1);
|
||||
send_delayed_tcp_ack(socket);
|
||||
unused_rc = socket->send_ack(true);
|
||||
socket->set_state(TCPSocket::State::SynReceived);
|
||||
return;
|
||||
case TCPFlags::ACK | TCPFlags::SYN:
|
||||
socket->set_ack_number(tcp_packet.sequence_number() + payload_size + 1);
|
||||
send_delayed_tcp_ack(socket);
|
||||
unused_rc = socket->send_ack(true);
|
||||
socket->set_state(TCPSocket::State::Established);
|
||||
socket->set_setup_state(Socket::SetupState::Completed);
|
||||
socket->set_connected(true);
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue