[Mesa-dev] [Mesa-stable] [PATCH] glx: use correct table offset for glAreTexturesResidentEXT

Andres Gomez agomez at igalia.com
Thu Aug 9 12:51:40 UTC 2018


Emil, this patch has been stalled in the -stable ML for quite some time
without update.

Unless you say otherwise, I will just ignore it at this point and trust
that you will also Cc -stable in the future, in case you come with
another version.

On Tue, 2017-09-26 at 18:52 +0200, Juan A. Suarez Romero wrote:
> On Tue, 2017-07-25 at 11:43 +0100, Emil Velikov wrote:
> > On 25 July 2017 at 09:12, Nicolai Hähnle <nhaehnle at gmail.com> wrote:
> > > On 21.07.2017 16:57, Emil Velikov wrote:
> > > > 
> > > > From: Emil Velikov <emil.velikov at collabora.com>
> > > > 
> > > > The correct offset is 322 as opposed to 332.
> > > > Broken since the rework of GET_DISPATCH back in ~2012.
> > > 
> > > 
> > > This is kind of amazing. How did you run into this?
> > > 
> > 
> > Looking at [truly] vendor neutral indirect libGLX, as opposed to the
> > current solution.
> > 
> > > Maybe add a simple touch test to piglit?
> > > 
> > 
> > Ack, will look into.
> > 
> > > Maybe use _gloffset_AreTexturesResident (from dispatch.h) instead of using a
> > > magic number?
> > > 
> > 
> > As the offending commit 99fee476a102 says "... can't use
> > src/mesa/main/dispatch.h ..."
> > 
> > At the same time, the function was moved outside of the python
> > generator(s) to address a out of bounds access - see commit
> > 13f96c5401f.
> > Sadly the same issue exists elsewhere - so I'm looking into addressing
> > that once and for all, and purging the handwritten function.
> > 
> > That's my evil plan ;-)
> 
> Emil, how's going your evil plan? :)
> 
> 	J.A.
> 
> > 
> > -Emil
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 
> _______________________________________________
> mesa-stable mailing list
> mesa-stable at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-stable
-- 
Br,

Andres


More information about the mesa-dev mailing list