Re: [Mesa-dev] [PATCH 2/2] gallium/os: fix os_time_get_nano() to roll over less

2017-08-18 Thread Jose Fonseca
On 18/08/17 08:23, Frank Richter wrote: Hi, On 17.08.2017 02:34, Brian Paul wrote: BTW, I wonder if we would win by using lldiv(). Because this is often use for performance measurements, so these extra division might add some impact. Frank, do you want to look into that? In the mean time,

Re: [Mesa-dev] [PATCH 2/2] gallium/os: fix os_time_get_nano() to roll over less

2017-08-18 Thread Frank Richter
Hi, On 17.08.2017 02:34, Brian Paul wrote: BTW, I wonder if we would win by using lldiv(). Because this is often use for performance measurements, so these extra division might add some impact. Frank, do you want to look into that? In the mean time, I'll push the patches as-is. AFAICS,

Re: [Mesa-dev] [PATCH 2/2] gallium/os: fix os_time_get_nano() to roll over less

2017-08-16 Thread Brian Paul
On 08/16/2017 04:42 PM, Jose Fonseca wrote: On 16/08/17 23:37, Jose Fonseca wrote: On 16/08/17 14:22, Brian Paul wrote: From: Frank Richter Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102241 Cc: mesa-sta...@lists.freedesktop.org Reviewed-by: Brian Paul

Re: [Mesa-dev] [PATCH 2/2] gallium/os: fix os_time_get_nano() to roll over less

2017-08-16 Thread Jose Fonseca
On 16/08/17 14:22, Brian Paul wrote: From: Frank Richter Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102241 Cc: mesa-sta...@lists.freedesktop.org Reviewed-by: Brian Paul --- src/gallium/auxiliary/os/os_time.c | 9 - 1 file

Re: [Mesa-dev] [PATCH 2/2] gallium/os: fix os_time_get_nano() to roll over less

2017-08-16 Thread Jose Fonseca
On 16/08/17 23:37, Jose Fonseca wrote: On 16/08/17 14:22, Brian Paul wrote: From: Frank Richter Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102241 Cc: mesa-sta...@lists.freedesktop.org Reviewed-by: Brian Paul ---

[Mesa-dev] [PATCH 2/2] gallium/os: fix os_time_get_nano() to roll over less

2017-08-16 Thread Brian Paul
From: Frank Richter Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102241 Cc: mesa-sta...@lists.freedesktop.org Reviewed-by: Brian Paul --- src/gallium/auxiliary/os/os_time.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-)