On 11/4/07, Gerrit Renker <gerrit@xxxxxxxxxxxxxx> wrote: > This comments out a problematic section comprising a half-finished algorithm: > > - The variable `ccid2hctx_ackloss' is never initialised to a value different from 0 and > hence in fact is a read-only constant. > - The `arsent' variable counts packets other than Acks (it is incremented for every packet), > and there is no test for Ack Loss. > - The concept of counting Acks as such leads to a complex calculation, and the calculation > at the moment is inconsistent with this concept. > The problem is that the number of Acks - rather than the number of windows - is counted, > which leads to a complex (cubic/quadratic) expression - this is not even implemented. > > In its current state, the commented-out algorithm interferes with normal processing by > changing Ack Ratio incorrectly, and at the wrong times. > > A new algorithm is necessary, which will not necessarily use the same variables as used by > the unfinished one; hence the old variables have been removed. > > Signed-off-by: Gerrit Renker <gerrit@xxxxxxxxxxxxxx> Acked-by: Ian McDonald <ian.mcdonald@xxxxxxxxxxx> - To unsubscribe from this list: send the line "unsubscribe dccp" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
