OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "gbm: mark surface buffers as explicit flushed"

Browse Source 
This reverts commit c49a71c03c.

It broke radeonsi.

GBM can't set __DRI_IMAGE_USE_BACKBUFFER if gbm itself doesn't use it as
a back buffer by rendering to it and calling SwapBuffers. If another
library uses it as a back buffer, that library should set
__DRI_IMAGE_USE_BACKBUFFER, not GBM. A different flag could be added
to indicate the behavior that the original commit expected.

Fixes: c49a71c03c - gbm: mark surface buffers as explicit flushed
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/11996
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32345>
This commit is contained in:
Marek Olšák
2024-11-26 02:18:57 -05:00
committed by Marge Bot
parent 1f954a207b
commit 1a7c54b840
2 changed files with 3 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/gbm/backends/dri/gbm_dri.c
View File

@@ -899,14 +899,8 @@ gbm_dri_bo_create(struct gbm_device *gbm,
dri_use |= __DRI_IMAGE_USE_LINEAR;
if (usage & GBM_BO_USE_PROTECTED)
dri_use |= __DRI_IMAGE_USE_PROTECTED;
if (usage & GBM_BO_USE_FRONT_RENDERING) {
assert (!(usage & GBM_BO_EXPLICIT_FLUSH));
if (usage & GBM_BO_USE_FRONT_RENDERING)
dri_use |= __DRI_IMAGE_USE_FRONT_RENDERING;
}
if (usage & GBM_BO_EXPLICIT_FLUSH) {
assert (!(usage & GBM_BO_USE_FRONT_RENDERING));
dri_use |= __DRI_IMAGE_USE_BACKBUFFER;
}
/* Gallium drivers requires shared in order to get the handle/stride */
dri_use |= __DRI_IMAGE_USE_SHARE;
@@ -1110,7 +1104,7 @@ gbm_dri_surface_create(struct gbm_device *gbm,
surf->base.v0.width = width;
surf->base.v0.height = height;
surf->base.v0.format = core->v0.format_canonicalize(format);
surf->base.v0.flags = flags | GBM_BO_EXPLICIT_FLUSH;
surf->base.v0.flags = flags;
if (!modifiers) {
assert(!count);
return &surf->base;

6
src/gbm/main/gbm.h
View File

@@ -330,11 +330,7 @@ enum gbm_bo_flags {
*/
GBM_BO_FIXED_COMPRESSION_12BPC = (13 << 7),
/**
* This buffer will be used with an API with well defined transition points
* between context internal usage and external visibility.
*/
GBM_BO_EXPLICIT_FLUSH = (1 << 11),
/* next available value is (1 << 11) */
};
#define GBM_BO_FIXED_COMPRESSION_MASK (((1 << 11) - 1) & ~((1 << 7) - 1))