Re: wined3d: Add newer ATI/AMD cards vendor recognition

[Henri Verbeet]

On 31 January 2010 14:59, Luca Bennati <lucak3(a)gmail.com> wrote:
> As seen in bug #21515, newer ATI cards used with radeon open driver return a
> GL_VENDOR string not present in wined3d_guess_vendor().
> This oneliner merely adds that so that the function return VENDOR_MESA
> instead of VENDOR_WINE that could make apps crash.

I think this patch is ok.

We will need to distinguish between card vendor and GL implementation
vendor, but that's beyond the scope of this patch, I'll fix that. Just
reporting VENDOR_ATI here without making that distinction first would
mean we potentially apply fglrx quirks for Mesa.

wined3d: Add newer ATI/AMD cards vendor recognition

[Stefan Dösinger]

Am 01.02.2010 um 11:35 schrieb Henri Verbeet:
> On 31 January 2010 14:59, Luca Bennati <lucak3(a)gmail.com> wrote:
> > As seen in bug #21515, newer ATI cards used with radeon open driver return a
> > GL_VENDOR string not present in wined3d_guess_vendor().
> > This oneliner merely adds that so that the function return VENDOR_MESA
> > instead of VENDOR_WINE that could make apps crash.
>
> I think this patch is ok.
>
> We will need to distinguish between card vendor and GL implementation
> vendor, but that's beyond the scope of this patch, I'll fix that. Just
> reporting VENDOR_ATI here without making that distinction first would
> mean we potentially apply fglrx quirks for Mesa
> Agreed