----- On Nov 9, 2018, at 5:34 PM, Jonathan Rajotte 
jonathan.rajotte-jul...@efficios.com wrote:
[...]
>> +
>> +LTTNG_HIDDEN
>> +struct timespec timespec_abs_diff(struct timespec t1, struct timespec t2)
>> +{
>> +    uint64_t ts1 = (uint64_t) t1.tv_sec * INT64_C(1000000000) + (uint64_t)
>> t1.tv_nsec;
>> +    uint64_t ts2 = (uint64_t) t2.tv_sec * INT64_C(1000000000) + (uint64_t)
>> t2.tv_nsec;
> 
> Use NSEC_PER_SEC here.
> 
>> +    uint64_t diff = max(ts1, ts2) - min(ts1, ts2);
>> +    struct timespec res;
>> +
>> +    res.tv_sec = diff / INT64_C(1000000000);
> 
> Use NSEC_PER_SEC here.
> 
>> +    res.tv_nsec = diff % INT64_C(1000000000);
> 
> Use NSEC_PER_SEC here.

Fixing in v2, thanks,

Mathieu

> 
>> +    return res;
>> +}
>> --
>> 2.11.0
>> 
> 
> --
> Jonathan Rajotte-Julien
> EfficiOS

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Reply via email to