[PATCH 34/37] drm/ttm: fix fence locking in ttm_buffer_object_transfer

2012-12-12 Thread Daniel Vetter
On Wed, Dec 12, 2012 at 3:48 PM, Jerome Glisse wrote: > Instead of that i would just move the call to > ttm_buffer_object_transfer to happen before releasing the fence_lock > in ttm_bo_move_accel_cleanup , something like : Yeah, looks better. Fixed up locally. -Daniel -- Daniel Vetter Software E

[PATCH 34/37] drm/ttm: fix fence locking in ttm_buffer_object_transfer

2012-12-12 Thread Daniel Vetter
Noticed while reviewing the fence locking in the radone pageflip handler. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/ttm/ttm_bo_util.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c b/drivers/gpu/drm/ttm/ttm_bo_util.c index b9c4e51..5c8b207 1006

[PATCH 34/37] drm/ttm: fix fence locking in ttm_buffer_object_transfer

2012-12-12 Thread Jerome Glisse
On Wed, Dec 12, 2012 at 8:07 AM, Daniel Vetter wrote: > Noticed while reviewing the fence locking in the radone pageflip > handler. > > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/ttm/ttm_bo_util.c |2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/ttm/ttm_bo

Re: [PATCH 34/37] drm/ttm: fix fence locking in ttm_buffer_object_transfer

2012-12-12 Thread Daniel Vetter
On Wed, Dec 12, 2012 at 3:48 PM, Jerome Glisse wrote: > Instead of that i would just move the call to > ttm_buffer_object_transfer to happen before releasing the fence_lock > in ttm_bo_move_accel_cleanup , something like : Yeah, looks better. Fixed up locally. -Daniel -- Daniel Vetter Software E

Re: [PATCH 34/37] drm/ttm: fix fence locking in ttm_buffer_object_transfer

2012-12-12 Thread Jerome Glisse
On Wed, Dec 12, 2012 at 8:07 AM, Daniel Vetter wrote: > Noticed while reviewing the fence locking in the radone pageflip > handler. > > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/ttm/ttm_bo_util.c |2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/ttm/ttm_bo_

[PATCH 34/37] drm/ttm: fix fence locking in ttm_buffer_object_transfer

2012-12-12 Thread Daniel Vetter
Noticed while reviewing the fence locking in the radone pageflip handler. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/ttm/ttm_bo_util.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c b/drivers/gpu/drm/ttm/ttm_bo_util.c index b9c4e51..5c8b207 1006