Henri Verbeet wrote: > 2009/9/5 Stefan Dösinger <[email protected]>: > >> + /* 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
