Re: [Freedreno] [PATCH] drm/msm: fix an integer overflow test

2017-06-30 Thread Jordan Crouse
On Fri, Jun 30, 2017 at 10:59:15AM +0300, Dan Carpenter wrote: > We recently added an integer overflow check but it needs an additional > tweak to work properly on 32 bit systems. > > The problem is that we're doing the right hand side of the assignment as > type unsigned long so the max it will

Re: [Freedreno] [Intel-gfx] [PATCH 1/2] drm/atomic: Change drm_atomic_helper_swap_state to return an error.

2017-06-30 Thread Daniel Vetter
On Wed, Jun 28, 2017 at 03:28:11PM +0200, Maarten Lankhorst wrote: > We want to change swap_state to wait indefinitely, but to do this > swap_state should wait interruptibly. This requires propagating > the error to each driver. All drivers have changes to deal with the > clean up. In order to

[Freedreno] [PATCH] drm/msm: fix an integer overflow test

2017-06-30 Thread Dan Carpenter
We recently added an integer overflow check but it needs an additional tweak to work properly on 32 bit systems. The problem is that we're doing the right hand side of the assignment as type unsigned long so the max it will have an integer overflow instead of being larger than SIZE_MAX. That