[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-17 Thread Christian König
On 17.03.2015 16:19, Alex Deucher wrote: > On Mon, Mar 16, 2015 at 11:48 PM, Michel Dänzer > wrote: >> On 17.03.2015 07:32, Alex Deucher wrote: >>> On Thu, Mar 12, 2015 at 10:55 PM, Michel Dänzer >>> wrote: On 12.03.2015 22:09, Alex Deucher wrote: > On Thu, Mar 12, 2015 at 5:23 AM, C

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-17 Thread Michel Dänzer
On 17.03.2015 07:32, Alex Deucher wrote: > On Thu, Mar 12, 2015 at 10:55 PM, Michel Dänzer > wrote: >> On 12.03.2015 22:09, Alex Deucher wrote: >>> On Thu, Mar 12, 2015 at 5:23 AM, Christian König >>> wrote: On 12.03.2015 10:02, Michel Dänzer wrote: > > On 12.03.2015 06:14, Alex

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-17 Thread Christian König
On 16.03.2015 23:32, Alex Deucher wrote: > On Thu, Mar 12, 2015 at 10:55 PM, Michel Dänzer > wrote: >> On 12.03.2015 22:09, Alex Deucher wrote: >>> On Thu, Mar 12, 2015 at 5:23 AM, Christian König >>> wrote: On 12.03.2015 10:02, Michel Dänzer wrote: > On 12.03.2015 06:14, Alex Deuche

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-17 Thread Alex Deucher
On Tue, Mar 17, 2015 at 11:43 AM, Christian König wrote: > On 17.03.2015 16:19, Alex Deucher wrote: > > On Mon, Mar 16, 2015 at 11:48 PM, Michel Dänzer > wrote: > > On 17.03.2015 07:32, Alex Deucher wrote: > > On Thu, Mar 12, 2015 at 10:55 PM, Michel Dänzer > wrote: > > On 12.03.2015 22:09,

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-17 Thread Alex Deucher
On Mon, Mar 16, 2015 at 11:48 PM, Michel Dänzer wrote: > On 17.03.2015 07:32, Alex Deucher wrote: >> On Thu, Mar 12, 2015 at 10:55 PM, Michel Dänzer >> wrote: >>> On 12.03.2015 22:09, Alex Deucher wrote: On Thu, Mar 12, 2015 at 5:23 AM, Christian König wrote: > On 12.03.2015 10

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-17 Thread Alex Deucher
On Tue, Mar 17, 2015 at 7:40 AM, Christian König wrote: > On 16.03.2015 23:32, Alex Deucher wrote: >> >> On Thu, Mar 12, 2015 at 10:55 PM, Michel Dänzer >> wrote: >>> >>> On 12.03.2015 22:09, Alex Deucher wrote: On Thu, Mar 12, 2015 at 5:23 AM, Christian König wrote: >

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-16 Thread Alex Deucher
On Thu, Mar 12, 2015 at 10:55 PM, Michel Dänzer wrote: > On 12.03.2015 22:09, Alex Deucher wrote: >> On Thu, Mar 12, 2015 at 5:23 AM, Christian König >> wrote: >>> On 12.03.2015 10:02, Michel Dänzer wrote: On 12.03.2015 06:14, Alex Deucher wrote: > > On Wed, Mar 11, 2015 at 4

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-15 Thread Oded Gabbay
On 03/12/2015 11:36 AM, Christian König wrote: > On 12.03.2015 10:30, Oded Gabbay wrote: >> >> On 03/12/2015 11:23 AM, Christian König wrote: >>> On 12.03.2015 10:02, Michel Dänzer wrote: On 12.03.2015 06:14, Alex Deucher wrote: > On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher >

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-13 Thread Michel Dänzer
On 13.03.2015 18:11, Daniel Vetter wrote: > On Thu, Mar 12, 2015 at 06:02:56PM +0900, Michel Dänzer wrote: >> On 12.03.2015 06:14, Alex Deucher wrote: >>> On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher >>> wrote: On Wed, Mar 11, 2015 at 2:21 PM, Christian König wrote: > On 11.03.2

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-13 Thread Daniel Vetter
On Fri, Mar 13, 2015 at 06:46:33PM +0900, Michel Dänzer wrote: > On 13.03.2015 18:11, Daniel Vetter wrote: > > On Thu, Mar 12, 2015 at 06:02:56PM +0900, Michel Dänzer wrote: > >> On 12.03.2015 06:14, Alex Deucher wrote: > >>> On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher > >>> wrote: > On

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-13 Thread Alex Deucher
On Fri, Mar 13, 2015 at 12:36 PM, Daniel Vetter wrote: > On Fri, Mar 13, 2015 at 06:46:33PM +0900, Michel Dänzer wrote: >> On 13.03.2015 18:11, Daniel Vetter wrote: >> > On Thu, Mar 12, 2015 at 06:02:56PM +0900, Michel Dänzer wrote: >> >> On 12.03.2015 06:14, Alex Deucher wrote: >> >>> On Wed, M

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-13 Thread Michel Dänzer
On 12.03.2015 22:09, Alex Deucher wrote: > On Thu, Mar 12, 2015 at 5:23 AM, Christian König > wrote: >> On 12.03.2015 10:02, Michel Dänzer wrote: >>> >>> On 12.03.2015 06:14, Alex Deucher wrote: On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher wrote: > > On Wed, Mar 11, 2015

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-13 Thread Daniel Vetter
On Thu, Mar 12, 2015 at 06:02:56PM +0900, Michel Dänzer wrote: > On 12.03.2015 06:14, Alex Deucher wrote: > > On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher > > wrote: > >> On Wed, Mar 11, 2015 at 2:21 PM, Christian König > >> wrote: > >>> On 11.03.2015 16:44, Alex Deucher wrote: > >

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-12 Thread Michel Dänzer
On 12.03.2015 06:14, Alex Deucher wrote: > On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher > wrote: >> On Wed, Mar 11, 2015 at 2:21 PM, Christian König >> wrote: >>> On 11.03.2015 16:44, Alex Deucher wrote: radeon_bo_create() calls radeon_ttm_placement_from_domain() before ttm_bo_i

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-12 Thread Lauri Kasanen
On Thu, 12 Mar 2015 18:02:56 +0900 Michel Dänzer wrote: > struct ttm_place::lpfn is honoured even with TTM_PL_FLAG_TOPDOWN, so > latter should work with RADEON_GEM_CPU_ACCESS. It sounds like the > problem is really that some BOs are expected to be within a certain > range from the beginning of V

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-12 Thread Oded Gabbay
On 03/12/2015 11:23 AM, Christian König wrote: > On 12.03.2015 10:02, Michel Dänzer wrote: >> On 12.03.2015 06:14, Alex Deucher wrote: >>> On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher >>> wrote: On Wed, Mar 11, 2015 at 2:21 PM, Christian König wrote: > On 11.03.2015 16:44, Al

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-12 Thread Christian König
On 12.03.2015 10:30, Oded Gabbay wrote: > > On 03/12/2015 11:23 AM, Christian König wrote: >> On 12.03.2015 10:02, Michel Dänzer wrote: >>> On 12.03.2015 06:14, Alex Deucher wrote: On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher wrote: > On Wed, Mar 11, 2015 at 2:21 PM, Christian KÃ

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-12 Thread Christian König
On 12.03.2015 10:02, Michel Dänzer wrote: > On 12.03.2015 06:14, Alex Deucher wrote: >> On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher >> wrote: >>> On Wed, Mar 11, 2015 at 2:21 PM, Christian König >>> wrote: On 11.03.2015 16:44, Alex Deucher wrote: > radeon_bo_create() calls radeon_tt

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-12 Thread Alex Deucher
On Thu, Mar 12, 2015 at 5:23 AM, Christian König wrote: > On 12.03.2015 10:02, Michel Dänzer wrote: >> >> On 12.03.2015 06:14, Alex Deucher wrote: >>> >>> On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher >>> wrote: On Wed, Mar 11, 2015 at 2:21 PM, Christian König wrote: >

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-11 Thread Christian König
On 11.03.2015 16:44, Alex Deucher wrote: > radeon_bo_create() calls radeon_ttm_placement_from_domain() > before ttm_bo_init() is called. radeon_ttm_placement_from_domain() > uses the ttm bo size to determine when to select top down > allocation but since the ttm bo is not initialized yet the > che

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-11 Thread Alex Deucher
On Wed, Mar 11, 2015 at 4:51 PM, Alex Deucher wrote: > On Wed, Mar 11, 2015 at 2:21 PM, Christian König > wrote: >> On 11.03.2015 16:44, Alex Deucher wrote: >>> >>> radeon_bo_create() calls radeon_ttm_placement_from_domain() >>> before ttm_bo_init() is called. radeon_ttm_placement_from_domain()

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-11 Thread Alex Deucher
On Wed, Mar 11, 2015 at 2:21 PM, Christian König wrote: > On 11.03.2015 16:44, Alex Deucher wrote: >> >> radeon_bo_create() calls radeon_ttm_placement_from_domain() >> before ttm_bo_init() is called. radeon_ttm_placement_from_domain() >> uses the ttm bo size to determine when to select top down

[PATCH] drm/radeon: fix TOPDOWN handling for bo_create

2015-03-11 Thread Alex Deucher
radeon_bo_create() calls radeon_ttm_placement_from_domain() before ttm_bo_init() is called. radeon_ttm_placement_from_domain() uses the ttm bo size to determine when to select top down allocation but since the ttm bo is not initialized yet the check is always false. Noticed-by: Oded Gabbay Signe