Re: [PATCH] drm/etnaviv: only reject timeouts with tv_nsec >= 2 seconds

2020-01-28 Thread Arnd Bergmann
On Fri, Jan 24, 2020 at 9:56 AM Guido Günther wrote: > On Wed, Jan 22, 2020 at 10:35:53AM +, Russell King - ARM Linux admin > wrote: > > On Wed, Jan 22, 2020 at 11:30:34AM +0100, Guido Günther wrote: > > I think it would probably be better for the kernel to print a > > warning once when noti

Re: [PATCH] drm/etnaviv: only reject timeouts with tv_nsec >= 2 seconds

2020-01-24 Thread Guido Günther
Hi Russel, On Wed, Jan 22, 2020 at 10:35:53AM +, Russell King - ARM Linux admin wrote: > On Wed, Jan 22, 2020 at 11:30:34AM +0100, Guido Günther wrote: > > Hi, > > On Tue, Jan 21, 2020 at 08:05:27PM +0100, Arnd Bergmann wrote: > > > On Tue, Jan 21, 2020 at 5:10 PM Lucas Stach > > > wrote: > >

Re: [PATCH] drm/etnaviv: only reject timeouts with tv_nsec >= 2 seconds

2020-01-22 Thread Russell King - ARM Linux admin
On Wed, Jan 22, 2020 at 11:30:34AM +0100, Guido Günther wrote: > Hi, > On Tue, Jan 21, 2020 at 08:05:27PM +0100, Arnd Bergmann wrote: > > On Tue, Jan 21, 2020 at 5:10 PM Lucas Stach wrote: > > > > > > Hi Guido, > > > > > > On Di, 2020-01-21 at 13:55 +0100, Guido Günther wrote: > > > > Hi, > > > >

Re: [PATCH] drm/etnaviv: only reject timeouts with tv_nsec >= 2 seconds

2020-01-22 Thread Guido Günther
Hi, On Tue, Jan 21, 2020 at 08:05:27PM +0100, Arnd Bergmann wrote: > On Tue, Jan 21, 2020 at 5:10 PM Lucas Stach wrote: > > > > Hi Guido, > > > > On Di, 2020-01-21 at 13:55 +0100, Guido Günther wrote: > > > Hi, > > > On Tue, Jan 21, 2020 at 12:45:25PM +0100, Arnd Bergmann wrote: > > > > As Guido G

Re: [PATCH] drm/etnaviv: only reject timeouts with tv_nsec >= 2 seconds

2020-01-21 Thread Arnd Bergmann
On Tue, Jan 21, 2020 at 5:10 PM Lucas Stach wrote: > > Hi Guido, > > On Di, 2020-01-21 at 13:55 +0100, Guido Günther wrote: > > Hi, > > On Tue, Jan 21, 2020 at 12:45:25PM +0100, Arnd Bergmann wrote: > > > As Guido Günther reported, get_abs_timeout() in the etnaviv user space > > > sometimes passes

Re: [PATCH] drm/etnaviv: only reject timeouts with tv_nsec >= 2 seconds

2020-01-21 Thread Lucas Stach
Hi Guido, On Di, 2020-01-21 at 13:55 +0100, Guido Günther wrote: > Hi, > On Tue, Jan 21, 2020 at 12:45:25PM +0100, Arnd Bergmann wrote: > > As Guido Günther reported, get_abs_timeout() in the etnaviv user space > > sometimes passes timeouts with nanosecond values larger than 10, > > which

Re: [PATCH] drm/etnaviv: only reject timeouts with tv_nsec >= 2 seconds

2020-01-21 Thread Russell King - ARM Linux admin
On Tue, Jan 21, 2020 at 01:55:46PM +0100, Guido Günther wrote: > Hi, > On Tue, Jan 21, 2020 at 12:45:25PM +0100, Arnd Bergmann wrote: > > As Guido Günther reported, get_abs_timeout() in the etnaviv user space > > sometimes passes timeouts with nanosecond values larger than 10, > > which get

Re: [PATCH] drm/etnaviv: only reject timeouts with tv_nsec >= 2 seconds

2020-01-21 Thread Guido Günther
Hi, On Tue, Jan 21, 2020 at 12:45:25PM +0100, Arnd Bergmann wrote: > As Guido Günther reported, get_abs_timeout() in the etnaviv user space > sometimes passes timeouts with nanosecond values larger than 10, > which gets rejected after my first patch. > > To avoid breaking this, while also

[PATCH] drm/etnaviv: only reject timeouts with tv_nsec >= 2 seconds

2020-01-21 Thread Arnd Bergmann
As Guido Günther reported, get_abs_timeout() in the etnaviv user space sometimes passes timeouts with nanosecond values larger than 10, which gets rejected after my first patch. To avoid breaking this, while also not allowing completely arbitrary values, set the limit to 19 and use