WineHQ
Bug Tracking Database – Bug 30164

 Bugzilla

 

Last modified: 2021-02-06 14:29:59 UTC  

Multiple applications need KERNEL32.dll.InitializeConditionVariable implementation (Adobe Lightroom 4.x and 5.x, Sibelius 7, 64-bit Dolphin Emulator)

Bug 30164 - Multiple applications need KERNEL32.dll.InitializeConditionVariable implementation (Adobe Lightroom 4.x and 5.x, Sibelius 7, 64-bit Dolphin Emulator)
Multiple applications need KERNEL32.dll.InitializeConditionVariable implement...
Status: CLOSED FIXED
AppDB: Show Apps affected by this bug
Product: Wine
Classification: Unclassified
Component: kernel32
1.4
x86-64 Linux
: P3 normal
: ---
Assigned To: Mr. Bugs
https://web.archive.org/web/201403190...
: download
: 31415 33348 34703 39271 41839 44579 (view as bug list)
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2012-03-14 09:34 UTC by Robert Munteanu
Modified: 2021-02-06 14:29 UTC (History)
29 users (show)

See Also:
Regression SHA1:
Fixed by SHA1: 5472b9f9ca1a393c0153e256da67798a3afe53c7
Distribution: ---
Staged patchset:


Attachments
stub (2.12 KB, patch)
2012-03-14 19:53 UTC, Austin English
Details | Diff
Backtrace with a clean prefix, WINEARCH=32bit (7.97 KB, text/plain)
2012-03-15 03:16 UTC, Robert Munteanu
Details
KERNEL32.dll.InitializeConditionVariable (8.01 KB, text/plain)
2012-04-01 14:09 UTC, stgarf+winebugs
Details
Stack Trace for LR 4.1 (10.83 KB, text/plain)
2012-08-21 22:37 UTC, mtparsons87
Details
conditionvars.patch (7.44 KB, patch)
2013-01-05 07:28 UTC, Marcus Meissner
Details | Diff
LR 5 Beta startup log (9.25 KB, text/plain)
2013-04-16 03:38 UTC, Andreas Siegert
Details
LR 5, wine 1.7.2 (8.44 KB, text/plain)
2013-09-21 09:26 UTC, Igor Demyanov
Details
Patches for Lightroom 4.x and 5.x under wine 1.6 (3.53 KB, application/x-gzip)
2013-10-18 01:40 UTC, Roland Baudin
Details
InitializeConditionVariable patch for wine 1.7.9 (7.08 KB, patch)
2013-12-23 08:13 UTC, Jakub Kulesza
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Munteanu 2012-03-14 09:34:20 UTC
ixme:thread:GetThreadPreferredUILanguages 52, 0x23f6a0, 0x23f6b0 0x23f620
fixme:thread:GetThreadPreferredUILanguages 52, 0x23f400, 0x23f410 0x23f380
fixme:thread:GetThreadPreferredUILanguages 52, 0x23f460, 0x23f470 0x23f3e0
fixme:heap:HeapSetInformation 0x10000 0 0x23fca0 4
fixme:thread:GetThreadPreferredUILanguages 52, 0x23f010, 0x23f020 0x23ef90
fixme:shell:IShellLinkW_fnGetPath (0x57060): WIN32_FIND_DATA is not yet filled.
fixme:shell:IShellLinkW_fnGetPath (0x57060): WIN32_FIND_DATA is not yet filled.
wine: Call from 0x7f9f708d43fa to unimplemented function KERNEL32.dll.InitializeConditionVariable, aborting
wine: Unimplemented function KERNEL32.dll.InitializeConditionVariable called at address 0x7f9f708d43fa (thread 0024), starting debugger...
fixme:dbghelp_dwarf:compute_location Only supporting one breg (r8/336 -> rax/328)
fixme:dbghelp_dwarf:compute_location Only supporting one breg (r8/336 -> rcx/330)
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 10
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 9
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 8
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 7
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 6
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 10
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 9
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 8
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 7
fixme:dbghelp:interpret_function_table_entry PUSH_MACHFRAME 6
Comment 1 Dan Kegel 2012-03-14 12:42:29 UTC
According to http://forum.winehq.org/viewtopic.php?p=67085&sid=bb5380e0cfb50c4e9e0e175d66fb2170, also affects Sibelius 7.
Comment 2 Austin English 2012-03-14 19:53:44 UTC
Created attachment 39370 [details]
stub

Try this.
Comment 3 Vitaliy Margolen 2012-03-14 21:30:24 UTC
Doesn't look like you using 64-bit version of the app
Comment 4 Dmitry Timoshkov 2012-03-14 22:52:46 UTC
It would be helpful if you could run the app under 32-bit Wine and post
a real backtrace, so it would be clearer where that API gets called from.
Comment 5 Robert Munteanu 2012-03-15 03:16:33 UTC
Created attachment 39372 [details]
Backtrace with a clean prefix, WINEARCH=32bit

(In reply to comment #4)
> It would be helpful if you could run the app under 32-bit Wine and post
> a real backtrace, so it would be clearer where that API gets called from.

Here's the backtrace.
Comment 6 Dmitry Timoshkov 2012-03-15 03:37:43 UTC
(In reply to comment #5)
> Created attachment 39372 [details]
> Backtrace with a clean prefix, WINEARCH=32bit

Thanks, it's called from agkernel. What are official Windows version
requirements for this application?
Comment 7 Robert Munteanu 2012-03-15 03:41:38 UTC
Microsoft® Windows Vista® with Service Pack 2 or Windows® 7 with Service Pack 1 according to http://www.adobe.com/products/photoshop-lightroom/tech-specs.edu.html
Comment 8 Robert Munteanu 2012-03-15 05:46:27 UTC
(In reply to comment #2)
> Created attachment 39370 [details]
> stub
> 
> Try this.

That works, thanks!

The startup sequence is advanced but I still get errors. I'll file separate bugs if applicable.
Comment 9 Robert Munteanu 2012-03-15 05:50:45 UTC
Filed bug #30173 which looks very similar to this one.
Comment 10 stgarf+winebugs 2012-04-01 14:09:19 UTC
Created attachment 39636 [details]
KERNEL32.dll.InitializeConditionVariable

KERNEL32.dll.InitializeConditionVariable, aborting
Comment 11 Robert Munteanu 2012-04-13 03:09:12 UTC
(In reply to comment #2)
> Created attachment 39370 [details]
> stub
> 
> Try this.

Any chance of getting this committed? It does work for me.
Comment 12 Juergen 2012-04-29 13:56:30 UTC
Hello I'm a new wine user. 
Do you have a workaround for this issue?
I have exactly the same error during the start of Lightroom 4.

I have seen http://bugs.winehq.org/attachment.cgi?id=39370&action=diff but don't know what to do with ??

Thanks to all
Comment 13 Juergen 2012-04-29 14:45:41 UTC
Hi all,

on Wine 1.5.3. the bug still exist.
Comment 14 Robert Munteanu 2012-04-30 02:45:35 UTC
(In reply to comment #12)
> I have seen http://bugs.winehq.org/attachment.cgi?id=39370&action=diff but don't
> know what to do with ??

http://wiki.winehq.org/Patching
Comment 15 Bruno Jesus 2012-08-06 07:05:30 UTC
*** Bug 31415 has been marked as a duplicate of this bug. ***
Comment 16 mtparsons87 2012-08-21 22:37:17 UTC
Created attachment 41448 [details]
Stack Trace for LR 4.1

Had wine set to be Windows 7 when I ran into this issue
Comment 17 David Lichterman 2012-11-26 02:32:54 UTC
Looks like this is indeed the only thing stopping Adobe Lightroom 4.x from running on WINE, bringing it from Garbage to at least a Gold. See the thread here: http://www.pentaxforums.com/forums/digital-processing-software-printing/159894-lightroom-3-4-linux-its-alive-working-pretty-well-3.html#post2171005

As the app maintainer and someone that runs a windows VM just for Lightroom 4.x, all I can really do is beg for someone to finish up the patch initially submitted by Austin English, and worked on by Marcus Meissner here: http://www.winehq.org/pipermail/wine-patches/2012-August/117082.html or someone to tell me what is lacking from the patch and mentoring me as I implement that delta.
Comment 18 Marcus Meissner 2013-01-05 07:28:19 UTC
Created attachment 43089 [details]
conditionvars.patch

current version of my conditional variables patch
Comment 19 lubosz 2013-01-09 14:53:58 UTC
I can confirm that Marcus Meissner's patch works.
Adobe Photoshop Lightroom 4.3 does not start without it on wine 1.5.21
Comment 20 lhn 2013-01-13 23:28:05 UTC
Patch works for me too (Ubuntu 12.04). I followed chroot steps at http://wiki.winehq.org/WineOn64bit, applied patch per http://wiki.winehq.org/Patching, and then used /var/chroot/wine/wine to launch the patched Wine. I do get the filmstrip bug described at http://www.pentaxforums.com/forums/digital-processing-software-printing/159894-lightroom-3-4-linux-its-alive-working-pretty-well-3.html#post2174700, but that seems like a minor inconvenience.
Comment 21 smithabcdef3 2013-01-18 17:41:37 UTC
I have a question for those who have tried this patch successfully... are you running LR in 32 or 64 bit mode?  If 64, are you using WoW64 or pure 64 bit?

I have built a pure 64 bit version (from version 1.5.22), and while the patch definitely allows you to open LR 4.3, it seems to crash very frequently (so far it has happened when going to the Develop module).  The error is:

Unhandled Exception: page fault in 64-bit code (0x000000007976699c)

with a long backtrace (which I can supply if desired).

I still have a lot of work to do before I can even come close to filing an official bug, I just wanted to get an idea of how the rest of you were running this.

(FWIW, I am currently running in a testing VM containing Linux Mint 14 64 bit.  If I can get it working stably I will be installing onto my desktop, running the same distro).

Cheers
Comment 22 Marcus Meissner 2013-01-19 04:16:48 UTC
I also see those crashes in 64bit. I am not sure if they come from my patch or not :/
Comment 23 smithabcdef3 2013-01-21 14:01:37 UTC
Thanks for the reply!

I did get LR working without crashes on wine 1.5 32 bit.  Unfortunately the entire user experience is still not great (not the fault of this patch, mind you, just the state of wine + LR in general).  Developing seems to be slow, there are refresh issues when first importing (mouse over an image and it will go gray), and 1:1 zoom on either develop or library modes only shows a gray screen :-(  That was just the issues I found in the first 5 minutes of testing.

I will try some more, see if I can find a decent configuration, but at this point I am leaning towards playing with LR in a VM (see how much of a performance hit I experience).  Unfortunately it does not seem to be ready for production-level usage in Wine ATM.

Anyway, thanks again for your help and for making this patch in the first place... eventually I am sure things will get worked out!  Just wish I could help rather than just complaining ;-)

Cheers
Comment 24 Gary Thompson 2013-01-29 07:02:43 UTC
I'm having better success and don't have the 1:1 problem.  I pasted a comment on AppDB, and repeating it here (sorry for repeating myself, I'm very excited to have this working).

If you're not afraid to use the command line and follow instructions it was quite easy.

I'm on 64 Bit Ubuntu 12.04.1 and I have Wine already installed and using the Wine PPA. This helps as it brings in dependencies.

Steps I took are noted below but I will reference Roland65s post:

http://www.pentaxforums.com/forums/digital-processing-software-printing/159894-lightroom-3-4-linux-its-alive-working-pretty-well-3.html#post2171005

* Download Roland65s zip file in Step 1
* Follow the Chroot instructions on http://wiki.winehq.org/WineOn64bit but stop just after "git clone source.winehq.org/git/wine"
* type in "cd wine" without quotes
* Using a file explorer from your desktop, create a new file called "conditionvars.patch" under "/var/chroot/wine" and paste into it the contents from http://bugs.winehq.org/attachment.cgi?id=43089&action=diff&context=patch&collapsed=&headers=1&format=raw
* Note in the step above you might need use another terminal and use "gksu gedit" to create the file
* Once the file is created patch the file back in the chroot window using "patch -p1 < conditionvars.patch". Hopefully it is successful, depending on whether it is changed in the git main branch or not (hopefully when these instructions are out of date, the problem is solved).
* type in "make" in the chroot window
* follow Roland65s steps from step 3 onwards but whenever you use winetricks you'll need to type "WINE=/var/chroot/wine/wine winetricks" instead. Also winecfg is "/var/chroot/wine/wine winecfg".

Finally, wine needs to be executed above using "/var/chroot/wine/wine" following my instructions. I haven't gone as far as installing the newly compiled 32-bit libraries in my main computer as this is good enough for me for now as I test.
Comment 25 Lionel T. 2013-03-09 09:44:13 UTC
The patch seems to works very well. Any chance to have it integrated into a next Wine release ?
Comment 26 Marcus Meissner 2013-03-09 11:43:24 UTC
Alexandre was not happy with the current design.

- busy waiting should not be done
- uses too much events / win32 handles


Fixing the design to be acceptable is currently above my alloted timed and abilities.
Comment 27 monte 2013-04-05 14:09:52 UTC
Ok, now after some last updates for Ubuntu 12.04.2LTS Lightroom 4.2 doesn't start even on wine with proposed patch.

I had this thing before, when tried new 12.10. Back then I had to roll back to 12.04 to work with lightroom. Now again ubuntu seems to have broken my wine.
So what do I have is Lightroom starts, shows splash screen and after a few seconds freezes with it sending to console "fixme:sync:WakeConditionVariable (0x47eee50) semi-correct stub" continuously.

I've even tried to install 13.04 on VBox but no chance to run Lightroom again.

The first version of wine I was working on was 1.5.17 with old patch. Today I've got the last version from git, applied the latest patch given in this thread and compiled it hoping to get rid of this bug but Lightroom is still hanging on.

It is very bad for me and I believe not only for me. Unfortunately I'm not a programmer so all I can do is to hope that someone finally brings compatibility with Lightroom to wine.
Comment 28 Nikolay Sivov 2013-04-07 11:58:51 UTC
*** Bug 33348 has been marked as a duplicate of this bug. ***
Comment 29 Andreas Siegert 2013-04-16 03:38:53 UTC
Created attachment 44196 [details]
LR 5 Beta startup log

The LR 5 beta will not start. 
The same Wine instance created with the patches mentioned previously in this bug thread will still run LR 4.4.

cheers
afx
Comment 30 Marcus Meissner 2013-04-16 03:51:32 UTC
please open a new bug for the new missing function
Comment 31 Dan Kegel 2013-04-16 08:40:08 UTC
That's bug 31487
Comment 32 Luke 2013-07-14 09:28:07 UTC
I can also confirm this bug on 1.6-rc4, with Adobe Lightroom 4.1.
Comment 33 Pekka Helenius 2013-09-08 13:43:14 UTC
Will this ever be fixed? Still present in Wine 1.7.1

Someone has managed to solve the problem in Wine 1.5.17 using a custom patch, see post #34 (roland65):

http://www.pentaxforums.com/forums/digital-processing-software-printing/159894-lightroom-3-4-linux-its-alive-working-pretty-well-3.html
Comment 34 Igor Demyanov 2013-09-21 09:26:40 UTC
Created attachment 46015 [details]
LR 5, wine 1.7.2
Comment 35 Anastasius Focht 2013-10-12 04:24:47 UTC
*** Bug 34703 has been marked as a duplicate of this bug. ***
Comment 36 Roland Baudin 2013-10-18 01:40:51 UTC
Created attachment 46346 [details]
Patches for Lightroom 4.x and 5.x under wine 1.6
Comment 37 Roland Baudin 2013-10-18 01:41:28 UTC
It would be nice if this bug could be at last fixed.

I attach the three patches I use (patch 0 by me, and patches 1 and 2 found on the winehq patches mailing list). These patches are for wine 1.6 and Lightroom 4.x or 5.x.

Here is a recall of the recipe I use for 32 bits Lightroom on wine 1.6:

1. winetricks win7
2. Install 32 bits Lightroom
3. winetricks winxp
4. winetricks corefonts gdiplus ie7
5. cp "sRGB Color Space Profile.icm" ~/.wine/drive_c/windows/system32/spool/drivers/color

Item 3 is to get some menus work, otherwise they are corrupted.
Item 5 is only necessary if you don't have a display profile installed system wide in /etc/xdg/color. To install a display profile system wide, run dispcalGUI and go to the options menu to install a profile. If you do that, you'll get full color management in Lightroom (I carefully checked this important point).

At first start, you'll see some UI glitches with the greenish bands (user tips) but this disappeared after.

Ligtroom 5.2 is *very stable* on my Ubuntu 13.04 (gnome classic + compiz) box. Of course I didn't test everything, but the library and development modules work very well. 

RB
Comment 38 Roland Baudin 2013-10-18 01:45:35 UTC
Just to be complete, I also tested Lightroom 5.2 on a WoW64 wine 1.6 install. On Debian Wheezy, I managed to let it work quite well (except the Slideshow module). However, on Ubuntu 13.04 there are random crashes I wasn't able to fix.
RB
Comment 39 Tony Wasserka 2013-11-03 08:45:02 UTC
The 64 bit build of the Dolphin Emulator (http://dl-mirror.dolphin-emu.org/4.0.1/dolphin-x64-4.0.1.exe) fails to start at all due to this bug. I unfortunately can't build Wine from source at the moment, but maybe the fact that other applications are affected as well is some motivation to implement the function.
Comment 40 Anastasius Focht 2013-11-03 10:04:06 UTC
Hello folks,

revising summary because multiple applications are affected.
Also putting Toni's download in here for easier testing.
Commercial products, namely from Adobe may not be available for everyone for tests.

$ sha1sum dolphin-x64-4.0.1.exe 
f69409d7133cb337b49e91d82a43303661ac0f42  dolphin-x64-4.0.1.exe

$ du -sh dolphin-x64-4.0.1.exe 
9.7M	dolphin-x64-4.0.1.exe

$ wine --version
wine-1.7.5-251-gbcf4ded

Regards
Comment 41 Marcus Meissner 2013-11-03 16:47:04 UTC
I have also testcases in wine already if you want to verify other implementations.

The condition variable implementation itself will be a bit more difficult than in the current patches.
Comment 42 Roland Baudin 2013-11-08 01:24:49 UTC
Perhaps the Crossover guys would be glad to claim that Lightroom is a supported application?
Comment 43 Jakub Kulesza 2013-12-23 08:11:40 UTC
I'm posting a patch that is basically one of the files posted by Roland Baudin, but adapted to wine 1.7.9. LR 5.2 runs, Dolphin emulator as well. LR has got some issues with redrawing thou.
Comment 44 Jakub Kulesza 2013-12-23 08:13:13 UTC
Created attachment 46973 [details]
InitializeConditionVariable patch for wine 1.7.9
Comment 45 jason.braddock.69 2014-01-06 14:06:22 UTC
(In reply to comment #44)
> Created attachment 46973 [details]
> InitializeConditionVariable patch for wine 1.7.9

Hi Jakub,

will it be included upstream?

Regards
Comment 46 Jakub Kulesza 2014-01-06 15:47:46 UTC
(In reply to comment #45)
> (In reply to comment #44)
> > Created attachment 46973 [details]
> > InitializeConditionVariable patch for wine 1.7.9
> 
> will it be included upstream?
>

Hi!

I'm not the maintainer, just wanted to share it with people wanting to test it themselves. 

I hope so as well...
Comment 47 jason.braddock.69 2014-01-06 16:17:34 UTC
(In reply to comment #46)
> Hi!
> 
> I'm not the maintainer, just wanted to share it with people wanting to test
> it themselves. 
> 
> I hope so as well...

Do you know how to contact the maintainers and ask them to include it upstream?
I guess some of them may be in this bug's cc list?
Comment 48 Marcus Meissner 2014-01-07 03:03:14 UTC
I am the author of this patch, but the patch was rejected by the Wine maintainer for not meeting all functional criteria.

There is now yet another try to get a InitializeConditionalVariable/*Condition* patch integrated.

The functional requirements on this are really strict, thats why its not easy to do.
Comment 49 jason.braddock.69 2014-01-07 15:49:33 UTC
(In reply to comment #48)
> I am the author of this patch, but the patch was rejected by the Wine
> maintainer for not meeting all functional criteria.
> 
> There is now yet another try to get a
> InitializeConditionalVariable/*Condition* patch integrated.
> 
> The functional requirements on this are really strict, thats why its not
> easy to do.

Hi Marcus,

if I clearly understand, do you mean that there is still code to produce before tha patch can be accepted?

If yes, is there anybody on this?
Comment 50 Marcus Meissner 2014-01-07 17:04:26 UTC
There is currently someone else trying to get a version in, but it also does not yet meet approval.

http://source.winehq.org/patches/data/101427
http://source.winehq.org/patches/data/101426
Comment 51 Marcus Meissner 2014-01-08 13:57:34 UTC
Todays round of commits contained Sebastian Lackner Condition Variable implementation. will be in 1.7.11.
Comment 52 jason.braddock.69 2014-01-08 14:01:15 UTC
(In reply to comment #51)
> Todays round of commits contained Sebastian Lackner Condition Variable
> implementation. will be in 1.7.11.

Hi,

this is great news!
I'll test it as soon as 1.7.11 is out.

Indeed there is no open source alternative to lightroom (darktable doesn't support as many cameras as lightroom does)!

Again thanks for your help.
I will post an update with my tests results.

Regards
Comment 53 Alexandre Julliard 2014-01-17 13:57:59 UTC
Closing bugs fixed in 1.7.11.
Comment 54 Bruno Jesus 2015-09-14 06:26:25 UTC
*** Bug 39271 has been marked as a duplicate of this bug. ***
Comment 55 Bruno Jesus 2016-12-07 15:31:31 UTC
*** Bug 41839 has been marked as a duplicate of this bug. ***
Comment 56 Zeb Figura 2018-02-22 09:01:08 UTC
*** Bug 44579 has been marked as a duplicate of this bug. ***


Privacy Policy
If you have a privacy inquiry regarding this site, please write to [email protected]

Hosted By CodeWeavers