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

i965: Don't use MAX_SURFACES in mark_surface_used

Browse Source 
Vulkan doesn't respect MAX_SURFACES so this assert isn't valid in that
case.  It should, however, assert that it isn't insanely large.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
This commit is contained in:
Jason Ekstrand
2017-02-28 17:15:04 -08:00
parent 0d2c9ce1ce
commit 035616cb8e
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/mesa/drivers/dri/i965/brw_shader.cpp
View File

@@ -33,7 +33,10 @@ extern "C" void
brw_mark_surface_used(struct brw_stage_prog_data *prog_data, brw_mark_surface_used(struct brw_stage_prog_data *prog_data,
unsigned surf_index) unsigned surf_index)
{ {
assert(surf_index < BRW_MAX_SURFACES); /* A binding table index is 8 bits and the top 3 values are reserved for
* special things (stateless and SLM).
*/
assert(surf_index <= 252);
prog_data->binding_table.size_bytes = prog_data->binding_table.size_bytes =
MAX2(prog_data->binding_table.size_bytes, (surf_index + 1) * 4); MAX2(prog_data->binding_table.size_bytes, (surf_index + 1) * 4);