On Tue, May 02, 2017 at 11:14 -0600, Todd C. Miller wrote: > On Tue, 02 May 2017 11:12:58 -0600, "Todd C. Miller" wrote: > > > On Tue, 02 May 2017 18:59:44 +0200, Mike Belopuhov wrote: > > > > > After switching the ph_timestamp to int64_t, the implementation > > > can be re-adjusted to use int64_t timestamps throughout. > > > > > > I think I've got all of the conversion right, but will double- > > > check everything before getting this in. > > > > Would it be worth converting codel_intervals from us to ns so you > > don't need to multiply by 1000 in so many places? > > Nevermind, I see that most of codel is in terms of us not ns, so > it is probably simplest to leave it that way for now. > > - todd
The trade off is basically the double the size of the table. It's a bit above 1.5Kb right now and if I convert everything to ns, it'll be just below 4K. Which is also OK I guess.
