diff options
author | Jason Ekstrand <[email protected]> | 2016-11-07 08:07:43 -0800 |
---|---|---|
committer | Jason Ekstrand <[email protected]> | 2016-11-09 11:31:10 -0800 |
commit | 595400d57745fba198b42d95f3c4f5d855023c33 (patch) | |
tree | 8e3953d38208edd658eeac056a17ff793f6f02f2 /src/intel/vulkan/anv_private.h | |
parent | 0fe68294275a4ab1d9f71bb48d21a5e6b830e1ca (diff) |
anv/batch: Move last_ss_pool_bo_offset to the command buffer
The original reason for putting it in the batch_bo was to allow primaries
to share it across secondaries or something like that. However, the
relocation lists in secondary command buffers are are always left alone and
copied into the primary command buffer's relocation list. This means that
the offset really applies at the command buffer level and putting it in the
batch_bo doesn't make sense. This fixes a couple of potential bugs around
re-submission of command buffers that are not likely to be hit but are bugs
none the less.
Signed-off-by: Jason Ekstrand <[email protected]>
Reviewed-by: Kristian H. Kristensen <[email protected]>
Cc: "13.0" <[email protected]>
Diffstat (limited to 'src/intel/vulkan/anv_private.h')
-rw-r--r-- | src/intel/vulkan/anv_private.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h index 1025714a990..a60cd3ce4ac 100644 --- a/src/intel/vulkan/anv_private.h +++ b/src/intel/vulkan/anv_private.h @@ -641,9 +641,6 @@ struct anv_batch_bo { /* Bytes actually consumed in this batch BO */ size_t length; - /* Last seen surface state block pool bo offset */ - uint32_t last_ss_pool_bo_offset; - struct anv_reloc_list relocs; }; @@ -1188,7 +1185,10 @@ struct anv_cmd_buffer { */ struct u_vector bt_blocks; uint32_t bt_next; + struct anv_reloc_list surface_relocs; + /** Last seen surface state block pool center bo offset */ + uint32_t last_ss_pool_center; struct anv_execbuf execbuf2; |