Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-08-05 Thread Brian Starkey
Hi Lowry, Based on Daniel's input, this patch looks fine: Reviewed-by: Brian Starkey I think there's some opportunity for improvement around prepare_signaling/complete_signaling, but that can be treated as separate from fixing this bug. Thanks, -Brian On Wed, Jul 31, 2019 at 11:04:45AM +,

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-08-02 Thread Daniel Vetter
On Fri, Aug 02, 2019 at 10:09:05AM +, Brian Starkey wrote: > Hi Daniel, > > On Fri, Aug 02, 2019 at 11:45:13AM +0200, Daniel Vetter wrote: > > On Fri, Aug 2, 2019 at 11:43 AM Daniel Vetter wrote: > > > > > > On Fri, Aug 2, 2019 at 11:29 AM Brian Starkey > > > wrote: > > > > > > > > Hi Lowry

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-08-02 Thread james qian wang (Arm Technology China)
On Fri, Aug 02, 2019 at 05:29:20PM +0800, Brian Starkey wrote: > Hi Lowry, > > On Thu, Aug 01, 2019 at 06:34:08AM +, Lowry Li (Arm Technology China) > wrote: > > Hi Brian, > > > > On Wed, Jul 31, 2019 at 09:20:04PM +0800, Brian Starkey wrote: > > > Hi Lowry, > > > > > > Thanks for this clea

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-08-02 Thread Brian Starkey
Hi Daniel, On Fri, Aug 02, 2019 at 11:45:13AM +0200, Daniel Vetter wrote: > On Fri, Aug 2, 2019 at 11:43 AM Daniel Vetter wrote: > > > > On Fri, Aug 2, 2019 at 11:29 AM Brian Starkey wrote: > > > > > > Hi Lowry, > > > > > > On Thu, Aug 01, 2019 at 06:34:08AM +, Lowry Li (Arm Technology China

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-08-02 Thread Daniel Vetter
On Fri, Aug 2, 2019 at 11:43 AM Daniel Vetter wrote: > > On Fri, Aug 2, 2019 at 11:29 AM Brian Starkey wrote: > > > > Hi Lowry, > > > > On Thu, Aug 01, 2019 at 06:34:08AM +, Lowry Li (Arm Technology China) > > wrote: > > > Hi Brian, > > > > > > On Wed, Jul 31, 2019 at 09:20:04PM +0800, Brian

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-08-02 Thread Daniel Vetter
On Fri, Aug 2, 2019 at 11:29 AM Brian Starkey wrote: > > Hi Lowry, > > On Thu, Aug 01, 2019 at 06:34:08AM +, Lowry Li (Arm Technology China) > wrote: > > Hi Brian, > > > > On Wed, Jul 31, 2019 at 09:20:04PM +0800, Brian Starkey wrote: > > > Hi Lowry, > > > > > > Thanks for this cleanup. > > >

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-08-02 Thread Brian Starkey
Hi Lowry, On Thu, Aug 01, 2019 at 06:34:08AM +, Lowry Li (Arm Technology China) wrote: > Hi Brian, > > On Wed, Jul 31, 2019 at 09:20:04PM +0800, Brian Starkey wrote: > > Hi Lowry, > > > > Thanks for this cleanup. > > > > On Wed, Jul 31, 2019 at 11:04:45AM +, Lowry Li (Arm Technology Chi

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-08-01 Thread Liviu Dudau
On Thu, Aug 01, 2019 at 06:31:13AM +, Lowry Li (Arm Technology China) wrote: > Hi Liviu, > > On Wed, Jul 31, 2019 at 01:15:25PM +, Liviu Dudau wrote: > > Hi Lowry, > > > > On Wed, Jul 31, 2019 at 11:04:45AM +, Lowry Li (Arm Technology China) > > wrote: > > > During it signals the com

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-07-31 Thread Lowry Li (Arm Technology China)
Hi Brian, On Wed, Jul 31, 2019 at 09:20:04PM +0800, Brian Starkey wrote: > Hi Lowry, > > Thanks for this cleanup. > > On Wed, Jul 31, 2019 at 11:04:45AM +, Lowry Li (Arm Technology China) > wrote: > > During it signals the completion of a writeback job, after releasing > > the out_fence, we

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-07-31 Thread Lowry Li (Arm Technology China)
Hi Liviu, On Wed, Jul 31, 2019 at 01:15:25PM +, Liviu Dudau wrote: > Hi Lowry, > > On Wed, Jul 31, 2019 at 11:04:45AM +, Lowry Li (Arm Technology China) > wrote: > > During it signals the completion of a writeback job, after releasing > > the out_fence, we'd clear the pointer. > > > > C

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-07-31 Thread Brian Starkey
Hi Lowry, Thanks for this cleanup. On Wed, Jul 31, 2019 at 11:04:45AM +, Lowry Li (Arm Technology China) wrote: > During it signals the completion of a writeback job, after releasing > the out_fence, we'd clear the pointer. > > Check if fence left over in drm_writeback_cleanup_job(), release

Re: [PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-07-31 Thread Liviu Dudau
Hi Lowry, On Wed, Jul 31, 2019 at 11:04:45AM +, Lowry Li (Arm Technology China) wrote: > During it signals the completion of a writeback job, after releasing > the out_fence, we'd clear the pointer. > > Check if fence left over in drm_writeback_cleanup_job(), release it. > > Signed-off-by: L

[PATCH v1 2/2] drm: Clear the fence pointer when writeback job signaled

2019-07-31 Thread Lowry Li (Arm Technology China)
During it signals the completion of a writeback job, after releasing the out_fence, we'd clear the pointer. Check if fence left over in drm_writeback_cleanup_job(), release it. Signed-off-by: Lowry Li (Arm Technology China) --- drivers/gpu/drm/drm_writeback.c | 23 +++ 1 fil