MP 1.13 Final freezes (Available BIN to TEST in first post) (1 Viewer)

[FantomFan]

Thanks, just tried that and it seems to fix the problem I reported.

New logs attached if there any help (WatchDog options1 and 2).

 

[Sebastiii]

Great but now why lol

So there is something strange it seems :
[2015-12-30 10:09:49,669] [Log ] [MPMain ] [DEBUG] - D3D: OnFormClosing()
[2015-12-30 10:09:49,728] [Log ] [MPMain ] [DEBUG] - D3D CleanupEnvironment()
[2015-12-30 10:09:49,729] [Log ] [MPMain ] [WARN ] - Main: OnDeviceLost()
[2015-12-30 10:09:49,729] [Log ] [MPMain ] [DEBUG] - Main: Application not ready - ignoring Event
It like there is reset of D3D device or something and like the loadskin code try to do something in sync with MP mainthread, deadlock seems to be present. But now you use the new dll, i didn't see loadskin error too, So i don't fully understand what happen lol

 

[Sebastiii]

In second log we can see the loadskin error
I remember that TsReader and DVDplayer code use some quite same code so it can explain why it can failed but really no clue

 

[Sebastiii]

Hi,

I have added a DLL in first post, i have re-enable loadskin but removed a 'lock (GUIGraphicsContext.RenderLock)' call from the loadskin method and where the lock happen.
Now that loadskin call is done in MP main thread, i think the second lock trigger the freeze, hopefully, i was able to reproduce it on my side and it seems to be ok now.

 

[killer8]

Will give your new DLL a try. Like I already said in my bug report https://forum.team-mediaportal.com/threads/mediaportal-screen-hangs-when-stopping-video.133215/ the first fix didn't work for me as it crashed when a quite watching tv

 

[Sebastiii]

Added V3 dll in first post based on : https://issues.team-mediaportal.com/browse/MP1-4741

 

[ravenrocks]

V3 works like a charm for me with the alt-enter problem. Don't think its been this stable even in 1.12. Real fast too.

 

[killer8]

V3 fix seems to work fine, thank you!

 

[Sebastiii]

Added Jira link to first post
From the result, we will integrate it in next MP release, until it we need to use the dll in first post

 

[richardc1983]

This looks like the fix @Adam86