Author: tuexen
Date: Mon May 18 19:35:46 2020
New Revision: 361226
URL: https://svnweb.freebsd.org/changeset/base/361226
Log:
Don't check an unsigned variable for being negative.
MFC after:3 days.
Modified:
head/sys/netinet/sctp_usrreq.c
Modified:
Author: tuexen
Date: Mon May 18 19:23:01 2020
New Revision: 361225
URL: https://svnweb.freebsd.org/changeset/base/361225
Log:
Remove redundant assignment.
MFC after:3 days
Modified:
head/sys/netinet/sctputil.c
Modified: head/sys/netinet/sctputil.c
> On 18. May 2020, at 20:23, Conrad Meyer wrote:
>
> On Mon, May 18, 2020 at 10:35 AM Michael Tuexen wrote:
>>
>>> On 18. May 2020, at 17:38, Conrad Meyer wrote:
>>>
>>> These changes are a bit odd. The only reason a standards-compliant
>>&
Author: tuexen
Date: Mon May 18 18:42:43 2020
New Revision: 361224
URL: https://svnweb.freebsd.org/changeset/base/361224
Log:
Cleanup, no functional change intended.
MFC after:3 days
Modified:
head/sys/netinet/sctp_pcb.c
Modified: head/sys/netinet/sctp_pcb.c
Author: tuexen
Date: Mon May 18 18:32:58 2020
New Revision: 361222
URL: https://svnweb.freebsd.org/changeset/base/361222
Log:
Avoid an integer underflow.
MFC after:3 days
Modified:
head/sys/netinet/sctp_asconf.c
Modified: head/sys/netinet/sctp_asconf.c
Author: tuexen
Date: Mon May 18 18:27:10 2020
New Revision: 361221
URL: https://svnweb.freebsd.org/changeset/base/361221
Log:
Remove redundant check.
MFC after:3 days
Modified:
head/sys/netinet/sctp_asconf.c
Modified: head/sys/netinet/sctp_asconf.c
ate a standards-compliant portability shim for that
>> platform instead of this more invasive change?
>>
>> FreeBSD's snprintf(9) does not have this bug, nor does its snprintf(3).
>>
>> Best regards,
>> Conrad
>>
>> On Mon, May 18, 2020
more invasive change?
If you want, I can revert the change and use the code only on non-FreeBSD
platforms.
Best regards
Michael
>
> FreeBSD's snprintf(9) does not have this bug, nor does its snprintf(3).
>
> Best regards,
> Conrad
>
> On Mon, May 18, 2020 at 3:07 AM Michael Tu
Author: tuexen
Date: Mon May 18 15:02:15 2020
New Revision: 361214
URL: https://svnweb.freebsd.org/changeset/base/361214
Log:
Fix logical condition by looking at usecs.
This issue was found by cpp-check running on the userland stack.
MFC after:3 days
Modified:
Author: tuexen
Date: Mon May 18 15:00:18 2020
New Revision: 361213
URL: https://svnweb.freebsd.org/changeset/base/361213
Log:
Whitespace change.
MFC after:3 days
Modified:
head/sys/netinet/sctputil.c
Modified: head/sys/netinet/sctputil.c
Author: tuexen
Date: Mon May 18 10:07:01 2020
New Revision: 361209
URL: https://svnweb.freebsd.org/changeset/base/361209
Log:
Handle failures of snprintf().
MFC after:3 days
Modified:
head/sys/netinet/sctp_asconf.c
head/sys/netinet/sctp_indata.c
Author: tuexen
Date: Sun May 17 22:31:38 2020
New Revision: 361145
URL: https://svnweb.freebsd.org/changeset/base/361145
Log:
Non-functional changes, cleanups.
MFC after:3 days
Modified:
head/sys/netinet/sctp_asconf.c
head/sys/netinet/sctp_os_bsd.h
Author: tuexen
Date: Sat May 16 19:26:39 2020
New Revision: 361116
URL: https://svnweb.freebsd.org/changeset/base/361116
Log:
Ensure that an stcb is not dereferenced when it is about to be
freed.
This issue was found by SYZKALLER.
MFC after:3 days
Modified:
Author: tuexen
Date: Fri May 15 14:06:37 2020
New Revision: 361081
URL: https://svnweb.freebsd.org/changeset/base/361081
Log:
Allow only IPv4 addresses in sendto() for TCP on AF_INET sockets.
This problem was found by looking at syzkaller reproducers for some other
problems.
Author: tuexen
Date: Mon May 11 22:47:20 2020
New Revision: 360942
URL: https://svnweb.freebsd.org/changeset/base/360942
Log:
Fix a copy and paste error introduced in r360878.
Reported-by: syzbot+a0863e972771f2f0d...@syzkaller.appspotmail.com
Reported-by:
Author: tuexen
Date: Sun May 10 22:54:30 2020
New Revision: 360885
URL: https://svnweb.freebsd.org/changeset/base/360885
Log:
Ensure that the SCTP iterator runs with an stcb and inp, which belong to
each other.
Reported by: syzbot+82d39d14f2f765e38...@syzkaller.appspotmail.com
MFC
Author: tuexen
Date: Sun May 10 17:43:42 2020
New Revision: 360879
URL: https://svnweb.freebsd.org/changeset/base/360879
Log:
Remove trailing whitespace.
Modified:
head/sys/netinet/tcp_usrreq.c
Modified: head/sys/netinet/tcp_usrreq.c
Author: tuexen
Date: Sun May 10 17:19:19 2020
New Revision: 360878
URL: https://svnweb.freebsd.org/changeset/base/360878
Log:
Ensure that we have a path when starting the T3 RXT timer.
Reported by: syzbot+f2321629047f89486...@syzkaller.appspotmail.com
MFC after:3 days
Modified:
Author: tuexen
Date: Sun May 10 10:03:10 2020
New Revision: 360869
URL: https://svnweb.freebsd.org/changeset/base/360869
Log:
Only drop DATA chunk with lower priorities as specified in RFC 7496.
This issue was found by looking at a reproducer generated by syzkaller.
MFC after:
Author: tuexen
Date: Tue May 5 19:54:30 2020
New Revision: 360671
URL: https://svnweb.freebsd.org/changeset/base/360671
Log:
Avoid underflowing a variable, which would result in taking more
data from the stream queues then needed.
Thanks to Timo Voelker for finding this bug and
Author: tuexen
Date: Tue May 5 17:52:44 2020
New Revision: 360662
URL: https://svnweb.freebsd.org/changeset/base/360662
Log:
Fix the computation of the numbers of entries of the mapping array to
look at when generating a SACK. This was wrong in case of sequence
numbers wrap arounds.
Author: tuexen
Date: Mon May 4 23:05:11 2020
New Revision: 360645
URL: https://svnweb.freebsd.org/changeset/base/360645
Log:
Add net epoch support back, which was taken out by accident in
https://svnweb.freebsd.org/changeset/base/360639
Reviewed by: rrs
Sponsored by:
Author: tuexen
Date: Mon May 4 22:02:49 2020
New Revision: 360641
URL: https://svnweb.freebsd.org/changeset/base/360641
Log:
Enter the net epoch before calling the output routine in TCP BBR.
This was only triggered when setting the IPPROTO_TCP level socket
option TCP_DELACK.
This issue
Author: tuexen
Date: Wed Apr 22 21:22:33 2020
New Revision: 360209
URL: https://svnweb.freebsd.org/changeset/base/360209
Log:
Improve input validation when processing AUTH chunks.
Thanks to Natalie Silvanovich from Google for finding and reporting the
issue found by her in the SCTP
Author: tuexen
Date: Wed Apr 22 12:47:46 2020
New Revision: 360193
URL: https://svnweb.freebsd.org/changeset/base/360193
Log:
Improve input validation when processing AUTH chunks.
Thanks to Natalie Silvanovich from Google for finding and reporting the
issue found by her in the SCTP
Author: tuexen
Date: Tue Apr 14 16:35:05 2020
New Revision: 359926
URL: https://svnweb.freebsd.org/changeset/base/359926
Log:
Improve the TCP blackhole detection. The principle is to reduce the
MSS in two steps and try each candidate two times. However, if two
candidates are the same (which
> On 11. Apr 2020, at 23:35, Conrad Meyer wrote:
>
> Hi Michael,
>
> On Sat, Apr 11, 2020 at 1:37 PM Michael Tuexen wrote:
>>
>> Author: tuexen
>> Date: Sat Apr 11 20:36:54 2020
>> New Revision: 359809
>> URL: https://svnweb.freebsd.org/changeset/
Author: tuexen
Date: Sun Apr 12 09:31:36 2020
New Revision: 359822
URL: https://svnweb.freebsd.org/changeset/base/359822
Log:
Revert https://svnweb.freebsd.org/changeset/base/359809
The intended change was
sp->next.tqe_next = NULL;
sp->next.tqe_prev = NULL;
which doesn't
Author: tuexen
Date: Sat Apr 11 20:36:54 2020
New Revision: 359809
URL: https://svnweb.freebsd.org/changeset/base/359809
Log:
Zero out pointers for consistency.
This was found by running syzkaller on an INVARIANTS kernel.
MFC after:3 days
Modified:
Author: tuexen
Date: Mon Apr 6 13:58:13 2020
New Revision: 359657
URL: https://svnweb.freebsd.org/changeset/base/359657
Log:
Do more argument validation under INVARIANTS when starting/stopping
an SCTP timer.
MFC after:1 week
Modified:
head/sys/netinet/sctputil.c
Author: tuexen
Date: Tue Mar 31 15:54:54 2020
New Revision: 359487
URL: https://svnweb.freebsd.org/changeset/base/359487
Log:
Allow the TCP backhole detection to be disabled at all, enabled only
for IPv4, enabled only for IPv6, and enabled for IPv4 and IPv6.
The current blackhole detection
Author: tuexen
Date: Sun Mar 29 15:43:00 2020
New Revision: 359422
URL: https://svnweb.freebsd.org/changeset/base/359422
Log:
Be a bit more precisly in the description of the sysctl variable
net.inet.tcp.pmtud_blackhole_detection. Also remove three entries,
which are not sysctl variables
Author: tuexen
Date: Sat Mar 28 22:35:04 2020
New Revision: 359410
URL: https://svnweb.freebsd.org/changeset/base/359410
Log:
Small cleanup by using a variable just assigned.
MFC after:1 week
Modified:
head/sys/netinet/sctp_usrreq.c
Modified: head/sys/netinet/sctp_usrreq.c
Author: tuexen
Date: Sat Mar 28 20:25:45 2020
New Revision: 359405
URL: https://svnweb.freebsd.org/changeset/base/359405
Log:
Handle integer overflows correctly when converting msecs and secs to
ticks and vice versa.
These issues were caught by recently added panic() calls on INVARIANTS
Author: tuexen
Date: Fri Mar 27 21:48:52 2020
New Revision: 359379
URL: https://svnweb.freebsd.org/changeset/base/359379
Log:
Some more uint32_t cleanups, no functional change.
MFC after:1 week
Modified:
head/sys/netinet/sctp_pcb.h
head/sys/netinet/sctp_structs.h
Author: tuexen
Date: Fri Mar 27 11:08:11 2020
New Revision: 359357
URL: https://svnweb.freebsd.org/changeset/base/359357
Log:
Use uint32_t where it is expected to be used. No functional change.
MFC after:1 week
Modified:
head/sys/netinet/sctp_pcb.h
Author: tuexen
Date: Wed Mar 25 18:14:12 2020
New Revision: 359305
URL: https://svnweb.freebsd.org/changeset/base/359305
Log:
Improve consistency in debug output.
MFC after:1 week
Modified:
head/sys/netinet/sctputil.c
Modified: head/sys/netinet/sctputil.c
Author: tuexen
Date: Wed Mar 25 18:20:37 2020
New Revision: 359306
URL: https://svnweb.freebsd.org/changeset/base/359306
Log:
Remove an optimization, which was incorrect a couple of times and
therefore doesn't seem worth to be there.
In this case COOKIE where not retransmitted anymore, when
Author: tuexen
Date: Wed Mar 25 15:29:01 2020
New Revision: 359302
URL: https://svnweb.freebsd.org/changeset/base/359302
Log:
Revert https://svnweb.freebsd.org/changeset/base/357829
This introduces a regression reported by koobs@ when running a pyhton
test suite on a loaded system.
Author: tuexen
Date: Wed Mar 25 13:19:41 2020
New Revision: 359301
URL: https://svnweb.freebsd.org/changeset/base/359301
Log:
Use consistent debug output.
MFC after:1 week
Modified:
head/sys/netinet/sctputil.c
Modified: head/sys/netinet/sctputil.c
Author: tuexen
Date: Wed Mar 25 13:18:37 2020
New Revision: 359300
URL: https://svnweb.freebsd.org/changeset/base/359300
Log:
Don't restore the vnet too early in error cases.
MFC after:1 week
Modified:
head/sys/netinet/sctputil.c
Modified: head/sys/netinet/sctputil.c
Author: tuexen
Date: Tue Mar 24 23:04:07 2020
New Revision: 359288
URL: https://svnweb.freebsd.org/changeset/base/359288
Log:
Only call panic when building with INVARIANTS.
MFC after:1 week
Modified:
head/sys/netinet/sctputil.c
Modified: head/sys/netinet/sctputil.c
Author: tuexen
Date: Tue Mar 24 22:44:36 2020
New Revision: 359287
URL: https://svnweb.freebsd.org/changeset/base/359287
Log:
Another cleanup of the timer code. Also be more pedantic about the
parameters of the timer start and stop routines. Several inconsistencies
have been fixed in
Author: tuexen
Date: Mon Mar 23 12:17:13 2020
New Revision: 359234
URL: https://svnweb.freebsd.org/changeset/base/359234
Log:
Cleanup the file and add two ASSERT variants for locks, which will be
used shortly.
MFC after:1 week
Modified:
head/sys/netinet/sctp_lock_bsd.h
Author: tuexen
Date: Sat Mar 21 16:12:19 2020
New Revision: 359195
URL: https://svnweb.freebsd.org/changeset/base/359195
Log:
More timer cleanups, no functional change.
MFC after:1 week
Modified:
head/sys/netinet/sctp_asconf.c
head/sys/netinet/sctp_input.c
Author: tuexen
Date: Fri Mar 20 14:49:44 2020
New Revision: 359162
URL: https://svnweb.freebsd.org/changeset/base/359162
Log:
Remove a set, but unused variable.
MFC after:1 week
Modified:
head/sys/netinet/sctp_usrreq.c
Modified: head/sys/netinet/sctp_usrreq.c
Author: tuexen
Date: Thu Mar 19 23:07:52 2020
New Revision: 359156
URL: https://svnweb.freebsd.org/changeset/base/359156
Log:
A a missing NET_EPOCH_ENTER/NET_EPOCH_EXIT pair. This was affecting
implicit connection setups via sendmsg().
Reported by:
Author: tuexen
Date: Thu Mar 19 21:01:16 2020
New Revision: 359152
URL: https://svnweb.freebsd.org/changeset/base/359152
Log:
Consistently provide arguments for timer start and stop routines.
This is another step in cleaning up timer handling.
MFC after:1 week
Modified:
Author: tuexen
Date: Thu Mar 19 18:55:54 2020
New Revision: 359151
URL: https://svnweb.freebsd.org/changeset/base/359151
Log:
Cleanup the stream reset and asconf timer.
MFC after:1 week
Modified:
head/sys/netinet/sctp_timer.c
head/sys/netinet/sctp_timer.h
Author: tuexen
Date: Thu Mar 19 14:37:28 2020
New Revision: 359131
URL: https://svnweb.freebsd.org/changeset/base/359131
Log:
The MTU candidates MUST be a multiple of 4, so make them so.
MFC after:1 week
Modified:
head/sys/netinet/sctputil.c
Modified:
Author: tuexen
Date: Tue Mar 17 19:20:12 2020
New Revision: 359048
URL: https://svnweb.freebsd.org/changeset/base/359048
Log:
Handle the timers in a consistent sequence according to the definition
of the timer type.
Just a cleanup, no functional change intended.
MFC after:1
Author: tuexen
Date: Fri Mar 13 18:38:28 2020
New Revision: 358965
URL: https://svnweb.freebsd.org/changeset/base/358965
Log:
sendfile() does currently not support SCTP sockets.
Therefore, fail the call.
Reviewed by: markj@
MFC after:1 week
Differential Revision:
Author: tuexen
Date: Thu Mar 12 15:37:41 2020
New Revision: 358911
URL: https://svnweb.freebsd.org/changeset/base/358911
Log:
Use KMOD_TCPSTAT_INC instead of TCPSTAT_INC for RACK and BBR, since
these are kernel modules. Also add a KMOD_TCPSTAT_ADD and use that
instead of TCPSTAT_ADD.
Author: tuexen
Date: Wed Mar 4 16:41:25 2020
New Revision: 358621
URL: https://svnweb.freebsd.org/changeset/base/358621
Log:
When using automatically generated flow labels and using TCP SYN
cookies, use the same flow label for the segments sent during the
handshake and after the handshake.
Author: tuexen
Date: Wed Mar 4 12:22:53 2020
New Revision: 358614
URL: https://svnweb.freebsd.org/changeset/base/358614
Log:
Don't send an uninitilised traffic class in the IPv6 header, when
sending a TCP segment from the TCP SYN cache (like a SYN-ACK).
This fix initialises it to zero.
Author: tuexen
Date: Sun Mar 1 12:34:27 2020
New Revision: 358482
URL: https://svnweb.freebsd.org/changeset/base/358482
Log:
Remove stale definitions. The removed definitions are not used right
now and are incompatible with the correct ones in RFC 3168.
Submitted by: Richard
Author: tuexen
Date: Thu Feb 20 15:37:44 2020
New Revision: 358169
URL: https://svnweb.freebsd.org/changeset/base/358169
Log:
Remove an unused timer type.
MFC after:1 week
Modified:
head/sys/netinet/sctp_constants.h
Modified: head/sys/netinet/sctp_constants.h
Author: tuexen
Date: Tue Feb 18 21:25:17 2020
New Revision: 358083
URL: https://svnweb.freebsd.org/changeset/base/358083
Log:
Epochify SCTP.
Modified:
head/sys/netinet/sctp_os_bsd.h
head/sys/netinet/sctp_output.c
head/sys/netinet/sctp_usrreq.c
head/sys/netinet/sctputil.c
Author: tuexen
Date: Tue Feb 18 19:41:55 2020
New Revision: 358080
URL: https://svnweb.freebsd.org/changeset/base/358080
Log:
Remove unused function.
Modified:
head/sys/netinet/sctp_bsd_addr.c
Modified: head/sys/netinet/sctp_bsd_addr.c
Author: tuexen
Date: Mon Feb 17 18:05:03 2020
New Revision: 358028
URL: https://svnweb.freebsd.org/changeset/base/358028
Log:
Fix the non-default stream schedulers such that do not interleave
user messages when it is now allowed.
Thanks to Christian Wright for reporting the issue for the
Author: tuexen
Date: Mon Feb 17 14:54:21 2020
New Revision: 358023
URL: https://svnweb.freebsd.org/changeset/base/358023
Log:
Don't use uninitialised stack memory if the sysctl variable
net.inet.tcp.hostcache.enable is set to 0.
The bug resulted in using possibly a too small MSS value or
Author: tuexen
Date: Thu Feb 13 15:14:46 2020
New Revision: 357858
URL: https://svnweb.freebsd.org/changeset/base/357858
Log:
sack_newdata and snd_recover hold the same value. Therefore, use only
a single instance: use snd_recover also where sack_newdata was used.
Submitted by:
Author: tuexen
Date: Thu Feb 13 13:58:34 2020
New Revision: 357856
URL: https://svnweb.freebsd.org/changeset/base/357856
Log:
Whitespace cleanup. No functional change.
Sponsored by: Netflix, Inc.
Modified:
head/sys/netinet/tcp_log_buf.h
Modified: head/sys/netinet/tcp_log_buf.h
Author: tuexen
Date: Wed Feb 12 17:05:10 2020
New Revision: 357830
URL: https://svnweb.freebsd.org/changeset/base/357830
Log:
Don't panic under INVARIANTS when we can't allocate memory for storing
a vtag in time wait.
This issue was found by running syzkaller.
MFC after:1
Author: tuexen
Date: Wed Feb 12 17:02:15 2020
New Revision: 357829
URL: https://svnweb.freebsd.org/changeset/base/357829
Log:
Mark the socket as disconnected when freeing the association the first
time.
This issue was found by running syzkaller.
MFC after:1 week
Modified:
> On 11. Feb 2020, at 20:52, Gleb Smirnoff wrote:
>
> Michael,
>
> On Tue, Feb 11, 2020 at 08:38:21PM +0100, Michael Tuexen wrote:
> M> I can revert it and get it working in a different way. However, the
> M> networking code uses ints for booleans in a lot of
Author: tuexen
Date: Tue Feb 11 20:02:20 2020
New Revision: 357775
URL: https://svnweb.freebsd.org/changeset/base/357775
Log:
Revert https://svnweb.freebsd.org/changeset/base/357761
This was suggested by cem@
Modified:
head/sys/netinet/sctp_pcb.c
head/sys/netinet/sctputil.c
> On 11. Feb 2020, at 19:18, Conrad Meyer wrote:
>
> Hi Michael,
>
> On Tue, Feb 11, 2020 at 6:00 AM Michael Tuexen wrote:
>>
>> Author: tuexen
>> Date: Tue Feb 11 14:00:27 2020
>> New Revision: 357761
>> URL: https://svnweb.freebsd.org/changeset/b
Author: tuexen
Date: Tue Feb 11 18:15:57 2020
New Revision: 357768
URL: https://svnweb.freebsd.org/changeset/base/357768
Log:
Don't start an SCTP timer using a net, which has been removed.
Submitted by: Taylor Brandstetter
MFC after:1 week
Modified:
Author: tuexen
Date: Tue Feb 11 14:00:27 2020
New Revision: 357761
URL: https://svnweb.freebsd.org/changeset/base/357761
Log:
Use an int instead of a bool variable, since bool is not supported
on all platforms the stack is running on in userland.
Modified:
head/sys/netinet/sctp_pcb.c
Author: tuexen
Date: Sun Feb 9 22:40:05 2020
New Revision: 357708
URL: https://svnweb.freebsd.org/changeset/base/357708
Log:
Stop the PMTU and HB timer when removing a net, not when freeing it.
Submitted by: Taylor Brandstetter
MFC after:1 week
Modified:
Author: tuexen
Date: Sun Feb 9 22:05:41 2020
New Revision: 357705
URL: https://svnweb.freebsd.org/changeset/base/357705
Log:
Cleanup timer handling.
Submitted by: Taylor Brandstetter
MFC after:1 week
Modified:
head/sys/netinet/sctp_indata.c
head/sys/netinet/sctp_output.c
> On 6. Feb 2020, at 21:51, Jeff Roberson wrote:
>
> Author: jeff
> Date: Thu Feb 6 20:51:46 2020
> New Revision: 357641
> URL: https://svnweb.freebsd.org/changeset/base/357641
>
> Log:
> Fix a race in smr_advance() that could result in unnecessary poll calls.
>
> This was relatively
Author: tuexen
Date: Tue Feb 4 14:01:07 2020
New Revision: 357501
URL: https://svnweb.freebsd.org/changeset/base/357501
Log:
Remove unused timer.
Submitted by: Taylor Brandstetter
Modified:
head/sys/netinet/sctp_pcb.c
head/sys/netinet/sctp_structs.h
Author: tuexen
Date: Tue Feb 4 12:34:16 2020
New Revision: 357500
URL: https://svnweb.freebsd.org/changeset/base/357500
Log:
Improve numbering of debug information.
Submitted by: Taylor Brandstetter
MFC after:1 week
Modified:
head/sys/netinet/sctp_constants.h
Author: tuexen
Date: Tue Jan 28 10:09:05 2020
New Revision: 357197
URL: https://svnweb.freebsd.org/changeset/base/357197
Log:
Fix build issues for the userland stack on 32-bit platforms.
Reported by: Felix Weinrank
MFC after:1 week
Modified:
Author: tuexen
Date: Sat Jan 25 13:45:10 2020
New Revision: 357116
URL: https://svnweb.freebsd.org/changeset/base/357116
Log:
Sending CWR after an RTO is according to RFC 3168 generally required
and not only for the DCTCP congestion control.
Submitted by: Richard Scheffenegger
Author: tuexen
Date: Sat Jan 25 13:34:29 2020
New Revision: 357115
URL: https://svnweb.freebsd.org/changeset/base/357115
Log:
Don't set the ECT codepoint on retransmitted packets during SACK loss
recovery. This is required by RFC 3168.
Submitted by: Richard Scheffenegger
Author: tuexen
Date: Sat Jan 25 13:11:14 2020
New Revision: 357114
URL: https://svnweb.freebsd.org/changeset/base/357114
Log:
As a TCP client only enable ECN when the corresponding sysctl variable
indicates that ECN should be negotiated for the client side.
Submitted by: Richard
Author: tuexen
Date: Fri Jan 24 22:50:23 2020
New Revision: 357101
URL: https://svnweb.freebsd.org/changeset/base/357101
Log:
Don't delay the ACK for a TCP segment with the CWR flag set.
This allows the data sender to increase the CWND faster.
Submitted by: Richard Scheffenegger
Author: tuexen
Date: Fri Jan 24 22:37:53 2020
New Revision: 357100
URL: https://svnweb.freebsd.org/changeset/base/357100
Log:
The server side of TCP fast open relies on the delayed ACK timer to allow
including user data in the SYN-ACK. When DSACK support was added in
r347382, an immediate
> On 22. Jan 2020, at 18:19, Gleb Smirnoff wrote:
>
> Author: glebius
> Date: Wed Jan 22 17:19:53 2020
> New Revision: 356989
> URL: https://svnweb.freebsd.org/changeset/base/356989
>
> Log:
> Plug possible calls into ip6?_output() without network epoch from SCTP
> bluntly adding epoch
Author: tuexen
Date: Thu Jan 16 17:15:06 2020
New Revision: 356796
URL: https://svnweb.freebsd.org/changeset/base/356796
Log:
Remove debug code not needed anymore.
Submitted by: Richard Scheffenegger
Reviewed by: tuexen@
MFC after:1 week
Differential
Author: tuexen
Date: Sun Jan 12 17:52:32 2020
New Revision: 356663
URL: https://svnweb.freebsd.org/changeset/base/356663
Log:
Fix race when accepting TCP connections.
When expanding a SYN-cache entry to a socket/inp a two step approach was
taken:
1) The local address was filled in,
Author: tuexen
Date: Sun Jan 12 15:45:27 2020
New Revision: 356660
URL: https://svnweb.freebsd.org/changeset/base/356660
Log:
Fix division by zero issue.
Thanks to Stas Denisov for reporting the issue for the userland stack
and providing a fix.
MFC after:3 days
Author: tuexen
Date: Sun Jan 5 14:08:01 2020
New Revision: 356378
URL: https://svnweb.freebsd.org/changeset/base/356378
Log:
Don't make the sendall iterator as being up if it could not be started.
MFC after:1 week
Modified:
head/sys/netinet/sctp_output.c
Modified:
Author: tuexen
Date: Sun Jan 5 14:06:40 2020
New Revision: 356377
URL: https://svnweb.freebsd.org/changeset/base/356377
Log:
Return -1 consistently if an error occurs.
MFC after:1 week
Modified:
head/sys/netinet/sctp_pcb.c
Modified: head/sys/netinet/sctp_pcb.c
Author: tuexen
Date: Sun Jan 5 13:56:32 2020
New Revision: 356376
URL: https://svnweb.freebsd.org/changeset/base/356376
Log:
Ensure that we don't miss a trigger for kicking off the SCTP iterator.
Reported by: nwhitehorn@
MFC after:1 week
Modified:
Author: tuexen
Date: Sat Jan 4 20:33:12 2020
New Revision: 356357
URL: https://svnweb.freebsd.org/changeset/base/356357
Log:
Make the message size limit used for SCTP_SENDALL configurable via
a sysctl variable instead of a compiled in constant.
This is based on a patch provided by
Author: tuexen
Date: Thu Jan 2 14:04:16 2020
New Revision: 356271
URL: https://svnweb.freebsd.org/changeset/base/356271
Log:
Remove empty line which was added in r356270 by accident.
MFC after:1 week
Modified:
head/sys/netinet/sctp.h
Modified: head/sys/netinet/sctp.h
Author: tuexen
Date: Thu Jan 2 13:55:10 2020
New Revision: 356270
URL: https://svnweb.freebsd.org/changeset/base/356270
Log:
Improve input validation of the spp_pathmtu field in the
SCTP_PEER_ADDR_PARAMS socket option. The code in the stack assumes
sane values for the MTU.
This issue
Author: tuexen
Date: Tue Dec 31 16:15:47 2019
New Revision: 356235
URL: https://svnweb.freebsd.org/changeset/base/356235
Log:
Fix delayed ACK generation for DCTCP.
Submitted by: Richard Scheffenegger
Reviewed by: che...@netapp.com, rgrimes@, tuexen@
Differential
Author: tuexen
Date: Tue Dec 31 14:32:48 2019
New Revision: 356227
URL: https://svnweb.freebsd.org/changeset/base/356227
Log:
Add flags for upcoming patches related to improved ECN handling.
No functional change.
Submitted by: Richard Scheffenegger
Reviewed by: rgrimes@,
Author: tuexen
Date: Tue Dec 31 14:23:52 2019
New Revision: 356226
URL: https://svnweb.freebsd.org/changeset/base/356226
Log:
Clear the flag indicating that the last received packet was marked CE also
in the case where a packet not marked was received.
Submitted by: Richard
Author: tuexen
Date: Tue Dec 31 12:29:01 2019
New Revision: 356224
URL: https://svnweb.freebsd.org/changeset/base/356224
Log:
Add curly braces missed in https://svnweb.freebsd.org/changeset/base/354773
Sponsored by: Netflix, Inc.
CID: 1407649
Modified:
Author: tuexen
Date: Fri Dec 20 15:25:08 2019
New Revision: 355931
URL: https://svnweb.freebsd.org/changeset/base/355931
Log:
Improve input validation for some parameters having a too small
reported length.
Thanks to Natalie Silvanovich from Google for finding one of these
issues in
Author: tuexen
Date: Sun Dec 1 21:01:33 2019
New Revision: 355273
URL: https://svnweb.freebsd.org/changeset/base/355273
Log:
Move all ECN related flags from the flags to the flags2 field.
This allows adding more ECN related flags in the future.
No functional change intended.
Submitted
Author: tuexen
Date: Sun Dec 1 20:45:48 2019
New Revision: 355272
URL: https://svnweb.freebsd.org/changeset/base/355272
Log:
Make the TF_* flags easier readable by humans by adding leading zeroes
to make them aligned.
Submitted by: Richard Scheffenegger
Reviewed by:
Author: tuexen
Date: Sun Dec 1 20:35:41 2019
New Revision: 355269
URL: https://svnweb.freebsd.org/changeset/base/355269
Log:
This is an initial step in implementing the new congestion window
validation as specified in RFC 7661.
Submitted by: Richard Scheffenegger
Reviewed by:
Author: tuexen
Date: Sun Dec 1 19:20:28 2019
New Revision: 355268
URL: https://svnweb.freebsd.org/changeset/base/355268
Log:
Add a description for the TCP sysctl variable rfc6675_pipe.
It was introduced by r290122, but no documentation was provided.
This is taken from
101 - 200 of 1306 matches
Mail list logo