On Wednesday, July 19, 2017 3:09:17 AM PDT Chris Wilson wrote: > Even if we are using older kernels that do not accept the batch in the > first slot, we can simplify our code by creating the batch with itself > in the first slot and moving it to the end on execbuf submission. > --- > src/mesa/drivers/dri/i965/intel_batchbuffer.c | 70 > ++++++++++++--------------- > 1 file changed, 31 insertions(+), 39 deletions(-)
Alternatively, instead of swapping them out, we could simply add_exec_bo the
batch at the end, and in execbuffer() do:
if (!use_batch_first) {
execbuf.buffers_ptr++;
execbuf.buffers_count--;
}
to skip over the batchbuffer entry at the beginning. That seems easier...
It might make sense to just take this approach right away, effectively
squashing patches 8 and 9...
--Ken
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
