[Mesa-dev] [PATCH 00/17] i965: Growing the batch buffer, separate state buffers

[Kenneth Graunke]

On Wednesday, September 6, 2017 2:21:51 AM PDT Chris Wilson wrote:
> Quoting Kenneth Graunke (2017-09-06 01:09:33)
> > A couple issues remain: the series drops the malloc'd shadow copy of
> > the batch for non-LLC systems.  I haven't checked how dire this is.
> > The last patch also dramatically impacts batch sizes, which we'll need
> > to benchmark.  However, I thought I'd get the code out there for review
> > before finishing that - I don't expect the mechanics to change much.
> 
> It's a slight regression atm (at least looking at piglit/drawoverhead)
> all due to that dastardly readback in brw_emit_surface_state().
> 
> I've been chicken and been switching it with a temporary alloca there.
> -Chris

Yeah, it looks to be about -3 to -4% on Synmark's OglBatch[67] as well.

I'll try and fix ISL to handle the relocs so we can avoid the readback.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170906/e28317f4/attachment.sig>