[5/5] WineD3D: Use GL_ATI_meminfo if available

James McKenzie jjmckenzie51 at earthlink.net
Sun Sep 6 23:57:58 CDT 2009


Henri Verbeet wrote:
> 2009/9/5 Stefan Dösinger <stefan at codeweavers.com>:
>   
>> +        /* 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




More information about the wine-devel mailing list