diff options
| author | Jason Ekstrand <[email protected]> | 2017-02-17 13:48:11 -0800 |
|---|---|---|
| committer | Jason Ekstrand <[email protected]> | 2017-02-23 12:10:42 -0800 |
| commit | 3885375195c9c62f7450beabb070a0e47cc11c58 (patch) | |
| tree | 50680251d74066eb81c9e9035846d1318538edc5 /src/intel/vulkan/anv_image.c | |
| parent | e6e8475b0f17e605e1c8251a076cc1d48734873b (diff) | |
intel/isl: Return surface creation success from aux helpers
The isl_surf_init call that each of these helpers make can, in theory,
fail. We should propagate that up to the caller rather than just
silently ignoring it.
Reviewed-by: Topi Pohjolainen <[email protected]>
Reviewed-by: Chad Versace <[email protected]>
Diffstat (limited to 'src/intel/vulkan/anv_image.c')
| -rw-r--r-- | src/intel/vulkan/anv_image.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index e2f7ca340fd..47d0a1ec333 100644 --- a/src/intel/vulkan/anv_image.c +++ b/src/intel/vulkan/anv_image.c @@ -209,8 +209,9 @@ make_surface(const struct anv_device *dev, anv_finishme("Test gen8 multisampled HiZ"); } else { assert(image->aux_surface.isl.size == 0); - isl_surf_get_hiz_surf(&dev->isl_dev, &image->depth_surface.isl, - &image->aux_surface.isl); + ok = isl_surf_get_hiz_surf(&dev->isl_dev, &image->depth_surface.isl, + &image->aux_surface.isl); + assert(ok); add_surface(image, &image->aux_surface); image->aux_usage = ISL_AUX_USAGE_HIZ; } |