Re: [Openh323-devel] MPEG4 patches

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Matthias Schneider wrote:
> I have raised this problem on the mailing list a while ago but unfortunately got no replies.
> Actually I dont have a clean solutions for this problem, if you take out that 5 ms sleep the video
> threat gets nasty and seems to block other useful functions like hanging up or other user
> interface interactions. I am circumventing the problem right now by using a faster machine where
> the 5 ms dont matter anymore, but I surely would like to hear about a better idea. Also in theory
> this would limit opal to 200 packets/s which would mean a peak rate of roughly 2.4mbit/s ,
> provided that we spent 0 time in things like encoding, decoding and other video processing... 
>
> Matthias
>   

Yeah this errors seem to trigger when testing in the 1.8 to 3.0
megabit/s range.  Strange, my development boxes are dual-core Athlon
4200s, they should have more than enough horsepower, but it even
triggers as superuser when running the app with a very high priority.

I've got a patch attached that fixes the problem in patch.cxx for me.  I
don't expect it to make CVS, but hopefully if anyone else runs into the
same problems they can try it out and maybe figure out a better
solution.  Unfortunately this version is prone to running up the CPU
quite a bit, and the receive queue still fills up, although it gets
cleared quickly.  At the very least, the video doesn't glitch out anymore.

Josh




--- patch.cxx.old	2007-05-09 16:08:54.000000000 -0400
+++ patch.cxx	2007-05-10 15:06:00.000000000 -0400
@@ -558,13 +558,12 @@
 		
     inUse.Signal();
 		
-    if (!isSynchronous || !sourceFrame.GetPayloadSize())
+    if (!sourceFrame.GetPayloadSize())
       PThread::Sleep(5); // Don't starve the CPU
 #if !defined(WIN32)
-    else
+    else if (isSynchronous)
       PThread::Sleep(5); // Permit to another thread to take the mutex
 #endif
-		
     if (len == 0)
       break;
 		
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Openh323-devel mailing list
Openh323-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/openh323-devel

[Index of Archives]     [IETF SIP]     [Gnu Gatekeeper]     [Asterisk PBX]     [Asterisk SS7]     [Fedora Linux]     [Gimp]     [Yosemite News]     [Yosemite Campsites]     [ISDN Cause Codes]

  Powered by Linux