intel/mi_builder: fix resolve call
authorLionel Landwerlin <lionel.g.landwerlin@intel.com>
Sat, 17 Apr 2021 15:21:26 +0000 (18:21 +0300)
committerMarge Bot <eric+marge@anholt.net>
Thu, 13 May 2021 17:02:41 +0000 (17:02 +0000)
Giving NULL for anv_combine_address() triggers an assert in that
function.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Fixes: 8525ebe6e375 ("intel/mi_builder: Return an address from __gen_get_batch_address")
Reviewed-by: Felix DeGrood <felix.j.degrood@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/10301>

src/intel/common/mi_builder.h

index a73f499..e95ef54 100644 (file)
@@ -1188,7 +1188,8 @@ _mi_resolve_address_token(struct mi_builder *b,
 {
    __gen_address_type addr = __gen_get_batch_address(b->user_data,
                                                     batch_location);
-   uint64_t addr_addr_u64 = __gen_combine_address(b->user_data, NULL, addr, 0);
+   uint64_t addr_addr_u64 = __gen_combine_address(b->user_data, batch_location,
+                                                  addr, 0);
    *(token.ptrs[0]) = addr_addr_u64;
    *(token.ptrs[1]) = addr_addr_u64 + 4;
 }