[Mesa-dev] [PATCH 3/3] i965/fs: Add a MAX_GRF_SIZE define and use it various places

Matt Turner mattst88 at gmail.com
Thu Oct 2 11:53:27 PDT 2014


On Wed, Oct 1, 2014 at 12:18 PM, Jason Ekstrand <jason at jlekstrand.net> wrote:
> Previously, we had a MAX_SAMPLER_MESSAGE_SIZE which we used instead.
> However, some FB write messages can validly be longer than this so we need
> something different.  Since MAX_SAMPLER_MESSAGE_SIZE is validly useful on
> its own, we leave it alone and add a new MAX_GRF_SIZE that's big enough for
> FB writes.

I might call it MAX_FB_WRITE_SIZE or something. MAX_GRF_SIZE is a
little generic, but in the register allocation code it seems like the
thing you want to convey is "this is the largest group of registers
we'll allocate" and MAX_FB_WRITE_SIZE doesn't obviously do that.

MAX_VGRF_SIZE? That seems better for a future where we want to
indirectly address a large block of registers. But we're not going to
want to register coalesce arrays. Maybe need two macros?

Ken?


More information about the mesa-dev mailing list