Page Menu
Home
FreeBSD
Search
Configure Global Search
Log In
Files
F141076317
D29411.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Flag For Later
Award Token
Size
3 KB
Referenced Files
None
Subscribers
None
D29411.diff
View Options
diff --git a/sys/netinet/tcp_input.c b/sys/netinet/tcp_input.c
--- a/sys/netinet/tcp_input.c
+++ b/sys/netinet/tcp_input.c
@@ -2576,47 +2576,7 @@
if (V_tcp_do_prr &&
IN_FASTRECOVERY(tp->t_flags) &&
(tp->t_flags & TF_SACK_PERMIT)) {
- int snd_cnt = 0, limit = 0;
- int del_data = 0, pipe = 0;
- /*
- * In a duplicate ACK del_data is only the
- * diff_in_sack. If no SACK is used del_data
- * will be 0. Pipe is the amount of data we
- * estimate to be in the network.
- */
- del_data = tp->sackhint.delivered_data;
- if (V_tcp_do_rfc6675_pipe)
- pipe = tcp_compute_pipe(tp);
- else
- pipe = (tp->snd_nxt - tp->snd_fack) +
- tp->sackhint.sack_bytes_rexmit;
- tp->sackhint.prr_delivered += del_data;
- if (pipe >= tp->snd_ssthresh) {
- if (tp->sackhint.recover_fs == 0)
- tp->sackhint.recover_fs =
- imax(1, tp->snd_nxt - tp->snd_una);
- snd_cnt = howmany((long)tp->sackhint.prr_delivered *
- tp->snd_ssthresh, tp->sackhint.recover_fs) -
- tp->sackhint.prr_out;
- } else {
- if (V_tcp_do_prr_conservative)
- limit = tp->sackhint.prr_delivered -
- tp->sackhint.prr_out;
- else
- limit = imax(tp->sackhint.prr_delivered -
- tp->sackhint.prr_out,
- del_data) + maxseg;
- snd_cnt = imin(tp->snd_ssthresh - pipe, limit);
- }
- snd_cnt = imax(snd_cnt, 0) / maxseg;
- /*
- * Send snd_cnt new data into the network in
- * response to this ACK. If there is a going
- * to be a SACK retransmission, adjust snd_cwnd
- * accordingly.
- */
- tp->snd_cwnd = imax(maxseg, tp->snd_nxt - tp->snd_recover +
- tp->sackhint.sack_bytes_rexmit + (snd_cnt * maxseg));
+ tcp_do_prr_ack(tp, th);
} else if ((tp->t_flags & TF_SACK_PERMIT) &&
(to.to_flags & TOF_SACK) &&
IN_FASTRECOVERY(tp->t_flags)) {
@@ -2814,9 +2774,13 @@
if (IN_FASTRECOVERY(tp->t_flags)) {
if (SEQ_LT(th->th_ack, tp->snd_recover)) {
if (tp->t_flags & TF_SACK_PERMIT)
- if (V_tcp_do_prr && to.to_flags & TOF_SACK)
- tcp_prr_partialack(tp, th);
- else
+ if (V_tcp_do_prr && to.to_flags & TOF_SACK) {
+ tcp_timer_activate(tp, TT_REXMT, 0);
+ tp->t_rtttime = 0;
+ tcp_do_prr_ack(tp, th);
+ tp->t_flags |= TF_ACKNOW;
+ (void) tcp_output(tp);
+ } else
tcp_sack_partialack(tp, th);
else
tcp_newreno_partial_ack(tp, th);
@@ -3946,15 +3910,13 @@
}
void
-tcp_prr_partialack(struct tcpcb *tp, struct tcphdr *th)
+tcp_do_prr_ack(struct tcpcb *tp, struct tcphdr *th)
{
int snd_cnt = 0, limit = 0, del_data = 0, pipe = 0;
int maxseg = tcp_maxseg(tp);
INP_WLOCK_ASSERT(tp->t_inpcb);
- tcp_timer_activate(tp, TT_REXMT, 0);
- tp->t_rtttime = 0;
/*
* Compute the amount of data that this ACK is indicating
* (del_data) and an estimate of how many bytes are in the
@@ -3994,8 +3956,6 @@
*/
tp->snd_cwnd = imax(maxseg, tp->snd_nxt - tp->snd_recover +
tp->sackhint.sack_bytes_rexmit + (snd_cnt * maxseg));
- tp->t_flags |= TF_ACKNOW;
- (void) tcp_output(tp);
}
/*
diff --git a/sys/netinet/tcp_var.h b/sys/netinet/tcp_var.h
--- a/sys/netinet/tcp_var.h
+++ b/sys/netinet/tcp_var.h
@@ -1063,7 +1063,7 @@
void tcp_clean_sackreport(struct tcpcb *tp);
void tcp_sack_adjust(struct tcpcb *tp);
struct sackhole *tcp_sack_output(struct tcpcb *tp, int *sack_bytes_rexmt);
-void tcp_prr_partialack(struct tcpcb *, struct tcphdr *);
+void tcp_do_prr_ack(struct tcpcb *, struct tcphdr *);
void tcp_sack_partialack(struct tcpcb *, struct tcphdr *);
void tcp_free_sackholes(struct tcpcb *tp);
int tcp_newreno(struct tcpcb *, struct tcphdr *);
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Thu, Jan 1, 1:46 PM (2 h, 1 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
27437560
Default Alt Text
D29411.diff (3 KB)
Attached To
Mode
D29411: Refactor PRR code
Attached
Detach File
Event Timeline
Log In to Comment