Re: [PATCH 01/16] math128: Introduce various 128bit primitives

From: Peter Zijlstra
Date: Fri Oct 26 2012 - 05:55:10 EST


On Fri, 2012-10-26 at 11:42 +0200, Ingo Molnar wrote:
> * Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
>
> > On Fri, 2012-10-26 at 11:24 +0200, Ingo Molnar wrote:
> >
> > > So can we control this by restricting the users and avoiding
> > > the overflow?
> > >
> > > A 2^64 result should be a *huge* amount of space already for
> > > just about anything.
> >
> > I _think_ something like: dl_runtime * dl_deadline < U64_MAX,
> > might do that. The question is, is this constraint usable?
> > Simplified that boils down to about 4 seconds each, which
> > sounds pretty much ok for most people -- but such statements
> > usually come back to bite you (640kb anybody...).
>
> We could constrain the precision, not the maximum value.
>
> Having a 4 seconds hard limit is one thing, only having 10 nsecs
> precision at 40 seconds is another.

That gets to be rather ugly I think.. for one it might surprise people,
secondly you get to have a bunch of conditionals and shifts in that code
path.

Personally I'd prefer to do the simple thing, esp. for a new interface.
So either do the hard limit or the u128 thing.

If we go with the hard limit, we can always address things when people
run into it and complain, at such a time we also have a better view of
people's uses and expectations methinks.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/