Re: [5/5] WineD3D: Use GL_ATI_meminfo if available

[James McKenzie]

Henri Verbeet wrote:
> 2009/9/5 Stefan Dösinger :
> > -
> >    /* All GL_ATI_meminfo enums return 4 ints, even the (undocumented)
> > -
> >     * GL_TOTAL_PHYSICAL_MEMORY_ATI one, which returns {mem, 0, 0, 0} */
> > -
> >    GLint mem[4];
> > -
> >    /* Returns the vidmem in KB */
> > -
> >    glGetIntegerv(GL_TOTAL_PHYSICAL_MEMORY_ATI, mem);
> > -
> >    checkGLcall("glGetIntegerv(GL_TOTAL_PHYSICAL_MEMORY_ATI, mem)");
> > -
> > -
> >    /* The first driver version that supports ATI_meminfo doesn't implement
> > -
> >     * GL_TOTAL_PHYSICAL_MEMORY_ATI - catch this situation as adviced by AMD
> > -
> >     */
> > -
> >    if(mem[0] < 65536)
>
> If the implementation doesn't recognise "GL_TOTAL_PHYSICAL_MEMORY_ATI"
> you're testing uninitialized memory here. I hope AMD didn't actually
> advice you to code it like that.

Stefan:

There is or was a lurker here from AMD.  Maybe you could forward your
patches to them to get the extension fixed?

James McKenzie