i965: Emitting 3DSTATE_SO_BUFFER of 0-size.
authorSergii Romantsov <sergii.romantsov@gmail.com>
Wed, 15 Aug 2018 12:21:47 +0000 (15:21 +0300)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 15 Aug 2018 20:15:28 +0000 (13:15 -0700)
Avoided filling of whole structure and bo-allocation if
size of surface is 0.

Signed-off-by: Sergii Romantsov <sergii.romantsov@globallogic.com>
src/mesa/drivers/dri/i965/genX_state_upload.c

index ea5ad55be59a0d5a185a0554606f62b2fba9f35f..c051848985f2865f1aca39ac06c7aba1c48a0e1f 100644 (file)
@@ -3787,19 +3787,20 @@ genX(upload_3dstate_so_buffers)(struct brw_context *brw)
    for (int i = 0; i < 4; i++) {
       struct intel_buffer_object *bufferobj =
          intel_buffer_object(xfb_obj->Buffers[i]);
+      uint32_t start = xfb_obj->Offset[i];
+      uint32_t end = ALIGN(start + xfb_obj->Size[i], 4);
+      uint32_t const size = end - start;
 
-      if (!bufferobj) {
+      if (!bufferobj || !size) {
          brw_batch_emit(brw, GENX(3DSTATE_SO_BUFFER), sob) {
             sob.SOBufferIndex = i;
          }
          continue;
       }
 
-      uint32_t start = xfb_obj->Offset[i];
       assert(start % 4 == 0);
-      uint32_t end = ALIGN(start + xfb_obj->Size[i], 4);
       struct brw_bo *bo =
-         intel_bufferobj_buffer(brw, bufferobj, start, end - start, true);
+         intel_bufferobj_buffer(brw, bufferobj, start, size, true);
       assert(end <= bo->size);
 
       brw_batch_emit(brw, GENX(3DSTATE_SO_BUFFER), sob) {