After a bit more testing, I found that this one has one bug.

|  @@ -593,22 +591,21 @@ EXPORT_SYMBOL_GPL(dccp_rcv_state_process);
|    * @t_recv: receive timestamp of packet with timestamp echo
|    * @t_hist: packet history timestamp or NULL
|    */
|  -u32 dccp_sample_rtt(struct sock *sk, struct timeval *t_recv,
|  -                                 struct timeval *t_hist)
|  +u32 dccp_sample_rtt(struct sock *sk, ktime_t t_recv, ktime_t *t_hist)
|   {
|       struct dccp_sock *dp = dccp_sk(sk);
|       struct dccp_options_received *or = &dp->dccps_options_received;
|  -    suseconds_t delta;
|  +    s64 delta;
|   
|       if (t_hist == NULL) {
|               if (!or->dccpor_timestamp_echo) {
|                       DCCP_WARN("packet without timestamp echo\n");
|                       return DCCP_SANE_RTT_MAX;
|               }
|  -            timeval_sub_usecs(t_recv, or->dccpor_timestamp_echo * 10);
|  -            delta = timeval_usecs(t_recv);
|  +            ktime_sub_us(t_recv, or->dccpor_timestamp_echo * 10);
|  +            delta = ktime_to_us(t_recv);
==> It should read
    t_recv = ktime_sub_us(t_recv, or->dccpor_timestamp_echo * 10);
    the statement is otherwise void.

-
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