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

anv: fix invalid binding table index computation

Browse Source 
The ++ operator strikes again.

Fixes: f92c5bc8f3 ("anv/device: fix maximum number of images supported")
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
This commit is contained in:
Lionel Landwerlin
2019-01-17 18:19:59 +00:00
committed by Mark Janes
parent c4c5c90255
commit 4149d41f2e
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/intel/vulkan/genX_cmd_buffer.c
View File

@@ -2178,8 +2178,6 @@ emit_binding_table(struct anv_cmd_buffer *cmd_buffer,
surface_state = sstate.state; surface_state = sstate.state;
assert(surface_state.alloc_size); assert(surface_state.alloc_size);
add_surface_state_relocs(cmd_buffer, sstate); add_surface_state_relocs(cmd_buffer, sstate);
image++;
if (devinfo->gen < 9) { if (devinfo->gen < 9) {
assert(image < MAX_GEN8_IMAGES); assert(image < MAX_GEN8_IMAGES);
struct brw_image_param *image_param = struct brw_image_param *image_param =
@@ -2188,6 +2186,7 @@ emit_binding_table(struct anv_cmd_buffer *cmd_buffer,
*image_param = *image_param =
desc->image_view->planes[binding->plane].storage_image_param; desc->image_view->planes[binding->plane].storage_image_param;
} }
image++;
break; break;
} }
@@ -2232,8 +2231,6 @@ emit_binding_table(struct anv_cmd_buffer *cmd_buffer,
assert(surface_state.alloc_size); assert(surface_state.alloc_size);
add_surface_reloc(cmd_buffer, surface_state, add_surface_reloc(cmd_buffer, surface_state,
desc->buffer_view->address); desc->buffer_view->address);
image++;
if (devinfo->gen < 9) { if (devinfo->gen < 9) {
assert(image < MAX_GEN8_IMAGES); assert(image < MAX_GEN8_IMAGES);
struct brw_image_param *image_param = struct brw_image_param *image_param =
@@ -2241,6 +2238,7 @@ emit_binding_table(struct anv_cmd_buffer *cmd_buffer,
*image_param = desc->buffer_view->storage_image_param; *image_param = desc->buffer_view->storage_image_param;
} }
image++;
break; break;
default: default: