rayantony / linux-track

Automatically exported from code.google.com/p/linux-track
MIT License
0 stars 0 forks source link

TrackIR.exe causes wineserver to leak file descriptors. #95

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?

1. Play IL-2 or any other game using the wine bridge for a long time.
2. Once wineserver runs out of fd's the whole wine sandbox operates abnormally 
- e.g. IL2 hangs when trying to exit mission. If you log console output, you 
may see messages about "ulimit -n" and not being able to create threads.

This is caused by a file handle leak in tryExclusiveLock which is called every 
second:

https://code.google.com/p/linux-track/source/browse/trunk/src/wine_bridge/client
/rest.c#168

Setting a very low ulimit -n before starting wine allows reproducing this a lot 
sooner.

Original issue reported on code.google.com by ANGavri...@gmail.com on 31 May 2015 at 11:19

GoogleCodeExporter commented 8 years ago
Hello,
first of all, thank you for such nice bug report - I really appreciate the 
effort you put into that.

I see the problem and I'm going to fix it now.

That said, is there a reason why you use the TrackIR.exe thingie? It is meant 
for use with applications that need TIRViews.dll (lib of hacks to make older 
games use the TrackIR); IL2 works well without any such thing (up to 6DOF for 
newer versions, I think 1946 and up) - just start the controller and that is 
it...

Kind regards,

Michal

Original comment by f.jo...@email.cz on 31 May 2015 at 7:19

GoogleCodeExporter commented 8 years ago
Says here it starts automatically, and it does, seeing as I don't:

https://code.google.com/p/linux-track/source/detail?spec=svn912&r=880

Original comment by ANGavri...@gmail.com on 31 May 2015 at 7:25

GoogleCodeExporter commented 8 years ago
You are absolutely right, I found that out while trying to confirm the fix 
actually works...

I just checked-in the fix into the trunk and will try to compile a new 
universal package for you to test.

Kind regards,

Michal

Original comment by f.jo...@email.cz on 31 May 2015 at 7:52

GoogleCodeExporter commented 8 years ago
Hello,
the fix is now in the repository; you can download new universal package to 
test it here:

http://linuxtrack.eu/test/linuxtrack-0.99.12fix-32.zip

http://linuxtrack.eu/test/linuxtrack-0.99.12fix-64.zip

Kind regards,

Michal

Original comment by f.jo...@email.cz on 1 Jun 2015 at 8:29