From 3885375195c9c62f7450beabb070a0e47cc11c58 Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Fri, 17 Feb 2017 13:48:11 -0800 Subject: 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 Reviewed-by: Chad Versace --- src/intel/vulkan/anv_image.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'src/intel/vulkan/anv_image.c') 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; } -- cgit v1.2.3