gpu: ion: Update debugfs show functions to show heap specific info

Message ID CAMcxFTQ6O9EYQSYjdZxStihtNL8AEX-zcZVGORm3nGgB-TZyyA@mail.gmail.com
State New
Headers show

Commit Message

Nishanth Peethambaran Dec. 12, 2012, 1:58 a.m.
The debugfs show functions for client and heap were showing info
for the heap type instead of showing info of the individual heap.

Change-Id: Id5afe7963c8ddfafae1f959ce48dd5c2a5fcca07
Signed-off-by: Nishanth Peethambaran <nishanth@broadcom.com>
---
 drivers/gpu/ion/ion.c |   18 +++++++++---------
 include/linux/ion.h   |    4 ++--
 2 files changed, 11 insertions(+), 11 deletions(-)

  *		to this allocation

Patch hide | download patch | download mbox

diff --git a/drivers/gpu/ion/ion.c b/drivers/gpu/ion/ion.c
index 6aa817a..48cda5d 100644
--- a/drivers/gpu/ion/ion.c
+++ b/drivers/gpu/ion/ion.c
@@ -413,7 +413,7 @@  struct ion_handle *ion_alloc(struct ion_client
*client, size_t len,
 		/* if the client doesn't support this heap type */
 		if (!((1 << heap->type) & client->heap_mask))
 			continue;
-		/* if the caller didn't specify this heap type */
+		/* if the caller didn't specify this heap id */
 		if (!((1 << heap->id) & heap_mask))
 			continue;
 		buffer = ion_buffer_create(heap, dev, len, align, flags);
@@ -597,11 +597,11 @@  static int ion_debug_client_show(struct seq_file
*s, void *unused)
 	for (n = rb_first(&client->handles); n; n = rb_next(n)) {
 		struct ion_handle *handle = rb_entry(n, struct ion_handle,
 						     node);
-		enum ion_heap_type type = handle->buffer->heap->type;
+		int id = handle->buffer->heap->id;

-		if (!names[type])
-			names[type] = handle->buffer->heap->name;
-		sizes[type] += handle->buffer->size;
+		if (!names[id])
+			names[id] = handle->buffer->heap->name;
+		sizes[id] += handle->buffer->size;
 	}
 	mutex_unlock(&client->lock);

@@ -1176,7 +1176,7 @@  static const struct file_operations ion_fops = {
 };

 static size_t ion_debug_heap_total(struct ion_client *client,
-				   enum ion_heap_type type)
+				   int id)
 {
 	size_t size = 0;
 	struct rb_node *n;
@@ -1186,7 +1186,7 @@  static size_t ion_debug_heap_total(struct
ion_client *client,
 		struct ion_handle *handle = rb_entry(n,
 						     struct ion_handle,
 						     node);
-		if (handle->buffer->heap->type == type)
+		if (handle->buffer->heap->id == id)
 			size += handle->buffer->size;
 	}
 	mutex_unlock(&client->lock);
@@ -1207,7 +1207,7 @@  static int ion_debug_heap_show(struct seq_file
*s, void *unused)
 	for (n = rb_first(&dev->clients); n; n = rb_next(n)) {
 		struct ion_client *client = rb_entry(n, struct ion_client,
 						     node);
-		size_t size = ion_debug_heap_total(client, heap->type);
+		size_t size = ion_debug_heap_total(client, heap->id);
 		if (!size)
 			continue;
 		if (client->task) {
@@ -1228,7 +1228,7 @@  static int ion_debug_heap_show(struct seq_file
*s, void *unused)
 	for (n = rb_first(&dev->buffers); n; n = rb_next(n)) {
 		struct ion_buffer *buffer = rb_entry(n, struct ion_buffer,
 						     node);
-		if (buffer->heap->type != heap->type)
+		if (buffer->heap->id != heap->id)
 			continue;
 		total_size += buffer->size;
 		if (!buffer->handle_count) {
diff --git a/include/linux/ion.h b/include/linux/ion.h
index a7d399c..d8168fb 100644
--- a/include/linux/ion.h
+++ b/include/linux/ion.h
@@ -135,7 +135,7 @@  void ion_client_destroy(struct ion_client *client);
  * @len:	size of the allocation
  * @align:	requested allocation alignment, lots of hardware blocks have
  *		alignment requirements of some kind
- * @heap_mask:	mask of heaps to allocate from, if multiple bits are set
+ * @heap_mask:	mask of heap ids to allocate from, if multiple bits are set
  *		heaps will be tried in order from lowest to highest order bit
  * @flags:	heap flags, the low 16 bits are consumed by ion, the high 16
  *		bits are passed on to the respective heap and can be heap
@@ -236,7 +236,7 @@  struct ion_handle *ion_import_dma_buf(struct
ion_client *client, int fd);
  * struct ion_allocation_data - metadata passed from userspace for allocations
  * @len:	size of the allocation
  * @align:	required alignment of the allocation
- * @heap_mask:	mask of heaps to allocate from
+ * @heap_mask:	mask of heap ids to allocate from
  * @flags:	flags passed to heap
  * @handle:	pointer that will be populated with a cookie to use to refer