Re: [PATCH 13/15] [DCCP] options: convert dccp_insert_option_timestamp to ktime_t

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Tue, Aug 21, 2007 at 02:25:27PM +0100, Gerrit Renker escreveu:
> Sorry this one has a bug also - it compresses a signed 64 bit quantity into a 32-bit unsigned
> value. When re-interpreted as microsecond value (in dccp_sample_rtt), this gives values which
> are way too small.
> 
> Maybe we should keep the dccp_epoch thing, but now with ktime_t, so as to avoid large microsecond values?

Argh, it seems so :-(
 
> In any case, the RTT sampling is not usable - it constantly produces overflow, since the timestamp
> option value is too small.

Reading code and RFC now...
 
- Arnaldo
-
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

[Index of Archives]     [Linux Kernel]     [IETF DCCP]     [Linux Networking]     [Git]     [Security]     [Linux Assembly]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux