Re: [Y2038] [PATCH 1/7] [media] dvb: use ktime_t for internal timeout

2015-09-15 Thread Arnd Bergmann
On Tuesday 15 September 2015 19:55:35 Andreas Oberritter wrote: > > /* show speed every SPEED_PKTS_INTERVAL packets */ > > if (!(demux->speed_pkts_cnt % SPEED_PKTS_INTERVAL)) { > > - cur_time = current_kernel_time(); > > + cur_time =

Re: [PATCH 1/7] [media] dvb: use ktime_t for internal timeout

2015-09-15 Thread Andreas Oberritter
Hello Arnd, On 15.09.2015 17:49, Arnd Bergmann wrote: > The dvb demuxer code uses a 'struct timespec' to pass a timeout > as absolute time. This will cause problems on 32-bit architectures > in 2038 when time_t overflows, and it is racy with a concurrent > settimeofday() call. > > This patch

[PATCH 1/7] [media] dvb: use ktime_t for internal timeout

2015-09-15 Thread Arnd Bergmann
The dvb demuxer code uses a 'struct timespec' to pass a timeout as absolute time. This will cause problems on 32-bit architectures in 2038 when time_t overflows, and it is racy with a concurrent settimeofday() call. This patch changes the code to use ktime_get() instead, using the monotonic time

Re: [PATCH 1/7] [media] dvb: use ktime_t for internal timeout

2015-09-15 Thread Andreas Oberritter
Hello Arnd, On 15.09.2015 17:49, Arnd Bergmann wrote: > The dvb demuxer code uses a 'struct timespec' to pass a timeout > as absolute time. This will cause problems on 32-bit architectures > in 2038 when time_t overflows, and it is racy with a concurrent > settimeofday() call. > > This patch

[PATCH 1/7] [media] dvb: use ktime_t for internal timeout

2015-09-15 Thread Arnd Bergmann
The dvb demuxer code uses a 'struct timespec' to pass a timeout as absolute time. This will cause problems on 32-bit architectures in 2038 when time_t overflows, and it is racy with a concurrent settimeofday() call. This patch changes the code to use ktime_get() instead, using the monotonic time

Re: [Y2038] [PATCH 1/7] [media] dvb: use ktime_t for internal timeout

2015-09-15 Thread Arnd Bergmann
On Tuesday 15 September 2015 19:55:35 Andreas Oberritter wrote: > > /* show speed every SPEED_PKTS_INTERVAL packets */ > > if (!(demux->speed_pkts_cnt % SPEED_PKTS_INTERVAL)) { > > - cur_time = current_kernel_time(); > > + cur_time =