Skip to content
This repository has been archived by the owner on Aug 28, 2023. It is now read-only.

clarify the meaning of "application-limited" #139

Merged
merged 7 commits into from Jan 26, 2022

Commits on Jan 18, 2022

  1. clarify the meaning of "application-limited"

    Clarify the meaning of "application-limited" to explicitly
    mention that this can include slow sender applicaitons
    or slow receiver applications.
    
    Matt Olson notes in Issue NTAP#137:
    
      It seems like usually "app-limited" refers only to the local app, 
      but if the peer's receive window limits the connection's send 
      rate then we have the same problem where CWnd can grow
     arbitrarily. So, is "app-limited" supposed to also refer to the peer's
     receive window? If so, then this should probably be spelled out.
    
    So we should be explicit that application-limited can refer to sender
    or receiver apps that are slower than what cwnd allows.
    
    We should particularly clarify this because the text on this topic
    in 7661 is buggy, when it says:
    
       "application-limited period" for the time when the sender sends less
      than is allowed by the congestion or receiver windows.
    
    This RFC 7661 text is buggy because if the receiver window is
    smaller than the congestion window, and the sender sends
    an amount equal to the receiver window, then the flow is
    application-limited (it is limited by the receiver and receiver
    advertised window instead of cwnd).
    nealcardwell committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    d308319 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    1c0f006 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    ad37705 View commit details
    Browse the repository at this point in the history

Commits on Jan 19, 2022

  1. fix typo

    Co-authored-by: Vidhi Goel <goel.vidhi07@gmail.com>
    nealcardwell and goelvidhi committed Jan 19, 2022
    Configuration menu
    Copy the full SHA
    72d174d View commit details
    Browse the repository at this point in the history
  2. tighten up: "less than is allowed" -> "less than what is allowed"

    Co-authored-by: Vidhi Goel <goel.vidhi07@gmail.com>
    nealcardwell and goelvidhi committed Jan 19, 2022
    Configuration menu
    Copy the full SHA
    c1f8e58 View commit details
    Browse the repository at this point in the history

Commits on Jan 25, 2022

  1. use "and thus sends less data than what is allowed by"

    From @larseggert and @goelvidhi.
    
    Co-authored-by: Lars Eggert <lars@eggert.org>
    nealcardwell and larseggert committed Jan 25, 2022
    Configuration menu
    Copy the full SHA
    9727140 View commit details
    Browse the repository at this point in the history
  2. fix line length lint error from previous patch

    The latest patch caused a lint error:
    
    draft-ietf-tcpm-rfc8312bis.md:1017:81 MD013/line-length Line length [Expected: 80; Actual: 99]
    
    Fixed by wrapping:
    
    "and thus sends less data than what is allowed by
    the sender's congestion window."
    nealcardwell committed Jan 25, 2022
    Configuration menu
    Copy the full SHA
    82d96ba View commit details
    Browse the repository at this point in the history