April 2010 - Wine-devel - list.winehq.org

[rfc] realtime priority patch based on rtkit
by Maarten Lankhorst 17 Apr '10

17 Apr '10

Hi all, I decided to dig through the sample documentation for rtkit and see if I could find a way to make it work for wine. There's still a few things missing: hardcoded libdbus soname, no checking if dbus is available, -I/usr/include/dbus-1.0 -I/usr/lib/dbus-1.0/include should be added to ntdll/Makefile and SIGXCPU should demote all threads of the application back to normal priority The server queues an APC to the target thread that signals it should try to acquire rt prio for it. the thread makes a request to rtkit over dbus to say it wants realtime. This is because rtkit requires the affected process to call dbus. Any moral objections to this approach? Cheers, Maarten

2 2

re: Wine 2.0
by Dan Kegel 17 Apr '10

17 Apr '10

I would be fine with calling this wine 2.0, since it adds a major new feature (64 bit compatibility).

1 0

Re: winecfg italian translation update
by Nikolay Sivov 17 Apr '10

17 Apr '10

On 4/17/2010 15:14, Davide Pizzetti wrote: > Here's an italian translation update of winecfg > > - LTEXT "Le Dynamic Link Libraries possono essere specificate indiv > + LTEXT "Le Dynamic Link Libraries possono essere specificate indiv > ,IDC_STATIC,16,16,220,32 > You mail client ate the rest of lines (or something else does).

1 0

Re: winegcc: Fix compiling of 32-bit Winelib Apps on shared 32/64-bit setups
by Marcus Meissner 16 Apr '10

16 Apr '10

On Fri, Apr 16, 2010 at 10:51:29PM +0200, André Hentschel wrote: > --- > tools/winegcc/winegcc.c | 8 ++++++++ > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c > index 7023ff4..f780a69 100644 > --- a/tools/winegcc/winegcc.c > +++ b/tools/winegcc/winegcc.c > @@ -1252,6 +1252,14 @@ int main(int argc, char **argv) > opts.unicode_app = 1; > else if (strcmp("-m32", argv[i]) == 0) > { > + if (opts.target_cpu == CPU_x86_64) > + { > + const char *stdlib32path[] = { "/usr/lib32/wine", "/usr/local/lib32/wine", "/lib32/wine" }; lib32 is almost never used in the Linux distros though e.g. for SUSE: /usr/lib is the 32bit path on x86, ppc, s390 /usr/lib64 the 64bit path on x86_64, ppc64, s390x Just ia64 is an exception and has /usr/lib (and the compat libs somewhere weird). Ciao, Marcus

2 1

Re: winegcc: Fix compiling of 32-bit Winelib Apps on shared 32/64-bit setups
by Charles Davis 16 Apr '10

16 Apr '10

On 4/16/10 2:51 PM, André Hentschel wrote: > --- > tools/winegcc/winegcc.c | 8 ++++++++ > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c > index 7023ff4..f780a69 100644 > --- a/tools/winegcc/winegcc.c > +++ b/tools/winegcc/winegcc.c > @@ -1252,6 +1252,14 @@ int main(int argc, char **argv) > opts.unicode_app = 1; > else if (strcmp("-m32", argv[i]) == 0) > { > + if (opts.target_cpu == CPU_x86_64) > + { > + const char *stdlib32path[] = { "/usr/lib32/wine", "/usr/local/lib32/wine", "/lib32/wine" }; That's great for Linux, FreeBSD, etc., but in a world of universal binaries (read: Mac OS X), this ain't gonna fly. I'm pretty sure that when 64-bit Wine support does come to Mac OS X, people are going to want to build it universal (i.e. 32-bit and 64-bit binaries in the same file); I know I do. Chip

1 0

FYI: Wine is used for TeamViewer
by André Hentschel 16 Apr '10

16 Apr '10

Hey, you might take that as spam, but i just saw that Wine seems to be used for the Linux Version of http://www.teamviewer.com I did not yet had a look if they use wine64 for their 64-bit packages... -- Best Regards, André Hentschel

1 0

ICM tests and top-down DIB
by Joerg-Cyril.Hoehle＠t-systems.com 16 Apr '10

16 Apr '10

Hi, any suggestions about how could one reasonably test the ICM codecs? I found no test at all in Wine (but the 4 Austin and I once wrote). I was considering adding the ability to deal with a negative biHeight values (native amstream seems to use this to work around bug #17893) but I feel uncomfortable at touching the always sensitive graphics area without good tests to start with. Upside-down support in the codecs could start a landslide in other places. Negative biHeight indicates top-down bitmaps, see http://msdn.microsoft.com/en-us/library/dd183376.aspx Grepping through dlls/* revealed only few places that deal with them. Thanks for your help, Jörg Höhle.

1 0

Re: [PATCH] ntdll: split out sort and handle win64 cdecl
by Alexandre Julliard 16 Apr '10

16 Apr '10

Marcus Meissner <meissner(a)suse.de> writes: > qsort() calls a cdecl routine, but Win64 cdecl is sadly > not UNIX cdecl... So we need a wrapper. > > Hope this kind of ifdef magic is acceptable, I don't want > it for win32. That looks quite inefficient, not to mention not thread-safe. It would be better to reimplement these functions, that's not hard. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 3/4] wined3d: Separate AMD Radeon HD2300 from other HD2xxx/HD3xxx GPUs since it is D3D9 instead of D3D10.
by Roderick Colenbrander 16 Apr '10

16 Apr '10

Obviously this patch and the same for 4/4 can be dropped. I'm still a bit new to git send-email and my out directory wasn't empty, so it picked these two up. Roderick On Fri, Apr 16, 2010 at 8:49 AM, Roderick Colenbrander <thunderbird2k(a)gmail.com> wrote: > Just some clever AMD rebranding for China I believe. > --- > dlls/wined3d/directx.c | 30 ++++++++++++++++++------------ > dlls/wined3d/wined3d_private.h | 2 +- > 2 files changed, 19 insertions(+), 13 deletions(-) > > diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c > index 6498859..7e22dbc 100644 > --- a/dlls/wined3d/directx.c > +++ b/dlls/wined3d/directx.c > @@ -1021,7 +1021,7 @@ static const struct driver_version_information driver_version_table[] = > {HW_VENDOR_ATI, CARD_ATI_RADEON_9500, "ATI Radeon 9500", 14, 10, 6764 }, > {HW_VENDOR_ATI, CARD_ATI_RADEON_X700, "ATI Radeon X700 SE", 14, 10, 6764 }, > {HW_VENDOR_ATI, CARD_ATI_RADEON_X1600, "ATI Radeon X1600 Series", 14, 10, 6764 }, > - {HW_VENDOR_ATI, CARD_ATI_RADEON_HD2300, "ATI Mobility Radeon HD 2300", 14, 10, 6764 }, > + {HW_VENDOR_ATI, CARD_ATI_RADEON_HD2350, "ATI Mobility Radeon HD 2350", 14, 10, 6764 }, > {HW_VENDOR_ATI, CARD_ATI_RADEON_HD2600, "ATI Mobility Radeon HD 2600", 14, 10, 6764 }, > {HW_VENDOR_ATI, CARD_ATI_RADEON_HD2900, "ATI Radeon HD 2900 XT", 14, 10, 6764 }, > {HW_VENDOR_ATI, CARD_ATI_RADEON_HD4350, "ATI Radeon HD 4350", 14, 10, 6764 }, > @@ -1581,16 +1581,17 @@ static enum wined3d_pci_device select_card_ati_binary(const struct wined3d_gl_in > return CARD_ATI_RADEON_HD2600; > } > > - /* Radeon R6xx HD2300/HD2400/HD3400 - lowend */ > - if (strstr(gl_renderer, "HD 2300") > + /* Radeon R6xx HD2350/HD2400/HD3400 - lowend > + * Note HD2300=DX9, HD2350=DX10 */ > + if (strstr(gl_renderer, "HD 2350") > || strstr(gl_renderer, "HD 2400") > || strstr(gl_renderer, "HD 3470") > || strstr(gl_renderer, "HD 3450") > || strstr(gl_renderer, "HD 3430") > || strstr(gl_renderer, "HD 3400")) > { > - *vidmem = 128; /* HD2300 uses at least 128MB, HD2400 uses 256MB */ > - return CARD_ATI_RADEON_HD2300; > + *vidmem = 256; /* HD2350/2400 use 256MB, HD34xx use 256-512MB */ > + return CARD_ATI_RADEON_HD2350; > } > > /* Radeon R6xx/R7xx integrated */ > @@ -1613,14 +1614,19 @@ static enum wined3d_pci_device select_card_ati_binary(const struct wined3d_gl_in > return CARD_ATI_RADEON_X1600; > } > > - /* Radeon R4xx + X1300/X1400/X1450/X1550/X2300 (lowend R5xx) */ > + /* Radeon R4xx + X1300/X1400/X1450/X1550/X2300/X2500/HD2300 (lowend R5xx) > + * Note X2300/X2500/HD2300 are R5xx GPUs with a 2xxx naming but they are still DX9-only */ > if (strstr(gl_renderer, "X700") > || strstr(gl_renderer, "X800") > || strstr(gl_renderer, "X850") > || strstr(gl_renderer, "X1300") > || strstr(gl_renderer, "X1400") > || strstr(gl_renderer, "X1450") > - || strstr(gl_renderer, "X1550")) > + || strstr(gl_renderer, "X1550") > + || strstr(gl_renderer, "X2300") > + || strstr(gl_renderer, "X2500") > + || strstr(gl_renderer, "HD 2300") > + ) > { > *vidmem = 128; /* x700/x8*0 use 128-256MB, >=x1300 128-512MB */ > return CARD_ATI_RADEON_X700; > @@ -1737,12 +1743,12 @@ static enum wined3d_pci_device select_card_ati_mesa(const struct wined3d_gl_info > return CARD_ATI_RADEON_HD2600; > } > > - /* Radeon R6xx HD2300/HD2400/HD3400 - lowend */ > + /* Radeon R6xx HD2350/HD2400/HD3400 - lowend */ > if (strstr(gl_renderer, "RV610") > || strstr(gl_renderer, "RV620")) > { > - *vidmem = 128; /* HD2300 uses at least 128MB, HD2400 uses 256MB */ > - return CARD_ATI_RADEON_HD2300; > + *vidmem = 256; /* HD2350/2400 use 256MB, HD34xx use 256-512MB */ > + return CARD_ATI_RADEON_HD2350; > } > > /* Radeon R6xx/R7xx integrated */ > @@ -1860,8 +1866,8 @@ static enum wined3d_pci_device select_card_ati_mesa(const struct wined3d_gl_info > if (strstr(gl_renderer, "(RV610") > || strstr(gl_renderer, "(RV620")) > { > - *vidmem = 128; /* HD2300 uses at least 128MB, HD2400 uses 256MB */ > - return CARD_ATI_RADEON_HD2300; > + *vidmem = 256; /* HD2350/2400 use 256MB, HD34xx use 256-512MB */ > + return CARD_ATI_RADEON_HD2350; > } > > /* Radeon R6xx/R7xx integrated */ > diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h > index 4652be5..53200ce 100644 > --- a/dlls/wined3d/wined3d_private.h > +++ b/dlls/wined3d/wined3d_private.h > @@ -1301,7 +1301,7 @@ enum wined3d_pci_device > CARD_ATI_RADEON_XPRESS_200M = 0x5955, > CARD_ATI_RADEON_X700 = 0x5e4c, > CARD_ATI_RADEON_X1600 = 0x71c2, > - CARD_ATI_RADEON_HD2300 = 0x7210, > + CARD_ATI_RADEON_HD2350 = 0x94c7, > CARD_ATI_RADEON_HD2600 = 0x9581, > CARD_ATI_RADEON_HD2900 = 0x9400, > CARD_ATI_RADEON_HD3200 = 0x9620, > -- > 1.6.3.3 > >

1 0

Re: loader: Fix French translation of wine manpage (resent)
by Frédéric Delanoy 15 Apr '10

15 Apr '10

Hi. Could someone please tell me why this patch was silently discarded ? Is there some something awfully wrong in the changes in tools/Makefile.in ? Or some other trivial mistake ? Frédéric

1 0