Chuck, all,

Thanks much for the updates to the draft to reflect a working flow control mechanism. I think the spirit is correct, but the description is unclear, my comments below are an attempt to clarify the text. Otherwise, this is almost there! Thanks and apologies for the delay in reviewing.

Comments:

1/ “A peer tracks a few critical values for each connection.” —> “A sender tracks a few critical values for each connection.”

2/ For each of the variables described, replace “peer” with “sender” or receiver” as appropriate. Flow control functions require actions at the send side and at the receive side, and articulating which side is maintaining a variable or performing an action is very important to not introduce ambiguity and potential deadlocks.

3/ “Remote credits” —> I would suggest changing this to “Advertised Credits”, to avoid confusion about which endpoint uses this. 

4/ “The sender MUST NOT post this message if the sender's "Send message counter" is greater than the current "Remote credits" value.”
—> Two issues: (i) What is the “send message counter”? Is this supposed to be “Sent message count”? Or a new variable? (ii) each endpoint has its “Remote credits” value, which one does this sentence refer to?

5/ “A receiver MAY adjust its credit limit” —> “A receiver MAY adjust its advertised credit limit”

6/ “For instance, a peer may” —> “For instance, a receiver may”

- jana