[Tipc-discussion] A Few Comments on Draft TIPC Document

[<all...@wi...>]

Hi Jon:
 
I thought I'd pass along my feedback about the draft TIPC document, since
you indicated that you're accepting input.  Most of these are simple grammar
or spelling issues, but there are a few others that relate to content.
 
page 2, bullet 5
You might want to add the word "ordered" (or some equivalent term) to the
list of properties relating to data transfer.  (It may seem obvious, but I
noted that you explicitly mentioned this later in the document.)
 
page 14
The 2nd sentence for "Link" is a bit misleading/outdated, based on what you
told me on Monday.  I suggest the following replacement text: "A node pair
may be interconnected by up to 2 active links; additional standby links of
lower priority may also be present."
 
page 22
The last sentence of section 2.2.6 is garbled.  Maybe remove "and considered
advantageous"?
 
page 23
Text for section 2.2.9 is missing.  Will this be filled in later?
 
page 24, paragraph 2
Replace "losses" with "loses".
 
page 35, section 2.5.4, paragraph 2, line 2
Replace "exists" with "exist".
 
page 37, section 2.5.5, bullet 2
Replace "N message" with "N messages".
 
page 46, state Reset-Reset
Replace "will will" with "will" at start of 3rd sentence.
 
page 50, state Active-Standby
Replace "numeral" with "numeric".
 
page 52, section 2.8
Replace "support" with "supports".
 
page 53, paragraph 1
Replace "nameing" with "naming" on line 3.
 
page 53, paragraph 2
Wouldn't each node have 199 x 2 + 4 x 2 = 406 links to maintain?  Also, I
was under the impression that each node creates a virtual link to itself to
handle traffic between two of its own ports; am I mistaken here or would
that mean there are really 407 links per node in the example?
 
page 60, paragraph 1
Replace "sner" with "sender" on line 2, I think.
 
page 62
The two paragraphs following Figure 21 should really appear at the end of
section 3 (i.e. just before the start of section 3.1), rather than in
section 3.1.1, since this info applies to both section 3.1 and section 3.2.
Having it it section 3.1.1 confused me a fair bit the first time I read the
document.
 
page 64, 2nd paragraph of text
Replace "TIPC_COMM_ERROR" with "TIPC_CONN_ERROR" on line 4, I think.
 
 
That's all for now.
 
Regards,
Al Stephens
Member of Technical Staff
Wind River Systems