[Mesa-dev] [PATCH] i965: update MaxTextureRectSize to match PRMs and comply with OpenGL 4.1+

Kenneth Graunke kenneth at whitecape.org
Sat Jun 17 18:36:17 UTC 2017


On Saturday, June 17, 2017 10:37:48 AM PDT Emil Velikov wrote:
> On 17 June 2017 at 18:09, Kenneth Graunke <kenneth at whitecape.org> wrote:
> > On Friday, June 16, 2017 2:04:03 AM PDT Iago Toral Quiroga wrote:
> >> We were exposing 4096, but we can do up to 8192 in Gen4-6 and up to
> >> 16384 in gen7+. OpenGL 4.1+ requires at least 16384.
> >> ---
> >>  src/mesa/drivers/dri/i965/brw_context.c | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c
> >> index 2d91b37..157d855 100644
> >> --- a/src/mesa/drivers/dri/i965/brw_context.c
> >> +++ b/src/mesa/drivers/dri/i965/brw_context.c
> >> @@ -503,7 +503,7 @@ brw_initialize_context_constants(struct brw_context *brw)
> >>     ctx->Const.Max3DTextureLevels = 12; /* 2048 */
> >>     ctx->Const.MaxArrayTextureLayers = brw->gen >= 7 ? 2048 : 512;
> >>     ctx->Const.MaxTextureMbytes = 1536;
> >> -   ctx->Const.MaxTextureRectSize = 1 << 12;
> >> +   ctx->Const.MaxTextureRectSize = brw->gen >= 7 ? 16384 : 8192;
> >>     ctx->Const.MaxTextureMaxAnisotropy = 16.0;
> >>     ctx->Const.MaxTextureLodBias = 15.0;
> >>     ctx->Const.StripTextureBorder = true;
> >>
> >
> > Good catch - thanks for fixing this!
> >
> > Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
> >
> Do we want it in -stable?
> 
> -Emil
> 

Sounds like a good idea to me.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170617/eabd96b3/attachment.sig>


More information about the mesa-dev mailing list