iris: leave another TODO
authorKenneth Graunke <kenneth@whitecape.org>
Sat, 10 Nov 2018 08:08:03 +0000 (00:08 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 21 Feb 2019 18:26:10 +0000 (10:26 -0800)
src/gallium/drivers/iris/iris_batch.c

index 203ca139f66c69c1a486e2f1da8c6c3bbc2f5a86..1d3a787d3652ecbc665bceb30d06ceb553f52dd9 100644 (file)
@@ -215,6 +215,9 @@ add_exec_bo(struct iris_batch *batch, struct iris_bo *bo)
     * those first.
     */
    for (int b = 0; b < ARRAY_SIZE(batch->other_batches); b++) {
+      // XXX: this is bad, we use the same state / instruction buffers for
+      // both batches, and if both of them are reading some dynamic state,
+      // we flush all the time.  check for writes vs. reads?
       if (iris_batch_references(batch->other_batches[b], bo))
          iris_batch_flush(batch->other_batches[b]);
    }