mirror of
https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git
synced 2026-10-12 07:37:31 -04:00
ion: Improve ION allocation paths
Clean up some of the ION heap interfaces so that they are more inline/uniform with respect to each other. Change-Id: I4edabc2c8ccb533898540ceda1fd6aacc2e2e56a Signed-off-by: Isaac J. Manjarres <isaacm@codeaurora.org>
This commit is contained in:
parent
7e5cfaa6da
commit
dcdd5691d9
3 changed files with 23 additions and 4 deletions
|
|
@ -5,7 +5,7 @@
|
|||
* Copyright (C) Linaro 2012
|
||||
* Author: <benjamin.gaignard@linaro.org> for ST-Ericsson.
|
||||
*
|
||||
* Copyright (c) 2016-2019, The Linux Foundation. All rights reserved.
|
||||
* Copyright (c) 2016-2020, The Linux Foundation. All rights reserved.
|
||||
*/
|
||||
|
||||
#include <linux/device.h>
|
||||
|
|
@ -29,6 +29,11 @@ struct ion_cma_heap {
|
|||
container_of(to_msm_ion_heap(x), struct ion_cma_heap, heap)
|
||||
|
||||
/* ION CMA heap operations functions */
|
||||
static bool ion_heap_is_cma_heap_type(enum ion_heap_type type)
|
||||
{
|
||||
return type == ION_HEAP_TYPE_DMA;
|
||||
}
|
||||
|
||||
static int ion_cma_allocate(struct ion_heap *heap, struct ion_buffer *buffer,
|
||||
unsigned long len,
|
||||
unsigned long flags)
|
||||
|
|
@ -42,6 +47,13 @@ static int ion_cma_allocate(struct ion_heap *heap, struct ion_buffer *buffer,
|
|||
int ret;
|
||||
struct device *dev = cma_heap->heap.dev;
|
||||
|
||||
if (ion_heap_is_cma_heap_type(buffer->heap->type) &&
|
||||
is_secure_allocation(buffer->flags)) {
|
||||
pr_err("%s: CMA heap doesn't support secure allocations\n",
|
||||
__func__);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
if (align > CONFIG_CMA_ALIGNMENT)
|
||||
align = CONFIG_CMA_ALIGNMENT;
|
||||
|
||||
|
|
@ -49,7 +61,7 @@ static int ion_cma_allocate(struct ion_heap *heap, struct ion_buffer *buffer,
|
|||
if (!pages)
|
||||
return -ENOMEM;
|
||||
|
||||
if (!(flags & ION_FLAG_SECURE)) {
|
||||
if (hlos_accessible_buffer(buffer)) {
|
||||
if (PageHighMem(pages)) {
|
||||
unsigned long nr_clear_pages = nr_pages;
|
||||
struct page *page = pages;
|
||||
|
|
@ -68,7 +80,7 @@ static int ion_cma_allocate(struct ion_heap *heap, struct ion_buffer *buffer,
|
|||
}
|
||||
|
||||
if (MAKE_ION_ALLOC_DMA_READY ||
|
||||
(flags & ION_FLAG_SECURE) ||
|
||||
(!hlos_accessible_buffer(buffer)) ||
|
||||
(!ion_buffer_cached(buffer)))
|
||||
ion_pages_sync_for_device(dev, pages, size,
|
||||
DMA_BIDIRECTIONAL);
|
||||
|
|
|
|||
|
|
@ -31,6 +31,11 @@ bool is_secure_vmid_valid(int vmid)
|
|||
(!ret && vmid == trusted_vm_vmid));
|
||||
}
|
||||
|
||||
bool is_secure_allocation(unsigned long flags)
|
||||
{
|
||||
return !!(flags & (ION_FLAGS_CP_MASK | ION_FLAG_SECURE));
|
||||
}
|
||||
|
||||
int get_secure_vmid(unsigned long flags)
|
||||
{
|
||||
int ret;
|
||||
|
|
|
|||
|
|
@ -1,6 +1,6 @@
|
|||
/* SPDX-License-Identifier: GPL-2.0-only */
|
||||
/*
|
||||
* Copyright (c) 2017-2019, The Linux Foundation. All rights reserved.
|
||||
* Copyright (c) 2017-2020, The Linux Foundation. All rights reserved.
|
||||
*/
|
||||
|
||||
#include "msm_ion_priv.h"
|
||||
|
|
@ -22,4 +22,6 @@ int ion_hyp_assign_from_flags(u64 base, u64 size, unsigned long flags);
|
|||
|
||||
bool hlos_accessible_buffer(struct ion_buffer *buffer);
|
||||
|
||||
bool is_secure_allocation(unsigned long flags);
|
||||
|
||||
#endif /* _ION_SECURE_UTIL_H */
|
||||
|
|
|
|||
Loading…
Reference in a new issue