Age | Commit message (Collapse) | Author |
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/google/gchips/+/16135719
Change-Id: Iafd00f78da3bb5bdd36cd6fddd4fdf8a00f153b8
|
|
* changes:
Revert "Add delayed logger to allow logging verbose/info logs as warnings"
Revert "Commit all gralloc logs to warning on high memory allocation"
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/google/gchips/+/16074853
Change-Id: I5c73a63d4db00e4b5fdbdcea1c5d99638b6b518b
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/google/gchips/+/16074852
Change-Id: I1a9de08f6fd847c8e488aa16ee55ee1d72964d8c
|
|
This reverts commit d91ca7bd043f4bae411f298000726176c083d5b3.
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: Ifb3fb740169047aace14cfe202c663bd54625a4e
|
|
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
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/google/gchips/+/16120899
Change-Id: I862ffbd38bd5d58a147cbb4d4a4d1f883188d950
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/google/gchips/+/16120899
Change-Id: I607f07c9bd3c432462f9fbf42ad7a856f69df866
|
|
The ndk_platform backend will soon be deprecated because the ndk backend
can serve the same purpose. This is to eliminate the confusion about
having two variants (ndk and ndk_platform) for the same ndk backend.
Bug: 161456198
Test: m
Merged-In: Icc9af3798ac89742fa56b1cb37d8116d99b4a9c2
Change-Id: Icc9af3798ac89742fa56b1cb37d8116d99b4a9c2
(cherry picked from commit 567dfa8a668dd271f739711ad7dd6889f33b4d1f)
|
|
Dump buffers are only per process and is not available at the allocator
level. Gralloc should not be holding reference to that while allocating
a buffer.
Fix: 199336554
Test: Do not observe leaks in heap profiling
Change-Id: If92cffb2db9ee30ab1a5cee0297d066138c60674
|
|
Bug: 199336554
Test: Do not observe leaks in heap profiling
Change-Id: Iacab8555a1f425d7746fc7ab0937ddf9b1cf23d8
|
|
|
|
Bug: 201933520
Test: Boot to home
Change-Id: I125493e6b06436e4daac73c208cd5ca5ab4b68bd
|
|
Bug: 197289575
Test: Capture shot from UW camera, notice no failure logs
Change-Id: If41bf5501fe5bdb25300b78c8cda34e1dd353645
|
|
Exynos encoder need 16 stride alignment for both luma and chroma
in YV12. Add 32 stride alignment for luma to ensure we meet the
requirement. This change will not impact the decoding case since
we use 64 stride alignment by default.
Bug: 201012738
Test: run vts -m VtsHalMediaC2V1_0TargetVideoEncTest
Change-Id: I7086366bfaf4f589e878c732b30b615034ccfa20
|
|
For some applications (TikTok, AliExpress, Shopee), the YUV layout
is not requested from GraphicBufferMapper API, causing a misalignment
if the resolution can't satisfy both 64 and 128 alignments.
To prevent luma stride misalignment with GPU stride alignment.
The luma plane will maintain the same `stride` size, and the
chroma plane will align to `stride/2`.
Bug: 199836131
Test: AliExpress/Shopee live streaming
Change-Id: Id8dd547ff60dec7be49d777f2a63081923d5a9cf
|
|
|
|
Some of the codec components run in minijail which blocks getppid calls.
We should update the rules in codec as well to actually allow this
syscall. For now this should work as a WA.
Bug: 195608566
Test: Video playback in photos app
Change-Id: I63cb1a74ebbef8e1fbc17a05a253f94c9f571402
|
|
* changes:
Commit all gralloc logs to warning on high memory allocation
Add delayed logger to allow logging verbose/info logs as warnings
Log the calling process's PID
|
|
Bug: 192821955
Test: Manually verified
Change-Id: Ib1f3cfdae6bf5280c5ee85fbbfa11f1e2da7cbdc
|
|
Bug: 192821955
Test: Manually verified
Change-Id: I3d0f30b0dad3a5784659c47ce70e239aff834a7d
|
|
Bug: 192821955
Test: Logcat
Change-Id: Ic93aa915d76f10dac0a9084240df8e403497090e
|
|
libGralloc4Wrapper currently exposes reference retain and free functions
which call into Gralloc's internal reference-counting mechanisms. These
actually aren't necessary: the only thing that clients need is a function
that undoes the effect of importBuffer() on a locally-created buffer.
Removed ::retain and ::release, and instead added freeImportedHandle(),
which calls freeBuffer(), unregistering the buffer, and then also
manually unmaps the buffer's handles (which is necessary because
freeBuffer() doesn't normally do that).
Bug: 193573573
Test: many photos with rear and front camera
Change-Id: Iba339d35279512d75da01388c294e6c2191ede6a
|
|
|
|
Bug: 191912915
Test: Full-range video playback with known calls to set_dataspace
Change-Id: I9ee1419922a316f3ce0e303ab46c2e46d1b72fec
|
|
This is a workaround as codec is not locking the buffer before accessing
it.
Bug: 187145254
Test: CtsMediaV2TestCases
Change-Id: Iee8e688d40a404e78447f6f90b001f34566b56a1
|
|
Bug: 190688388
Test: Removed a no-op
Change-Id: Id3ff97cefda191699a824a8fce4dd2f74c124624
|
|
|
|
This potentially should improve the importBuffer calls as not all
clients need CPU mapped buffers.
This patch needs to be thoroughly tested as there are chances it might
break metadata reporting. I did the smoke test and it seems fine.
Fix: 190688388
Test: Yet to be tested
Change-Id: I2f98b1f9646a1336ded3747d2a24c6851a883d73
|
|
Some older kernels don't have this UAPI. Fall back to a local version
if and only if absolutely necessary.
Fixes: 191494136
Test: build
Change-Id: I76879c009903cf2886cee6cc6d339c9c85d72080
|
|
Fix: 191275214
Test: Video playback using SW decoder
Change-Id: Ieec84b51ba2cbf689555262e9ccd180f22df55ad
|
|
into sc-dev
|
|
|
|
The process crashed when it operated metadata on an unimported buffer.
To enhance the robustness of the system, the patch creates the check on
mali reference to avoid crashing.
Bug: 186739698
Test: android.mediav2.cts.EncoderColorAspectsTes
Change-Id: Ia9c074dcee0d0f3a8ef18170ea6bce153fa734a5
|
|
|
|
Fix: 190074890
Test: Lens viewfinder
Change-Id: Ia03cb5a961a7402c0c129d8f1ca10332cc7c19e5
|
|
Work around the fact that external hardware buffers with AFBC are
color-swapped in Vulkan.
Bug: 189467474
Test: Lens mode in Camera app
Change-Id: I0d7b225d6de24012000b989d923af72a44c3c929
|
|
Map camera YUV420_888 to NV21 when the consumer is GPU.
It was previously mapped to NV12.
Test: Build
Bug: 190204149
Change-Id: I174bebb5b8012a11737c0396f031b73917f3cb23
|
|
This reverts commit e3a678186d514650cca2e0464c26cb10912e4e72.
Reason for revert: All IPs now allow 64-byte alignment.
Bug: 188600370
Bug: 184793812
Bug: 185836979
Bug: 188226310
Change-Id: I2b9b20d2f3de7794590a613f40e173e263b487aa
|
|
The INTERNAL_Y8 and INTERNAL_YV12 are used by Camera in recording use
cases. The patch enables the linear support in camera.
Bug: 185281551
Test: Recording with INTERNAL_Y8 buffer allocation
Change-Id: Ida5b15736454ee8bc842e0ad8a3ec9eafb93f49a
|
|
Currently, only Camera has been verified to handle 64-byte strides
properly when writing buffers for the GPU. Narrow this down and emit an
error in the case of problematic usage combinations.
Bug: 188226310
Test: play a 480p h.264 video, camera viewfinder, record/play video
Change-Id: I7e82394ea40a49018705009da8b7d8aafa4aea26
|
|
The camera requires 64-byte stride alignment for a variety of formats.
Set this alignment if GPU usage is detected.
Test: Boot, use viewfinder
Bug: 185836979
Bug: 185065640
Bug: 184793812
Change-Id: I96a5abf5465fb583d5f3393dece431d3b60a2cdb
|
|
into sc-dev
|
|
GPU process is still involved in camera EAF and EIF handling. The patch
respects to the GPU usage in the producer flag and disables SBWC
accordingly.
Bug: 186092174
Test: Enable Camera BlurRefinement with correct resolution
Change-Id: I3b8715aaede8594296bfd31ca737b882bdb7b005
|
|
Bug: 157775531
Test: AV1 video playback
Change-Id: I05046ea004630a80ae22ea6a2625e4a29cb052ce
|
|
mali_gralloc_ion_sync is called if BOARD_USES_GRALLOC_ION_SYNC is set,
but currently the implementation is empty. Added an implementation that
passes through to libdmabufheap's sync interface.
Test: Set camera permissions to "Ask every time"; reboot; launch camera
Test: RealCalc
Fixes: 185289225
Fixes: 186613737
Fixes: 183735780
Change-Id: I8425d2d94c0202fa2dabf8ff60d66bc8d40b6b86
|
|
There is still an architecture problem in media frameworks that the
buffer usage is limited to 32 bit. It results the MFC IP flag cannot be
successfully passed to Gralloc module. Before Media frameworks resolves
the limitation in usage flag, the patch considers MFC as the only
consumer when VIDEO_ENCODER is specified.
Bug: b/163821791
Test: Enable Front Cam and Rear Cam
Test: 3P App compatibility test.
Change-Id: I404d49ed33762d3b621523761be2c8bb7c383498
|
|
This is a port of ag/13246326 for gralloc4
Bug: 175341085
Test: Video playback
Test: gfx-gralloc-nv12-sbwc-test (no new failures)
Change-Id: I7dcb7ce4c6e45d3ae5fcae9c2aa6c339cc57761e
|
|
Fixes: 183621062
Test: take a photo with the camera, look at logcat
Change-Id: I9d6623e6d539aa4810fe6550a2ccaa41d000fcf0
|
|
Test: PTS SecurePreview test passing
Bug: 184608844
Change-Id: Ia1396cfcf5df9d5bcc56c6ee34043b8548a49453
|