Skip to content

Conversation

@yhe39
Copy link
Contributor

@yhe39 yhe39 commented Oct 23, 2025

No description provided.

Yiwei Zhang and others added 30 commits July 12, 2022 21:49
… am: e2509fe

Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2129812

Change-Id: I58fe1cbcda340a888887c62ec507bdf0eb076d51
Signed-off-by: Automerger Merge Worker <[email protected]>
…a158

Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2151993

Change-Id: Idd7eaa0c9b68f4cd7d59ff34469ce609dc9221bd
Signed-off-by: Automerger Merge Worker <[email protected]>
… am: e2509fe am: 332be8b

Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2129812

Change-Id: I4afe4a95b9a8b01207778c7735b73daae0709da8
Signed-off-by: Automerger Merge Worker <[email protected]>
…a158 am: e3c09fc

Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2151993

Change-Id: I2d2ccd026e00c15d4048be45b4d0cfb488037b12
Signed-off-by: Automerger Merge Worker <[email protected]>
… am: e2509fe am: 332be8b am: 28c5bc3

Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2129812

Change-Id: Ic92ec0c49e96a05483b5bd43c20a7dc3bdc778bc
Signed-off-by: Automerger Merge Worker <[email protected]>
…a158 am: e3c09fc am: d89f6ca

Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2151993

Change-Id: Icd2806bf7c28b8d4db75a0a296595d9309bf09f6
Signed-off-by: Automerger Merge Worker <[email protected]>
…-s ours

am skip reason: Merged-In I1257909949c4831fbbc3064cea269c429da16748 with SHA-1 722a08b is already in history

Original change: https://googleplex-android-review.googlesource.com/c/platform/external/minigbm/+/19250631

Change-Id: I3aa0980a617d057067b3f2d175754146b4b11fab
Signed-off-by: Automerger Merge Worker <[email protected]>
…o stage-aosp-master am: 21b565c -s ours

am skip reason: Merged-In I1257909949c4831fbbc3064cea269c429da16748 with SHA-1 722a08b is already in history

Original change: https://googleplex-android-review.googlesource.com/c/platform/external/minigbm/+/19250631

Change-Id: I6592c9204a44b121e90a2c75f44612b7e0fc2d7b
Signed-off-by: Automerger Merge Worker <[email protected]>
…-s ours am: a9406de -s ours

am skip reason: Merged-In I1257909949c4831fbbc3064cea269c429da16748 with SHA-1 722a08b is already in history

Original change: https://googleplex-android-review.googlesource.com/c/platform/external/minigbm/+/19250631

Change-Id: Ie9a7e041d347dcc2bf96bc2281ed3f6115d98105
Signed-off-by: Automerger Merge Worker <[email protected]>
…o stage-aosp-master am: 21b565c -s ours am: 5f1f1e1 -s ours

am skip reason: Merged-In I1257909949c4831fbbc3064cea269c429da16748 with SHA-1 722a08b is already in history

Original change: https://googleplex-android-review.googlesource.com/c/platform/external/minigbm/+/19250631

Change-Id: Ib6e32b6503b2eb5c0151b11a31d0f030b338bae0
Signed-off-by: Automerger Merge Worker <[email protected]>
…-s ours am: a9406de -s ours am: 7f5e827 -s ours

am skip reason: Merged-In I1257909949c4831fbbc3064cea269c429da16748 with SHA-1 722a08b is already in history

Original change: https://googleplex-android-review.googlesource.com/c/platform/external/minigbm/+/19250631

Change-Id: I7bec51694e248a1b3cde3e0f9127ae6deb9204f6
Signed-off-by: Automerger Merge Worker <[email protected]>
…o stage-aosp-master am: 21b565c -s ours am: 5f1f1e1 -s ours am: 4fb982f -s ours

am skip reason: Merged-In I1257909949c4831fbbc3064cea269c429da16748 with SHA-1 722a08b is already in history

Original change: https://googleplex-android-review.googlesource.com/c/platform/external/minigbm/+/19250631

Change-Id: I357bcc88770ec0c72f5a9508a7fd6e78fbb9beb5
Signed-off-by: Automerger Merge Worker <[email protected]>
Harmless initialization logging should be info instead of error.

TEST=build and check logcat
BUG=b:234143058

Change-Id: I41ff39b428feb85d01663eec74b3f826007337c1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3759415
Commit-Queue: Dominik Behr <[email protected]>
Reviewed-by: Dominik Behr <[email protected]>
Commit-Queue: Yiwei Zhang <[email protected]>
Reviewed-by: Jason Macnak <[email protected]>
Tested-by: Yiwei Zhang <[email protected]>
TEST=./presubmit.sh

Change-Id: Ia53da4264de6ab637a21ad55029024c52a253c0f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3761444
Tested-by: Yiwei Zhang <[email protected]>
Reviewed-by: Jason Macnak <[email protected]>
Auto-Submit: Yiwei Zhang <[email protected]>
Reviewed-by: Dominik Behr <[email protected]>
Commit-Queue: Yiwei Zhang <[email protected]>
TEST=build

Change-Id: I78b123f395fef8853f310a567ac0341b6958c535
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3761445
Reviewed-by: Jason Macnak <[email protected]>
Reviewed-by: Dominik Behr <[email protected]>
Commit-Queue: Yiwei Zhang <[email protected]>
Auto-Submit: Yiwei Zhang <[email protected]>
Tested-by: Yiwei Zhang <[email protected]>
TEST=build

Change-Id: I3f0021bfd05eb1ab7c3b35ae2f31d806bc0a58dc
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3761446
Reviewed-by: Jason Macnak <[email protected]>
Commit-Queue: Yiwei Zhang <[email protected]>
Tested-by: Yiwei Zhang <[email protected]>
Auto-Submit: Yiwei Zhang <[email protected]>
Blob formats (aka buffers) get represented as R8 textures with height
of one and width==size.  These actually have no particular stride
requirement, even though the host might ask for a pitch that would make
sense for a 2D R8 texture with height greater than one.

BUG=b:239335026
TEST=cts-tradefed run cts -m CtsNNAPITestCases

Change-Id: I293f03b1c4292ed50252ea272f7d9e47e6c64ee2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3770014
Reviewed-by: Yiwei Zhang <[email protected]>
Tested-by: Rob Clark <[email protected]>
Auto-Submit: Rob Clark <[email protected]>
Commit-Queue: Rob Clark <[email protected]>
Reviewed-by: Jason Macnak <[email protected]>
With arcvm, virtgpu_cross_domain.c in the host remaps YVU420_ANDROID to
YVU420, to avoid exposing custom DRM_FORMAT_x values in the API between
the host and guest.  Which means it expects identical behavior for both.
Fortunately, we can do this, provided that it is a single level+layer
(minigbm does not allocate anything more complex than non-mipmap 2D),
and not tiled/UBWC.  The latter constraint we can get away with because
NV12 is used for anything produced by hw.

BUG=b:238158981
TEST=cts-tradefed run cts -m CtsVideoTestCases

Change-Id: If1330340f84835643be7cf8cae95026cbcebf273
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3774456
Auto-Submit: Rob Clark <[email protected]>
Tested-by: Rob Clark <[email protected]>
Reviewed-by: Yiwei Zhang <[email protected]>
Commit-Queue: Rob Clark <[email protected]>
Commit-Queue: Yiwei Zhang <[email protected]>
This is required for modern Android atop a gralloc with the cross_domain
backend or other native backend.

BUG=b:238609372
TEST=build and AHB cts

Change-Id: Id963eeec62c4400cc009384e127c93588c99346e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3777566
Auto-Submit: Yiwei Zhang <[email protected]>
Tested-by: Yiwei Zhang <[email protected]>
Reviewed-by: Jason Macnak <[email protected]>
Commit-Queue: Yiwei Zhang <[email protected]>
BUG=b:238609372
TEST=build

Change-Id: If1aefed829b69070d5f01f20e7c6961afc15cd82
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3774933
Tested-by: Jason Macnak <[email protected]>
Reviewed-by: Yiwei Zhang <[email protected]>
Commit-Queue: Jason Macnak <[email protected]>
BUG=b:238609372
TEST= vts -m VtsHalSensorsV2_1TargetTest

Change-Id: I33d9afb41fa546c97faffdbbd996a4d65a5ee156
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3774934
Reviewed-by: Yiwei Zhang <[email protected]>
Commit-Queue: Jason Macnak <[email protected]>
Tested-by: Jason Macnak <[email protected]>
No need to introduce host storage and mapping to the guest for buffers
that only have CPU access.  Fixes slow performance of sw encoders.

BUG=b:238158981, b:239333100
TEST=cts-tradefed run cts -m CtsVideoTestCases -t android.video.cts.VideoEncoderDecoderTest#testAvcGoog0Perf0320x0240
TEST=cts-tradefed run cts -m CtsCameraTestCases -t android.hardware.camera2.cts.RecordingTest#testSupportedVideoSizes[1]

Change-Id: Iaf88a750ae215d3a02b8415ab93be06b08a7f550
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3777563
Reviewed-by: Yiwei Zhang <[email protected]>
Auto-Submit: Rob Clark <[email protected]>
Tested-by: Rob Clark <[email protected]>
Commit-Queue: Yiwei Zhang <[email protected]>
Commit-Queue: Rob Clark <[email protected]>
Test: build
Change-Id: I281daffb82b7e959cadd1073b15b45f8f0ec7722
Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2164289

Change-Id: I27b37026c2685b77c60f4a1cec3fb6eb81fc3d83
Signed-off-by: Automerger Merge Worker <[email protected]>
… am: abc4860

Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2164289

Change-Id: Ibe7387b75ee87cd36ad1c7110fc0652e405994bb
Signed-off-by: Automerger Merge Worker <[email protected]>
… am: abc4860 am: ae07d12

Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2164289

Change-Id: Ib971190bbc5eeee33acb7dd0c8911ba5439df7aa
Signed-off-by: Automerger Merge Worker <[email protected]>
… am: abc4860 am: ae07d12 am: 108625e

Original change: https://android-review.googlesource.com/c/platform/external/minigbm/+/2164289

Change-Id: I91286103d7ae550779a78515bff99168da1349ba
Signed-off-by: Automerger Merge Worker <[email protected]>
BUG=b:238609372
TEST= vts -m VtsHalSensorsV2_1TargetTest

Change-Id: Id50e8057a0cc3150bb6b29bd7bdaa31fb7faa1ef
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3774935
Tested-by: Jason Macnak <[email protected]>
Commit-Queue: Jason Macnak <[email protected]>
Reviewed-by: Yiwei Zhang <[email protected]>
BUG=b:238609372
TEST=vts -m VtsHalSensorsV2_1TargetTest

Change-Id: I37834f80d71e453e6e29b88198efc1584c109456
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3773926
Reviewed-by: Yiwei Zhang <[email protected]>
Commit-Queue: Jason Macnak <[email protected]>
Tested-by: Jason Macnak <[email protected]>
manxiaoliang and others added 29 commits March 24, 2025 02:29
* Logically dead code
* Invalid type in argument to printf format specifier

Tracked-On: OAM-131039
Signed-off-by: manxiaoliang <[email protected]>
Fixed: Memory crash occurs when the screencap command is executed.
Ported from projectceladon#206

Tracked-On: OAM-128783
Signed-off-by: Weifeng Liu <[email protected]>
Signed-off-by: Wan, Hao <[email protected]>
Raw pointers in handles_ may point to buffers_, so buffers_ must
be cleared after handles_ to avoid use-after-free.

Tracked-On: OAM-128790
Signed-off-by: Kuang, Zhonghao <[email protected]>
Also, remove TILING_Y/TILING_4 support for render formats
in hybrid gpu mode. iGPU doesn't support TILING_4, dGPU doesn't
support TILING_Y. When TILING_Y memory alloc by iGPU, but
composite or display by dGPU, there will be blur screen issue.

Do clflush when write often, otherwise it'll affect antutu video
decoding score. In this case, the buffer content is not changed.
So no need to write back.

Tracked-On: OAM-131309
Signed-off-by: ZhuChenyanX <[email protected]>
Use clflushopt if it is supported instead of clflush. clflushopt is more
optimized and suggested by the HW engineers. It has improved MTL unmap
operation times significantly.

BUG=b:274641129
TEST=camera preview should be minimum of 30fps

Tracked-On: OAM-131309
Signed-off-by: Nirmoy Das <[email protected]>
Signed-off-by: Sushma Venkatesh Reddy <[email protected]>
Change-Id: I9bf7e355d5ebdfb1647c578173123bb73db5571d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/4382538
Reviewed-by: Dominik Behr <[email protected]>
Signed-off-by: chenyanxzhu <[email protected]>
The codec bo alloc request contains VIDEO_ENCODER flag.
This flag will affect i915 to use legacy allocation, which will
not include local memory. And when SF use dgpu for composition,
it'll take much longer time. Add an estimation: when it's hw codec
format, use local memory.

Tracked-On: OAM-131309
Signed-off-by: chenyanxzhu <[email protected]>
If has the GRALLOC_USAGE_PRIVATE_2, use local memory,
therwise use the system memroy.
By default use the sys memory, dGPU only changes
to use local memory to improve performance.

Tracked-On: OAM-130613
Signed-off-by: HeYue <[email protected]>
Add BMG device 0xE20B, 0xE20C, 0xE210, 0xE211

Tracked-On: OAM-131719
Signed-off-by: Kui Wen <[email protected]>
Tracked-On: OAM-131934
Signed-off-by: chenyanxzhu <[email protected]>
…) and cros_gralloc_driver::~cros_gralloc_driver() when meet race condition which would triggered by running screencap cmd

Tracked-On: OAM-130910
Signed-off-by: Gong Pu <[email protected]>
Changes done:
- Added xe version check to set gpu type
- Indentation fixes
- Query fake offset for subsequence mmap call irrespective of tiling
mode.

Tests done:
- Video playback
- Video encoding verified in MTL NUC using scrcpy

Tracked-On: OAM-133702
Signed-off-by: Jeevaka Prabu Badrappan <[email protected]>
Tracked-On: OAM-133702
Signed-off-by: HeYue <[email protected]>
The xe kernel check whether the size align with PAGE.

Tracked-On: OAM-133702
Signed-off-by: HeYue <[email protected]>
Tracked-On: OAM-133702
Signed-off-by: Kui Wen <[email protected]>
Fix low decoding score on antutu ux in lunarlake.

Tracked-On: OAM-133730
Signed-off-by: ZhuChenyanX <[email protected]>
Signed-off-by: HeYue <[email protected]>
Scrcpy needs WC bo, otherwise it'll has mos_bo_create_from_prime_xe
fail error. The AB24 bo uses VIDEO_ENCODER flag, but no CPU_READ_OFTEN
flag.

Tracked-On: OAM-133730
Signed-off-by: ZhuChenyanX <[email protected]>
Tracked-On: OAM-133773
Signed-off-by: ZhuChenyanX <[email protected]>
Tracked-On: OAM-133826
Signed-off-by: HeYue <[email protected]>
Tracked-On: OAM-133857
Signed-off-by: ZhuChenyanX <[email protected]>
RPL MTL use i915 driver

Tracked-On: OAM-134007
Signed-off-by: Kanli Hu <[email protected]>
Tracked-On: OAM-134096
Signed-off-by: ZhuChenyanX <[email protected]>
There is no DRM_FORMAT_ABGR10101010 defined.
Transfer the HAL_PIXEL_FORMAT_RGBA_10101010 to
DRM_FORMAT_ABGR2101010 so far.

Tracked-On: OAM-134109
Signed-off-by: HeYue <[email protected]>
Fix the unsupported fomat 0x3c error.

Tracked-On: OAM-134133
Signed-off-by: HeYue <[email protected]>
Fix low decoding score on antutu ux in lunarlake.

Tracked-On: OAM-134069
Signed-off-by: ZhuChenyanX <[email protected]>
Signed-off-by: HeYue <[email protected]>
Scrcpy needs WC bo, otherwise it'll has mos_bo_create_from_prime_xe
fail error. The AB24 bo uses VIDEO_ENCODER flag, but no CPU_READ_OFTEN
flag.

Tracked-On: OAM-134069
Signed-off-by: ZhuChenyanX <[email protected]>
The ARL use i915 backend, add the AB24 VIDEO_ENCODER flag support

Tracked-On: OAM-134179
Signed-off-by: HeYue <[email protected]>
On A16, integer valid range of enumeration type
'gralloc1_function_descriptor_t' should be in [0, 31].

Tracked-On: OAM-134305
Change-Id: I53ad1eeff73c8c0995b40137fd02a90a057e3f73
Signed-off-by: HeYue <[email protected]>
@yhe39 yhe39 closed this Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.