d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

tcp_lro: When processing compressed acks lets support the new early wake feature for rack.

Browse Source 
During compressed ack and mbuf queuing we determine if we need to wake up. A
new function was added that is optional to the tfb so that the stack itself can also
be asked if a wakeup should happen. This helps compensate for late hpts calls.

Reviewed by: tuexen
Sponsored by: Netflix Inc
Differential Revision:https://reviews.freebsd.org/D39502
This commit is contained in:
Randall Stewart 2023-04-12 11:35:14 -04:00
parent b4f1582f3b
commit 1073f41657
1 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
sys/netinet/tcp_lro.c
View File

@ -1195,6 +1195,19 @@ tcp_queue_pkts(struct inpcb *inp, struct tcpcb *tp, struct lro_entry *le)
le->m_last_mbuf = NULL;
}
static bool
tcp_lro_check_wake_status(struct inpcb *inp)
{
struct tcpcb *tp;
tp = intotcpcb(inp);
if (__predict_false(tp == NULL))
return (true);
if (tp->t_fb->tfb_early_wake_check != NULL)
return ((tp->t_fb->tfb_early_wake_check)(tp));
return (false);
}
static struct mbuf *
tcp_lro_get_last_if_ackcmp(struct lro_ctrl *lc, struct lro_entry *le,
struct inpcb *inp, int32_t *new_m, bool can_append_old_cmp)
@ -1751,6 +1764,9 @@ do_bpf_strip_and_compress(struct inpcb *inp, struct lro_ctrl *lc,
* not care about SACKS, then we should wake up.
*/
*should_wake = true;
} else if (*should_wake == false) {
/* Wakeup override check if we are false here */
*should_wake = tcp_lro_check_wake_status(inp);
}
/* Is the ack compressable? */
if (can_compress == false)