DCCP protocol discussions
 help / color / mirror / Atom feed
* Re: [PATCH 13/15] [DCCP] options: convert
@ 2007-08-21 14:58 Arnaldo Carvalho de Melo
  0 siblings, 0 replies; only message in thread
From: Arnaldo Carvalho de Melo @ 2007-08-21 14:58 UTC (permalink / raw)
  To: dccp

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-08-21 14:58 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-21 14:58 [PATCH 13/15] [DCCP] options: convert Arnaldo Carvalho de Melo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox