mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-09 20:32:04 +09:00
ANDROID: staging: ion: Obey kptr_restrict
Some debugging messages did not use %pK, but since those messages are not very useful and have been removed upstream, just remove them instead. Bug: 77937819 Change-Id: Ie45897fe2d6ec3f842a02883e8ec929ed2e76933 Signed-off-by: Alistair Strachan <astrachan@google.com>
This commit is contained in:
committed by
Amit Pundir
parent
d581f4e010
commit
e95d38f9c3
@@ -856,7 +856,6 @@ void ion_client_destroy(struct ion_client *client)
|
||||
struct ion_device *dev = client->dev;
|
||||
struct rb_node *n;
|
||||
|
||||
pr_debug("%s: %d\n", __func__, __LINE__);
|
||||
while ((n = rb_first(&client->handles))) {
|
||||
struct ion_handle *handle = rb_entry(n, struct ion_handle,
|
||||
node);
|
||||
@@ -947,9 +946,6 @@ static void ion_buffer_sync_for_device(struct ion_buffer *buffer,
|
||||
int pages = PAGE_ALIGN(buffer->size) / PAGE_SIZE;
|
||||
int i;
|
||||
|
||||
pr_debug("%s: syncing for device %s\n", __func__,
|
||||
dev ? dev_name(dev) : "null");
|
||||
|
||||
if (!ion_buffer_fault_user_mappings(buffer))
|
||||
return;
|
||||
|
||||
@@ -1003,7 +999,6 @@ static void ion_vm_open(struct vm_area_struct *vma)
|
||||
mutex_lock(&buffer->lock);
|
||||
list_add(&vma_list->list, &buffer->vmas);
|
||||
mutex_unlock(&buffer->lock);
|
||||
pr_debug("%s: adding %p\n", __func__, vma);
|
||||
}
|
||||
|
||||
static void ion_vm_close(struct vm_area_struct *vma)
|
||||
@@ -1011,14 +1006,12 @@ static void ion_vm_close(struct vm_area_struct *vma)
|
||||
struct ion_buffer *buffer = vma->vm_private_data;
|
||||
struct ion_vma_list *vma_list, *tmp;
|
||||
|
||||
pr_debug("%s\n", __func__);
|
||||
mutex_lock(&buffer->lock);
|
||||
list_for_each_entry_safe(vma_list, tmp, &buffer->vmas, list) {
|
||||
if (vma_list->vma != vma)
|
||||
continue;
|
||||
list_del(&vma_list->list);
|
||||
kfree(vma_list);
|
||||
pr_debug("%s: deleting %p\n", __func__, vma);
|
||||
break;
|
||||
}
|
||||
mutex_unlock(&buffer->lock);
|
||||
@@ -1375,7 +1368,6 @@ static int ion_release(struct inode *inode, struct file *file)
|
||||
{
|
||||
struct ion_client *client = file->private_data;
|
||||
|
||||
pr_debug("%s: %d\n", __func__, __LINE__);
|
||||
ion_client_destroy(client);
|
||||
return 0;
|
||||
}
|
||||
@@ -1387,7 +1379,6 @@ static int ion_open(struct inode *inode, struct file *file)
|
||||
struct ion_client *client;
|
||||
char debug_name[64];
|
||||
|
||||
pr_debug("%s: %d\n", __func__, __LINE__);
|
||||
snprintf(debug_name, 64, "%u", task_pid_nr(current->group_leader));
|
||||
client = ion_client_create(dev, debug_name);
|
||||
if (IS_ERR(client))
|
||||
|
||||
@@ -49,8 +49,6 @@ static int ion_cma_allocate(struct ion_heap *heap, struct ion_buffer *buffer,
|
||||
struct device *dev = cma_heap->dev;
|
||||
struct ion_cma_buffer_info *info;
|
||||
|
||||
dev_dbg(dev, "Request buffer allocation len %ld\n", len);
|
||||
|
||||
if (buffer->flags & ION_FLAG_CACHED)
|
||||
return -EINVAL;
|
||||
|
||||
@@ -78,7 +76,6 @@ static int ion_cma_allocate(struct ion_heap *heap, struct ion_buffer *buffer,
|
||||
goto free_table;
|
||||
/* keep this for memory release */
|
||||
buffer->priv_virt = info;
|
||||
dev_dbg(dev, "Allocate buffer %p\n", buffer);
|
||||
return 0;
|
||||
|
||||
free_table:
|
||||
@@ -96,7 +93,6 @@ static void ion_cma_free(struct ion_buffer *buffer)
|
||||
struct device *dev = cma_heap->dev;
|
||||
struct ion_cma_buffer_info *info = buffer->priv_virt;
|
||||
|
||||
dev_dbg(dev, "Release buffer %p\n", buffer);
|
||||
/* release memory */
|
||||
dma_free_coherent(dev, buffer->size, info->cpu_addr, info->handle);
|
||||
/* release sg table */
|
||||
|
||||
Reference in New Issue
Block a user