[Intel-gfx] [PATCH v2] drm/i915: Expand ggtt_view paramters for debugfs

[Tvrtko Ursulin]

On 12/01/2017 11:06, Chris Wilson wrote:
> When dumping the VMA, include the parameters of the different GGTT views
> so that we can distinguish them.
>
> v2: Contract output and add MISSING_CASE for any unknown types.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin at linux.intel.com>
> ---
>  drivers/gpu/drm/i915/i915_debugfs.c | 28 ++++++++++++++++++++++++++--
>  1 file changed, 26 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index 9d7b5a8c8dea..72bec5eb2f18 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -159,8 +159,32 @@ describe_obj(struct seq_file *m, struct drm_i915_gem_object *obj)
>  		seq_printf(m, " (%sgtt offset: %08llx, size: %08llx",
>  			   i915_vma_is_ggtt(vma) ? "g" : "pp",
>  			   vma->node.start, vma->node.size);
> -		if (i915_vma_is_ggtt(vma))
> -			seq_printf(m, ", type: %u", vma->ggtt_view.type);
> +		if (i915_vma_is_ggtt(vma)) {
> +			switch (vma->ggtt_view.type) {
> +			case I915_GGTT_VIEW_NORMAL:
> +				seq_puts(m, ", normal");
> +				break;
> +
> +			case I915_GGTT_VIEW_PARTIAL:
> +				seq_printf(m, ", partial [%08llx+%x]",
> +					   vma->ggtt_view.params.partial.offset,
> +					   vma->ggtt_view.params.partial.size << PAGE_SHIFT);
> +				break;
> +
> +			case I915_GGTT_VIEW_ROTATED:
> +				seq_printf(m, ", rotated [(%ux%u, %d), (%ux%u, %d)]",

It's all %u AFAICS. Also there is offset which I missed previously. If 
you add it we probably need to name the stride and offset fields then? 
Like (%ux%u, stride=%u, offset=%u)?

> +					   vma->ggtt_view.params.rotated.plane[0].width,
> +					   vma->ggtt_view.params.rotated.plane[0].height,
> +					   vma->ggtt_view.params.rotated.plane[0].stride,
> +					   vma->ggtt_view.params.rotated.plane[1].width,
> +					   vma->ggtt_view.params.rotated.plane[1].height,
> +					   vma->ggtt_view.params.rotated.plane[1].stride);
> +				break;
> +			default:
> +				MISSING_CASE(vma->ggtt_view.type);
> +				break;
> +			}
> +		}
>  		if (vma->fence)
>  			seq_printf(m, " , fence: %d%s",
>  				   vma->fence->id,
>

With the above you can add my r-b.

Regards,

Tvrtko