Page MenuHomeFreeBSD

rscheff_gmx.at (Richard Scheffenegger)
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 18 2018, 9:44 PM (56 w, 4 d)

Recent Activity

Yesterday

rscheff_gmx.at created D22438: Improve NewReno after_idle ssthresh per RFC2861 Sec.2.
Mon, Nov 18, 5:28 PM
rscheff_gmx.at created D22436: Add access to TOS(ECN bits) to the TCP syncache.
Mon, Nov 18, 3:07 AM

Sun, Nov 17

rscheff_gmx.at added a comment to D22429: Addition of flag bit definitions in preparation of AccECN.

Something really strange here with this diff

Sun, Nov 17, 8:47 AM
rscheff_gmx.at updated the diff for D22429: Addition of flag bit definitions in preparation of AccECN.
  • minimizing delta vs. HEAD (see D22428)
Sun, Nov 17, 6:30 AM
rscheff_gmx.at created D22429: Addition of flag bit definitions in preparation of AccECN.
Sun, Nov 17, 6:24 AM
rscheff_gmx.at created D22428: Editorial change to enhance readability of the TF_* bit constants.
Sun, Nov 17, 5:39 AM
rscheff_gmx.at updated the summary of D22426: Add access to TOS(ECN bits) byte in TCP RACK.
Sun, Nov 17, 4:18 AM
rscheff_gmx.at created D22426: Add access to TOS(ECN bits) byte in TCP RACK.
Sun, Nov 17, 3:36 AM

Sat, Nov 16

rscheff_gmx.at added a reviewer for D19118: Add Boundary and Overflow checks in Cubic formulas: tuexen.
Sat, Nov 16, 3:50 AM
rscheff_gmx.at added a comment to D21798: Restrict cwnd growth on app-limited flows.

Should be ready to land now.

Sat, Nov 16, 3:34 AM
rscheff_gmx.at updated the diff for D21798: Restrict cwnd growth on app-limited flows.
  • minor typo in man
  • add sysctl variable properly
  • bump man page timestamp
Sat, Nov 16, 3:32 AM

Oct 14 2019

rscheff_gmx.at updated the diff for D21798: Restrict cwnd growth on app-limited flows.
  • adding new sysctl to control NewCWV related behavior, disabled by default. extend man page
Oct 14 2019, 11:06 AM

Oct 10 2019

rscheff_gmx.at added a comment to D21798: Restrict cwnd growth on app-limited flows.

from the transport call:

Oct 10 2019, 2:11 PM

Oct 7 2019

rscheff_gmx.at added a comment to D21619: Update SIFTR man page, and minor trailing whitespace nit..

Removed the filter logic changes as they were logically equivalent with the already implemented logic - could not figure out why I though this may not work properly (possibly not clearing the siftr logs properly when checking).

Oct 7 2019, 9:24 AM
rscheff_gmx.at retitled D21619: Update SIFTR man page, and minor trailing whitespace nit. from Update SIFTR man page, and make siftr port_filter work as intended to Update SIFTR man page, and minor trailing whitespace nit..
Oct 7 2019, 9:22 AM
rscheff_gmx.at updated the diff for D21619: Update SIFTR man page, and minor trailing whitespace nit..
  • update siftr man page timestamp and fix trailing whitespace
Oct 7 2019, 9:18 AM
rscheff_gmx.at added a comment to D21907: Move request window scaling computation to tcp_output().

Looking through the history of that comment (to move this to tcp_output), it seems that the context was something different. In rS166403 that comment was added, when the advertised receive window scale was simply adjusted up to have a minimum granularity of just above 1 MSS. But only part of that code was kept subsequently... With other words, that comment to have this section moved appeared under a different context, and seems to be a left-over without merit not (thus remove that line from the comment instead, perhaps)?

Oct 7 2019, 8:53 AM

Oct 1 2019

rscheff_gmx.at added a comment to D21840: Make TCP_INFO report TFO.

TCP_INFO is not portable afaik; however, I wonder why the linux varian has a flag for "SYN_DATA" (which may get delivered to the appliacation if the TFO is present), but no flag for TFO, while this change is signaling the presence of TFO, but no SYN_DATA... Just curious...

Oct 1 2019, 1:21 PM

Sep 26 2019

rscheff_gmx.at added inline comments to D21665: Don't send TCP segments when the IP header chain and the TCP header doesn't fit in a packet.
Sep 26 2019, 2:18 PM
rscheff_gmx.at added a comment to D21798: Restrict cwnd growth on app-limited flows.

The uperf profile to simulate an application transitioning from app-limited into bulk transfer mode.

Sep 26 2019, 10:07 AM
rscheff_gmx.at created D21798: Restrict cwnd growth on app-limited flows.
Sep 26 2019, 10:01 AM

Sep 12 2019

rscheff_gmx.at updated the diff for D21619: Update SIFTR man page, and minor trailing whitespace nit..
  • bumping .Dd date
Sep 12 2019, 4:09 PM
rscheff_gmx.at updated the diff for D21619: Update SIFTR man page, and minor trailing whitespace nit..
  • man page typos
Sep 12 2019, 3:46 PM
rscheff_gmx.at added a reviewer for D21619: Update SIFTR man page, and minor trailing whitespace nit.: bcr.
Sep 12 2019, 3:20 PM
rscheff_gmx.at created D21619: Update SIFTR man page, and minor trailing whitespace nit..
Sep 12 2019, 3:17 PM
rscheff_gmx.at abandoned D19234: fix cwnd increase for empty acks when not doing ABC.

@tuexen correctly pointed out, that newreno_ack_received is only called by the wrapper cc_ack_received during input processing, if an ACK actually moves snd_una right (acks new data). This patch would therefore only increase the codepath length, but not functionally change anything.

Sep 12 2019, 9:40 AM

Sep 9 2019

rscheff_gmx.at added a comment to D21567: Improve DSACK computation.

Checked this patch with various "pure FIN after hole" and "data FIN after hole" combinations; the FIN bit becomes part of the SACK (and DSACK) blocks, as it should be when considering that FIN is part of the sequence number space.

I don't think the above is true with the code right now. But changing it that the FIN is dealt with in SACK/DSACK is a separate commit, I think,

Sep 9 2019, 2:31 PM
rscheff_gmx.at accepted D21567: Improve DSACK computation.
Sep 9 2019, 11:51 AM
rscheff_gmx.at added a comment to D21567: Improve DSACK computation.

Checked this patch with various "pure FIN after hole" and "data FIN after hole" combinations; the FIN bit becomes part of the SACK (and DSACK) blocks, as it should be when considering that FIN is part of the sequence number space.

Sep 9 2019, 10:49 AM

Aug 27 2019

rscheff_gmx.at added inline comments to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
Aug 27 2019, 11:08 AM

Aug 8 2019

rscheff_gmx.at added inline comments to D19047: TCP Dynamic Burst Limit.
Aug 8 2019, 11:40 PM

Aug 7 2019

rscheff_gmx.at added inline comments to D21179: Add header definition for RFC4340, Datagram Congestion Control Protocol.
Aug 7 2019, 6:30 PM
rscheff_gmx.at added inline comments to D21179: Add header definition for RFC4340, Datagram Congestion Control Protocol.
Aug 7 2019, 12:09 PM
D21178: Rename IPPROTO 33 from SEP to DCCP is now accepted and ready to land.
Aug 7 2019, 11:46 AM

Aug 3 2019

rscheff_gmx.at added a comment to D21038: DSACK patch to conform fully with RFC2883.

synced up my branch to add the dsack change on drop in rack_bbr_common, as pointed out. Thx!

Aug 3 2019, 8:12 PM
rscheff_gmx.at added a comment to D21038: DSACK patch to conform fully with RFC2883.
  • add dsack drop block in rack_bbr_common
  • lint - breaking some lines longer than 80 char
Aug 3 2019, 8:10 PM

Jul 30 2019

rscheff_gmx.at created D21117: Reduce size of t_rttupdated in tcpcb.
Jul 30 2019, 5:16 PM

Jul 29 2019

rscheff_gmx.at added inline comments to D21038: DSACK patch to conform fully with RFC2883.
Jul 29 2019, 6:22 PM
rscheff_gmx.at added a comment to D18811: nuke sack_newdata.

Randall, since you seem to be the most knowledgeable person when it comes to tcpcb optimization, I think this one should be reviewed by you for any potential cache line implications.

Jul 29 2019, 6:09 PM
rscheff_gmx.at added a reviewer for D18811: nuke sack_newdata: rrs. rscheff_gmx.at removed 1 blocking reviewer(s) for D18811: nuke sack_newdata: transport.
Jul 29 2019, 6:08 PM
rscheff_gmx.at added a comment to D21038: DSACK patch to conform fully with RFC2883.

This tgz file contains packetdrill tests for dsack, with the non-duplicate packets getting delivered in various orderings, before the overlapping DSACK is sent.

Jul 29 2019, 6:04 PM
rscheff_gmx.at added a comment to D19118: Add Boundary and Overflow checks in Cubic formulas.

I learned that the SCE project group is looking into qualification of TCP Cubic; As R. Grimes is part of that group, adding him as a reviewer, to validate the functional changes to Cubic are sound.

Jul 29 2019, 5:57 PM
rscheff_gmx.at added a reviewer for D19118: Add Boundary and Overflow checks in Cubic formulas: rgrimes.
Jul 29 2019, 5:57 PM
rscheff_gmx.at added a comment to D18954: Implement Cubic-specific After-Idle reaction.

I learned that the SCE project group is looking into qualification of TCP Cubic; As R. Grimes is part of that group, adding him as a reviewer, to validate the functional changes to Cubic are sound.

Jul 29 2019, 5:55 PM
rscheff_gmx.at added a reviewer for D18954: Implement Cubic-specific After-Idle reaction: rgrimes.
Jul 29 2019, 5:55 PM
rscheff_gmx.at added a comment to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
  • bump dctcp reference to RFC8257 in the man page
Jul 29 2019, 3:32 AM

Jul 28 2019

rscheff_gmx.at added a comment to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
  • fixing two nits in the man page, and setting dctcp alpha default to conservative 1024
Jul 28 2019, 9:27 PM
rscheff_gmx.at added inline comments to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
Jul 28 2019, 4:32 PM
rscheff_gmx.at added a comment to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
  • cleaned up sysctl handlers, fixed sysctl name in man
Jul 28 2019, 4:29 PM
rscheff_gmx.at added a comment to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.

Addressed all comments so far.

Jul 28 2019, 12:48 PM
rscheff_gmx.at added a comment to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
  • remove .Pp from man page
  • fixing lint issues of man update
Jul 28 2019, 12:41 PM

Jul 27 2019

rscheff_gmx.at added a comment to D21011: Functional implementation of Accurate ECN in FreeBSD.

Adjust IW if SYN or SYN,ACK were delivered with [CE] set, as indicated by the SYN,ACK or final ACK of the 3WHS.

Jul 27 2019, 3:41 PM
rscheff_gmx.at added a comment to D21011: Functional implementation of Accurate ECN in FreeBSD.
  • adding AccECN settings of net.inet.tcp.ecn.enable to man page
Jul 27 2019, 3:37 PM
rscheff_gmx.at added a comment to D19118: Add Boundary and Overflow checks in Cubic formulas.
  • put magic number as #define
Jul 27 2019, 12:55 PM
rscheff_gmx.at added a comment to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
  • parathentis in #define
  • updating man page
Jul 27 2019, 11:00 AM
rscheff_gmx.at added a comment to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.

Regarding the man page and defaults:

Jul 27 2019, 10:53 AM

Jul 26 2019

rscheff_gmx.at added a comment to D21011: Functional implementation of Accurate ECN in FreeBSD.
  • adding CC reaction on <SYN(,ACK)> that are CE marked
  • limiting initwin to 2mss on a [CE] delivered for <SYN> or <SYN,ACK>
  • Set the initial window explicitly to 2*MSS
Jul 26 2019, 3:41 PM
rscheff_gmx.at added a comment to D21011: Functional implementation of Accurate ECN in FreeBSD.
  • adding debug printout, fixing AccECN final ACK signaling
  • add ipv6 version check and #ifdef on ipv6 in t4_listen
  • final ACK of 3WHS working, proper initial s_cep, r_cep values
  • move all ECN related tcpcb flags to flags2
  • whitespace, remove debug logging code
  • missed trailing space
  • fix ipv6 version check
Jul 26 2019, 2:43 AM

Jul 24 2019

rscheff_gmx.at added a comment to D21011: Functional implementation of Accurate ECN in FreeBSD.
  • adding TH_AE to tcp_log_addr string
  • fixing initial cep couter on [ce]SYN to 6;
  • removal of debug code
  • remove debug code
Jul 24 2019, 6:33 PM
rscheff_gmx.at added a comment to D21038: DSACK patch to conform fully with RFC2883.
  • fix indentation
Jul 24 2019, 1:10 PM
rscheff_gmx.at added a comment to D21038: DSACK patch to conform fully with RFC2883.

Added DSACK to RACK again. Validated by running packet drill DSACK testsuite against the rack stack:

Jul 24 2019, 2:28 AM
rscheff_gmx.at added a comment to D21038: DSACK patch to conform fully with RFC2883.

update rack stack with dsack (again)

Jul 24 2019, 2:23 AM

Jul 23 2019

rscheff_gmx.at added a comment to D21038: DSACK patch to conform fully with RFC2883.

Hopefully the tab/space mixup is fixed now too.

Jul 23 2019, 9:46 PM
rscheff_gmx.at added a comment to D21038: DSACK patch to conform fully with RFC2883.
  • fix whitespace mixup and missing bracket
Jul 23 2019, 9:45 PM
rscheff_gmx.at created D21038: DSACK patch to conform fully with RFC2883.
Jul 23 2019, 5:45 PM

Jul 21 2019

rscheff_gmx.at created D21011: Functional implementation of Accurate ECN in FreeBSD.
Jul 21 2019, 12:08 PM

Jul 11 2019

D20908: Fixing the DSACK code to properly clean up after sending a DSACK. is now accepted and ready to land.

Sorry for the oversight; Indeed I tested this with continous data from the receiver only, not when the data direction changes :(

Jul 11 2019, 4:18 AM

Jun 17 2019

rscheff_gmx.at added inline comments to D20550: Fix bugs in DCTCP on TCP events.
Jun 17 2019, 2:36 PM
rscheff_gmx.at added a comment to D20550: Fix bugs in DCTCP on TCP events.
  • - missed rename of variable
Jun 17 2019, 2:32 PM
rscheff_gmx.at added inline comments to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
Jun 17 2019, 2:20 PM
rscheff_gmx.at updated the summary of D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
  • fix comment to be in-line with RFC
Jun 17 2019, 2:19 PM

Jun 11 2019

rscheff_gmx.at added inline comments to D20550: Fix bugs in DCTCP on TCP events.
Jun 11 2019, 8:37 AM
rscheff_gmx.at added a comment to D20550: Fix bugs in DCTCP on TCP events.

Comment from Midori (original author of this code):

Jun 11 2019, 8:20 AM

Jun 7 2019

rscheff_gmx.at created D20550: Fix bugs in DCTCP on TCP events.
Jun 7 2019, 12:49 PM
rscheff_gmx.at added a comment to D20549: fixing sysctl interface for DCTCP and types of dctcp variables.

Note: DCTCP alpha is initially at zero - thus there will be no reduction in cwnd after the first window, only the 2nd window will have some response, if an elephant flow is tested.

Jun 7 2019, 12:08 PM
rscheff_gmx.at updated the summary of D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
Jun 7 2019, 12:07 PM
rscheff_gmx.at updated the diff for D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
  • aligning sysctl with linux for alpha
Jun 7 2019, 12:01 PM
rscheff_gmx.at added a reviewer for D20549: fixing sysctl interface for DCTCP and types of dctcp variables: transport.
Jun 7 2019, 11:56 AM
rscheff_gmx.at created D20549: fixing sysctl interface for DCTCP and types of dctcp variables.
Jun 7 2019, 11:56 AM

Jun 6 2019

rscheff_gmx.at added a comment to D19622: Fix panic in network stack due memory use after free in relation to fragmented packets.

@bz - once a packet (fragment in this case) has made it to it's destination, I would argue that you shouldn't needlessly drop that packet any more. (Host vs. Router behavior). An interface flapping frequently, and removing all enqueued fragments while doing this, would reduce the usability of such an interface from marginal to completely inoperable.

Jun 6 2019, 2:57 PM

Jun 5 2019

rscheff_gmx.at accepted D20372: Fix TCP RACK after adding DSACK support.

Looks good to me

Jun 5 2019, 10:38 AM

Apr 26 2019

rscheff_gmx.at updated the diff for D18811: nuke sack_newdata.
  • bump tcp_log_buf version
Apr 26 2019, 8:40 AM

Apr 25 2019

rscheff_gmx.at added a reviewer for D18954: Implement Cubic-specific After-Idle reaction: transport.
Apr 25 2019, 2:36 PM
rscheff_gmx.at added reviewers for D18985: New SACK (RFC6675): tuexen, lstewart, slavash, jtl.
Apr 25 2019, 2:28 PM
rscheff_gmx.at added a comment to D18811: nuke sack_newdata.

Removal of the variable in a cacheline > 4 is not that critical (to retain the alignment of tcpcb at that place still.)

Apr 25 2019, 2:19 PM
rscheff_gmx.at added a comment to D17614: RFC6582 - prevent cwnd to collapse down to 1 mss after exiting recovery.

Lawrence reviewed this during IETF104, Michael volunteered to follup up with the full commit process.

Apr 25 2019, 2:15 PM · transport

Mar 29 2019

rscheff_gmx.at added inline comments to D18624: improvements to support code for RFC6675.
Mar 29 2019, 9:45 AM
rscheff_gmx.at updated the diff for D18624: improvements to support code for RFC6675.
  • fixing curly bracket if branch
  • semantically moving check for new sack data to tcp_input
  • fixed wrong deref to TO struct
Mar 29 2019, 9:42 AM
rscheff_gmx.at updated the diff for D18624: improvements to support code for RFC6675.
  • fixing curly bracket if branch, semantically moving check for new sack data to tcp_input
Mar 29 2019, 9:01 AM
rscheff_gmx.at added a comment to D19334: Receiver Side DSACK (RFC2883) implementation.

Ready to land.

Mar 29 2019, 8:37 AM

Mar 27 2019

rscheff_gmx.at added a comment to D19334: Receiver Side DSACK (RFC2883) implementation.

Need to look into RACK stack and apply similar changes.

Mar 27 2019, 5:58 PM

Mar 25 2019

rscheff_gmx.at added a reviewer for D18892: Phase 2 to add Proportional Rate Reduction (RFC6937) to FreeBSD: hiren.
Mar 25 2019, 3:43 PM

Feb 27 2019

rscheff_gmx.at added a reviewer for D19118: Add Boundary and Overflow checks in Cubic formulas: chengc_netapp.com.
Feb 27 2019, 8:56 PM
rscheff_gmx.at added a reviewer for D18954: Implement Cubic-specific After-Idle reaction: chengc_netapp.com.
Feb 27 2019, 8:56 PM

Feb 24 2019

rscheff_gmx.at added a comment to D19334: Receiver Side DSACK (RFC2883) implementation.

packetdrill test script

Feb 24 2019, 9:51 PM
rscheff_gmx.at created D19334: Receiver Side DSACK (RFC2883) implementation.
Feb 24 2019, 9:46 PM
rscheff_gmx.at retitled D19329: address possible illegal SACK block after FIN when client keeps sending data from don't update rcv_nxt if socket can not receive. to address possible illegal SACK block after FIN when client keeps sending data.
Feb 24 2019, 2:06 PM
rscheff_gmx.at created D19329: address possible illegal SACK block after FIN when client keeps sending data.
Feb 24 2019, 1:52 PM

Feb 23 2019

rscheff_gmx.at added a comment to D18960: Patch to MFC TCP reassembly rewrite to stable/11.

Quick comment: DSACKs are not working in HEAD. Potentially, the tlenp is set to zero for the duplicate segment, so that tcp_update_sack_list is not called to build the DSACK entry to the next ACK. If this is the reason, this patch may break DSACK in BSD11

Feb 23 2019, 10:06 AM

Feb 18 2019

rscheff_gmx.at created D19234: fix cwnd increase for empty acks when not doing ABC.
Feb 18 2019, 10:15 AM