[PATCH] [v3] drm: pre allocate node for create_block

2013-07-05 Thread Daniel Vetter
On Thu, Jul 04, 2013 at 10:32:27PM +0200, David Herrmann wrote: > Hi > > On Thu, Jul 4, 2013 at 10:14 PM, Ben Widawsky wrote: > > For an upcoming patch where we introduce the i915 VMA, it's ideal to > > have the drm_mm_node as part of the VMA struct (ie. it's pre-allocated). > > Part of the conve

[PATCH] [v3] drm: pre allocate node for create_block

2013-07-05 Thread Ben Widawsky
On Fri, Jul 05, 2013 at 09:25:35PM +0200, Daniel Vetter wrote: > On Thu, Jul 04, 2013 at 10:32:27PM +0200, David Herrmann wrote: > > Hi > > > > On Thu, Jul 4, 2013 at 10:14 PM, Ben Widawsky wrote: > > > For an upcoming patch where we introduce the i915 VMA, it's ideal to > > > have the drm_mm_nod

Re: [PATCH] [v3] drm: pre allocate node for create_block

2013-07-05 Thread Ben Widawsky
On Fri, Jul 05, 2013 at 09:25:35PM +0200, Daniel Vetter wrote: > On Thu, Jul 04, 2013 at 10:32:27PM +0200, David Herrmann wrote: > > Hi > > > > On Thu, Jul 4, 2013 at 10:14 PM, Ben Widawsky wrote: > > > For an upcoming patch where we introduce the i915 VMA, it's ideal to > > > have the drm_mm_nod

Re: [PATCH] [v3] drm: pre allocate node for create_block

2013-07-05 Thread Daniel Vetter
On Thu, Jul 04, 2013 at 10:32:27PM +0200, David Herrmann wrote: > Hi > > On Thu, Jul 4, 2013 at 10:14 PM, Ben Widawsky wrote: > > For an upcoming patch where we introduce the i915 VMA, it's ideal to > > have the drm_mm_node as part of the VMA struct (ie. it's pre-allocated). > > Part of the conve

[PATCH] [v3] drm: pre allocate node for create_block

2013-07-04 Thread David Herrmann
Hi On Thu, Jul 4, 2013 at 10:14 PM, Ben Widawsky wrote: > For an upcoming patch where we introduce the i915 VMA, it's ideal to > have the drm_mm_node as part of the VMA struct (ie. it's pre-allocated). > Part of the conversion to VMAs is to kill off obj->gtt_space. Doing this > will break a bunch

Re: [PATCH] [v3] drm: pre allocate node for create_block

2013-07-04 Thread David Herrmann
Hi On Thu, Jul 4, 2013 at 10:14 PM, Ben Widawsky wrote: > For an upcoming patch where we introduce the i915 VMA, it's ideal to > have the drm_mm_node as part of the VMA struct (ie. it's pre-allocated). > Part of the conversion to VMAs is to kill off obj->gtt_space. Doing this > will break a bunch

[PATCH] [v3] drm: pre allocate node for create_block

2013-07-04 Thread Ben Widawsky
For an upcoming patch where we introduce the i915 VMA, it's ideal to have the drm_mm_node as part of the VMA struct (ie. it's pre-allocated). Part of the conversion to VMAs is to kill off obj->gtt_space. Doing this will break a bunch of code, but amongst them are 2 callers of drm_mm_create_block(),

[PATCH] [v3] drm: pre allocate node for create_block

2013-07-04 Thread Ben Widawsky
For an upcoming patch where we introduce the i915 VMA, it's ideal to have the drm_mm_node as part of the VMA struct (ie. it's pre-allocated). Part of the conversion to VMAs is to kill off obj->gtt_space. Doing this will break a bunch of code, but amongst them are 2 callers of drm_mm_create_block(),