drm/msm: Clean up and enhance the output of the 'gem' debugfs node

Add headers for the 'gem' debugfs file to make it easier to remember
what all the values mean and move the list of virtual address regions
to the next line and add the name and map status to make it clearer
what we are looking at.

Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org>
Signed-off-by: Rob Clark <robdclark@gmail.com>
This commit is contained in:
Jordan Crouse 2018-11-07 15:35:49 -07:00 committed by Rob Clark
parent c0ee979469
commit 575f048550

View File

@ -763,16 +763,23 @@ void msm_gem_describe(struct drm_gem_object *obj, struct seq_file *m)
break; break;
} }
seq_printf(m, "%08x: %c %2d (%2d) %08llx %p\t", seq_printf(m, "%08x: %c %2d (%2d) %08llx %p",
msm_obj->flags, is_active(msm_obj) ? 'A' : 'I', msm_obj->flags, is_active(msm_obj) ? 'A' : 'I',
obj->name, kref_read(&obj->refcount), obj->name, kref_read(&obj->refcount),
off, msm_obj->vaddr); off, msm_obj->vaddr);
/* FIXME: we need to print the address space here too */ seq_printf(m, " %08zu%9s\n", obj->size, madv);
list_for_each_entry(vma, &msm_obj->vmas, list)
seq_printf(m, " %08llx", vma->iova);
seq_printf(m, " %zu%s\n", obj->size, madv); if (!list_empty(&msm_obj->vmas)) {
seq_puts(m, " vmas:");
list_for_each_entry(vma, &msm_obj->vmas, list)
seq_printf(m, " [%s: %08llx,%s]", vma->aspace->name,
vma->iova, vma->mapped ? "mapped" : "unmapped");
seq_puts(m, "\n");
}
rcu_read_lock(); rcu_read_lock();
fobj = rcu_dereference(robj->fence); fobj = rcu_dereference(robj->fence);
@ -799,9 +806,10 @@ void msm_gem_describe_objects(struct list_head *list, struct seq_file *m)
int count = 0; int count = 0;
size_t size = 0; size_t size = 0;
seq_puts(m, " flags id ref offset kaddr size madv\n");
list_for_each_entry(msm_obj, list, mm_list) { list_for_each_entry(msm_obj, list, mm_list) {
struct drm_gem_object *obj = &msm_obj->base; struct drm_gem_object *obj = &msm_obj->base;
seq_printf(m, " "); seq_puts(m, " ");
msm_gem_describe(obj, m); msm_gem_describe(obj, m);
count++; count++;
size += obj->size; size += obj->size;