[Differential] [Request, 103 lines] D5173: Rework initial congestion window calculation.

[hiren (hiren panchasara)]

hiren created this revision.
hiren added a reviewer: lstewart.
hiren added a subscriber: transport.
Herald added a subscriber: imp.
Herald added a reviewer: transport.

REVISION SUMMARY
  Rework initial congestion window calculation.
  
  The patch is my first (barely compiling) attempt at:
  
  1. add ability to specify initcwnd in number of bytes
  2. move entire calculation into a separate function
  3. add a new sysctl to enable RFC6928 which is ON by default
  4. https://reviews.freebsd.org/D5124's fix included.
  
  Approach here is, user can specify initcwnd of choice in number of segments or bytes. If none is present, we default to RFC6928 and have initcwnd as 10 segments.
  
  AFAIK, this is not changing the default behavior of what we have in FreeBSD-head right now. OR at least, that is the plan.
  This change also makes enabling RFC3390 (to get initcwnd at 4 segments) more obvious/easier. (You just have to disable RFC6928.)
  
  Suggestions?
  
  (I need to remember to do any manpage updates with this.)

TEST PLAN
  I'll work on it once we have a general consensus on the approach.

REPOSITORY
  rS FreeBSD src repository

BRANCH
  rework_initcwnd

REVISION DETAIL
  https://reviews.freebsd.org/D5173

AFFECTED FILES
  sys/netinet/cc/cc_newreno.c
  sys/netinet/tcp_input.c
  sys/netinet/tcp_var.h

EMAIL PREFERENCES
  https://reviews.freebsd.org/settings/panel/emailpreferences/

To: hiren, lstewart, transport
Cc: imp, transport
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5173.12966.patch
Type: text/x-patch
Size: 5956 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-transport/attachments/20160203/2bd28047/attachment.bin>