[Bug 31590] New: AMD Radeon HD 7850 with 2GB video memory is recognized as Radeon HD 3200 with 128MB video memory.
http://bugs.winehq.org/show_bug.cgi?id=31590
Bug #: 31590 Summary: AMD Radeon HD 7850 with 2GB video memory is recognized as Radeon HD 3200 with 128MB video memory. Product: Wine Version: unspecified Platform: x86 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: futureway@asia.com Classification: Unclassified
AMD Radeon HD 7850 with 2GB video memory is recognized as Radeon HD 3200 with 128MB video memory by Wine. Some games can't run, because they require a more advanced graphics card and more video memory. When will Wine add AMD Radeon HD 7000 series to the list of supported graphics cards, and automatically recognize them and their video memory correctly? I don't think that it is a good solution to manually add some registry keys such as "VideoMemorySize", "VideoPciDeviceID" and "VideoPciVendorID".
http://bugs.winehq.org/show_bug.cgi?id=31590
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |00cpxxx@gmail.com
--- Comment #1 from Bruno Jesus 00cpxxx@gmail.com 2012-08-31 20:20:00 CDT --- I think you could try to follow the example below to try fixing that.
http://source.winehq.org/git/wine.git/commitdiff/ada3dff134a1d711deafc495da2...
http://bugs.winehq.org/show_bug.cgi?id=31590
--- Comment #2 from Jiang Yike futureway@asia.com 2012-08-31 21:55:04 CDT --- Thank you for your suggestion!
But I hope that there will be an official version of Wine which can solve the problem instead of applying a patch.
http://bugs.winehq.org/show_bug.cgi?id=31590
--- Comment #3 from Bruno Jesus 00cpxxx@gmail.com 2012-08-31 22:29:37 CDT --- You can create the patch and submit it following the guidelines from http://wiki.winehq.org/SubmittingPatches
http://bugs.winehq.org/show_bug.cgi?id=31590
--- Comment #4 from Jiang Yike futureway@asia.com 2012-09-01 05:12:56 CDT --- I just wrote a patch text according to your posts. I don't use Git, so I don't know whether the text is correct, especially "index" and line numbers. Will you please have a look at it and modify it? Can you help me to submit it?
Thank you for your help!
Here is the patch text:
wined3d: Recognize AMD Radeon HD 7800 Series renderer string. ---
diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -1156,6 +1156,7 @@ static const struct gpu_description gpu_description_table[] = {HW_VENDOR_AMD, CARD_AMD_RADEON_HD7800, "AMD Radeon HD 7800 Series", DRIVER_AMD_RadeonSI, 2048}, /* Intel cards */ @@ -1711,6 +1711,7 @@ static enum wined3d_pci_device select_card_amd_binary(const struct wined3d_gl_in /* Southern Islands */ {"HD 7800", CARD_AMD_RADEON_HD7800}, /* Northern Islands */ diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1346,6 +1346,7 @@ enum wined3d_pci_device CARD_AMD_RADEON_HD7800 = 0x6819,
http://bugs.winehq.org/show_bug.cgi?id=31590
--- Comment #5 from Bruno Jesus 00cpxxx@gmail.com 2012-09-01 05:53:21 CDT --- Created attachment 41545 --> http://bugs.winehq.org/attachment.cgi?id=41545 Add Radeon 7800 series
Patch created from information in last comment and http://en.wikipedia.org/wiki/Southern_Islands_%28GPU_family%29#Radeon_HD_780...
http://bugs.winehq.org/show_bug.cgi?id=31590
--- Comment #6 from Jiang Yike futureway@asia.com 2012-09-01 06:36:49 CDT --- Thank you for your help! The patch doesn't seem to be listed in http://source.winehq.org/patches .
The open source driver of AMD Radeon HD 7000 series is named "RadeonSI" instead of "R600". Should the string "DRIVER_AMD_R600" be modified to "DRIVER_AMD_RadeonSI"?
http://bugs.winehq.org/show_bug.cgi?id=31590
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |hverbeet@gmail.com
--- Comment #7 from Bruno Jesus 00cpxxx@gmail.com 2012-09-01 07:06:07 CDT --- Henri, can you please take a look at the patch and in the question in the message above?
http://bugs.winehq.org/show_bug.cgi?id=31590
--- Comment #8 from Jiang Yike futureway@asia.com 2012-09-03 18:02:40 CDT --- If the AMD driver uses "RadeonSI", the file "dlls/wined3d/directx.c" should add a line:
41 enum wined3d_display_driver 42 { 43 DRIVER_AMD_RAGE_128PRO, 44 DRIVER_AMD_R100, 45 DRIVER_AMD_R300, 46 DRIVER_AMD_R600,
+ 47 DRIVER_AMD_RADEONSI,
"RadeonSI" is being developed, and it is not released yet. It will be the open source driver for AMD Radeon 7000 series. Is there any difference when choosing DRIVER_AMD_R600 or DRIVER_AMD_RADEONSI in the file "dlls/wined3d/directx.c"?
http://bugs.winehq.org/show_bug.cgi?id=31590
--- Comment #9 from Henri Verbeet hverbeet@gmail.com 2012-09-04 01:37:54 CDT --- (In reply to comment #7)
Henri, can you please take a look at the patch and in the question in the message above?
Looks ok at first sight, but please make sure you verified those values on an actual Windows system.
(In reply to comment #8)
If the AMD driver uses "RadeonSI", the file "dlls/wined3d/directx.c" should add a line:
It doesn't work like that. DRIVER_AMD_R600 refers to the information in driver_version_table[]. You should verify that matches Windows for this hardware.
http://bugs.winehq.org/show_bug.cgi?id=31590
--- Comment #10 from Jiang Yike futureway@asia.com 2012-09-04 04:17:35 CDT --- I don't use Windows, so I can't verify it. However, the present values (AMD Radeon HD 6000 series and HD 7900 series have been added to the two files) may be samples, can't they?
http://bugs.winehq.org/show_bug.cgi?id=31590
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |a308bde74aadaba2a6b742faf2a | |34b6bbbb3bf1e Status|UNCONFIRMED |RESOLVED Resolution| |FIXED
--- Comment #11 from Bruno Jesus 00cpxxx@gmail.com 2012-09-07 22:17:24 CDT --- Patch commited source.winehq.org/git/wine.git/commitdiff/a308bde74aadaba2a6b742faf2a34b6bbbb3bf1e
http://bugs.winehq.org/show_bug.cgi?id=31590
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #12 from Alexandre Julliard julliard@winehq.org 2012-09-14 13:36:59 CDT --- Closing bugs fixed in 1.5.13.
https://bugs.winehq.org/show_bug.cgi?id=31590
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |focht@gmx.net Component|-unknown |directx-d3d Version|unspecified |1.5.12
-
wine-bugs＠winehq.org