[PATCH 1/6] drm: pre allocate node for create_block

2013-07-04 Thread Ben Widawsky
On Thu, Jul 04, 2013 at 11:19:58AM +0200, David Herrmann wrote: > Hi > > On Wed, Jul 3, 2013 at 11:45 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

[PATCH 1/6] drm: pre allocate node for create_block

2013-07-04 Thread David Herrmann
Hi On Wed, Jul 3, 2013 at 11:45 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

[PATCH 1/6] drm: pre allocate node for create_block

2013-07-04 Thread David Herrmann
Hi On Wed, Jul 3, 2013 at 11:45 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

Re: [PATCH 1/6] drm: pre allocate node for create_block

2013-07-04 Thread David Herrmann
Hi On Wed, Jul 3, 2013 at 11:45 PM, Ben Widawsky b...@bwidawsk.net 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

Re: [PATCH 1/6] drm: pre allocate node for create_block

2013-07-04 Thread David Herrmann
Hi On Wed, Jul 3, 2013 at 11:45 PM, Ben Widawsky b...@bwidawsk.net 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

Re: [PATCH 1/6] drm: pre allocate node for create_block

2013-07-04 Thread Ben Widawsky
On Thu, Jul 04, 2013 at 11:19:58AM +0200, David Herrmann wrote: Hi On Wed, Jul 3, 2013 at 11:45 PM, Ben Widawsky b...@bwidawsk.net 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

[PATCH 1/6] drm: pre allocate node for create_block

2013-07-03 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

[PATCH 1/6] drm: pre allocate node for create_block

2013-07-03 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(),