fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-09 04:38:03 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

iris: Don't zero the CCS in an already zeroed BO

Browse source 
Avoid redundant work. Includes a refactor that will be helpful for HiZ
in the next patch.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/36383>
This commit is contained in:
Nanley Chery 2025-07-25 08:06:08 -04:00 • committed by Marge Bot
parent ef4f4d3f84
commit 5964c31429
1 changed files with 14 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
src/gallium/drivers/iris/iris_resource.c
View file

@ -980,23 +980,27 @@ iris_resource_init_aux_buf(struct iris_screen *screen,
{
const struct intel_device_info *devinfo = screen->devinfo;
if (isl_aux_usage_has_ccs(res->aux.usage) && devinfo->ver <= 11) {
/* Initialize the CCS on BDW-ICL to the PASS_THROUGH state. This avoids
* the need to ambiguate in some cases.
*/
bool zero_aux = res->bo->zeroed;
/* Initialize CCS on BDW-ICL to the PASS_THROUGH state. We don't make use
* of the RESOLVED state for CCS, so this ensures that we won't have to
* perform an ambiguate operation.
*/
if (isl_aux_usage_has_ccs(res->aux.usage) && devinfo->ver <= 11)
zero_aux = true;
if (zero_aux != res->bo->zeroed) {
void* map = iris_bo_map(NULL, res->bo, MAP_WRITE | MAP_RAW);
if (!map)
return false;
memset((char*)map + res->aux.offset, 0, res->aux.surf.size_B);
iris_bo_unmap(res->bo);
res->aux.state = create_aux_state_map(res, ISL_AUX_STATE_PASS_THROUGH);
} else {
const enum isl_aux_state initial_state =
isl_aux_get_initial_state(devinfo, res->aux.usage, res->bo->zeroed);
res->aux.state = create_aux_state_map(res, initial_state);
}
const enum isl_aux_state initial_state =
isl_aux_get_initial_state(devinfo, res->aux.usage, zero_aux);
res->aux.state = create_aux_state_map(res, initial_state);
if (!res->aux.state)
return false;