Re: [Intel-gfx] [PATCH 3/5] drm/i915: Track page table backing store usage

["Iddamsetty, Aravind" <aravind.iddamsetty@intel.com>]

On 07-07-2023 18:32, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> 
> Account page table backing store against the owning client memory usage
> stats.
> 
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> ---
>  drivers/gpu/drm/i915/gt/intel_gtt.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/gt/intel_gtt.c b/drivers/gpu/drm/i915/gt/intel_gtt.c
> index 2f6a9be0ffe6..126269a0d728 100644
> --- a/drivers/gpu/drm/i915/gt/intel_gtt.c
> +++ b/drivers/gpu/drm/i915/gt/intel_gtt.c
> @@ -58,6 +58,9 @@ struct drm_i915_gem_object *alloc_pt_lmem(struct i915_address_space *vm, int sz)
>  	if (!IS_ERR(obj)) {
>  		obj->base.resv = i915_vm_resv_get(vm);
>  		obj->shares_resv_from = vm;
> +
> +		if (vm->fpriv)
> +			i915_drm_client_add_object(vm->fpriv->client, obj);
>  	}
>  
>  	return obj;
> @@ -79,6 +82,9 @@ struct drm_i915_gem_object *alloc_pt_dma(struct i915_address_space *vm, int sz)
>  	if (!IS_ERR(obj)) {
>  		obj->base.resv = i915_vm_resv_get(vm);
>  		obj->shares_resv_from = vm;
> +
> +		if (vm->fpriv)
> +			i915_drm_client_add_object(vm->fpriv->client, obj);
>  	}
>  
>  	return obj;

Reviewed-by: Aravind Iddamsetty <aravind.iddamsetty@intel.com>

Thanks,
Aravind.