[Intel-gfx] [PATCH 6/6] drm/i915: Include active flag when describing objects in debugfs

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Thu Mar 19 10:41:26 PDT 2015


On 03/09/2015 09:55 AM, Chris Wilson wrote:
> Since we use obj->active as a hint in many places throughout the code,
> knowing its state in debugfs is extremely useful.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---
>   drivers/gpu/drm/i915/i915_debugfs.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index 042ad2fec484..809f6eadc10c 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -123,8 +123,9 @@ describe_obj(struct seq_file *m, struct drm_i915_gem_object *obj)
>   	struct i915_vma *vma;
>   	int pin_count = 0;
>
> -	seq_printf(m, "%pK: %s%s%s %8zdKiB %02x %02x %x %x %x%s%s%s",
> +	seq_printf(m, "%pK: %s%s%s%s %8zdKiB %02x %02x %x %x %x%s%s%s",
>   		   &obj->base,
> +		   obj->active ? "*" : " ",

%c etc would maybe be more compact code? (Hey I have to earn my 
bike-shedding badge somehow! ;) Anyway,

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>

This now completes the series apart from I think two respins - rebase 
for possible extra unpin and missing lrc destructor.

Regards,

Tvrtko


More information about the Intel-gfx mailing list