diff options
author | Ankit Goyal <layog@google.com> | 2021-10-28 14:19:53 +0800 |
---|---|---|
committer | Ankit Goyal <layog@google.com> | 2021-10-29 19:00:33 +0800 |
commit | aa174c859bbaf62d794f9763bcdf925d959453b2 (patch) | |
tree | e19c333764d2e828d7ecc0909a251531bd39a1e2 | |
parent | 4a222191824349d125db714c281b9c09d88c2d0a (diff) | |
download | gchips-aa174c859bbaf62d794f9763bcdf925d959453b2.tar.gz |
Revert "Commit all gralloc logs to warning on high memory allocation"
This reverts commit 56831666f59b6bf33e1837669b6f752f973d0755.
Delayed logging leads to computing all the logging macros (ALOGV etc)
when they should be skipped. This leads to performance bottlenecks.
Bug: 204428645
Test: Builds
Change-Id: Id59480e77d0004b802d8dbe1fe26c69cd8550141
-rw-r--r-- | gralloc4/src/allocator/mali_gralloc_ion.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/gralloc4/src/allocator/mali_gralloc_ion.cpp b/gralloc4/src/allocator/mali_gralloc_ion.cpp index dd003c9..3123848 100644 --- a/gralloc4/src/allocator/mali_gralloc_ion.cpp +++ b/gralloc4/src/allocator/mali_gralloc_ion.cpp @@ -608,12 +608,6 @@ int mali_gralloc_ion_allocate(const gralloc_buffer_descriptor_t *descriptors, if (ion_fd >= 0 && fidx == 0) { fds[fidx] = ion_fd; } else { - // Warn of high memory allocation on size request greater than 200M - if (bufDescriptor->alloc_sizes[fidx] > static_cast<uint64_t>(200ull << 20)) { - log_info_verbose_as_warning(); - MALI_GRALLOC_LOGW("Huge memory allocation: %" PRIu64, bufDescriptor->alloc_sizes[fidx]); - } - fds[fidx] = dev->alloc_from_ion_heap(usage, bufDescriptor->alloc_sizes[fidx], ion_flags, &min_pgsz); } if (fds[fidx] < 0) |