Quoting Ian McDonald:
|  >  ==> However, I am not sure that the way  ccid3_hc_rx_update_li() is 
called is as
|  >      intentioned.
|  >
|  Not sure what you mean here. Can you explain?
The parameters of ccid3_hc_rx_update_li() are:

ccid3_hc_rx_update_li(struct sock *sk, u64 seq_loss, u8 win_loss)
{
        // ... 
        // The following is executed if hcrx->ccid3hcrx_li_hist is non-empty:

        /* calculate last interval length */
        seq_temp = dccp_delta_seqno(head->dccplih_seqno, seq_loss)

        /* Create the newest interval */
        entry->dccplih_seqno = seq_loss;
        entry->dccplih_interval = seq_temp;
        entry->dccplih_win_count = win_loss;
}

>From this it appears that seq_loss / win_loss are associated with _loss_.
However, in ccid3_hc_rx_detect_loss they are called as:


        ccid3_hc_rx_update_li(sk, hcrx->ccid3hcrx_seqno_nonloss,
                   hcrx->ccid3hcrx_ccval_nonloss)

which refers to the _nonloss_ variants. Loss or non_loss?
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to