[Mesa-dev] i965: Remove surface state Tile-X / Tile-Y offsets (GEN8 enabling)

[Chris Forbes]

Jordan, Abdiel:

Could you provide some context -- the impression I get is that using
the tile offsets is undesirable for reasons other than sharing code
with Gen8, but not familiar enough with the details to know why.

Just curious :)

-- Chris

On Tue, Dec 10, 2013 at 7:21 AM, Jordan Justen <jljusten at gmail.com> wrote:
> On Mon, Dec 9, 2013 at 8:18 AM, Abdiel Janulgue
> <abdiel.janulgue at linux.intel.com> wrote:
>> (cover letter seems to be lost in the way for the patch series but here it is)
>>
>> Gen 6/7 hardware support for tile_x and tile_y offset fields in SURFACE_STATE
>> has been removed in Gen8. All code dependencies must be removed. This patch
>> series completely eliminates the tile_x and y fields in the normal render
>> paths and BLORP. The LOD fields should be used instead.
>
> What about brw_update_renderbuffer_surface?
>
> I think we should be able to enable AMD_vertex_shader_layer for gen6
> once the offsets are gone.
>
> Did you get a chance to test piglit on gen6 too?
>
> -Jordan
>
>> This fixes https://bugs.freedesktop.org/show_bug.cgi?id=72068.
>>
>> CC: kenneth at whitecape.org
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev