fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2025-12-27 04:00:10 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

iris: Modify the comment about zeroing CCS

Browse source 
Among other changes, we highlight the fact that we'll map the CCS -
something we can't do on XeHP.

Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13555>
This commit is contained in:
Nanley Chery 2021-10-21 15:49:11 -07:00 • committed by Marge Bot
parent cba6d6cad3
commit eef4399afd
1 changed files with 5 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -831,18 +831,15 @@ iris_resource_configure_aux(struct iris_screen *screen,
case ISL_AUX_USAGE_GFX12_CCS_E:
case ISL_AUX_USAGE_STC_CCS:
case ISL_AUX_USAGE_MC:
/* When CCS_E is used, we need to ensure that the CCS starts off in
* a valid state. From the Sky Lake PRM, "MCS Buffer for Render
* Target(s)":
/* When CCS is used, we need to ensure that it starts off in a valid
* state. From the Sky Lake PRM, "MCS Buffer for Render Target(s)":
*
* "If Software wants to enable Color Compression without Fast
* clear, Software needs to initialize MCS with zeros."
*
* A CCS value of 0 indicates that the corresponding block is in the
* pass-through state which is what we want.
*
* For CCS_D, do the same thing. On Gfx9+, this avoids having any
* undefined bits in the aux buffer.
* A CCS surface initialized to zero is in the pass-through state. This
* state can avoid the need to ambiguate in some cases. We'll map and
* zero the CCS later on in iris_resource_init_aux_buf.
*/
if (imported) {
assert(res->aux.usage != ISL_AUX_USAGE_STC_CCS);