Re: [PATCH 2/2] drm/i915: Remove i915_vma_create from VMA API

[Joonas Lahtinen <joonas.lahtinen@linux.intel.com>]

On ma, 2017-01-16 at 09:49 +0000, Chris Wilson wrote:
> With the introduce of i915_vma_instance() for obtaining the VMA
> singleton for a (obj, vm, view) tuple, we can remove the
> i915_vma_create() in favour of a single entry point. We do incur a
> lookup onto an empty tree, but the i915_vma_create() were being called
> infrequently and during initialisation, so the small overhead is
> negligible.
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

<SNIP>

>  struct i915_vma *
> -i915_vma_create(struct drm_i915_gem_object *obj,
> -		struct i915_address_space *vm,
> -		const struct i915_ggtt_view *view);
> -
> -struct i915_vma *
>  i915_vma_lookup(struct drm_i915_gem_object *obj,
>  		struct i915_address_space *vm,
>  		const struct i915_ggtt_view *view);

I'd say our de-facto is to drop "i915_" prefix from locals.

Would add to the code understandability, but anyway;

Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx