[PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-24 Thread Jerome Glisse
On Thu, 2012-02-23 at 10:17 -0500, Alex Deucher wrote: > 2012/2/23 Christian K?nig : > > The function radeon_bo_list_validate can cause a > > bo to move, resulting in a different sync_obj > > and a dependency to wait for this move to finish. > > > > Signed-off-by: Christian K?nig > > Reviewed-by:

Re: [PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-24 Thread Jerome Glisse
On Thu, 2012-02-23 at 10:17 -0500, Alex Deucher wrote: > 2012/2/23 Christian König : > > The function radeon_bo_list_validate can cause a > > bo to move, resulting in a different sync_obj > > and a dependency to wait for this move to finish. > > > > Signed-off-by: Christian König > > Reviewed-by:

[PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Christian König
On 23.02.2012 18:32, Jerome Glisse wrote: > 2012/2/23 Mathias Fr?hlich > > > > Christian, > > On Thursday, February 23, 2012 15:18:42 Christian K?nig wrote: > > The function radeon_bo_list_validate can cause a > > bo to move, resulting in a diff

[PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Mathias Fröhlich
Christian, On Thursday, February 23, 2012 15:18:42 Christian K?nig wrote: > The function radeon_bo_list_validate can cause a > bo to move, resulting in a different sync_obj > and a dependency to wait for this move to finish. > > Signed-off-by: Christian K?nig > Reviewed-by: Alex Deucher I am

[PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Christian König
The function radeon_bo_list_validate can cause a bo to move, resulting in a different sync_obj and a dependency to wait for this move to finish. Signed-off-by: Christian K?nig Reviewed-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon.h|1 - drivers/gpu/drm/radeon/radeon_cs.c | 21 ++

[PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Jerome Glisse
2012/2/23 Mathias Fr?hlich > > Christian, > > On Thursday, February 23, 2012 15:18:42 Christian K?nig wrote: > > The function radeon_bo_list_validate can cause a > > bo to move, resulting in a different sync_obj > > and a dependency to wait for this move to finish. > > > > Signed-off-by: Christia

Re: [PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Christian König
On 23.02.2012 18:32, Jerome Glisse wrote: 2012/2/23 Mathias Fröhlich > Christian, On Thursday, February 23, 2012 15:18:42 Christian König wrote: > The function radeon_bo_list_validate can cause a > bo to move, resulting in a different sync_obj

[PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Alex Deucher
2012/2/23 Christian K?nig : > The function radeon_bo_list_validate can cause a > bo to move, resulting in a different sync_obj > and a dependency to wait for this move to finish. > > Signed-off-by: Christian K?nig > Reviewed-by: Alex Deucher For the series: Reviewed-by: Alex Deucher > --- > ?

Re: [PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Jerome Glisse
2012/2/23 Mathias Fröhlich > > Christian, > > On Thursday, February 23, 2012 15:18:42 Christian König wrote: > > The function radeon_bo_list_validate can cause a > > bo to move, resulting in a different sync_obj > > and a dependency to wait for this move to finish. > > > > Signed-off-by: Christia

Re: [PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Mathias Fröhlich
Christian, On Thursday, February 23, 2012 15:18:42 Christian König wrote: > The function radeon_bo_list_validate can cause a > bo to move, resulting in a different sync_obj > and a dependency to wait for this move to finish. > > Signed-off-by: Christian König > Reviewed-by: Alex Deucher I am

Re: [PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Alex Deucher
2012/2/23 Christian König : > The function radeon_bo_list_validate can cause a > bo to move, resulting in a different sync_obj > and a dependency to wait for this move to finish. > > Signed-off-by: Christian König > Reviewed-by: Alex Deucher For the series: Reviewed-by: Alex Deucher > --- >  

[PATCH 1/4] drm/radeon: move ring syncing after bo validation

2012-02-23 Thread Christian König
The function radeon_bo_list_validate can cause a bo to move, resulting in a different sync_obj and a dependency to wait for this move to finish. Signed-off-by: Christian König Reviewed-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon.h|1 - drivers/gpu/drm/radeon/radeon_cs.c | 21 ++