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.

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to