-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Eric Anholt wrote: > On Wed, 2009-04-01 at 18:30 -0700, Ian Romanick wrote: >> Track whether or not front-buffer rendering may have occured. If >> there might be any pending front-buffer rendering waiting to get to >> the screen, flush it from glFlush and related paths in the driver. >> >> Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> >> --- >> src/mesa/drivers/dri/intel/intel_buffers.c | 2 ++ >> src/mesa/drivers/dri/intel/intel_context.c | 11 +++++++++++ >> src/mesa/drivers/dri/intel/intel_context.h | 10 ++++++++++ >> 3 files changed, 23 insertions(+), 0 deletions(-) >> >> diff --git a/src/mesa/drivers/dri/intel/intel_buffers.c >> b/src/mesa/drivers/dri/intel/intel_buffers.c >> index f1908cb..26dd818 100644 >> --- a/src/mesa/drivers/dri/intel/intel_buffers.c >> +++ b/src/mesa/drivers/dri/intel/intel_buffers.c >> @@ -204,6 +204,8 @@ intel_draw_buffer(GLcontext * ctx, struct gl_framebuffer >> *fb) >> intel_batchbuffer_flush(intel->batch); >> intel->front_cliprects = GL_TRUE; >> colorRegions[0] = intel_get_rb_region(fb, BUFFER_FRONT_LEFT); >> + >> + intel->front_buffer_dirty = GL_TRUE; > > Why is DrawBuffers the right place for this? It looks like some > triangles, then glFlush, then some triangles, then glFlush, the last set > of triangles wouldn't show up.
/me puts a paper bag over his head. My goal was to set the bit in as few places as possible. I don't really care if we do extra flushes. I want to avoid setting the bit in 47 different places and missing one. I think setting the flag in DrawBuffers and only clearing it in intelFlush when the draw buffer is no long GL_FRONT is the right way to accomplish this. Thoughts? > Instead, I think I'd put the "we've done some rendering" flag setting > in: > intel_flush_primitive > intel_flush_inline_prim > brw_try_draw_prims > do_blit_bitmap() > do_blit_copypixels() > intelClearWithBlit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iEYEARECAAYFAknUSeUACgkQX1gOwKyEAw+coACfWL3Ajuj/tzdldp/VWnXESc+E 4wAAoI3l4kyQXx2S0vtovLGDKT5txOb6 =PVsT -----END PGP SIGNATURE----- ------------------------------------------------------------------------------ _______________________________________________ Mesa3d-dev mailing list Mesa3d-dev@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mesa3d-dev