| The code was too complicated, if p > 0 in ccid3_hc_tx_no_feedback_timer the | timestamp was being obtained to be passed to ccid3_hc_tx_update_x, where only | if p > 0 the timestamp was needed, so just leave it to ccid3_hc_tx_update_x to | obtain the timestamp if needed. The `too complicated' is there for a reason, it served to reduce the number of timestamps per packet. With your patch, the code is now back at taking two timestamps per each received packet: * first a timestamp is taken for the RTT sample * whenever p == 0 (normal condition without loss), the second timestamp is now taken in ccid3_hc_tx_update_x() Since this code is in the receive path, it is executed for each received packet, i.e. two timestamps per each incoming packet. The original code recycled the first timestamp; the timestamp in no_feedback_timer() (which is only executed in exception cases, when there is no feedback for over 4 RTTs) was taken only when absolutely necessary, if two `if' conditions held. But this is not a big deal - I will send a patch shortly. - 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
