[Mesa-dev] [PATCH] i965: Fix missing BRW_NEW_FS_PROG_DATA in gen6_renderbuffer_surfaces.
emil.l.velikov at gmail.com
Sat Aug 1 09:43:26 PDT 2015
On 6 July 2015 at 21:40, Kenneth Graunke <kenneth at whitecape.org> wrote:
> On Monday, July 06, 2015 12:18:18 PM Matt Turner wrote:
>> On Mon, Jul 6, 2015 at 9:55 AM, Kenneth Graunke <kenneth at whitecape.org> wrote:
>> > It looks like this was forgotten in commit 3c9dc2d31b80fc73bffa1f40a
>> > (i965: Make a brw_stage_prog_data for storing the SURF_INDEX
>> > information.) In other words, it's been missing since we moved to
>> > dynamic binding table slot assignments.
>> Author: Eric Anholt <eric at anholt.net>
>> AuthorDate: Wed Oct 2 14:07:40 2013 -0700
>> CommitDate: Tue Oct 15 10:18:42 2013 -0700
>> How did you find this?
> After reading Eero's latest performance analysis observations, I decided
> to try marking render targets uncached (MOCS = UC) when blending and
> logic ops (i.e. things that cause render target reads) are disabled.
> This necessitated adding _NEW_COLOR. Then I realized
> BRW_NEW_FS_PROG_DATA was just plain missing.
Seems like this patch has fallen between the cracks. Can anyone take a
look or let me know if it's no longer applicable.
More information about the mesa-dev