Request for comments on patch

Erich Hoover ehoover at mines.edu
Sun Jun 14 14:38:58 CDT 2009


In an effort to come up with a (hopefully palatable) solution to a
couple of bugs I've put together the attached patch.  Only the first
of the two issues is currently handled, though the patch paves the way
for the second issue.

1) Bug #15839: (not this title) Wine does not report the manufacturer
driver (for example, "nv4_disp.dll" for nVidia users)
2) Wine does not always report the correct amount of video RAM (not
sure if there is a bug number for this)

The approach handles graphics requests like this:
"manufacturer" driver  -->   "OS" driver (normally winex11.drv)

Where the manufacturer driver (at present) forwards all requests to
the OS driver.  Later additions could provide driver-specific code for
returning information such as the correct amount of video RAM.  Also,
the manufacturer driver is skipped (reverting to the current behavior)
unless "HKEY_CURRENT_USER\Software\Wine\Drivers\ReportNativeVideoDriver"
is set to "Y".

Note: patch makes changes to configure.ac, so cover your bases:
autoheader && autoconf && ./configure && make depend && make

I'm particularly interested in comments related to how the driver is
selected (__wine_get_video_driver in gdi32), but all feedback is
welcome.

Erich Hoover
ehoover at mines.edu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-manufacturer-driver-on-top-of-OS-driver-winex.patch
Type: text/x-diff
Size: 35871 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20090614/b061ca27/attachment-0001.patch>


More information about the wine-devel mailing list