wined3d: Add Nvidia 8800GTX detection
shelnutt2 at gmail.com
Tue Jul 20 18:57:30 CDT 2010
For ati there is also GL_ATI_mem_info  or WGL_AMD_gpu_association  . I
believe that  is now obsolete as from reading some of the wine-dev
mailing list , and some other site it seems AMD is now pushing the using
of WGL_AMD_gpu_association instead. A patch could be written in the exact
same manor as your nvidia patch for ati. There doesn't appear to be an intel
or via extension to get video memory, although it's much less of a problem
for those integrated graphics.
One thing with your patch is that only since 190.x drivers has the extension
been introduced. So any card that is older than the 6xxx series won't
support the extension. So you could remove the code for the FX card and
below. Besides that I think your implementation is good; but I'm no dev, so
I'll let Roderick and Henri make the real comments.
On Tue, Jul 20, 2010 at 9:54 PM, Luke Bratch <l_bratch at yahoo.co.uk> wrote:
> --- On Tue, 20/7/10, Henri Verbeet <hverbeet at gmail.com> wrote:
> > Negative on using experimental extensions I'm afraid.
> > Writing a proof
> > of concept patch is ok, of course, and I'd be willing to
> > review that,
> > but it can't go in until the extension is finalized. Note
> > that this
> > wouldn't allow you to get rid of the current code though,
> > you still
> > need a fallback. It would just allow for more accurate
> > reporting on
> > cards that do support that extension.
> The attached uses that experimental extension  to report the correct
> total memory for cards that support it, while sticking with the existing
> hard coded values and code layout for those that don't.
> I would appreciate any comments.
> Luke Bratch
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the wine-devel