Kenneth Graunke <kenn...@whitecape.org> writes:

> In brw_update_renderbuffer_surfaces(), if there are no color draw
> buffers, we always set up a null render target at surface index 0 so we
> have something to use with the FB write marking the end of thread.
>
> However, when we recently began computing surface indexes dynamically,
> we failed to reserve space for it.  This meant that the first texture
> would be assigned surface index 0, and our closing FB write would
> clobber the texture.
>
> Fixes Piglit's EXT_packed_depth_stencil/fbo-blit-d24s8 test on Gen4-5,
> which regressed as of commit 4e5306453da6a1c076309e543ec92d999e02f67a
> ("i965/fs: Dynamically set up the WM binding table offsets.")
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70605
> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
> Cc: Eric Anholt <e...@anholt.net>
> Cc: "10.0" mesa-sta...@lists.freedesktop.org

Reviewed-by: Eric Anholt <e...@anholt.net>

Attachment: pgpwvCC6JgyKo.pgp
Description: PGP signature

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

Reply via email to