Re: [PATCH v2 3/4] drm/i915: pre-alloc instead of drm_mm search/get_block

On Sat, Jul 27, 2013 at 01:38:42PM +0200, David Herrmann wrote:

i915 is the last user of the weird search+get_block drm_mm API. Convert it to an explicit kmalloc()+insert_node(). This drops the last user of the node-cache in drm_mm. We can remove it now in a follow-up patch.

v2:

• simplify error path in i915_setup_compression()

You only applied it to the err path, I was expecting err_fb to do the kfree(compressed_llb) as well. Feel free to rename those to err_fb and err_llb respectively if you think that helps. -Chris