From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gerrit Renker Date: Sat, 16 Jun 2007 15:00:25 +0000 Subject: Re: [PATCH 1/3]: Inline for time delta Message-Id: <200706161600.25348@strip-the-willow> List-Id: References: <200706091932.15559@strip-the-willow> In-Reply-To: <200706091932.15559@strip-the-willow> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: dccp@vger.kernel.org Quoting Arnaldo Carvalho de Melo: | > I am going to change that in my tree as well, since I am busy updating with regard | > to most recent changes anyway. | | I just checked with Thomas Gleixner, the ktime_t guy and he is ok with | adding ktime_us_delta() to ktime.h, which I have in my tree already. Ok when that comes through, the patch can simply be dropped. | I'm now staring at this: |    /* set the nominal send time for the next following packet */ | -        timeval_add_usecs(&hctx->ccid3hctx_t_nom, hctx->ccid3hctx_t_ipi); | +       hctx->ccid3hctx_t_nom = ktime_add_ns(hctx->ccid3hctx_t_nom, | +                                            hctx->ccid3hctx_t_ipi * 1000); | | I'm thinking about just keeping t_ipi in nanoseconds to avoid doing | the multiplies back and forth, but haven't fully looked at the other | t_ipi uses, quick thoughts? Stupid idea? Please keep it for the moment. The packet scheduler needs an overhaul in any case, so the above IMO is the simplest way of aligning the old code with the new interface. Overhauling the packet scheduler will cost some work, but it is one of the next items that should be addressed when through with the current set of patches.