Re: [PATCH] time: Avoid signed overflow in timekeeping_delta_to_ns()

2016-11-15 Thread John Stultz
On Tue, Nov 15, 2016 at 2:03 PM, John Stultz wrote: > On Tue, Nov 15, 2016 at 1:53 PM, Thomas Gleixner wrote: >> On Mon, 14 Nov 2016, John Stultz wrote: >> >>> On Mon, Nov 14, 2016 at 11:42 AM, Chris Metcalf >>> wrote: >>> > This bugfix was originally made in commit 35a4933a8959 ("time: >>> > A

Re: [PATCH] time: Avoid signed overflow in timekeeping_delta_to_ns()

2016-11-15 Thread John Stultz
On Tue, Nov 15, 2016 at 1:53 PM, Thomas Gleixner wrote: > On Mon, 14 Nov 2016, John Stultz wrote: > >> On Mon, Nov 14, 2016 at 11:42 AM, Chris Metcalf >> wrote: >> > This bugfix was originally made in commit 35a4933a8959 ("time: >> > Avoid signed overflow in timekeeping_get_ns()"). When the cod

Re: [PATCH] time: Avoid signed overflow in timekeeping_delta_to_ns()

2016-11-15 Thread Thomas Gleixner
On Mon, 14 Nov 2016, John Stultz wrote: > On Mon, Nov 14, 2016 at 11:42 AM, Chris Metcalf wrote: > > This bugfix was originally made in commit 35a4933a8959 ("time: > > Avoid signed overflow in timekeeping_get_ns()"). When the code was > > refactored in commit 6bd58f09e1d8 ("time: Add cycles to n

Re: [PATCH] time: Avoid signed overflow in timekeeping_delta_to_ns()

2016-11-14 Thread Laurent Vivier
On 14/11/2016 20:42, Chris Metcalf wrote: > This bugfix was originally made in commit 35a4933a8959 ("time: > Avoid signed overflow in timekeeping_get_ns()"). When the code was > refactored in commit 6bd58f09e1d8 ("time: Add cycles to nanoseconds > translation") the signed overflow fix was lost.

Re: [PATCH] time: Avoid signed overflow in timekeeping_delta_to_ns()

2016-11-14 Thread David Gibson
On Mon, Nov 14, 2016 at 02:42:49PM -0500, Chris Metcalf wrote: > This bugfix was originally made in commit 35a4933a8959 ("time: > Avoid signed overflow in timekeeping_get_ns()"). When the code was > refactored in commit 6bd58f09e1d8 ("time: Add cycles to nanoseconds > translation") the signed over

Re: [PATCH] time: Avoid signed overflow in timekeeping_delta_to_ns()

2016-11-14 Thread John Stultz
On Mon, Nov 14, 2016 at 11:42 AM, Chris Metcalf wrote: > This bugfix was originally made in commit 35a4933a8959 ("time: > Avoid signed overflow in timekeeping_get_ns()"). When the code was > refactored in commit 6bd58f09e1d8 ("time: Add cycles to nanoseconds > translation") the signed overflow fi

[PATCH] time: Avoid signed overflow in timekeeping_delta_to_ns()

2016-11-14 Thread Chris Metcalf
This bugfix was originally made in commit 35a4933a8959 ("time: Avoid signed overflow in timekeeping_get_ns()"). When the code was refactored in commit 6bd58f09e1d8 ("time: Add cycles to nanoseconds translation") the signed overflow fix was lost. Re-introduce it. Signed-off-by: Chris Metcalf ---