Ilia Mirkin <imir...@alum.mit.edu> writes: > On Thu, Aug 13, 2015 at 7:19 PM, Eric Anholt <e...@anholt.net> wrote: >> Ilia Mirkin <imir...@alum.mit.edu> writes: >> >>> Signed-off-by: Ilia Mirkin <imir...@alum.mit.edu> >>> --- >>> >>> The blending-in-shader piglit test passed even without the flush, >>> which doesn't inspire me with confidence, but those piglit_draw_rect >>> things are pretty heavy so perhaps cause a flush on their own anyways. >> >> I think you don't actually need the hook, because of the >> brw_render_cache_set_check_flush(brw, tex_obj->mt->bo) in >> intel_update_state(). You should be able to confirm that you're getting >> those pipe controls between your draws using INTEL_DEBUG=batch > > I'm a complete newbie when it comes to the i965 driver, but why would > intel_update_state() get called in between sequential draws? The idea > is that you can do things like > > tex0 = tex; > fb = tex; > draw; > barrier; > draw; > barrier; > draw; > > which is actually exactly what the blending-in-shader piglit does. So > there should be no extra state updates between them. Am I missing > something?
Not sure, was just giving my best guess as to why the test was working already. You'd probably need to trace and see if you always get an updatestate between draws.
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
